Attachment 90457 Details for Bug 11642 – Bug 11642: (follow-up) Batch patron deletion/anonymization should not allow permanent destruction of patron data

[patch] Bug 11642: (follow-up) Batch patron deletion/anonymization should not allow permanent destruction of patron data

Bug-11642-follow-up-Batch-patron-deletionanonymiza.patch (text/plain), 8.41 KB, created by Owen Leonard on 2019-06-10 13:53:33 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Owen Leonard

Created: 2019-06-10 13:53:33 UTC

Size: 8.41 KB

patch

obsolete

>From f4d784eaabf4a73067a9c2bc8aabb43cd8e5dfe0 Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Mon, 10 Jun 2019 13:30:56 +0000
>Subject: [PATCH] Bug 11642: (follow-up) Batch patron deletion/anonymization
> should not allow permanent destruction of patron data
>
>This patch makes some template changes to try to make this interface
>more consistent with other areas of Koha and to make warnings more
>clear.
>
>I have also changed the phrase "deletion/anonymization" to "deletion and
>anonymization."
>
>To test, apply the patch and test the patron deletion and anonymization
>process and confirm that the interface is clear and works well.
>---
> .../intranet-tmpl/prog/en/includes/tools-menu.inc  |  2 +-
> .../prog/en/modules/tools/cleanborrowers.tt        | 74 +++++++++++++---------
> 2 files changed, 46 insertions(+), 30 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/tools-menu.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/tools-menu.inc
>index 67ccf55..3fce8ea 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/includes/tools-menu.inc
>+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/tools-menu.inc
>@@ -32,7 +32,7 @@
>     <li><a href="/cgi-bin/koha/patroncards/home.pl">Patron card creator</a></li>
>     [% END %]
>     [% IF ( CAN_user_tools_delete_anonymize_patrons ) %]
>-    <li><a href="/cgi-bin/koha/tools/cleanborrowers.pl">Batch patron deletion/anonymization</a></li>
>+    <li><a href="/cgi-bin/koha/tools/cleanborrowers.pl">Batch patron deletion and anonymization</a></li>
>     [% END %]
>     [% IF ( CAN_user_tools_edit_patrons ) %]
>     <li><a href="/cgi-bin/koha/tools/modborrowers.pl">Batch patron modification</a></li>
>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 0ba2d02..6bb3ed2 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/cleanborrowers.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/cleanborrowers.tt
>@@ -5,7 +5,7 @@
> [% USE Branches %]
> [% SET footerjs = 1 %]
> [% INCLUDE 'doc-head-open.inc' %]
>-<title>Koha &rsaquo; Tools &rsaquo; Batch patron deletion/anonymization [% IF step == 2 %]&rsaquo; Confirm[% END %][% IF step == 3 %]&rsaquo; Finished[% END %]</title>
>+<title>Koha &rsaquo; Tools &rsaquo; Batch patron deletion and anonymization [% IF step == 2 %]&rsaquo; Confirm[% END %][% IF step == 3 %]&rsaquo; Finished[% END %]</title>
> [% INCLUDE 'doc-head-close.inc' %]
> </head>
> 
>@@ -17,9 +17,9 @@
>     <a href="/cgi-bin/koha/mainpage.pl">Home</a> &rsaquo;
>     <a href="/cgi-bin/koha/tools/tools-home.pl">Tools</a> &rsaquo;
>     [% IF step == 1 %]
>-        Batch patron deletion/anonymization
>+        Batch patron deletion and anonymization
>     [% ELSE %]
>-        <a href="/cgi-bin/koha/tools/cleanborrowers.pl">Batch patron deletion/anonymization</a> &rsaquo;
>+        <a href="/cgi-bin/koha/tools/cleanborrowers.pl">Batch patron deletion and anonymization</a> &rsaquo;
>     [% END %]
>     [% IF step == 2 %] Confirm [% END %]
>     [% IF step == 3 %] Finished [% END %]
>@@ -31,6 +31,12 @@
>             <main>
> 
> [% IF !OnlyMine %]
>+  [% IF current_branch == '*' %]
>+    <h1>Batch patron deletion and anonymization</h1>
>+  [% ELSE %]
>+    <h1>Batch patron deletion and anonymization for [% Branches.GetName( current_branch ) | html %]</h1>
>+  [% END %]
>+    [% IF step == 1 %]
>     <form method="get" action="/cgi-bin/koha/tools/cleanborrowers.pl" id="selectlibrary">
>     Select a library :
>         <select name="branch" id="branch" style="width:20em;">
>@@ -44,13 +50,9 @@
>         [% END %]
>         </select>
>     </form>
>-  [% IF current_branch == '*' %]
>-    <h1>Batch patron deletion/anonymization</h1>
>-  [% ELSE %]
>-    <h1>Batch patron deletion/anonymization for [% Branches.GetName( current_branch ) | html %]</h1>
>-  [% END %]
>+    [% END %]
> [% ELSE %]
>-    <h1>Batch patron deletion/anonymization for [% Branches.GetName( Branches.GetLoggedInBranchcode ) | html %]</h1>
>+    <h1>Batch patron deletion and anonymization for [% Branches.GetName( Branches.GetLoggedInBranchcode ) | html %]</h1>
> [% END %]
> 
> [% IF step == 1 %]
>@@ -138,30 +140,45 @@
> 
> <div id="step2">
>         <form name="f2" action="/cgi-bin/koha/tools/cleanborrowers.pl" method="post">
>-    <fieldset>
>-        <legend>Warnings</legend>
>-        <ul>
>-            <li>[% patrons_to_delete.size || 0 | html %] patrons will be deleted</li>
>-            <li>[% patrons_to_anonymize.count || 0 | html %] patron's checkout histories will be anonymized</li>
>-        </ul>
>-
>-        <br />
>-            [% IF patrons_to_delete.size %]
>-                <fieldset><legend>What do you want to do for deleted patrons?</legend>
>-                <input id="delete" type="radio" name="radio" value="delete" title="Delete patrons directly from the database, not recoverable"/>
>-                <label for="delete">Permanently delete these patrons</label>
> 
>-                <br /><input id="trash" type="radio" name="radio" value="trash" title="Move patrons to the deleted borrowers table, they can be deleted permanently by the cleanup_database script"/>
>-                <label for="trash">Move these patrons to the trash</label>
>-
>-                <br /><input id="testrun" type="radio" name="radio" value="testrun" checked="checked" />
>-                <label for="testrun">Do not remove any patrons (test run)</label>
>-                <input type="hidden" name="do_delete" value="[% patrons_to_delete.size | html %]" /></fieldset>
>+        <div class="dialog alert">
>+            <h3>Warning</h3>
>+            <ul>
>+                <li>[% patrons_to_delete.size || 0 | html %] patrons will be deleted</li>
>+                <li>[% patrons_to_anonymize.count || 0 | html %] patrons' checkout histories will be anonymized</li>
>+            </ul>
>+        </div>
> 
>+            [% IF patrons_to_delete.size %]
>+                <fieldset>
>+                    <legend>How should patrons be deleted?</legend>
>+                    <p>
>+                        <input id="delete" type="radio" name="radio" value="delete" />
>+                        <label for="delete">Permanently delete these patrons</label>
>+                        <div class="hint">
>+                            Delete patrons directly from the database. Patron data will not be recoverable.
>+                        </div>
>+                    </p>
>+                    <p>
>+                        <input id="trash" type="radio" name="radio" value="trash" />
>+                        <label for="trash">Move these patrons to the trash</label>
>+                        <div class="hint">
>+                            Move patrons to the deleted patrons table. They can be deleted permanently by the <code>cleanup_database</code> script.
>+                        </div>
>+                    </p>
>+                    <p>
>+                        <input id="testrun" type="radio" name="radio" value="testrun" checked="checked" />
>+                        <label for="testrun">Test run: Do not remove any patrons.</label>
>+                        <input type="hidden" name="do_delete" value="[% patrons_to_delete.size | html %]" /></fieldset>
>+                    </p>
>+                </fieldset>
>             [% END %]
>+
>             [% IF patrons_to_anonymize.count %]
>+                <fieldset>
>                 Checkout history for [% patrons_to_anonymize.count | html %] patrons will be anonymized
>                 <input type="hidden" name="do_anonym" value="[% patrons_to_anonymize.count | html %]" />
>+                </fieldset>
>             [% END %]
> 
>             <input type="hidden" name="step" value="3" />
>@@ -174,7 +191,6 @@
>             <input type="hidden" name="borrower_categorycode" value="[% borrower_categorycode | html %]" />
>             <input type="hidden" name="patron_list_id" value="[% patron_list_id | html %]" />
>             <input type="hidden" name="branch" value="[% current_branch | html %]" />
>-    </fieldset>
>     <fieldset class="action"><input type="submit" value="Finish" /> <a class="cancel" href="/cgi-bin/koha/tools/cleanborrowers.pl">Cancel</a></fieldset>
>         </form>
> </div>
>@@ -232,7 +248,7 @@
>                 $('#selectlibrary').submit();
>             });
>             $("form[name='f2']").on('submit',function(){
>-                if( $("#delete").attr("checked") ){
>+                if( $("#delete").prop("checked") ){
>                     if( !confirm(_("These patrons will be permanently removed from the database and cannot be recovered")) ){
>                         return false;
>                     }
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 11642: 24891 | 90381 | 90457 | 90496 | 90497 | 90498 | 90499 | 90556 | 90557 | 90558 | 91511 | 91512 | 91513