From 3d106f2bca642ca8e9b607989a25bfcb1905f20f 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 --- C4/Breeding.pm | 2 ++ 1 file changed, 2 insertions(+) diff --git a/C4/Breeding.pm b/C4/Breeding.pm index 8287cf4b6f..c19e87b1e3 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.11.0