[Koha-bugs] [Bug 29771] Get rid of CGI::param in list context warnings
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29771 Katrin Fischer changed: What|Removed |Added Blocks||25790 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25790 [Bug 25790] [OMNIBUS] warnings removal -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://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 29771] Get rid of CGI::param in list context warnings
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29771 Katrin Fischer changed: What|Removed |Added Blocks|25790 | Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25790 [Bug 25790] [OMNIBUS] warnings removal -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://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 29771] Get rid of CGI::param in list context warnings
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29771 Victor Grousset/tuxayo changed: What|Removed |Added Resolution|--- |FIXED CC||vic...@tuxayo.net Status|Pushed to oldstable |RESOLVED --- Comment #16 from Victor Grousset/tuxayo --- Not backported to oldoldstable (20.11.x). Feel free to ask if it's needed. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://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 29771] Get rid of CGI::param in list context warnings
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29771 Andrew Fuerste-Henry changed: What|Removed |Added CC||and...@bywatersolutions.com Version(s)|22.05.00,21.11.04 |22.05.00,21.11.04,21.05.14 released in|| Status|Pushed to stable|Pushed to oldstable --- Comment #15 from Andrew Fuerste-Henry --- Pushed to 21.05.x for 21.05.14 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://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 29771] Get rid of CGI::param in list context warnings
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29771 Kyle M Hall changed: What|Removed |Added Status|Pushed to master|Pushed to stable CC||k...@bywatersolutions.com Version(s)|22.05.00|22.05.00,21.11.04 released in|| --- Comment #14 from Kyle M Hall --- Pushed to 21.11.x for 21.11.04 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://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 29771] Get rid of CGI::param in list context warnings
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29771 --- Comment #13 from Fridolin Somers --- Pushed to master for 22.05, thanks to everybody involved 濾 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://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 29771] Get rid of CGI::param in list context warnings
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29771 Fridolin Somers changed: What|Removed |Added Version(s)||22.05.00 released in|| Status|Passed QA |Pushed to master -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://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 29771] Get rid of CGI::param in list context warnings
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29771 Fridolin Somers changed: What|Removed |Added Depends on|17239 | See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=17239 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17239 [Bug 17239] Vulnerabilitiy warning for package C4::Biblio -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://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 29771] Get rid of CGI::param in list context warnings
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29771 --- Comment #12 from Fridolin Somers --- (In reply to Tomás Cohen Arazi from comment #11) > (In reply to Fridolin Somers from comment #10) > > Dependency with Bug 17239 ? isn't it the opposite relation ? > > I think it is wrong. Probably a duplicate instead OK I move to See Also -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://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 29771] Get rid of CGI::param in list context warnings
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29771 --- Comment #11 from Tomás Cohen Arazi --- (In reply to Fridolin Somers from comment #10) > Dependency with Bug 17239 ? isn't it the opposite relation ? I think it is wrong. Probably a duplicate instead -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://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 29771] Get rid of CGI::param in list context warnings
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29771 Fridolin Somers changed: What|Removed |Added CC||fridolin.som...@biblibre.co ||m --- Comment #10 from Fridolin Somers --- Dependency with Bug 17239 ? isn't it the opposite relation ? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://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 29771] Get rid of CGI::param in list context warnings
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29771 Martin Renvoize changed: What|Removed |Added Status|Signed Off |Passed QA CC||martin.renvoize@ptfs-europe ||.com --- Comment #9 from Martin Renvoize --- Great call! This fixes all obvious cases and many non-obvious ones too.. No regressions found, lets get this pushed and deal with any further occurrences as we spot them. It would be nice to have a QA script test catch new introductions of such warnings.. but that can be handled outside of this bug. Passing QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://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 29771] Get rid of CGI::param in list context warnings
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29771 Martin Renvoize changed: What|Removed |Added Attachment #131130|0 |1 is obsolete|| --- Comment #8 from Martin Renvoize --- Created attachment 131188 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=131188=edit Bug 29771: Scalar context for split This patch tackles a very specific scenario. Calling split(..., CGI::param) makes it be called in list context. The split docs say: split /PATTERN/,EXPR,LIMIT this means the first CGI param will be used as EXPR and the second one as LIMIT, which is wrong anyway. So the fix is to just force scalar context. To test: 1. Not sure, just make sure nothing breaks when using the scripts in the browser. Signed-off-by: Tomas Cohen Arazi Signed-off-by: Nick Clemens Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://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 29771] Get rid of CGI::param in list context warnings
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29771 Martin Renvoize changed: What|Removed |Added Attachment #131129|0 |1 is obsolete|| --- Comment #7 from Martin Renvoize --- Created attachment 131187 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=131187=edit Bug 29771: Remove trivial cases I squashed the patches because they are too trivial to have a test plan. Or it is too much work to write the test plan for such trivial cases. I leave the original commit messages just in case. Generally, this are all cases in which CGI::param is being called in a trivially identifiable _list context_. i.e. they are assigned to a @variable. I left one case out on purpose: admin/auth_subfields_structure.pl Paul introduced this: my @kohafield = ''.$input->param('kohafield'); and then: my $kohafield = $kohafield[$i]; My intuition says it is forcing scalar context on the first assignment so the list contains a single element and then inside the loop some $kohafield assignments should lead to undef, and even warnings. I leave it for a separate patch because it is not that easy testable and is a sensible area. Bug 29771: Remove warning from acqui/finishreceive.pl This patch removes a warning that shows when receiving. To test: 1. Do the acq workflow up to the receive step. 2. Once you choose the items and click on Finish => FAIL: There's a warning in the logs 3. Revert receipt 4. Apply this patch 5. Receive => SUCCESS: No more warnings 6. Sign off :-D Bug 29771: Remove warning from svc/members/add_to_list To test: 1. Run: $ tail -f /var/log/koha/kohadev/*-error.log 2. Generate a patron list 3. Perform a patron search that gives you a few 4. Select some, and choose to add them to the list => FAIL: The logs show the infamous warn: CGI::param called in list context from /kohadevbox/koha/svc/members/add_to_list 5. Apply this patch 6. Restart plack and repeat 4 => SUCCESS: No warn! 7. Sign off :-D Signed-off-by: Tomas Cohen Arazi Signed-off-by: Nick Clemens Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://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 29771] Get rid of CGI::param in list context warnings
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29771 Nick Clemens changed: What|Removed |Added Attachment #128910|0 |1 is obsolete|| --- Comment #6 from Nick Clemens --- Created attachment 131130 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=131130=edit Bug 29771: Scalar context for split This patch tackles a very specific scenario. Calling split(..., CGI::param) makes it be called in list context. The split docs say: split /PATTERN/,EXPR,LIMIT this means the first CGI param will be used as EXPR and the second one as LIMIT, which is wrong anyway. So the fix is to just force scalar context. To test: 1. Not sure, just make sure nothing breaks when using the scripts in the browser. Signed-off-by: Tomas Cohen Arazi Signed-off-by: Nick Clemens -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://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 29771] Get rid of CGI::param in list context warnings
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29771 Nick Clemens changed: What|Removed |Added Attachment #128909|0 |1 is obsolete|| --- Comment #5 from Nick Clemens --- Created attachment 131129 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=131129=edit Bug 29771: Remove trivial cases I squashed the patches because they are too trivial to have a test plan. Or it is too much work to write the test plan for such trivial cases. I leave the original commit messages just in case. Generally, this are all cases in which CGI::param is being called in a trivially identifiable _list context_. i.e. they are assigned to a @variable. I left one case out on purpose: admin/auth_subfields_structure.pl Paul introduced this: my @kohafield = ''.$input->param('kohafield'); and then: my $kohafield = $kohafield[$i]; My intuition says it is forcing scalar context on the first assignment so the list contains a single element and then inside the loop some $kohafield assignments should lead to undef, and even warnings. I leave it for a separate patch because it is not that easy testable and is a sensible area. Bug 29771: Remove warning from acqui/finishreceive.pl This patch removes a warning that shows when receiving. To test: 1. Do the acq workflow up to the receive step. 2. Once you choose the items and click on Finish => FAIL: There's a warning in the logs 3. Revert receipt 4. Apply this patch 5. Receive => SUCCESS: No more warnings 6. Sign off :-D Bug 29771: Remove warning from svc/members/add_to_list To test: 1. Run: $ tail -f /var/log/koha/kohadev/*-error.log 2. Generate a patron list 3. Perform a patron search that gives you a few 4. Select some, and choose to add them to the list => FAIL: The logs show the infamous warn: CGI::param called in list context from /kohadevbox/koha/svc/members/add_to_list 5. Apply this patch 6. Restart plack and repeat 4 => SUCCESS: No warn! 7. Sign off :-D Signed-off-by: Tomas Cohen Arazi Signed-off-by: Nick Clemens -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://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 29771] Get rid of CGI::param in list context warnings
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29771 Nick Clemens 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 https://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 29771] Get rid of CGI::param in list context warnings
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29771 Bug 29771 depends on bug 18320, which changed state. Bug 18320 Summary: patroncards/edit-layout.pl raises warnings https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18320 What|Removed |Added Status|Pushed to oldstable |RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://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 29771] Get rid of CGI::param in list context warnings
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29771 Bug 29771 depends on bug 29758, which changed state. Bug 29758 Summary: CGI::param in list context in boraccount.pl warning https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29758 What|Removed |Added Status|Pushed to oldstable |RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://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 29771] Get rid of CGI::param in list context warnings
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29771 Jonathan Druart changed: What|Removed |Added QA Contact|testo...@bugs.koha-communit |jonathan.druart+koha@gmail. |y.org |com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://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 29771] Get rid of CGI::param in list context warnings
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29771 Bug 29771 depends on bug 21051, which changed state. Bug 21051 Summary: CGI::param called in list context when editing a patron card layout https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21051 What|Removed |Added Status|NEW |RESOLVED Resolution|--- |DUPLICATE -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://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 29771] Get rid of CGI::param in list context warnings
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29771 Jonathan Druart changed: What|Removed |Added CC||jonathan.druart+koha@gmail. ||com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://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 29771] Get rid of CGI::param in list context warnings
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29771 Tomás Cohen Arazi changed: What|Removed |Added Summary|Get rid of CGI::param in|Get rid of CGI::param in |list context warnings |list context warnings |[UMBRELLA] | Blocks||25790 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25790 [Bug 25790] [OMNIBUS] warnings removal -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://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 29771] Get rid of CGI::param in list context warnings [UMBRELLA]
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29771 --- Comment #4 from Tomás Cohen Arazi --- Created attachment 128910 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=128910=edit Bug 29771: Scalar context for split This patch tackles a very specific scenario. Calling split(..., CGI::param) makes it be called in list context. The split docs say: split /PATTERN/,EXPR,LIMIT this means the first CGI param will be used as EXPR and the second one as LIMIT, which is wrong anyway. So the fix is to just force scalar context. To test: 1. Not sure, just make sure nothing breaks when using the scripts in the browser. Signed-off-by: Tomas Cohen Arazi -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://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 29771] Get rid of CGI::param in list context warnings [UMBRELLA]
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29771 --- Comment #3 from Tomás Cohen Arazi --- Created attachment 128909 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=128909=edit Bug 29771: Remove trivial cases I squashed the patches because they are too trivial to have a test plan. Or it is too much work to write the test plan for such trivial cases. I leave the original commit messages just in case. Generally, this are all cases in which CGI::param is being called in a trivially identifiable _list context_. i.e. they are assigned to a @variable. I left one case out on purpose: admin/auth_subfields_structure.pl Paul introduced this: my @kohafield = ''.$input->param('kohafield'); and then: my $kohafield = $kohafield[$i]; My intuition says it is forcing scalar context on the first assignment so the list contains a single element and then inside the loop some $kohafield assignments should lead to undef, and even warnings. I leave it for a separate patch because it is not that easy testable and is a sensible area. Bug 29771: Remove warning from acqui/finishreceive.pl This patch removes a warning that shows when receiving. To test: 1. Do the acq workflow up to the receive step. 2. Once you choose the items and click on Finish => FAIL: There's a warning in the logs 3. Revert receipt 4. Apply this patch 5. Receive => SUCCESS: No more warnings 6. Sign off :-D Bug 29771: Remove warning from svc/members/add_to_list To test: 1. Run: $ tail -f /var/log/koha/kohadev/*-error.log 2. Generate a patron list 3. Perform a patron search that gives you a few 4. Select some, and choose to add them to the list => FAIL: The logs show the infamous warn: CGI::param called in list context from /kohadevbox/koha/svc/members/add_to_list 5. Apply this patch 6. Restart plack and repeat 4 => SUCCESS: No warn! 7. Sign off :-D Signed-off-by: Tomas Cohen Arazi -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://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 29771] Get rid of CGI::param in list context warnings [UMBRELLA]
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29771 Tomás Cohen Arazi changed: What|Removed |Added CC||eve...@bywatersolutions.com ||, l...@bywatersolutions.com Assignee|koha-b...@lists.koha-commun |tomasco...@gmail.com |ity.org | -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://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 29771] Get rid of CGI::param in list context warnings [UMBRELLA]
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29771 Tomás Cohen Arazi changed: What|Removed |Added Attachment #128908|0 |1 is obsolete|| -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://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 29771] Get rid of CGI::param in list context warnings [UMBRELLA]
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29771 Tomás Cohen Arazi changed: What|Removed |Added Status|NEW |Needs Signoff Patch complexity|--- |Small patch CC||tomasco...@gmail.com -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://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 29771] Get rid of CGI::param in list context warnings [UMBRELLA]
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29771 --- Comment #2 from Tomás Cohen Arazi --- Created attachment 128908 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=128908=edit Bug 29771: Remove warning from acqui/finishreceive.pl This patch removes a warning that shows when receiving. To test: 1. Do the acq workflow up to the receive step. 2. Once you choose the items and click on Finish => FAIL: There's a warning in the logs 3. Revert receipt 4. Apply this patch 5. Receive => SUCCESS: No more warnings 6. Sign off :-D Signed-off-by: Tomas Cohen Arazi -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://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 29771] Get rid of CGI::param in list context warnings [UMBRELLA]
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29771 --- Comment #1 from Tomás Cohen Arazi --- $ git grep '\->param' | grep -v -e 'template' -e 'multi_param' | grep '@' | wc -l 54 -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://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 29771] Get rid of CGI::param in list context warnings [UMBRELLA]
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29771 Tomás Cohen Arazi changed: What|Removed |Added Depends on||21051, 18320, 17239 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17239 [Bug 17239] Vulnerabilitiy warning for package C4::Biblio https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18320 [Bug 18320] patroncards/edit-layout.pl raises warnings https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21051 [Bug 21051] CGI::param called in list context when editing a patron card layout -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://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 29771] Get rid of CGI::param in list context warnings [UMBRELLA]
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29771 Tomás Cohen Arazi changed: What|Removed |Added Summary|Get rid of CGI::param in|Get rid of CGI::param in |list context warnings |list context warnings ||[UMBRELLA] Depends on||29758 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29758 [Bug 29758] CGI::param in list context in boraccount.pl warning -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://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/