In the Checkouts tab of the patron record, the counts of renewals and unseen renewals need some punctuation. Right now they say something like "(W of X renewals remainingY of Z unseen renewals remaining." That needs at least a comma and a space in the middle, for ""(W of X renewals remaining, Y of Z unseen renewals remaining."
Created attachment 137348 [details] [review] Bug 29050: Added a punctuation between renewal messages Test plan: 1. First step is to go to administration > then system preferences 2. Set unseenrenewal to "allow" 3. Go to administatrion > circulation and fines rule 4. Set unseenrenewals count to any number 5. Check out an item to a patron and go to the checkouts tab 6. Renew the item 7. Notice that the renewal messages have no seperation 8. Apply the patch 9. Notice there is now a "&" in the spacing between renewal messages Sponsored-by: Catalyst IT
Created attachment 137350 [details] [review] Bug 29050: Added a punctuation between renewal messages Test plan: 1. First step is to go to administration > then system preferences 2. Set unseenrenewal to "allow" 3. Go to administatrion > circulation and fines rule 4. Set unseenrenewals count to any number 5. Check out an item to a patron and go to the checkouts tab 6. Renew the item 7. Notice that the renewal messages have no seperation 8. Apply the patch 9. Notice there is now a "&" in the spacing between renewal messages Sponsored-by: Catalyst IT Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz> Does what it says it does, still needs independent sign off
Created attachment 137385 [details] [review] Bug 29050: Added a punctuation between renewal messages Test plan: 1. First step is to go to administration > then system preferences 2. Set unseenrenewal to "allow" 3. Go to administatrion > circulation and fines rule 4. Set unseenrenewals count to any number 5. Check out an item to a patron and go to the checkouts tab 6. Renew the item 7. Notice that the renewal messages have no seperation 8. Apply the patch 9. Notice there is now a "&" in the spacing between renewal messages Sponsored-by: Catalyst IT Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz> Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
Thank you for your patch, Nisha! Going straight to PQA for this one.
Created attachment 137403 [details] [review] Bug 29050: (follow-up) Replace & with and See the terminology list https://wiki.koha-community.org/wiki/Terminology Signed-off-by: David Nind <david@davidnind.com>
Pushed to master for 22.11. Nice work everyone, thanks!
Backported to 22.05.x for 22.05.04
pushed to 21.11.x for 21.11.11
Not backported to oldoldstable (21.05.x). Feel free to ask if it's needed. Nothing to document, marking resolved.