Koha-community/Koha
13
7
Fork 0
Code Issues Releases Wiki Activity

Bug 31595: Don't process extended attributes if none are inlcuded in the file

Browse source 
This patch simply adds a check to the import process to only process attrbutes if
they have been found in the data

To test:
1 - Import some patrons without extended attributes
2 - Import patrons including extended attributes
3 - Confirm both work unchanged

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
This commit is contained in:
Nick Clemens 2022-09-21 16:10:34 +00:00 committed by Tomas Cohen Arazi
parent 0d974073db
commit 40cff1a774
Signed by: tomascohen
GPG key ID: 0A272EA1B2F3C15F
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
Koha/Patrons/Import.pm
View file

@ -331,7 +331,7 @@ sub import_patrons {
}
}
}
if ($extended) {
if ($extended && @$patron_attributes) {
if ($ext_preserve) {
$patron_attributes = $patron->extended_attributes->merge_and_replace_with( $patron_attributes );
}
@ -391,7 +391,7 @@ sub import_patrons {
);
}
if ($extended) {
if ($extended && @$patron_attributes) {
# FIXME Hum, we did not filter earlier and now we do?
$patron->extended_attributes->filter_by_branch_limitations->delete;
$patron->extended_attributes($patron_attributes);