Attachment 103690 Details for Bug 25277 – Bug 25277: Don't specify a field in query if none passed

[patch] Bug 25277: Don't specify a field in query if none passed

Bug-25277-Dont-specify-a-field-in-query-if-none-pa.patch (text/plain), 2.33 KB, created by Nick Clemens (kidclamp) on 2020-04-24 13:34:50 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Nick Clemens (kidclamp)

Created: 2020-04-24 13:34:50 UTC

Size: 2.33 KB

patch

obsolete

>From 8e74563ea44a2f75a516d45bccb8419a03890580 Mon Sep 17 00:00:00 2001
>From: Nick Clemens <nick@bywatersolutions.com>
>Date: Fri, 24 Apr 2020 13:34:31 +0000
>Subject: [PATCH] Bug 25277: Don't specify a field in query if none passed
>
>---
> Koha/Z3950Responder/RPN.pm        | 13 +++++++++----
> etc/z3950/attribute_mappings.yaml |  2 --
> 2 files changed, 9 insertions(+), 6 deletions(-)
>
>diff --git a/Koha/Z3950Responder/RPN.pm b/Koha/Z3950Responder/RPN.pm
>index 63fbbdd83d..c931cec29d 100644
>--- a/Koha/Z3950Responder/RPN.pm
>+++ b/Koha/Z3950Responder/RPN.pm
>@@ -40,7 +40,7 @@ sub to_koha {
>     my ($self, $mappings) = @_;
> 
>     my $attrs = $self->{'attributes'};
>-    my $fields = $mappings->{use}{default} // '_all';
>+    my $fields = $mappings->{use}{default};
>     my $split = 0;
>     my $prefix = '';
>     my $suffix = '';
>@@ -62,7 +62,7 @@ sub to_koha {
>         }
>     }
> 
>-    $fields = [$fields] unless ref($fields) eq 'ARRAY';
>+    $fields = [$fields] unless !defined $fields || ref($fields) eq 'ARRAY';
> 
>     if ($split) {
>         my @terms;
>@@ -72,8 +72,12 @@ sub to_koha {
>             next if (!$word);
>             $word = $self->escape($word);
>             my @words;
>-            foreach my $field (@{$fields}) {
>-                push(@words, "$field:($prefix$word$suffix)");
>+            if( $fields ) {
>+                foreach my $field (@{$fields}) {
>+                    push(@words, "$field:($prefix$word$suffix)");
>+                }
>+            } else {
>+                push(@words, "($prefix$word$suffix)");
>             }
>             push (@terms, join(' OR ', @words));
>         }
>@@ -82,6 +86,7 @@ sub to_koha {
> 
>     my @terms;
>     $term = $self->escape($term);
>+    return ("($prefix$term$suffix)") unless $fields;
>     foreach my $field (@{$fields}) {
>         push(@terms, "$field:($prefix$term$suffix)");
>     }
>diff --git a/etc/z3950/attribute_mappings.yaml b/etc/z3950/attribute_mappings.yaml
>index 21d7e8f3f6..c4923fc532 100644
>--- a/etc/z3950/attribute_mappings.yaml
>+++ b/etc/z3950/attribute_mappings.yaml
>@@ -8,7 +8,6 @@ authorities:
>         3: Heading
>         9: LC-card-number
>         12: Local-number
>-        default: _all
> biblios:
>     # BIB-1 use attributes to index fields
>     use:
>@@ -43,4 +42,3 @@ biblios:
>         1031: itype
>         1033: Host-Item-Number
>         1045: control-number
>-        default: _all
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 25277: 103690 | 103698 | 104293 | 104294 | 104942 | 104943