http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7839
Paul Poulain <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA CC| |[email protected] QA Contact|[email protected] |[email protected] --- Comment #10 from Paul Poulain <[email protected]> --- QA comments: 1 the sub "get" in "getAttachedSubscriptions" is improperly spelled. It should be GetAttachedSubscriptions 2 the sub name is wrong I think: - Get...Subscriptions let think it returns a list of subscriptions, that's the case, that's OK - the "Attached" keyword is related to nothing. And there is no reference to the borrowernumber or routinglist thing. This sub should have another name. Something like GetSubscriptionsFromRoutinglistSubscriber or GetSubscriptionsFromBorrower 3 the + JOIN subscriptionroutinglist ON subscriptionroutinglist.subscriptionid = subscription.subscriptionid could be written + JOIN subscriptionroutinglist ON USING(subscriptionid) (not requested for passed QA status, but shorter and should be used as much as possible in the code) 4 the new script and C4/Serials.pm passes perlcritic 5 prove xt/author/translatable-templates.t xt/author/valid-templates.t is OK as well Please provide a follow-up or an updated fix for point 1, 2 and optionnally 3 to earn passed QA status (and be the 183th committer to join the list of Koha committers ;-) ) -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
