[Koha-bugs] [Bug 10291] Misleading Variable names in batchMod.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10291 Galen Charlton changed: What|Removed |Added Status|Passed QA |Pushed to Master CC||gmcha...@gmail.com --- Comment #8 from Galen Charlton --- Pushed to master. Thanks, Colin! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 10291] Misleading Variable names in batchMod.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10291 Jonathan Druart changed: What|Removed |Added Attachment #18634|0 |1 is obsolete|| --- Comment #7 from Jonathan Druart --- Created attachment 19076 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=19076&action=edit Bug 10291: Clarify misleading variable name Remove the unrequired extra processing and pass the array of itemnumbers as itemnumbers_array Signed-off-by: Srdjan Signed-off-by: Jonathan Druart -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 10291] Misleading Variable names in batchMod.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10291 Jonathan Druart changed: What|Removed |Added Status|Signed Off |Passed QA CC||jonathan.dru...@biblibre.co ||m Patch complexity|--- |Trivial patch QA Contact||jonathan.dru...@biblibre.co ||m --- Comment #6 from Jonathan Druart --- QA comment: Variable renaming. Need to modify the max amount of items (l. 113) and see the html code source for testing this patch. No regression, no change detected. Marked as Passed QA. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 10291] Misleading Variable names in batchMod.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10291 Srdjan Jankovic changed: What|Removed |Added Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 10291] Misleading Variable names in batchMod.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10291 Srdjan Jankovic changed: What|Removed |Added Attachment #18629|0 |1 is obsolete|| --- Comment #5 from Srdjan Jankovic --- Created attachment 18634 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=18634&action=edit [SIGNED-OFF] Bug 10291: Clarify misleading variable name Remove the unrequired extra processing and pass the array of itemnumbers as itemnumbers_array Signed-off-by: Srdjan -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 10291] Misleading Variable names in batchMod.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10291 Colin Campbell changed: What|Removed |Added Status|ASSIGNED|Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 10291] Misleading Variable names in batchMod.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10291 Colin Campbell changed: What|Removed |Added Attachment #18259|0 |1 is obsolete|| --- Comment #4 from Colin Campbell --- Created attachment 18629 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=18629&action=edit Updated patch Simplified the patch by not processing an array to look like a hash just to save a variable name. Passes an array of itemnumbers as exactly that -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 10291] Misleading Variable names in batchMod.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10291 Colin Campbell changed: What|Removed |Added Status|Needs Signoff |ASSIGNED --- Comment #3 from Colin Campbell --- (In reply to comment #2) > It looks pointless to me to have hashrefs with only one key 'itemnumber'. > Would you consider changing those arrays to be pure itemnumbers please, > while you're at it? Yes in fact that simplifies the patch. Will post an update -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 10291] Misleading Variable names in batchMod.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10291 Srdjan Jankovic changed: What|Removed |Added CC||srd...@catalyst.net.nz --- Comment #2 from Srdjan Jankovic --- It looks pointless to me to have hashrefs with only one key 'itemnumber'. Would you consider changing those arrays to be pure itemnumbers please, while you're at it? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 10291] Misleading Variable names in batchMod.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10291 Colin Campbell changed: What|Removed |Added Status|NEW |Needs Signoff Assignee|gmcha...@gmail.com |colin.campbell@ptfs-europe. ||com --- Comment #1 from Colin Campbell --- Created attachment 18259 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=18259&action=edit proposed patch -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/