]> git.koha-community.org Git - koha.git/commit
Bug 28052: Do not check SYSENV values only keys in loop
authorColin Campbell <colin.campbell@ptfs-europe.com>
Tue, 30 Mar 2021 11:13:27 +0000 (12:13 +0100)
committerFridolin Somers <fridolin.somers@biblibre.com>
Thu, 15 Apr 2021 10:35:31 +0000 (12:35 +0200)
commit0a7cf00d207369c3d46d2dc7078fe8e5574abeb2
tree7caef4cb4482c20e66fdb4cd3a7d7606cafb8b1f
parent42db687f67885fd0858f5afb19af082f42c187a6
Bug 28052: Do not check SYSENV values only keys in loop

Loop is only interested in key values, because 'keys' is omitted
it is checking values as well. This can lead to misleading errors
being logged (e.g. key is undefined)
Extract keys only to loop array as in the other loops below this

Testing:
1 - tail /var/lib/koha/kohadev/sip-output.log
2 - in another tab use the sip cli tester:
   perl misc/sip_cli_emulator.pl -a localhost -p 6001 -su term1 -sp term1 -l CPL -m checkin --item CHESS1
3 - Note errors:
Use of uninitialized value $key in index at /kohadevbox/koha/C4/SIP/Sip/MsgType.pm line 337, <STDIN> line 1.
Use of uninitialized value $key in index at /kohadevbox/koha/C4/SIP/Sip/MsgType.pm line 337, <STDIN> line 2.
4 - Apply patch and restart all the things
5 - Repeat 2
6 - Errors mentioned are gone

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
(cherry picked from commit cd40febe67b4d75e57311ff1673d0c34299b58fe)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
C4/SIP/Sip/MsgType.pm