Attachment 49682 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), 1.92 KB, created by Alex Arnaud on 2016-03-30 12:41:52 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Alex Arnaud

Created: 2016-03-30 12:41:52 UTC

Size: 1.92 KB

patch

obsolete

>From f51061902454d70f09d929d690ecb328498fe51d 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.
>---
> 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