Bugzilla – Attachment 14476 Details for
Bug 9360
Clean borrowers tool wording and layout problematic
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
bug 9360 - Clean borrowers tool wording and layout problematic
bug-9360---Clean-borrowers-tool-wording-and-layout.patch (text/plain), 6.63 KB, created by
Liz Rea
on 2013-01-08 21:48:44 UTC
(
hide
)
Description:
bug 9360 - Clean borrowers tool wording and layout problematic
Filename:
MIME Type:
Creator:
Liz Rea
Created:
2013-01-08 21:48:44 UTC
Size:
6.63 KB
patch
obsolete
>From e50d4d08f3ddfaf5f53adc873da5358c955c9205 Mon Sep 17 00:00:00 2001 >From: Liz Rea <liz@catalyst.net.nz> >Date: Tue, 8 Jan 2013 16:51:45 +1300 >Subject: [PATCH] bug 9360 - Clean borrowers tool wording and layout > problematic > >To test: > >Verify that the text has been changed to "Batch patron deletion/anonymization in the following places: > >* tools-home >* the page title >* the heading on cleanborrowers >* the menu on cleanborrowers > >Verify that the tool still works as intended. > >This (finally) fixes the things Owen pointed out. >--- > .../intranet-tmpl/prog/en/includes/tools-menu.inc | 2 +- > .../prog/en/modules/tools/cleanborrowers.tt | 54 +++++++++++--------- > .../prog/en/modules/tools/tools-home.tt | 2 +- > 3 files changed, 33 insertions(+), 25 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 c209de4..e4fd8c7 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/includes/tools-menu.inc >+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/tools-menu.inc >@@ -33,7 +33,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">Patrons (anonymize, bulk-delete)</a></li> >+ <li><a href="/cgi-bin/koha/tools/cleanborrowers.pl">Batch patron deletion/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 5aa1865..525a106 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/cleanborrowers.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/cleanborrowers.tt >@@ -1,5 +1,5 @@ > [% INCLUDE 'doc-head-open.inc' %] >-<title>Koha › Tools › Clean patron records [% IF ( step2 ) %]› Confirm[% END %][% IF ( step3 ) %]› Finished[% END %]</title> >+<title>Koha › Tools › Batch patron deletion/anonymization [% IF ( step2 ) %]› Confirm[% END %][% IF ( step3 ) %]› Finished[% END %]</title> > [% INCLUDE 'doc-head-close.inc' %] > [% INCLUDE 'calendar.inc' %] > <script type="text/javascript"> >@@ -47,38 +47,46 @@ > <div id="yui-main"> > <div class="yui-b"> > >-<h1>Delete some old patrons/Anonymize some check-out history</h1> >- >+<h1>Batch patron deletion/anonymization</h1> >+<div class="help"> >+ <p>This tool allows you to delete patrons and anonymize check-out history. For deleting patrons, any combination of limits can be used.</p> >+</div> > [% IF ( step1 ) %] > <!-- step 1 START --> > <div id="step1"> > <form name="f1" onsubmit="return checkForm(this);" action="/cgi-bin/koha/tools/cleanborrowers.pl" method="post"> > <fieldset> >- <legend>What do you want to do ?</legend> >- <h3><input id="checkborrower" type="checkbox" name="checkbox" value="borrower" /><label for="checkborrower"> Delete borrowers</label></h3> >- >- <label for="date1">Who have not borrowed since:</label> >- <input size="10" id="date1" name="filterdate1" type="text" class="datepicker" /> >- <span class="hint">[% INCLUDE 'date-format.inc' %]</span></p> >- >- <label for="borrower_dateexpiry">Whose expiration date is before:</label> >- <input size="10" id=borrower_dateexpiry" name="borrower_dateexpiry" type="text" class="datepicker" /> >- <span class="hint">[% INCLUDE 'date-format.inc' %]</span></p> >- >- <label for="borrower_categorycode">Whose patron category is:</label> >- <select id="borrower_categorycode" name="borrower_categorycode"> >- <option value="" selected="selected">Any</option> >+ <legend>Delete patrons</legend> >+ <h3><input id="checkborrower" type="checkbox" name="checkbox" value="borrower" /><label for="checkborrower"> Verify you want to delete patrons</label></h3> >+ <br /> >+ <h5>Delete patrons who meet the following criteria:</h5> >+ <ul> >+ <li><label for="date1">who have not borrowed since:</label> >+ <input size="10" id="date1" name="filterdate1" type="text" class="datepicker" /> >+ <span class="hint">[% INCLUDE 'date-format.inc' %]</span></li> >+ <li><label for="borrower_dateexpiry">whose expiration date is before:</label> >+ <input size="10" id="borrower_dateexpiry" name="borrower_dateexpiry" type="text" class="datepicker" /> >+ <span class="hint">[% INCLUDE 'date-format.inc' %]</span></li> >+ <li><label for="borrower_categorycode">whose patron category is:</label> >+ <select id="borrower_categorycode" name="borrower_categorycode"> >+ <option value="" selected="selected">Any</option> > [% FOREACH bc IN borrower_categorycodes %] > [% UNLESS bc.categorycode == 'S' %] > <option value="[% bc.categorycode %]">[% bc.description %]</option> > [% END %] > [% END %] >- </select> >- >- <h3><input id="checkissue" type="checkbox" name="checkbox" value="issue" /><label for="checkissue"> Anonymize history</label></h3> >- <label for="date2">Anonymize check-out history older than</label> >- <input size="10" id="date2" name="filterdate2" type="text" class="datepicker" /> >- <span class="hint">[% INCLUDE 'date-format.inc' %]</span></p> >+ </select></li> >+ </ul> >+ </fieldset> >+ <fieldset> >+ <legend>Anonymize check-out history</legend> >+ <h3><input id="checkissue" type="checkbox" name="checkbox" value="issue" /><label for="checkissue"> Verify you want to anonymize patron check-out history</label></h3> >+ <br /> >+ <ul> >+ <li><label for="date2">Permanently delete check-out history older than</label> >+ <input size="10" id="date2" name="filterdate2" type="text" class="datepicker" /> >+ <span class="hint">[% INCLUDE 'date-format.inc' %]</span></li> >+ </ul> > > <!-- hidden here --> > <input type="hidden" name="step2" value="1" /> >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/tools-home.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/tools-home.tt >index 52cc1e6..47a5c9e 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/tools-home.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/tools-home.tt >@@ -41,7 +41,7 @@ > [% END %] > > [% IF ( CAN_user_tools_delete_anonymize_patrons ) %] >- <dt><a href="/cgi-bin/koha/tools/cleanborrowers.pl">Patrons (anonymize, bulk-delete)</a></dt> >+ <dt><a href="/cgi-bin/koha/tools/cleanborrowers.pl">Batch patron deletion/anonymization</a></dt> > <dd>Delete old borrowers and anonymize circulation history (deletes borrower reading history)</dd> > [% END %] > >-- >1.7.9.5
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 9360
:
14456
|
14473
|
14476
|
14507
|
14533
|
14534
|
14551
|
14554
|
14558