Bug 11465 - Show counts and amounts on OPAC your summary tabs
Summary: Show counts and amounts on OPAC your summary tabs
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: OPAC (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low enhancement (vote)
Assignee: David Cook
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-01-02 05:43 UTC by David Cook
Modified: 2015-06-04 23:30 UTC (History)
4 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Bug 11465 - Show counts and amounts on OPAC your summary tabs (3.64 KB, patch)
2014-01-02 05:53 UTC, David Cook
Details | Diff | Splinter Review
[SIGNED-OFF] Bug 11465 - Show counts and amounts on OPAC your summary tabs (3.81 KB, patch)
2014-01-31 15:39 UTC, Nicole C. Engard
Details | Diff | Splinter Review
[PASSED QA] Bug 11465 - Show counts and amounts on OPAC your summary tabs (4.00 KB, patch)
2014-02-02 11:01 UTC, Katrin Fischer
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description David Cook 2014-01-02 05:43:59 UTC
Currently, when you enter your patron account on the OPAC, you see the "Checked out" tab on the "your summary" page.

Currently, to see the number of holds, number of waiting items, number of check outs, and the amount of fines, you have to click on the tabs.

However, one of our librarians has pointed out that it might be better to see the numbers at a glance (especially in regards to fines), as patrons might not realize that they need to click through to view counts and amounts.

I propose adding the counts and amounts to the tab headings themselves. This way, patrons will know at a glance where they stand. They can then click the tabs for more details.

The counts/amounts are already being passed to the template, so it's an easy change.

--

My main question is...do I do patches for bootstrap, ccsr, and prog?
Comment 1 David Cook 2014-01-02 05:53:53 UTC Comment hidden (obsolete)
Comment 2 Nicole C. Engard 2014-01-31 15:39:40 UTC Comment hidden (obsolete)
Comment 3 Katrin Fischer 2014-02-02 10:59:11 UTC
I think "Waiting" should appear as a separate tab according to the test plan, but this is not working for me before AND after appyling the patch. The waiting holds always appear on the "Holds" tab. As I can't detect a regression, I am going to pass this. David, could you maybe take a look why that doesn't work?
Comment 4 Katrin Fischer 2014-02-02 11:01:19 UTC
Created attachment 24981 [details] [review]
[PASSED QA] Bug 11465 - Show counts and amounts on OPAC your summary tabs

This patch adds counts and amounts to the tabs in the 'your summary'
page of the OPAC patron account.

_TEST PLAN_

Before applying:
1) Choose a test patron
2) Place a few holds
3) Check in one of the holds so that it is waiting for the patron
4) Check out a few items
5) Check out another item that can generate fines (backdate it a few weeks/months)
6) Run fines.pl to generate fines (make sure your finesmode syspref is set to prod)
7) Check your OPAC patron account.
8) Note the tabs "Checked out, Fines, Holds, and Waiting"

Apply the patch
1) Switch your OPAC to the bootstrap theme
2) Refresh your browser for the OPAC patron account screen
3) Note that the applicable counts and amounts appear on the tabs

--

At the moment, I just have a Bootstrap patch. This is an easy patch though, so
I can duplicate to the other themes if necessary...

Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
All tests on bootstrap pass

Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
All tests and QA script pass.
"Waiting" doesn't appear to work for me with and without the patch.
"Checkouts", "Holds" and "Fines" work fine.
Comment 5 Galen Charlton 2014-03-10 21:47:27 UTC
Pushed to master.  Thanks, David!
Comment 6 Fridolin Somers 2014-06-04 09:04:53 UTC
Pushed to 3.14.x, will be in 3.14.08