]> git.koha-community.org Git - koha.git/commit
Bug 33205: (bug 28268 follow-up) Method call $row->authid inside quotes - produces...
authorJanusz Kaczmarek <januszop@gmail.com>
Sat, 11 Mar 2023 10:45:32 +0000 (11:45 +0100)
committerKatrin Fischer <katrin.fischer@bsz-bw.de>
Fri, 12 Apr 2024 12:02:45 +0000 (14:02 +0200)
commit9c2510ea2ef1db46bf42cf948b6c599250d6dd9e
tree22888a91af4f9ed97ec5b78ff77a9663d3ee909c
parentcc834ba187b7d4a4b817d73f5c4be0eb0ebdaaf4
Bug 33205: (bug 28268 follow-up) Method call $row->authid inside quotes - produces meaningless warning

Bug 28268, in
Koha::MetadataRecord::Authority::get_all_authorities_iterator,
introduced a warning:

warn "Something went wrong reading record for authority $row->authid: $@\n";

But a method cannot be called from within quotes in Perl - extrapolation
does not work for functions/methods.  So, if something really goes
wrong, this line of code produces a warning like:

Something went wrong reading record for authority
Koha::Schema::Result::AuthHeader=HASH(0x55cf7e0d0958)->authid: Empty
String at /usr/share/perl5/MARC/File/XML.pm line 450.

which is not very informative -- authid is missing.

Test plan
=========
Hard to reproduce.  This problem occurs only in rare cases of some weird problem
in stores authority data or if a auth record had been deleted after a ES reindex
action has been initiated.  But the problem in the code and the fix seem obvious.

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
Koha/MetadataRecord/Authority.pm