Attachment 15739 Details for Bug 9430 – [SIGNED-OFF] Bug 9430 [Revised] Use DataTables on library transfer limits page

[patch] [SIGNED-OFF] Bug 9430 [Revised] Use DataTables on library transfer limits page

SIGNED-OFF-Bug-9430-Revised-Use-DataTables-on-libr.patch (text/plain), 3.88 KB, created by Bernardo Gonzalez Kriegel on 2013-02-27 23:20:51 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Bernardo Gonzalez Kriegel

Created: 2013-02-27 23:20:51 UTC

Size: 3.88 KB

patch

obsolete

>From 4f69fd7194c96d208c5cbe5662c3d9d316f0acb6 Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Fri, 18 Jan 2013 14:55:59 -0500
>Subject: [PATCH] [SIGNED-OFF] Bug 9430 [Revised] Use DataTables on library
> transfer limits page
>
>Replace the tablesorter plugin with the DataTables plugin on the
>library transfer limits page.
>
>Also made minor changes to "check/uncheck all" function to prevent page
>from jumping on click, and removed unused function.
>
>To test, open the library transfer limits page (Administration ->
>Library transfer limits). Confirm that table sorting works correctly for
>all tables.
>
>Test that the "check all/uncheck all" links still work correctly.
>
>Revision corrects template path to datatables assets.
>
>Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>
>
>Comment: Work as described. No errors.
>---
> .../en/modules/admin/branch_transfer_limits.tt     |   25 +++++++++++---------
> 1 file changed, 14 insertions(+), 11 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/branch_transfer_limits.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/branch_transfer_limits.tt
>index 1c2a7ac..bb1f3ee 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/branch_transfer_limits.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/branch_transfer_limits.tt
>@@ -1,15 +1,16 @@
> [% INCLUDE 'doc-head-open.inc' %]
> <title>Koha &rsaquo; Administration &rsaquo; Library checkin and transfer policy</title>
> [% INCLUDE 'doc-head-close.inc' %]
>-<script type="text/javascript" src="[% themelang %]/lib/jquery/plugins/jquery.tablesorter.min.js"></script>
>-<script type="text/javascript" src="[% themelang %]/lib/jquery/plugins/jquery.tablesorter.pager.js"></script>
>+<link rel="stylesheet" type="text/css" href="[% themelang %]/css/datatables.css" />
>+<script type="text/javascript" src="[% themelang %]/lib/jquery/plugins/jquery.dataTables.min.js"></script>
>+[% INCLUDE 'datatables-strings.inc' %]
>+<script type="text/javascript" src="[% themelang %]/js/datatables.js"></script>
> <script type="text/javascript" src="[% themelang %]/lib/jquery/plugins/jquery.checkboxes.min.js"></script>
> <script type="text/javascript">
> //<![CDATA[
> 	$(document).ready(function(){
>-		$("#CheckAll").click(function(){ $("#transferlimit_tabs").checkCheckboxes();});
>-		$("#UncheckAll").click(function(){ $("#transferlimit_tabs").unCheckCheckboxes();});
>-		$("#SwapAll").click(function(){ $("#transferlimit_tabs").toggleCheckboxes();});
>+        $("#CheckAll").click(function(){ $("#transferlimit_tabs").checkCheckboxes(); return false; });
>+        $("#UncheckAll").click(function(){ $("#transferlimit_tabs").unCheckCheckboxes(); return false; });
>         $('#transferlimit_tabs').tabs();
> 
> 		 $('#branchselect').change(function() {
>@@ -32,14 +33,17 @@
> 			$("#"+tid).unCheckCheckboxes();
> 			return false;
> 		});
>-		$(".sorted").tablesorter({
>-			sortList: [[0,0]],
>-			headers: { 1: { sorter: false}}
>-		}).tablesorterPager({container: $(".pager"),positionFixed: false,size: 10});
>+        $(".sorted").dataTable($.extend(true, {}, dataTablesDefaults, {
>+            "aoColumnDefs": [
>+                { "aTargets": [ -1 ], "bSortable": false, "bSearchable": false },
>+            ],
>+            "iDisplayLength": 20,
>+            "sPaginationType": "four_button"
>+        }));
> 	});
> //]]>
> </script>
>-<style type="text/css">td { text-align: center; }</style>
>+<style type="text/css">td { text-align: center; } .sorted { min-width: 50%; }</style>
> </head>
> <body id="admin_branch_transfer_limits" class="admin">
> [% INCLUDE 'header.inc' %]
>@@ -85,7 +89,6 @@
>     <p><a id="CheckAll[% codes_loo.code %]table" class="checkall" href="#">Check all</a> | <a id="UncheckAll[% codes_loo.code %]table" class="uncheckall" href="#">Uncheck all</a></p>
> 	
> 	<div id="pager[% codes_loo.code %]table">
>-	[% INCLUDE 'table-pager.inc' perpage='20' %]
> 	</div>
> 		<table id="[% codes_loo.code %]table" class="sorted">
> 			<thead>
>-- 
>1.7.9.5

Actions: View | Diff | Splinter Review

Attachments on bug 9430: 14713 | 14749 | 14882 | 15693 | 15739 | 15880