[Koha-bugs] [Bug 10494] KohaBranchName sends useless warnings to log if supplied library code is null or not present
bugzilla-daemon at bugs.koha-community.org
bugzilla-daemon at bugs.koha-community.org
Sun Jun 30 19:17:56 CEST 2013
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10494
--- Comment #6 from Katrin Fischer <katrin.fischer at bsz-bw.de> ---
Created attachment 19302
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=19302&action=edit
[PASSED QA] Bug 10494: remove spurious warnings from the KohaBranchName plugin
The KohaBranchName plugin now returns an empty string if
given a NULL (undefined) library code or a library code that
is not present in the database (e.g., the '__ANY__' placeholder
that purchase suggestions can use).
To test:
[1] Create a purchase suggestion, setting the library
to 'Any'. Create a second one setting the library to
a specific one.
[2] Go to the suggestions overview page in the staff interface.
When you do so, the Apache logs should contain a warning
that looks like this:
[Wed Jun 19 07:13:06 2013] [error] [client 192.168.1.10] [Wed Jun 19 07:13:06
2013] suggestion.pl: Use of uninitialized value in concatenation (.) or string
at /home/koha/koha-tmpl/intranet-tmpl/prog/en/modules/suggestion/suggestion.tt
line 487., referer:
http://koha-staff.metaquarry/cgi-bin/koha/acqui/acqui-home.pl
[3] Apply the patch.
[4] Refresh the suggestions overview page. This time, there should be
no new warning logged.
[5] Verify that the library names for the two pending suggestions
created in step 1 are displayed correctly.
Signed-off-by: Galen Charlton <gmc at esilibrary.com>
Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
Passes all tests and QA script. Also adds new tests!
Note: For the test plan to work you have to add the suggestion
from the intranet, as Any is not an available option in OPAC.
--
You are receiving this mail because:
You are watching all bug changes.
More information about the Koha-bugs
mailing list