From eda2847ba24357ad337015c0a88021c8ff75f8c5 Mon Sep 17 00:00:00 2001 From: Didier Gautheron Date: Thu, 24 Sep 2020 10:13:08 +0200 Subject: [PATCH] Bug 26528: Z39.50/SRU ignore invalid replies When searching for authorities if an authorities server reply has invalid records none are displayed. At least french BNF SRU server doesn't fully follow norm and can return an error confusing Koha protocol handler which then returns an empty MARC record. This patch silently removed bogus records. To Test: 1- Add BNF SRU server 2- Go to authorities page 3- Add an authority 4- Search for keyword(any) droits de l'homme 5- No result (Internal Server Error) 6- Apply patch 7- restart starman 8- redo 4 9- Many records are displayed Signed-off-by: Victor Grousset/tuxayo --- C4/Breeding.pm | 2 ++ 1 file changed, 2 insertions(+) diff --git a/C4/Breeding.pm b/C4/Breeding.pm index 2a4ce3c2ac..6b103b3cdb 100644 --- a/C4/Breeding.pm +++ b/C4/Breeding.pm @@ -618,6 +618,8 @@ sub Z3950SearchAuth { my $heading; my $heading_authtype_code; $heading_authtype_code = GuessAuthTypeCode($marcrecord); + next if ( not defined $heading_authtype_code ) ; + $heading = C4::AuthoritiesMarc::GetAuthorizedHeading({ record => $marcrecord }); my $breedingid = ImportBreedingAuth( $marcrecord, $serverhost[$k], $encoding[$k], $heading ); -- 2.24.3 (Apple Git-128)