Bug 39960

Summary: Improve messages in the OPAC ask for a discharge page (opac-discharge.tt)
Product: Koha Reporter: Tomás Cohen Arazi (tcohen) <tomascohen>
Component: TemplatesAssignee: Tomás Cohen Arazi (tcohen) <tomascohen>
Status: Signed Off --- QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: david, katrin.fischer, tomascohen
Version: Main   
Hardware: All   
OS: All   
GIT URL: Change sponsored?: ---
Patch complexity: --- Documentation contact:
Documentation submission: Text to go in the release notes:
This enhancement improves the messages in the OPAC on the ask for a discharge page (Your account > Ask for discharge, when the useDischarge system preference is enabled). Improvements: - Improved wording for the number of items checked out: instead of "..2 item(s).." and "..1 item(s)..." the text changes based on the actual number of checkouts - "...2 items..." and "...an item...". - More succinct text: for example, instead of "Please pay your charges before reapplying.", "Please pay them before reapplying."
Version(s) released in:
Circulation function:
Bug Depends on: 14250    
Bug Blocks:    
Attachments: Bug 39960: Improve messages in opac-discharge.tt
Bug 39960: Improve messages in opac-discharge.tt

Description Tomás Cohen Arazi (tcohen) 2025-05-21 18:59:25 UTC
There's a use of 'item(s)' that can be avoided, and the messages contain a redundant 'checked out items' that could just be 'them'.
Comment 1 Tomás Cohen Arazi (tcohen) 2025-05-21 19:08:44 UTC
Created attachment 182700 [details] [review]
Bug 39960: Improve messages in opac-discharge.tt

This patch makes the messages more streamlined.

To test:
1. Enable `useDischarge`
2. Have a patron with two checkouts
3. Log into the OPAC and try to request a discharge
=> FAIL: You get a message that says '2 item(s)' and has redundant
words.
4. Check one of the items in
5. Repeat 3
=> FAIL: Same, but says '1 item(s)'.
6. Check it in
7. Add a manual fee
8. Repeat 3
=> FAIL: Another redundant text about charges.
9. Apply this patch and repeat 2-8
=> SUCCESS: Things read better
10. Sign off :-D
Comment 2 David Nind 2025-05-21 20:15:28 UTC
Created attachment 182707 [details] [review]
Bug 39960: Improve messages in opac-discharge.tt

This patch makes the messages more streamlined.

To test:
1. Enable `useDischarge`
2. Have a patron with two checkouts
3. Log into the OPAC and try to request a discharge
=> FAIL: You get a message that says '2 item(s)' and has redundant
words.
4. Check one of the items in
5. Repeat 3
=> FAIL: Same, but says '1 item(s)'.
6. Check it in
7. Add a manual fee
8. Repeat 3
=> FAIL: Another redundant text about charges.
9. Apply this patch and repeat 2-8
=> SUCCESS: Things read better
10. Sign off :-D

Signed-off-by: David Nind <david@davidnind.com>