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: Pushed to main --- QA Contact: Lucas Gass (lukeg) <lucas>
Severity: enhancement    
Priority: P5 - low CC: david, katrin.fischer, lucas, tomascohen
Version: Main   
Hardware: All   
OS: All   
GIT URL: Change sponsored?: ---
Patch complexity: Trivial patch 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:
25.11.00
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
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>
Comment 3 Lucas Gass (lukeg) 2025-07-25 16:56:23 UTC
Created attachment 184655 [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>
Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
Comment 4 Lucas Gass (lukeg) 2025-07-25 20:38:48 UTC
Nice work everyone!

Pushed to main for 25.11