From 9b784bb91a7cea29c53f83761a3a30df25233be7 Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Wed, 10 May 2017 11:46:36 -0400 Subject: [PATCH] Bug 18574 - Clean Patron Records tool doesn't limit to the selected library The library selector simply does nothing because at some point the javascript that powers it was removed. This is very bad because the librarian may batch delete patrons assuming it is limiting by branch correctly and instead delete patrons for all the libraries! Test Plan: 1) Note that the library pulldown on tools/cleanborrowers.pl does nothing 2) Apply this patch 3) Note that chaning the value in the pulldown now refreshed the page and sets the library correctly! Signed-off-by: Katrin Fischer Signed-off-by: Julian Maurice Signed-off-by: Kyle M Hall (cherry picked from commit 7a4f3c17a8fe518534658009d075dcdc1d537b62) Signed-off-by: Katrin Fischer --- .../intranet-tmpl/prog/en/modules/tools/cleanborrowers.tt | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/cleanborrowers.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/cleanborrowers.tt index e47a85e888..7e8cf0ae85 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/cleanborrowers.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/cleanborrowers.tt @@ -11,6 +11,7 @@ return checkForm( this ); }); }); + /** * checkForm(form) * This function check the form is correctly filled. @@ -34,6 +35,12 @@ } return true; } + + $(document).ready(function() { + $('#branch').change(function() { + $('#selectlibrary').submit(); + }); + }); -- 2.39.5