Bug 36976: Warning 'Argument "" isn't numeric in numeric' in log when merging bibliog...
authorJanusz Kaczmarek <januszop@gmail.com>
Tue, 28 May 2024 12:42:20 +0000 (12:42 +0000)
committerKatrin Fischer <katrin.fischer@bsz-bw.de>
Fri, 28 Jun 2024 11:49:24 +0000 (13:49 +0200)
commitfb2e54b0c31970dd539fd4356edf106eeab422cb
treee1c2f8a09a12fd67cb2ed11bb795fa94767f4b71
parentfd7130becaef6af7da837df9fcbfcaf254fe3c48
Bug 36976: Warning 'Argument "" isn't numeric in numeric' in log when merging bibliographic records

When merging bibliographic records, Koha generates warning:

[WARN] Argument "" isn't numeric in numeric ge (>=) at /kohadevbox/koha/Koha/Util/MARC.pm line 81.

in [plack-]intranet-error.log.

This is because $tagslib->{$fieldtag}->{'tab'} -- for a tag, not for a
subfield --is (always?) an empty string (cf.
C4::Biblio::GetMarcStructure : $res->{$tag}->{tab} = ""), and in
Koha::Util::MARC::createMergeHash it is compared numerically with 0.

Test plan:
==========
1. From biblio search result page, selact two records with the checkboxes
   and start a merge (Edit -> Merge records).  Proceed with merging.
   After the merge, controll the content of the plack-intranet-error.log
   (when using plack -- standard in ktd).  Note the lines:
   [WARN] Argument "" isn't numeric in numeric ge (>=) at /kohadevbox/koha/Koha/Util/MARC.pm line 81.
2. Apply the patch; restart_all.
3. Repeat p. 1.  There should be no fresh warnings.

Sponsored-by: Ignatianum University in Cracow
Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
Koha/Util/MARC.pm