]> git.koha-community.org Git - koha.git/commit
Bug 36917: Remove some warnings from t/db_dependent/Authority/Merge.t
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Wed, 22 May 2024 08:56:10 +0000 (10:56 +0200)
committerKatrin Fischer <katrin.fischer@bsz-bw.de>
Fri, 24 May 2024 13:36:41 +0000 (15:36 +0200)
commit1d394dcdcc8f354be0ff48efdcfcbc56b35263fd
tree8268fd85deb110542e62925c3fc85104a4a5b21f
parent09d2c0ac102f54b6f65ca7d4b92e218f8d64196d
Bug 36917: Remove some warnings from t/db_dependent/Authority/Merge.t

Cannot determine authority type for record: 1709 at /kohadevbox/koha/Koha/SearchEngine/Elasticsearch.pm line 589.
Use of uninitialized value $subfields in pattern match (m//) at /kohadevbox/koha/C4/Heading/MARC21.pm line 412.
Use of uninitialized value $subfields in pattern match (m//) at /kohadevbox/koha/C4/Heading/MARC21.pm line 448.

This change removes the 2 "Use of uninitialized value $subfields in
pattern match (m//)" warnings

The warnings are gnerated because the subfields for headings are hardcoded variables in C4::Heading::MARC21
for any genuine authority record we should receive the correct list of subfields. The tests in this case are
mocking a new authtype, and so do not find the correct subfields. Skipping the generation here silences the
warnings and will prevent user defined types from throwing errors as well.

NOTE: Koha allows the user to define their own authority types, however, we have various features hardcoded
that will prevent them from working as expected

WNC amended commit message

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
C4/Heading/MARC21.pm