Created attachment 52865 [details] no fines warning This screenshot from an earlier manual : http://manual.koha-community.org/3.22/en/images/circ/patronnotes.png shows the fines note to the right of the checkout box, but I don't see it anymore in master. My patron owes over $10 but I'm not seeing the note.
Appears to be a regression from bug 14497 - the entire block is inside this if: [% IF NoIssuesCharge && fines > NoIssuesCharge %] So you don't get the warning about having any fines unless you have fines over the NoIssuesCharge
Same issue on master.
Created attachment 53125 [details] [review] Bug 16810 - Fines note not showing on checkout Fines did not show on patron checkout and detail pages unless amout was greater than syspref NoIssueCharges. This patch fixes the logic in koha-tmpl/intranet-tmpl/prog/en/includes/blocked-fines.inc Additionally, it changes links to buttons and makes some small text changes to ease translation. To test: - Apply patch - Display checkout or detail page and verify proper display for a patron who has - no fines (no message) - fines not exceeding syspref NoissueCharges (message) - fines exceeding syspref NoIssueCharges (message)
Created attachment 53137 [details] [review] [SIGNED-OFF]Bug 16810: Fines note not showing on checkout Fines did not show on patron checkout and detail pages unless amout was greater than syspref NoIssueCharges. This patch fixes the logic in koha-tmpl/intranet-tmpl/prog/en/includes/blocked-fines.inc Additionally, it changes links to buttons and makes some small text changes to ease translation. To test: - Apply patch - Display checkout or detail page and verify proper display for a patron who has - no fines (no message) - fines not exceeding syspref NoissueCharges (message) - fines exceeding syspref NoIssueCharges (message) Signed-off-by: Hector Castro <hector.hecaxmmx@gmail.com> Works as advertised
Hm, a fine not exceeding noissuecharge isn't showing for me, but it should.
Sorry... didn't test this right (you should actually apply the patch... right?). But good news is, that this works fine.
Created attachment 53301 [details] [review] [PASSED QA] Bug 16810: Fines note not showing on checkout Fines did not show on patron checkout and detail pages unless amout was greater than syspref NoIssueCharges. This patch fixes the logic in koha-tmpl/intranet-tmpl/prog/en/includes/blocked-fines.inc Additionally, it changes links to buttons and makes some small text changes to ease translation. To test: - Apply patch - Display checkout or detail page and verify proper display for a patron who has - no fines (no message) - fines not exceeding syspref NoissueCharges (message) - fines exceeding syspref NoIssueCharges (message) Signed-off-by: Hector Castro <hector.hecaxmmx@gmail.com> Works as advertised Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Pushed to master for 16.11, thanks Marc!
Pushed in 16.05. Will be in 16.05.02.
*** Bug 16957 has been marked as a duplicate of this bug. ***
Not needed in 3.22.x
*** Bug 17442 has been marked as a duplicate of this bug. ***