[Koha-bugs] [Bug 14186] Noisy warns in opac-reserve.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14186 Mason James m...@kohaaloha.com changed: What|Removed |Added CC||m...@kohaaloha.com --- Comment #14 from Mason James m...@kohaaloha.com --- Pushed to 3.16.x, will be in 3.16.13 -- 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 14186] Noisy warns in opac-reserve.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14186 Liz Rea wizzy...@gmail.com changed: What|Removed |Added CC||wizzy...@gmail.com --- Comment #13 from Liz Rea wizzy...@gmail.com --- Patch pushed to 3.18.x will be in 3.18.08 -- 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 14186] Noisy warns in opac-reserve.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14186 Chris Cormack ch...@bigballofwax.co.nz changed: What|Removed |Added CC||ch...@bigballofwax.co.nz Severity|enhancement |minor Status|Pushed to Master|Pushed to Stable --- Comment #12 from Chris Cormack ch...@bigballofwax.co.nz --- Pushed to 3.20.x will be in 3.20.1 -- 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 14186] Noisy warns in opac-reserve.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14186 Tomás Cohen Arazi tomasco...@gmail.com changed: What|Removed |Added Status|Passed QA |Pushed to Master CC||tomasco...@gmail.com --- Comment #11 from Tomás Cohen Arazi tomasco...@gmail.com --- Patch pushed to master. Thanks Aleisha and congratulations Indranil for your first 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/
[Koha-bugs] [Bug 14186] Noisy warns in opac-reserve.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14186 Kyle M Hall k...@bywatersolutions.com changed: What|Removed |Added Attachment #39400|0 |1 is obsolete|| --- Comment #9 from Kyle M Hall k...@bywatersolutions.com --- Created attachment 39401 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=39401action=edit [PASSED QA] Bug 14186: Undefined $reservedfor causes warn in opac-reserve.pl This patch sets $reservedfor to an empty string. Test plan = 1/ in a terminal, run `tail -f ` on your instance's opac-error.log 2/ go to the opac and search from an item that exists on the Koha instance. 3/ Select the title (if more than one title is returned) and click on 'Place hold' link to go to opac-reserve.pl 4/ notice the warning - opac-reserve.pl: Use of uninitialized value $reservedfor appear in the `tail`ed opac-error.log 5/ apply the patch 6/ reload the page (opac-reserve.pl) 7/ page works but the warning in step #4 is no longer thrown up 8/ run qa test (i.e. koha-qa.pl -c 1 -v 2), there should be no error Remarks: The QA test failed - forbidden pattern: tab char (line 470). Marking this as 'FAILED QA' Signed-off-by: Kyle M Hall k...@bywatersolutions.com -- 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 14186] Noisy warns in opac-reserve.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14186 Kyle M Hall k...@bywatersolutions.com changed: What|Removed |Added Attachment #39399|0 |1 is obsolete|| --- Comment #8 from Kyle M Hall k...@bywatersolutions.com --- Created attachment 39400 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=39400action=edit [PASSED QA] Bug 14186: Undefined $reservedfor causes warn in opac-reserve.pl This is a followup for Bug 14186 that removes the extraneous tab char on line 470, so that the patch can clear QA tools. This patch sets $reservedfor to an empty string. Test plan = 1/ in a terminal, run `tail -f ` on your instance's opac-error.log 2/ go to the opac and search from an item that exists on the Koha instance. 3/ Select the title (if more than one title is returned) and click on 'Place hold' link to go to opac-reserve.pl 4/ notice the warning - opac-reserve.pl: Use of uninitialized value $reservedfor appear in the `tail`ed opac-error.log 5/ apply the patch 6/ reload the page (opac-reserve.pl) 7/ page works but the warning in step #4 is no longer thrown up 8/ run qa test (i.e. koha-qa.pl -c 1 -v 2), there should be no error Remarks: Testing result match expected test plan output. The QA tests pass with OK for the commit. Signed-off-by: Indranil Das Gupta (L2C2 Technologies) indr...@gmail.com Signed-off-by: Kyle M Hall k...@bywatersolutions.com -- 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 14186] Noisy warns in opac-reserve.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14186 Kyle M Hall k...@bywatersolutions.com changed: What|Removed |Added Status|Signed Off |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 14186] Noisy warns in opac-reserve.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14186 Kyle M Hall k...@bywatersolutions.com changed: What|Removed |Added Attachment #39175|0 |1 is obsolete|| Attachment #39176|0 |1 is obsolete|| --- Comment #7 from Kyle M Hall k...@bywatersolutions.com --- Created attachment 39399 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=39399action=edit [PASSED QA] Bug 14186: Undefined $reservedfor causes warn in opac-reserve.pl This patch sets $reservedfor to an empty string. Test plan = 1/ in a terminal, run `tail -f ` on your instance's opac-error.log 2/ go to the opac and search from an item that exists on the Koha instance. 3/ Select the title (if more than one title is returned) and click on 'Place hold' link to go to opac-reserve.pl 4/ notice the warning - opac-reserve.pl: Use of uninitialized value $reservedfor appear in the `tail`ed opac-error.log 5/ apply the patch 6/ reload the page (opac-reserve.pl) 7/ page works but the warning in step #4 is no longer thrown up 8/ run qa test (i.e. koha-qa.pl -c 1 -v 2), there should be no error Remarks: The QA test failed - forbidden pattern: tab char (line 470). Marking this as 'FAILED QA' Signed-off-by: Kyle M Hall k...@bywatersolutions.com -- 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 14186] Noisy warns in opac-reserve.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14186 --- Comment #10 from Kyle M Hall k...@bywatersolutions.com --- Created attachment 39402 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=39402action=edit [PASSED QA] Bug 14186 [QA Followup]: Undefined $reservedfor causes warn in opac-reserve.pl This is a followup for Bug 14186 that removes the extraneous tab char on line 470, so that the patch can clear QA tools. This patch sets $reservedfor to an empty string. Test plan = 1/ in a terminal, run `tail -f ` on your instance's opac-error.log 2/ go to the opac and search from an item that exists on the Koha instance. 3/ Select the title (if more than one title is returned) and click on 'Place hold' link to go to opac-reserve.pl 4/ notice the warning - opac-reserve.pl: Use of uninitialized value $reservedfor appear in the `tail`ed opac-error.log 5/ apply the patch 6/ reload the page (opac-reserve.pl) 7/ page works but the warning in step #4 is no longer thrown up 8/ run qa test (i.e. koha-qa.pl -c 1 -v 2), there should be no error Remarks: Testing result match expected test plan output. The QA tests pass with OK for the commit. Signed-off-by: Indranil Das Gupta (L2C2 Technologies) indr...@gmail.com Signed-off-by: Kyle M Hall k...@bywatersolutions.com -- 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 14186] Noisy warns in opac-reserve.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14186 --- Comment #2 from Indranil Das Gupta indr...@gmail.com --- Comment on attachment 39057 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=39057 Bug 14186: Undefined $reservedfor causes warn in opac-reserve.pl Review of attachment 39057: -- (http://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.htmlbug=14186attachment=39057) - Hi Aleisha, Would you be considering making the small change mtomp...@hotmail.com suggested in bug 14185, comment 3 for this patch? If you do, then I'll wait for your update, before I sign off on this. -- 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 14186] Noisy warns in opac-reserve.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14186 Indranil Das Gupta indr...@gmail.com changed: What|Removed |Added Assignee|oleon...@myacpl.org |aleishaamo...@hotmail.com CC||indr...@gmail.com -- 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 14186] Noisy warns in opac-reserve.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14186 Indranil Das Gupta indr...@gmail.com changed: What|Removed |Added Version|unspecified |master --- Comment #3 from Indranil Das Gupta indr...@gmail.com --- (In reply to Indranil Das Gupta from comment #2) Comment on attachment 39057 [details] [review] Bug 14186: Undefined $reservedfor causes warn in opac-reserve.pl Review of attachment 39057 [details] [review]: - Hi Aleisha, Would you be considering making the small change mtomp...@hotmail.com suggested in bug 14185, comment 3 for this patch? If you do, then I'll wait for your update, before I sign off on this. ^^^ the question is no longer relevant. Will proceed with testing. -- 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 14186] Noisy warns in opac-reserve.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14186 Indranil Das Gupta indr...@gmail.com changed: What|Removed |Added Attachment #39057|0 |1 is obsolete|| --- Comment #4 from Indranil Das Gupta indr...@gmail.com --- Created attachment 39175 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=39175action=edit Bug 14186: Undefined $reservedfor causes warn in opac-reserve.pl This patch sets $reservedfor to an empty string. Test plan = 1/ in a terminal, run `tail -f ` on your instance's opac-error.log 2/ go to the opac and search from an item that exists on the Koha instance. 3/ Select the title (if more than one title is returned) and click on 'Place hold' link to go to opac-reserve.pl 4/ notice the warning - opac-reserve.pl: Use of uninitialized value $reservedfor appear in the `tail`ed opac-error.log 5/ apply the patch 6/ reload the page (opac-reserve.pl) 7/ page works but the warning in step #4 is no longer thrown up 8/ run qa test (i.e. koha-qa.pl -c 1 -v 2), there should be no error Remarks: The QA test failed - forbidden pattern: tab char (line 470). Marking this as 'FAILED 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 14186] Noisy warns in opac-reserve.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14186 Indranil Das Gupta indr...@gmail.com changed: What|Removed |Added Status|Needs Signoff |Failed 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 14186] Noisy warns in opac-reserve.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14186 --- Comment #5 from Indranil Das Gupta indr...@gmail.com --- Comment on attachment 39175 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=39175 Bug 14186: Undefined $reservedfor causes warn in opac-reserve.pl Result of QA test FAILopac/opac-reserve.pl OK critic FAIL forbidden patterns forbidden pattern: tab char (line 470) OK pod OK valid -- 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 14186] Noisy warns in opac-reserve.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14186 Indranil Das Gupta indr...@gmail.com changed: What|Removed |Added Status|Failed QA |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 14186] Noisy warns in opac-reserve.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14186 --- Comment #6 from Indranil Das Gupta indr...@gmail.com --- Created attachment 39176 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=39176action=edit [FOLLOWUP] Bug 14186: Undefined $reservedfor causes warn in opac-reserve.pl This is a followup for Bug 14186 that removes the extraneous tab char on line 470, so that the patch can clear QA tools. This patch sets $reservedfor to an empty string. Test plan = 1/ in a terminal, run `tail -f ` on your instance's opac-error.log 2/ go to the opac and search from an item that exists on the Koha instance. 3/ Select the title (if more than one title is returned) and click on 'Place hold' link to go to opac-reserve.pl 4/ notice the warning - opac-reserve.pl: Use of uninitialized value $reservedfor appear in the `tail`ed opac-error.log 5/ apply the patch 6/ reload the page (opac-reserve.pl) 7/ page works but the warning in step #4 is no longer thrown up 8/ run qa test (i.e. koha-qa.pl -c 1 -v 2), there should be no error Remarks: Testing result match expected test plan output. The QA tests pass with OK for the commit. Signed-off-by: Indranil Das Gupta (L2C2 Technologies) indr...@gmail.com -- 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 14186] Noisy warns in opac-reserve.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14186 --- Comment #1 from Aleisha Amohia aleishaamo...@hotmail.com --- Created attachment 39057 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=39057action=edit Bug 14186: Undefined $reservedfor causes warn in opac-reserve.pl This patch sets $reservedfor to an empty string. To test: 1) On the OPAC, do a simple search for an item that will return results (ie by Keyword, search for a) 2) Click Place Hold on any item 3) Notice warn in error log 4) Apply patch 5) Reload page (attempt to Place Hold) 6) Notice page still works but no warns -- 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 14186] Noisy warns in opac-reserve.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14186 Aleisha Amohia aleishaamo...@hotmail.com changed: What|Removed |Added Status|NEW |Needs Signoff Patch complexity|--- |Trivial 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/
