Attachment 63374 Details for Bug 18574 – [SIGNED OFF] Bug 18574 - Clean Patron Records tool doesn't limit to the selected library

[patch] [SIGNED OFF] Bug 18574 - Clean Patron Records tool doesn't limit to the selected library

SIGNED-OFF-Bug-18574---Clean-Patron-Records-tool-d.patch (text/plain), 1.62 KB, created by Katrin Fischer on 2017-05-10 22:16:16 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2017-05-10 22:16:16 UTC

Size: 1.62 KB

patch

obsolete

>From d4c9cc9cc9d04e6d83fcdd5aeb78fffdc0095703 Mon Sep 17 00:00:00 2001
>From: Kyle M Hall <kyle@bywatersolutions.com>
>Date: Wed, 10 May 2017 11:46:36 -0400
>Subject: [PATCH] [SIGNED OFF] 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 <katrin.fischer.83@web.de>
>---
> koha-tmpl/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 7da2ff7..f973d06 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();
>+        });
>+    });
> </script>
> 
> </head>
>-- 
>2.7.4

Actions: View | Diff | Splinter Review

Attachments on bug 18574: 63355 | 63374 | 63378