Attachment 110638 Details for Bug 26528 – Bug 26528: Z39.50/SRU ignore invalid replies

[patch] Bug 26528: Z39.50/SRU ignore invalid replies

Bug-26528-Z3950SRU-ignore-invalid-replies.patch (text/plain), 1.46 KB, created by Didier Gautheron on 2020-09-24 08:22:32 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Didier Gautheron

Created: 2020-09-24 08:22:32 UTC

Size: 1.46 KB

patch

obsolete

>From 3d106f2bca642ca8e9b607989a25bfcb1905f20f Mon Sep 17 00:00:00 2001
>From: Didier Gautheron <didier.gautheron@biblibre.com>
>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

Actions: View | Diff | Splinter Review

Attachments on bug 26528: 110638 | 119033 | 119733