Attachment 90371 Details for Bug 23061 – Bug 23061: Add column/print/export feature on checkout history page

[patch] Bug 23061: Add column/print/export feature on checkout history page

Bug-23061-Add-columnprintexport-feature-on-checkou.patch (text/plain), 4.44 KB, created by Owen Leonard on 2019-06-06 13:38:56 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Owen Leonard

Created: 2019-06-06 13:38:56 UTC

Size: 4.44 KB

patch

obsolete

>From 1353854ea1c8066626ebbac2b89c203711e07bff Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Thu, 6 Jun 2019 12:24:41 +0000
>Subject: [PATCH] Bug 23061: Add column/print/export feature on checkout
> history page
>
>This patch adds columns configuration and export features to the
>checkout history page.
>
>This patch also removes some inline CSS: "overflow:hidden" from both the
>checkout and holds history pages. This was breaking the table controls
>tooltips, and didn't appear to me to be necessary.
>
>To test, apply the patch and restart Plack (or restart_all on
>kohadevbox).
>
> - Open a patron's checkout history page in the staff client.
> - The table of checkouts should have controls at the top.
>   for columns configuration and export/print.
> - These controls should work correctly.
>---
> admin/columns_settings.yml                         | 27 ++++++++++++++++++++++
> .../prog/en/modules/members/holdshistory.tt        |  2 +-
> .../prog/en/modules/members/readingrec.tt          | 13 +++++++----
> 3 files changed, 37 insertions(+), 5 deletions(-)
>
>diff --git a/admin/columns_settings.yml b/admin/columns_settings.yml
>index f484ed9..213ee00 100644
>--- a/admin/columns_settings.yml
>+++ b/admin/columns_settings.yml
>@@ -590,6 +590,33 @@ modules:
>           columnname: itemtype
>         -
>           columnname: status
>+
>+    checkouthistory:
>+      checkouthistory-table:
>+        - columnname: type
>+          cannot_be_toggled: 1
>+          cannot_be_modified: 1
>+        -
>+          columnname: date
>+        -
>+          columnname: title
>+        -
>+          columnname: author
>+        -
>+          columnname: call_number
>+        -
>+          columnname: barcode
>+        -
>+          columnname: number_of_renewals
>+        -
>+          columnname: checked_out_on
>+        -
>+          columnname: checked_out_from
>+        -
>+          columnname: date_due
>+        -
>+          columnname: return_date
>+
>     patron-lists:
>       patron-list-table:
>         -
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/holdshistory.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/holdshistory.tt
>index 87b4417..e030588 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/holdshistory.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/holdshistory.tt
>@@ -36,7 +36,7 @@
> 
> [% SET show_itemtype_column = Koha.Preference('AllowHoldItemTypeSelection') %]
> 
>-<div id="holdshistory" style="overflow:hidden">
>+<div id="holdshistory">
>   <table id="table_holdshistory">
>       <thead>
>         <tr>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/readingrec.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/readingrec.tt
>index ccd72b7..097dbf0 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/readingrec.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/readingrec.tt
>@@ -4,6 +4,7 @@
> [% USE Koha %]
> [% USE AuthorisedValues %]
> [% USE Branches %]
>+[% USE ColumnsSettings %]
> [% SET footerjs = 1 %]
> [% INCLUDE 'doc-head-open.inc' %]
> <title>Circulation History for [% INCLUDE 'patron-title.inc' no_html = 1 %]</title>
>@@ -44,7 +45,7 @@
>     <li><a href="#readingrec" id="tab_onsite_checkout">On-site checkouts</a></li>
>   </ul>
>   [% END %]
>-  <div id="readingrec" style="overflow:hidden">
>+  <div id="readingrec">
>     <table id="table_readingrec">
>       <thead>
>         <tr>
>@@ -127,16 +128,20 @@
>     [% INCLUDE 'str/members-menu.inc' %]
>     [% Asset.js("js/members-menu.js") | $raw %]
>     [% INCLUDE 'datatables.inc' %]
>+    [% INCLUDE 'columns_settings.inc' %]
>     <script id="js">
>         $(document).ready(function() {
>-            var table = $("#table_readingrec").dataTable($.extend(true, {}, dataTablesDefaults, {
>+            var columns_settings = [% ColumnsSettings.GetColumns('members', 'checkouthistory', 'checkouthistory-table', 'json') %];
>+            var table = KohaTable("table_readingrec", {
>                 "sPaginationType": "four_button",
>-                "aaSorting": [],
>+                "aaSorting": [[10, 'desc']],
>+                "sDom": 'C<"top pager"ilpfB><"#filter_c">tr<"bottom pager"ip>',
>                 "aoColumnDefs": [
>                     { "sType": "anti-the", "aTargets" : [ "anti-the" ] },
>                     { "sType": "title-string", "aTargets" : [ "title-string" ] }
>                 ]
>-            }));
>+            }, columns_settings);
>+
>             var tabs = $("#tabs").tabs({
>                 activate: function(e, ui) {
>                     var active = tabs.tabs("option", "active" );
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 23061: 90369 | 90371 | 90375 | 90449