]> git.koha-community.org Git - koha.git/commit
Bug 21036: Fix warnings from C4/Biblio
authorMarcel de Rooy <m.de.rooy@rijksmuseum.nl>
Thu, 5 Jul 2018 07:53:57 +0000 (09:53 +0200)
committerLucas Gass <lucas@bywatersolutions.com>
Sun, 26 May 2019 12:58:20 +0000 (12:58 +0000)
commit7d29b1c7a830323f28a2f15a07b95130523ba45e
tree3b6f2807e0abb397b58ff810b4d5b559ecb21d18
parent51d7f8114b983e4add6f1aba257e381683394ffe
Bug 21036: Fix warnings from C4/Biblio

Use of uninitialized value $isbn in string ne at /usr/share/koha/prodclone/C4/Biblio.pm line 1794. (16.11 line number)
Trivial edit.

And these warnings from TransformHtmlToXml (with 16.11 line numbers):
Use of uninitialized value in substr at /usr/share/koha/prodclone/C4/Biblio.pm line 2527.
Use of uninitialized value in substr at /usr/share/koha/prodclone/C4/Biblio.pm line 2528.
substr outside of string at /usr/share/koha/prodclone/C4/Biblio.pm line 2528.
Indicator in 952 is empty at /usr/share/koha/prodclone/C4/Biblio.pm line 2534.

The last warning is not needed and can be removed.
Note that the code used the construct @$indicator[$j] for $$indicator[$j].
The first is an array slice. This worked in list context. But apparently
the second was meant to be used. And can be rewritten as $indicator->[$j]
which generally is considered more readable.
The code around indicator1/2 and ind1/2 is simplified. This change is applied
twice in the same sub.

Test plan:
Read the changes.
Run t/Biblio/TransformHtmlToXml.t

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
(cherry picked from commit 295e9b1054386cf188c5fc1eb4fd590e5c451513)
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
(cherry picked from commit 6413f40738185f5319c66b763cb5bedc6fba8745)

Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
C4/Biblio.pm