Attachment 93347 Details for Bug 20888 – Bug 20888: Allow use of boolean operator 'not' in item search

[patch] Bug 20888: Allow use of boolean operator 'not' in item search

Bug-20888-Allow-use-of-boolean-operator-not-in-ite.patch (text/plain), 2.26 KB, created by Matthias Meusburger on 2019-10-01 09:20:54 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Matthias Meusburger

Created: 2019-10-01 09:20:54 UTC

Size: 2.26 KB

patch

obsolete

>From e7fb59e4c6232815fe5202c7ec687a99d0079fd6 Mon Sep 17 00:00:00 2001
>From: Matthias Meusburger <matthias.meusburger@biblibre.com>
>Date: Thu, 10 Jan 2019 10:48:38 +0100
>Subject: [PATCH] Bug 20888: Allow use of boolean operator 'not' in item search
>
>Test plan:
>
> - Apply the patch
> - Check that you have a 'is not' option in the third box in item search
> - Check that you can exclude items from the search results with this 'is not' option
> - Check that you can still combine parameters with 'and' and 'or'
> - Check that the exclusion works for custom fields (see Administration > Item search fields)
> - Check that the exclusion works for custom fields linked to authorised values
>---
> .../prog/en/modules/catalogue/itemsearch.tt            | 18 ++++++++++++++++--
> 1 file changed, 16 insertions(+), 2 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/itemsearch.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/itemsearch.tt
>index 7adecfe..725fa24 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/itemsearch.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/itemsearch.tt
>@@ -81,8 +81,22 @@
>         </optgroup>
>       [% END %]
>     </select>
>+    [% IF params.exists('op') %]
>+        <select name="op" class="form-field-not">
>+           <option value="like">is</option>
>+            [% IF params.op == '!=' %]
>+               <option value="!=" selected="selected">is not</option>
>+            [% ELSE %]
>+               <option value="!=">is not</option>
>+            [% END %]
>+        </select>
>+    [% ELSE %]
>+        <select name="op" class="form-field-not">
>+           <option value="like">is</option>
>+           <option value="!=">is not</option>
>+        </select>
>+    [% END %]
>     <input type="text" name="q" class="form-field-value" value="" />
>-    <input type="hidden" name="op" value="like" />
>   </div>
> [% END %]
> 
>@@ -441,7 +455,7 @@
>       form_field.append(button_field_new);
> 
>             // If a field is linked to an authorised values list, display the list.
>-            $('div.form-field-select-text select[name="f"]').change(function() {
>+            $('div.form-field-select-text select.form-field-column').change(function() {
>                 loadAuthorisedValuesSelect($(this));
>             }).change();
> 
>-- 
>2.7.4

Actions: View | Diff | Splinter Review

Attachments on bug 20888: 84692 | 85281 | 93347 | 93370 | 97626 | 97627 | 111265 | 111266 | 111267 | 112453 | 112454 | 112455 | 112691 | 112692 | 112693