[Koha-bugs] [Bug 14145] Noisy warns in admin/preferences.pl

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Sat May 9 07:39:35 CEST 2015


http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14145

--- Comment #3 from M. Tompsett <mtompset at hotmail.com> ---
Created attachment 38999
  -->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=38999&action=edit
Bug 14145: Nicer counter-patch

The problem with the odd number of hashes happens when not everything is
selected.
The grep returns undef, because it isn't found in @values.
By turning the grep into a ternary-operator truth value, we can set a value (1
or 0) expressly.

The next problem is when nothing is selected in these multiple lists, $value is
undefined, so you can't split it. By splitting the definition of @values from
the actual splitting, we can split only if $value is defined, thus eliminating
the
warning message.

TEST PLAN
---------
1) back up your koha error log file
2) blank your koha error log file
3) log in to the staff client
4) Home -> Koha administration -> Global system preferences
5) Click on every tab, EXCEPT local use.
6) notice the koha error log file has warnings.
7) blank the koha error log file again
8) apply this patch
9) Click on every tab, EXCEPT local use, again.
10) notice the koha error log file has no warnings.
11) koha qa test tools.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.


More information about the Koha-bugs mailing list