From 9ac83cdaafb267e1a76aa957b86b7e7b8955d819 Mon Sep 17 00:00:00 2001
From: Owen Leonard <oleonard@myacpl.org>
Date: Mon, 3 Feb 2014 12:03:14 -0500
Subject: [PATCH] Bug 11670 - Remove sorting from patron lists action columns

The list of patron lists has sortable columns for the add, edit, and
delete buttons. These columns should not be sortable. This patch removes
sorting.

This patch also adds the "btn-mini" class to the in-table buttons to
make the table more compact.

To test you must have existing patron lists. Apply the patch and
navigate to Tools -> Patron lists. The columns containing the add, edit,
and delete buttons should not be sortable. Other columns should sort
correctly.

---------------------------------------------------------------------

Created to patron lists and confirmed the collums 'add patrons', 'edit'
and 'delete' are no longer sortable. Column 'Name' and 'Patrons in list'
are still sortable. The size of buttons are smaller.
I also agree this is an improvement.

Signed-off-by: Petter Goksoyr Asen <boutrosboutrosboutros@gmail.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
---
 .../prog/en/modules/patron_lists/lists.tt            | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/patron_lists/lists.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/patron_lists/lists.tt
index 901b9256ed..ec8c2b496b 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/patron_lists/lists.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/patron_lists/lists.tt
@@ -8,7 +8,11 @@
 <script type="text/javascript">
 //<![CDATA[
     $(document).ready(function() {
-        $('#patron-lists-table').dataTable($.extend(true, {}, dataTablesDefaults));
+        $('#patron-lists-table').dataTable($.extend(true, {}, dataTablesDefaults, {
+            "aoColumnDefs": [
+                { "aTargets": [ -1, -2, -3 ], "bSortable": false, "bSearchable": false },
+            ]
+        } ));
     });
 
     function ConfirmDelete( list ) {
@@ -49,17 +53,17 @@
                         <td>[% l.name %]</td>
                         <td>[% l.patron_list_patrons.size || 0 %]</td>
                         <td>
-                            <a class="btn" href="list.pl?patron_list_id=[% l.patron_list_id %]">
+                            <a class="btn btn-mini" href="list.pl?patron_list_id=[% l.patron_list_id %]">
                                 <i class="icon-plus-sign"></i> Add patrons <i class="icon-user"></i>
                             </a>
                         </td>
                         <td>
-                            <a class="btn" href="add-modify.pl?patron_list_id=[% l.patron_list_id %]">
+                            <a class="btn btn-mini" href="add-modify.pl?patron_list_id=[% l.patron_list_id %]">
                                 <i class="icon-edit"></i> Edit
                             </a>
                         </td>
                         <td>
-                            <a class="btn" href="delete.pl?patron_list_id=[% l.patron_list_id %]" onclick='return ConfirmDelete("[% l.name | html %]")'>
+                            <a class="btn btn-mini" href="delete.pl?patron_list_id=[% l.patron_list_id %]" onclick='return ConfirmDelete("[% l.name | html %]")'>
                                 <i class="icon-trash"></i> Delete
                             </a>
                         </td>
-- 
2.39.2