Bug 12258 [Template follow-up] Datatable in Patrons Account Fines
authorOwen Leonard <oleonard@myacpl.org>
Thu, 30 Oct 2014 13:57:49 +0000 (09:57 -0400)
committerTomas Cohen Arazi <tomascohen@gmail.com>
Tue, 11 Nov 2014 18:20:49 +0000 (15:20 -0300)
commit84de1f143be9a91682c58e34116d91a24fb465ff
tree8b98ca3187883d314e200a51e9c72dbb3d7aa618
parent214473b0e0c8705ab51e97b4dec33c7d5550f295
Bug 12258 [Template follow-up] Datatable in Patrons Account Fines

This follow-up corrects some of the QA issues affecting the other patch,
as well as changing the way the "filter" option is displayed:

- Added the use of the DataTables include file
- Removed redundant document.ready
- Fixed single quotes
- Fixed default sort (should be date descending to match the current
  functionality)
- Adding missing <tr>
- Converted filter button to a link

The last change is a judgement call, but the button in the DataTables
toolbar looked awkward and caused ugly wrapping at narrower viewport
sizes. I think a link is more keeping with the pattern established by
"select all / clear all" links.

To test, apply both patches and view the account page
(members/boraccount.pl) for a patron who has paid and unpaid fines (the
more the better).

- Confirm that the default sort is by date descending.
- Confirm that DataTables controls (paging, search, result count) work
  correctly.
- Confirm that clicking the "Filter paid transactions" link works
  correctly to toggle display of paid transactions.

Works as expected. Passed koha-qa.pl.
Signed-off-by: Marc VĂ©ron <veron@veron.ch>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Works as described, no problems found.
Passes QA script and tests.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
koha-tmpl/intranet-tmpl/prog/en/modules/members/boraccount.tt