Summary: | Break down fee display by types on checkout tab | ||
---|---|---|---|
Product: | Koha | Reporter: | Jane Wagner <jwagner> |
Component: | Circulation | Assignee: | Bugs List <koha-bugs> |
Status: | NEW --- | QA Contact: | Bugs List <koha-bugs> |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | chris, gmcharlt, kyle.m.hall, veron |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
See Also: | http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5419 | ||
Change sponsored?: | Sponsored | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Attachments: |
Fee breakdown screenshot
screenshot |
Description
Chris Cormack
2010-05-21 01:11:14 UTC
I've cherry-picked these changes into a branch, ip-bug-3484-fee-breakdown-2010-12-31, in my development repo at http://gitorious.org/koha-dev/koha-dev. The only problem I see with it is that it doesn't include an update to the database to properly handle the addition of a new preference, CircFinesBreakdown I've pushed an update to my repo which includes updates to the database for the new system preference. This is now ready for some real testing. Pushed for testing to new/awaiting_qa/enh/bug_3484 please test and sign off This bug is mentioned in: Pull request for Bug 3484 http://lists.koha-community.org/pipermail/koha-patches/2011-January/013552.html Pull request for Bug 3484 http://lists.koha-community.org/pipermail/koha-patches/2011-January/013559.html Created attachment 3358 [details]
screenshot
I18N problem: Patches are hardcoding $ into the templates.
+ <!-- TMPL_IF NAME="lost_fines_total" --><li><span class="circ-hlt">Replacement fees:</span> $<!-- T...
+ <!-- TMPL_IF NAME="overdue_fines_total" --><li><span class="circ-hlt">Overdue fines:</span> $<!-- T...
+ <!-- TMPL_IF NAME="reserve_fees_total" --><li><span class="circ-hlt">Hold fees:</span> $<!-- TMPL_V...
+ <!-- TMPL_IF NAME="other_fees_total" --><li><span class="circ-hlt">Other fees:</span> $<!-- TMPL_VA...
New patches using Template::Toolkit are needed. Resetting to ASSIGNED. Still valid? It might be nice to have this as an option - we only list the totals currently. I still like the idea of this. |