From e362644218d56a5247552e1f29ccf4d0b25f725c Mon Sep 17 00:00:00 2001
From: Owen Leonard <oleonard@myacpl.org>
Date: Mon, 11 Feb 2013 15:29:08 -0500
Subject: [PATCH] Bug 7969 - select all / unselect all on fines page

This patch adds "select all" and "clear all" links to the pay fines page
so that checkboxes which are all checked by default can be easily
unchecked and selected individually.

This patch also moves the column of checkboxes to the start of the table
in order to conform to common practices.

To test, vew the "Pay fines" page for a patron who has outstanding
charges on their account. Confirm that the "select all" and "clear all"
links work correctly and that payment and writeoff functionality is not
changed.

Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
All tests pass

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
---
 .../prog/en/modules/members/pay.tt            | 19 +++++++++++--------
 1 file changed, 11 insertions(+), 8 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/pay.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/pay.tt
index 2780c3f3da..171337d224 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/pay.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/pay.tt
@@ -1,6 +1,7 @@
 [% INCLUDE 'doc-head-open.inc' %]
 <title>Koha &rsaquo; Patrons &rsaquo; Pay Fines for  [% borrower.firstname %] [% borrower.surname %]</title>
 [% INCLUDE 'doc-head-close.inc' %]
+<script type="text/javascript" src="[% themelang %]/lib/jquery/plugins/jquery.checkboxes.min.js"></script>
 <script type= "text/javascript">
 //<![CDATA[
     $(document).ready(function(){
@@ -10,6 +11,8 @@
                     event.preventDefault();
                 }
         });
+        $('#CheckAll').click(function(){ $("#finest").checkCheckboxes(); return false;});
+        $('#CheckNone').click(function(){ $("#finest").unCheckCheckboxes(); return false;});
     });
 //]]>
 </script>
@@ -40,12 +43,12 @@
 [% IF ( accounts ) %]
     <form action="/cgi-bin/koha/members/pay.pl" method="post">
 	<input type="hidden" name="borrowernumber" id="borrowernumber" value="[% borrower.borrowernumber %]" />
-
-<table>
+<p><span class="checkall"><a id="CheckAll" href="#">Select all</a></span> | <span class="clearall"><a id="CheckNone" href="#">Clear all</a></span></p>
+<table id="finest">
 <thead>
 <tr>
+    <th>&nbsp;</th>
     <th>Fines &amp; charges</th>
-    <th>Sel</th>
 	<th>Description</th>
     <th>Account type</th>
 	<th>Notify id</th>
@@ -64,6 +67,11 @@
 [% FOREACH account_grp IN accounts %]
     [% FOREACH line IN account_grp.accountlines %]
 <tr>
+    <td>
+    [% IF ( line.amountoutstanding > 0 ) %]
+        <input type="checkbox" checked="checked" name="incl_par_[% line.accountno %]" />
+    [% END %]
+    </td>
     <td>
     [% IF ( line.amountoutstanding > 0 ) %]
         <input type="submit" name="pay_indiv_[% line.accountno %]" value="Pay" />
@@ -81,11 +89,6 @@
     <input type="hidden" name="notify_level[% line.accountno %]" value="[% line.notify_level %]" />
     <input type="hidden" name="totals[% line.accountno %]" value="[% line.totals %]" />
     </td>
-    <td>
-    [% IF ( line.amountoutstanding > 0 ) %]
-        <input type="checkbox" checked="checked" name="incl_par_[% line.accountno %]" />
-    [% END %]
-    </td>
     <td>[% line.description %] ([% line.title |html_entity %])</td>
     <td>[% line.accounttype %]</td>
     <td>[% line.notify_id %]</td>
-- 
2.39.2