Bugzilla – Attachment 82757 Details for
Bug 21913
Clean up payment details page
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 21913: Clean up payment details page
Bug-21913-Clean-up-payment-details-page.patch (text/plain), 5.71 KB, created by
Owen Leonard
on 2018-11-29 16:50:36 UTC
(
hide
)
Description:
Bug 21913: Clean up payment details page
Filename:
MIME Type:
Creator:
Owen Leonard
Created:
2018-11-29 16:50:36 UTC
Size:
5.71 KB
patch
obsolete
>From 6f306ec38ce537d177d1fce2f9fbf81868485b22 Mon Sep 17 00:00:00 2001 >From: Owen Leonard <oleonard@myacpl.org> >Date: Wed, 24 Oct 2018 16:09:12 +0000 >Subject: [PATCH] Bug 21913: Clean up payment details page > >This patch makes various corrections and improvements to the payment >details page: > > - Grid converted to Bootstrap > - Correct title tag > - Add patron toolbar > - Adding missing patron-related JavaScript > - Improve some language > >To test, apply the patch and locate a patron who has one or more >payments on their account. > > - Open Fines -> Account -> Details of a transaction > - Confirm that the page looks correct and adjusts well at various > browser widths. > - Confirm that patron toolbar items work correctly > - Confirm that the payments table only appears when there are payments > to list. >--- > .../prog/en/modules/members/accountline-details.tt | 78 ++++++++++++++-------- > 1 file changed, 51 insertions(+), 27 deletions(-) > >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/accountline-details.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/accountline-details.tt >index ae66f7e..fc0aec3 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/accountline-details.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/accountline-details.tt >@@ -1,29 +1,40 @@ >+[% USE raw %] >+[% USE Asset %] > [%- USE Price -%] > [%- USE KohaDates -%] > [%- USE AuthorisedValues -%] > [%- USE Branches -%] >- >+[% SET footerjs = 1 %] > [% PROCESS 'accounts.inc' %] >- > [% INCLUDE 'doc-head-open.inc' %] >-<title>Koha › Patrons › Account for [% INCLUDE 'patron-title.inc' %]</title> >+<title>Koha › Patrons › [% INCLUDE 'patron-title.inc' no_html = 1 %] › Details of fee</title> > [% INCLUDE 'doc-head-close.inc' %] > </head> >-<body id="pat_discharges" class="pat"> >+ >+<body id="pat_accountline_details" class="pat"> > [% INCLUDE 'header.inc' %] > [% INCLUDE 'patron-search.inc' %] > >-<div id="breadcrumbs"><a href="/cgi-bin/koha/mainpage.pl">Home</a> › <a href="/cgi-bin/koha/members/members-home.pl">Patrons</a> › <a href="/cgi-bin/koha/members/boraccount.pl?borrowernumber=[% patron.borrowernumber | html %]">Account for [% INCLUDE 'patron-title.inc' %]</a> › Details for account line [% accountline.id | html %]</div> >+<div id="breadcrumbs"><a href="/cgi-bin/koha/mainpage.pl">Home</a> › <a href="/cgi-bin/koha/members/members-home.pl">Patrons</a> › <a href="/cgi-bin/koha/members/boraccount.pl?borrowernumber=[% patron.borrowernumber | html %]">Account for [% INCLUDE 'patron-title.inc' %]</a> › >+ [% IF type == 'credit' %] >+ Details of payment >+ [% ELSIF type == 'debit' %] >+ Details of fee >+ [% END %] >+</div> >+ >+<div class="main container-fluid"> >+ <div class="row"> >+ <div class="col-sm-10 col-sm-push-2"> >+ <main> >+ >+ [% INCLUDE 'members-toolbar.inc' borrowernumber=patron.borrowernumber %] > >-<div id="doc3" class="yui-t2"> >-<div id="bd"> >- <div id="yui-main"> >- <div class="yui-b"> > [% IF accountline %] > [% IF type == 'credit' %] >- <h2>Details for payment</h2> >+ <h2>Details of payment</h2> > [% ELSIF type == 'debit' %] >- <h2>Details for fee</h2> >+ <h2>Details of fee</h2> > [% END %] > > <table id="table_account_fines"> >@@ -73,13 +84,14 @@ > </tbody> > </table> > >- [% IF type == 'credit' %] >- <h3>Fees paid</h3> >- [% ELSIF type == 'debit' %] >- <h3>Payments</h3> >- [% END %] > >- [% IF account_offsets %] >+ [% IF account_offsets.count > 0 %] >+ [% IF type == 'credit' %] >+ <h3>Fees paid</h3> >+ [% ELSIF type == 'debit' %] >+ <h3>Payments</h3> >+ [% END %] >+ > <table class="accountline-offsets-table" id="accountline-debits-table"> > <thead> > <tr> >@@ -90,7 +102,7 @@ > <th>Type</th> > <th>Note</th> > <th>Transacting librarian</th> >- <th>Date/Time of change</th> >+ <th>Date/time of change</th> > <th>Amount of change</th> > <th>Type of change</th> > <th> </th> >@@ -123,19 +135,31 @@ > [% END %] > </tbody> > </table> >- [% ELSE %] >- No details available for this payment. > [% END %] > [% ELSE %] > <div class="dialog message"> >- <p>Account line not found.</p> >+ [% IF type == 'credit' %] >+ Payment not found >+ [% ELSIF type == 'debit' %] >+ Fee not found >+ [% END %] > </div> > [% END %] >- </div> >- </div> > >- <div class="yui-b"> >- [% INCLUDE 'circ-menu.inc' %] >- </div> >-</div> >+ </main> >+ </div> <!-- /.col-sm-10.col-sm-push-2 --> >+ >+ <div class="col-sm-2 col-sm-pull-10"> >+ <aside> >+ [% INCLUDE 'circ-menu.inc' %] >+ </aside> >+ </div> <!-- /.col-sm-2.col-sm-pull-10 --> >+ </div> <!-- /.row --> >+ >+ >+[% MACRO jsinclude BLOCK %] >+ [% INCLUDE 'str/members-menu.inc' %] >+ [% Asset.js("js/members-menu.js") | $raw %] >+[% END %] >+ > [% INCLUDE 'intranet-bottom.inc' %] >-- >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 21913
:
82757
|
83793
|
83828
|
83848