Attachment 49892 Details for Bug 15832 – Bug 15832 - Fix filter and items split-up in pendingreserves.tt

[patch] Bug 15832 - Fix filter and items split-up in pendingreserves.tt

Bug-15832---Fix-filter-and-items-split-up-in-pendi.patch (text/plain), 2.05 KB, created by Jonathan Druart on 2016-04-05 06:41:38 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2016-04-05 06:41:38 UTC

Size: 2.05 KB

patch

obsolete

>From 265ff98df64448bd6217ba925433a134c76f1eec Mon Sep 17 00:00:00 2001
>From: Alex Arnaud <alex.arnaud@biblibre.com>
>Date: Tue, 16 Feb 2016 16:33:17 +0100
>Subject: [PATCH] Bug 15832 - Fix filter and items split-up in
> pendingreserves.tt
>
>Test plan:
>
>- Go to circ/pendingreserves.pl (Ensure that there are biblios with many
>  items on different branches),
>- Check the libraries filter at the bottom of datatable. There should be
>  duplicates.
>- Apply this patch and return to circ/pendingreserves.pl,
>- check that libraries filter should not contain duplicate,
>- check that the filter works.
>
>Signed-off-by: Owen Leonard <oleonard@myacpl.org>
>
>Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>---
> koha-tmpl/intranet-tmpl/prog/en/modules/circ/pendingreserves.tt | 8 +++-----
> 1 file changed, 3 insertions(+), 5 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/pendingreserves.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/pendingreserves.tt
>index 919f0c3..46f3088 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/pendingreserves.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/pendingreserves.tt
>@@ -28,9 +28,10 @@ $(document).ready(function() {
>     for ( j=0 ; j<cD.length ; j++ ) {
>         var split_array = cD[j].split(/<br>/gi);
>         for ( k=0 ; k<split_array.length ; k++ ){
>-            var check_array = $.inArray(split_array[k], new_array);
>+            var str = $.trim(split_array[k].replace(/[\n\r]/g, ''));
>+            var check_array = $.inArray(str, new_array);
>             if (check_array == -1) {
>-                new_array.push(split_array[k]);
>+                new_array.push(str);
>             }
>         }
>     }
>@@ -50,9 +51,6 @@ $(document).ready(function() {
>       $(this).html( createSelect( holdst.fnGetColumnData(4) ) );
>       $('select', this).change( function () {
>           var filter_value = $(this).val();
>-          if(filter_value){
>-                filter_value = "(^|>)"+filter_value+"($|<)";
>-          }
>           holdst.fnFilter( filter_value, 4, true );
>       } );
>   } );
>-- 
>2.7.0

Actions: View | Diff | Splinter Review

Attachments on bug 15832: 48141 | 48497 | 48694 | 49682 | 49870 | 49892 | 49893