Attachment 116885 Details for Bug 27706 – Bug 27706: (bug 24488 follow-up) Fix libraries column filter

[patch] Bug 27706: (bug 24488 follow-up) Fix libraries column filter

Bug-27706-bug-24488-follow-up-Fix-libraries-column.patch (text/plain), 1.71 KB, created by Andrii Nugged on 2021-02-15 13:57:06 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Andrii Nugged

Created: 2021-02-15 13:57:06 UTC

Size: 1.71 KB

patch

obsolete

>From 23c4c731bb784e1dd1c89ca1e30a9c8f09810e18 Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>Date: Mon, 15 Feb 2021 10:34:15 +0100
>Subject: [PATCH] Bug 27706: (bug 24488 follow-up) Fix libraries column filter
>
>From commit 2285c2d65752dcc899d3fdcff634e6927a1cbbf0
>     Bug 24488: Show correct first patron details on Holds to pull
>
>               $('select', this).change( function () {
>-                  var filter_value = $(this).val();
>-                  holdst.fnFilter( filter_value, 5, true );
>+                  holdst.fnFilter( $(this).child().val(), 5 );
>               });
>
>This does not make sense, we need to filter on $(this).val(), it's a
>select element.
>
>Test plan:
>Place some holds
>Hit the pending holds page
>Use the "libraries" column filter and confirm that it works correctly
>
>Signed-off-by: Andrew Nugged <nugged@gmail.com>
>---
> koha-tmpl/intranet-tmpl/prog/en/modules/circ/pendingreserves.tt | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>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 c4bdf7f901..5909cfdbac 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/pendingreserves.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/pendingreserves.tt
>@@ -311,7 +311,7 @@
>           $("#homebranchfilter").each( function () {
>               $(this).html( createSelect( holdst.fnGetColumnData(5) ) );
>               $('select', this).change( function () {
>-                  holdst.fnFilter( $(this).child().val(), 5 );
>+                  holdst.fnFilter( $(this).val(), 5 );
>               });
>           });
>           $("#itemtype-filter").each( function () {
>-- 
>2.30.1

Actions: View | Diff | Splinter Review

Attachments on bug 27706: 116880 | 116885 | 116886