From 06f640241a4c9e3c9c9b7cdaa774819573347023 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Wed, 27 Nov 2019 15:49:36 +0100 Subject: [PATCH] Bug 20443: Fix Patrons/Import.t ->get_patrons has been removed but not replaced! Signed-off-by: Nick Clemens --- Koha/Patrons/Import.pm | 10 ++++++++-- t/db_dependent/Koha/Patrons/Import.t | 1 - 2 files changed, 8 insertions(+), 3 deletions(-) diff --git a/Koha/Patrons/Import.pm b/Koha/Patrons/Import.pm index 78938d699f..6bd13edbb2 100644 --- a/Koha/Patrons/Import.pm +++ b/Koha/Patrons/Import.pm @@ -174,8 +174,14 @@ sub import_patrons { elsif ($extended) { if ( defined($matchpoint_attr_type) ) { foreach my $attr (@$patron_attributes) { - if ( $attr->{code} eq $matchpoint and $attr->{value} ne '' ) { - my @borrowernumbers = $matchpoint_attr_type->get_patrons( $attr->{value} ); + if ( $attr->{code} eq $matchpoint and $attr->{attribute} ne '' ) { + my @borrowernumbers = Koha::Patron::Attributes->search( + { + code => $matchpoint_attr_type->code, + attribute => $attr->{attribute} + } + )->get_column('borrowernumber'); + $borrowernumber = $borrowernumbers[0] if scalar(@borrowernumbers) == 1; $patron = Koha::Patrons->find( $borrowernumber ); last; diff --git a/t/db_dependent/Koha/Patrons/Import.t b/t/db_dependent/Koha/Patrons/Import.t index d879aad278..fade0929a1 100644 --- a/t/db_dependent/Koha/Patrons/Import.t +++ b/t/db_dependent/Koha/Patrons/Import.t @@ -214,7 +214,6 @@ my $params_4 = { file => $handle_4, matchpoint => $attribute->{code}, }; # When ... my $result_4 = $patrons_import->import_patrons($params_4); -use Data::Printer colored => 1; warn p $result_4; # Then ... is($result_4->{already_in_db}, 0, 'Got the expected 0 already_in_db from import_patrons with extended user'); -- 2.20.1