Attachment 34331 Details for Bug 13442 – Bug 13442 - Facet links broken when collection code limit is used

[patch] Bug 13442 - Facet links broken when collection code limit is used

Bug-13442---Facet-links-broken-when-collection-cod.patch (text/plain), 1.24 KB, created by Kyle M Hall (khall) on 2014-12-11 16:54:35 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2014-12-11 16:54:35 UTC

Size: 1.24 KB

patch

obsolete

>From 930ce0cd21f1550c25164dbb43e4aa716dc6d8e7 Mon Sep 17 00:00:00 2001
>From: Kyle M Hall <kyle@bywatersolutions.com>
>Date: Thu, 11 Dec 2014 09:47:01 -0500
>Subject: [PATCH] Bug 13442 - Facet links broken when collection code limit is used
>
>Any search limits including a ccode will break the search facts.
>
>Test Plan:
>1) Run an advanced search using a ccode limit
>2) Try using any of the facet links of the left
>3) Note they are broken
>4) Apply this patch
>5) Refresh the results
>6) Note the facet links are no longer broken
>
>Note: I have not been able to reproduce this issue on my own test
>system, but have noted the problem on at least a dozen Koha servers.
>---
> C4/Search.pm |    2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
>diff --git a/C4/Search.pm b/C4/Search.pm
>index 1326f69..c7577b4 100644
>--- a/C4/Search.pm
>+++ b/C4/Search.pm
>@@ -1705,7 +1705,7 @@ sub buildQuery {
>             if ( $k !~ /mc-i(tem)?type/ ) {
>                 # in case the mc-ccode value has complicating chars like ()'s inside it we wrap in quotes
>                 $this_limit =~ tr/"//d;
>-                $this_limit = $k.":\"".$v."\"";
>+                $this_limit = $k.":'".$v."'";
>             }
> 
>             $group_OR_limits{$k} .= " or " if $group_OR_limits{$k};
>-- 
>1.7.2.5

Actions: View | Diff | Splinter Review

Attachments on bug 13442: 34331 | 34594 | 34602