Bugzilla – Attachment 78922 Details for
Bug 15139
Show non-public item note in overdues report
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 15139: Add non-public item notes to overdue report
Bug-15139-Add-non-public-item-notes-to-overdue-rep.patch (text/plain), 4.20 KB, created by
Katrin Fischer
on 2018-09-16 01:02:48 UTC
(
hide
)
Description:
Bug 15139: Add non-public item notes to overdue report
Filename:
MIME Type:
Creator:
Katrin Fischer
Created:
2018-09-16 01:02:48 UTC
Size:
4.20 KB
patch
obsolete
>From d6383850e4e40a1967178d2dfab3a31b627b0266 Mon Sep 17 00:00:00 2001 >From: Katrin Fischer <katrin.fischer.83@web.de> >Date: Sat, 11 Aug 2018 18:47:23 +0000 >Subject: [PATCH] Bug 15139: Add non-public item notes to overdue report > >Sometimes libraries want to see internal notes on the overdue >reports page, this patch adds the notes. > >Additional fixes: >- Library column was always empty, now library displays >- Replacement price was not formatted according to CurrencyFormat > >To test: >- Make sure you have some overdue issues in your system >- Circulation > Overdues report >- Make sure everything displays as it should > - Verify internal item note shows > - Verify price is correctly formatted (syspref CurrencyFormat) > - Library shows >- Download list of overdues > - Verify the new column displays >--- > circ/overdue.pl | 7 +++++-- > koha-tmpl/intranet-tmpl/prog/en/modules/circ/overdue.tt | 5 ++++- > 2 files changed, 9 insertions(+), 3 deletions(-) > >diff --git a/circ/overdue.pl b/circ/overdue.pl >index fe3fdd0..a4d9c44 100755 >--- a/circ/overdue.pl >+++ b/circ/overdue.pl >@@ -238,7 +238,8 @@ if ($noreport) { > borrowers.branchcode, > items.itemcallnumber, > items.replacementprice, >- items.enumchron >+ items.enumchron, >+ items.itemnotes_nonpublic > FROM issues > LEFT JOIN borrowers ON (issues.borrowernumber=borrowers.borrowernumber ) > LEFT JOIN items ON (issues.itemnumber=items.itemnumber) >@@ -308,8 +309,10 @@ if ($noreport) { > author => $data->{author}, > homebranchcode => $data->{homebranchcode}, > holdingbranchcode => $data->{holdingbranchcode}, >+ branchcode => $data->{branchcode}, > itemcallnumber => $data->{itemcallnumber}, > replacementprice => $data->{replacementprice}, >+ itemnotes_nonpublic => $data->{itemnotes_nonpublic}, > enumchron => $data->{enumchron}, > patron_attr_value_loop => \@patron_attr_value_loop, > }; >@@ -359,7 +362,7 @@ sub build_csv { > # build header ... > my @keys = > qw ( duedate title author borrowertitle firstname surname phone barcode email address address2 zipcode city country >- branchcode itemcallnumber biblionumber borrowernumber itemnum issuedate replacementprice streetnumber streettype); >+ branchcode itemcallnumber biblionumber borrowernumber itemnum issuedate replacementprice itemnotes_nonpublic streetnumber streettype); > my $csv = Text::CSV_XS->new(); > $csv->combine(@keys); > push @lines, $csv->string(); >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 872faf9..2a75cb3 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/overdue.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/overdue.tt >@@ -2,6 +2,7 @@ > [% USE Asset %] > [% USE KohaDates %] > [%- USE Branches -%] >+[%- USE Price -%] > [% SET footerjs = 1 %] > [% INCLUDE 'doc-head-open.inc' %] > <title>Koha › Circulation › Items overdue as of [% todaysdate | html %]</title> >@@ -53,6 +54,7 @@ > <th>Barcode</th> > <th>Call number</th> > <th>Price</th> >+ <th>Non-public note</th> > </tr> > </thead> > [%- BLOCK subject -%]Overdue:[%- END -%] >@@ -70,7 +72,8 @@ > <td>[% INCLUDE 'biblio-default-view.inc' biblionumber = overdueloo.biblionumber %][% overdueloo.title | html %] [% overdueloo.subtitle | html %]</a> [% IF ( overdueloo.author ) %], by [% overdueloo.author | html %][% END %][% IF ( overdueloo.enumchron ) %], [% overdueloo.enumchron | html %][% END %]</td> > <td><a href="/cgi-bin/koha/catalogue/moredetail.pl?biblionumber=[% overdueloo.biblionumber | html %]&itemnumber=[% overdueloo.itemnum | html %]#item[% overdueloo.itemnum | html %]">[% overdueloo.barcode | html %]</a></td> > <td>[% overdueloo.itemcallnumber | html %]</td> >- <td>[% overdueloo.replacementprice | html %]</td> >+ <td>[% overdueloo.replacementprice | $Price %]</td> >+ <td>[% overdueloo.itemnotes_nonpublic | html %]</td> > </tr> > [% END %] > </tbody> >-- >2.1.4
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 15139
:
77730
|
77731
|
78871
|
78922
|
79034
|
79784