Bugzilla – Attachment 92823 Details for
Bug 21852
Add more columns and column configuration to overdues report
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 21852: Add more columns and column configuration to overdues report
Bug-21852-Add-more-columns-and-column-configuratio.patch (text/plain), 7.90 KB, created by
Katrin Fischer
on 2019-09-15 20:35:46 UTC
(
hide
)
Description:
Bug 21852: Add more columns and column configuration to overdues report
Filename:
MIME Type:
Creator:
Katrin Fischer
Created:
2019-09-15 20:35:46 UTC
Size:
7.90 KB
patch
obsolete
>From ba577149db25cc7aa97ad19573d9a51b07edf967 Mon Sep 17 00:00:00 2001 >From: Owen Leonard <oleonard@myacpl.org> >Date: Fri, 16 Nov 2018 14:51:48 +0000 >Subject: [PATCH] Bug 21852: Add more columns and column configuration to > overdues report > >This patch adds some new columns to the overdue report: Patron category, >item home library, item holding library, item type. Adding these columns >makes available columns better match the available filter fields in the >sidebar form. > >Column configuration is added to this DataTable, with the new columns >defined as hidden by default in order to preserve current functionality. > >This patch also fixes a bug where the home and holding branch template >variable names didn't match what was being used in the template for >pre-selecting the searched-for value in the sidebar filter menu. > >To test, apply the patch and go to Circulation -> Overdues. > > - The list of overdues should look correct, with no new columns shown. > - Test the column visibility button, showing the columns which are > hidden by default. Confirm that the data in these columns displays > correctly. > - Update the default configuration for this table in Administration > -> Configure columns. Confirm that your changes are applied to the > overdues report. > >Signed-off-by: Maryse Simard <maryse.simard@inlibro.com> > >Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> > >Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> >--- > admin/columns_settings.yml | 31 +++++++++++++++++++++ > circ/overdue.pl | 8 ++++-- > .../intranet-tmpl/prog/en/modules/circ/overdue.tt | 32 ++++++++++++++++------ > 3 files changed, 60 insertions(+), 11 deletions(-) > >diff --git a/admin/columns_settings.yml b/admin/columns_settings.yml >index 3c5d2a9181..1ea975dce2 100644 >--- a/admin/columns_settings.yml >+++ b/admin/columns_settings.yml >@@ -920,6 +920,37 @@ modules: > - > columnname: items_needed > >+ overdues: >+ circ-overdues: >+ - >+ columnname: due_date >+ - >+ columnname: patron >+ - >+ columnname: patron_category >+ is_hidden: 1 >+ - >+ columnname: patron_library >+ - >+ columnname: title >+ - >+ columnname: home_library >+ is_hidden: 1 >+ - >+ columnname: holding_library >+ is_hidden: 1 >+ - >+ columnname: barcode >+ - >+ columnname: call_number >+ - >+ columnname: item_type >+ is_hidden: 1 >+ - >+ columnname: price >+ - >+ columnname: non-public_note >+ > opac: > biblio-detail: > holdingst: >diff --git a/circ/overdue.pl b/circ/overdue.pl >index 0f286bc563..0e558dce1e 100755 >--- a/circ/overdue.pl >+++ b/circ/overdue.pl >@@ -248,7 +248,8 @@ if ($noreport) { > items.itemcallnumber, > items.replacementprice, > items.enumchron, >- items.itemnotes_nonpublic >+ items.itemnotes_nonpublic, >+ items.itype > FROM issues > LEFT JOIN borrowers ON (issues.borrowernumber=borrowers.borrowernumber ) > LEFT JOIN items ON (issues.itemnumber=items.itemnumber) >@@ -331,12 +332,13 @@ if ($noreport) { > biblionumber => $data->{biblionumber}, > title => $data->{title}, > author => $data->{author}, >- homebranchcode => $data->{homebranchcode}, >- holdingbranchcode => $data->{holdingbranchcode}, >+ homebranchcode => $data->{homebranch}, >+ holdingbranchcode => $data->{holdingbranch}, > itemcallnumber => $data->{itemcallnumber}, > replacementprice => $data->{replacementprice}, > itemnotes_nonpublic => $data->{itemnotes_nonpublic}, > enumchron => $data->{enumchron}, >+ itemtype => $data->{itype}, > patron_attr_value_loop => \@patron_attr_value_loop, > }; > } >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/overdue.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/overdue.tt >index c1cecc6891..1e29484a70 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/overdue.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/overdue.tt >@@ -3,6 +3,9 @@ > [% USE KohaDates %] > [%- USE Branches -%] > [%- USE Price -%] >+[%- USE ItemTypes -%] >+[%- USE Categories -%] >+[%- USE ColumnsSettings -%] > [% SET footerjs = 1 %] > [% INCLUDE 'doc-head-open.inc' %] > <title>Koha › Circulation › Items overdue as of [% todaysdate | html %]</title> >@@ -51,10 +54,14 @@ > <tr> > <th class="title-string">Due date</th> > <th>Patron</th> >- <th>Library</th> >- <th>Title</th> >+ <th>Patron category</th> >+ <th>Patron library</th> >+ <th class="anti-the">Title</th> >+ <th>Home library</th> >+ <th>Holding library</th> > <th>Barcode</th> > <th>Call number</th> >+ <th>Item type</th> > <th>Price</th> > <th>Non-public note</th> > </tr> >@@ -70,10 +77,14 @@ > [% IF ( overdueloo.email ) %][<a href="mailto:[% overdueloo.email | uri %]?subject=[% INCLUDE subject %] [% overdueloo.title | uri %]">email</a>][% END %] > [% IF ( overdueloo.phone ) %]([% overdueloo.phone | html %])[% ELSIF ( overdueloo.mobile ) %]([% overdueloo.mobile | html %])[% ELSIF ( overdueloo.phonepro ) %]([% overdueloo.phonepro | html %])[% END %]</td> > [% END %] >- <td>[% Branches.GetName( overdueloo.patron.branchcode ) | html %]</td> >+ <td>[% Categories.GetName( overdueloo.patron.categorycode ) | html %]</td> >+ <td>[% Branches.GetName( overdueloo.homebranchcode ) | html %]</td> > <td>[% INCLUDE 'biblio-default-view.inc' biblionumber = overdueloo.biblionumber %][% INCLUDE 'biblio-title.inc' biblio=overdueloo %]</a> [% IF ( overdueloo.author ) %], by [% overdueloo.author | html %][% END %][% IF ( overdueloo.enumchron ) %], [% overdueloo.enumchron | html %][% END %]</td> >+ <td>[% Branches.GetName( overdueloo.holdingbranchcode ) | html %]</td> >+ <td>[% Branches.GetName( overdueloo.patron.branchcode ) | html %]</td> > <td><a href="/cgi-bin/koha/catalogue/moredetail.pl?biblionumber=[% overdueloo.biblionumber | uri %]&itemnumber=[% overdueloo.itemnum | uri %]#item[% overdueloo.itemnum | uri %]">[% overdueloo.barcode | html %]</a></td> > <td>[% overdueloo.itemcallnumber | html %]</td> >+ <td>[% ItemTypes.GetDescription( overdueloo.itemtype ) | html %]</td> > <td>[% overdueloo.replacementprice | $Price %]</td> > <td>[% overdueloo.itemnotes_nonpublic | html %]</td> > </tr> >@@ -213,7 +224,8 @@ > [% MACRO jsinclude BLOCK %] > [% INCLUDE 'calendar.inc' %] > [% INCLUDE 'datatables.inc' %] >- <script type="text/javascript"> >+ [% INCLUDE 'columns_settings.inc' %] >+ <script> > function clone_parent(node) { > var parent = node.parentNode; > var clone = parent.cloneNode(true); >@@ -266,13 +278,17 @@ > } > > $(document).ready(function(){ >- $("#overduest").dataTable($.extend(true, {}, dataTablesDefaults, { >+ var columns_settings = [% ColumnsSettings.GetColumns( 'circ', 'overdues', 'circ-overdues', 'json' ) | $raw %]; >+ KohaTable("overduest", { > "sPaginationType": "full", > "aaSorting": [[0, 'asc']], > "aoColumnDefs": [ >- { "sType": "title-string", "aTargets": [ "title-string" ] } >- ] >- })); >+ { "sType": "title-string", "aTargets": [ "title-string" ] }, >+ { "sType": "anti-the", "aTargets" : [ "anti-the" ] } >+ ], >+ "autoWidth": false, >+ "stateSave": true >+ }, columns_settings); > }); > </script> > [% END %] >-- >2.11.0
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 21852
:
82550
|
84281
|
84638
|
84639
|
84648
|
84649
|
84663
|
84664
|
84735
|
90269
|
90270
|
90271
|
92369
|
92370
|
92371
|
92373
|
92374
|
92375
|
92575
| 92823