Attachment 31048 Details for Bug 6536 – Bug 6536: [QA Follow-up] Remove surrounding spaces in sru_fields

[patch] Bug 6536: [QA Follow-up] Remove surrounding spaces in sru_fields

Bug-6536-QA-Follow-up-Remove-surrounding-spaces-in.patch (text/plain), 1.29 KB, created by Marcel de Rooy on 2014-08-21 12:53:36 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Marcel de Rooy

Created: 2014-08-21 12:53:36 UTC

Size: 1.29 KB

patch

obsolete

>From 6e9132d7f5e8b21ac124edae9e8319dbb0a3f119 Mon Sep 17 00:00:00 2001
>From: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>Date: Thu, 21 Aug 2014 14:47:06 +0200
>Subject: [PATCH] Bug 6536: [QA Follow-up] Remove surrounding spaces in
> sru_fields
>Content-Type: text/plain; charset=utf-8
>
>This patch only removes surrouding spaces at comma and equals-sign while
>passing the options in sru_fields to the ZOOM object.
>
>Test plan:
>If you add spaces between options in sru_fields, searching should still work.
>E.g. sru_fields= sru = get , sru_version = 1.1
>
>Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>---
> C4/Breeding.pm |    5 ++++-
> 1 files changed, 4 insertions(+), 1 deletions(-)
>
>diff --git a/C4/Breeding.pm b/C4/Breeding.pm
>index 7125cb2..293b933 100644
>--- a/C4/Breeding.pm
>+++ b/C4/Breeding.pm
>@@ -360,7 +360,10 @@ sub _create_connection {
> 
>     if( $server->{servertype} eq 'sru' ) {
>         foreach( split ',', $server->{sru_options}//'' ) {
>-            my @temp= split '=';
>+            #first remove surrounding spaces at comma and equals-sign
>+            s/^\s+|\s+$//g;
>+            my @temp= split '=', $_, 2;
>+            @temp= map { s/^\s+|\s+$//g; $_; } @temp;
>             $option1->option( $temp[0] => $temp[1] ) if @temp;
>         }
>     } elsif( $server->{servertype} eq 'zed' ) {
>-- 
>1.7.7.6

Actions: View | Diff | Splinter Review

Attachments on bug 6536: 4540 | 4541 | 4959 | 6901 | 6919 | 6920 | 6924 | 13462 | 26169 | 26170 | 26171 | 26172 | 29754 | 29755 | 29756 | 29779 | 29780 | 29788 | 29790 | 29791 | 29812 | 29813 | 29814 | 29815 | 29816 | 29817 | 29818 | 29819 | 30189 | 30190 | 30191 | 30192 | 30193 | 30194 | 30900 | 30901 | 30902 | 30903 | 30904 | 30905 | 30906 | 30907 | 30929 | 30930 | 30931 | 30932 | 30933 | 30934 | 30935 | 30936 | 31046 | 31048 | 31137 | 31193 | 31194 | 31195 | 31227 | 31247 | 31248 | 37264