]> git.koha-community.org Git - koha.git/commit
Bug 31441: Fix Koha::Item::as_marc_field when kohafield = ''
authorJulian Maurice <julian.maurice@biblibre.com>
Tue, 23 Aug 2022 11:47:56 +0000 (13:47 +0200)
committerArthur Suzuki <arthur.suzuki@biblibre.com>
Wed, 14 Dec 2022 09:44:49 +0000 (10:44 +0100)
commit1f926a8772dae1e9a3fb686db153730435abb72c
tree88f981b289dbfef24a8a566d58137f2071038e69
parentfce44867acdcfe34bea0cf165601d51081c85c5c
Bug 31441: Fix Koha::Item::as_marc_field when kohafield = ''

marc_subfield_structure.kohafield can be NULL, but it can also be an
empty string. But in that case, Koha::Item::as_marc_field ignores them,
which means the resulting MARC::Field object has missing data.
This can produce a bug in OPAC ISBD view (and probably other places
where this method is used)

Test plan:
1. Edit the default biblio MARC framework for the item field: find or
   create a subfield that is not linked to a DB column.
   Save even if you made no changes to make sure that
   marc_subfield_structure.kohafield is set to an empty string.
   I'll use 995$Z as an example for the following steps.
2. Add the following to syspref OPACISBD:
    #995|<br>Item:|{995Z}|
3. Create a biblio with an item and put a value into 995$Z
4. Go to the ISBD detail page for this record at OPAC. Confirm that the
   value you entered in 995$Z is not visible
5. Apply patch and restart koha
6. Refresh the ISBD detail page. Confirm that the 995$Z is now visible.
7. Run `prove t/db_dependent/Koha/Item.t`

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
(cherry picked from commit 389b9dfaf43e889612990401ab83cc1f75972333)

Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
(cherry picked from commit 272510faf747a0d7da49b3c5ae3b491fc83ba36d)
Signed-off-by: Arthur Suzuki <arthur.suzuki@biblibre.com>
Koha/Item.pm
t/db_dependent/Koha/Item.t