The message "No renewals allowed" is incorrect when it is too soon for renewal.
Created attachment 147792 [details] [review] Bug 33150 Add specific message for informing about too_soon for renewal.
Test plan for koha-testing-docker: - Make sure the system preference SCOAllowCheckin is set to "allow" - Go to Koha administration -> Circulation and fine rules - Edit the rule for all items and change "No renewal before" to 1 and save. - Check out an item to a borrower (for instance barcode 39999000000719 to the user koha with cardnumber 42). - Go to self checkout (http://localhost:8080/cgi-bin/koha/sco/sco-main.pl) - Log in as the user (koha/koha) - Verify that the error message in the Renew column is "This item cannot yet be renewed."
(In reply to Andreas Jonsson from comment #2) > Test plan for koha-testing-docker: > > - Make sure the system preference SCOAllowCheckin is set to "allow" > - Go to Koha administration -> Circulation and fine rules > - Edit the rule for all items and change "No renewal before" to 1 and save. > - Check out an item to a borrower (for instance barcode 39999000000719 to > the user koha with cardnumber 42). > - Go to self checkout (http://localhost:8080/cgi-bin/koha/sco/sco-main.pl) > - Log in as the user (koha/koha) > - Verify that the error message in the Renew column is "This item cannot yet > be renewed." Works as expected! Signed-off-by: Laura Escamilla <laura.escamilla@bywatersolutions.com>
Andreas, can you fix your commit message? * Commit title does not start with 'Bug XXXXX: ' - 022c622905 Other than that, this patch looks/works good.
Created attachment 148037 [details] [review] Bug 33150: Add specific message for informing about too_soon for renewal. The message "No renewals allowed" is incorrect when it is too soon for renewal, this patch adds a specific error message. To test: - Make sure the system preference SCOAllowCheckin is set to "allow" - Go to Koha administration -> Circulation and fine rules - Edit the rule for all items and change "No renewal before" to 1 and save. - Check out an item to a borrower (for instance barcode 39999000000719 to the user koha with cardnumber 42). - Go to self checkout (http://localhost:8080/cgi-bin/koha/sco/sco-main.pl) - Log in as the user (koha/koha) - Verify that the error message in the Renew column is "This item cannot yet be renewed." Signed-off-by: Laura Escamilla <laura.escamilla@bywatersolutions.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Hi Andreas, I've fixed the mistake pointed out by Lucas, but I also fixed your commit message. Please make sure to always add a short problem description and the test plan into your commit message. See here: https://wiki.koha-community.org/wiki/Commit_messages Also added Laura's SO line.
Pushed to master for 23.05. Nice work everyone, thanks!
Nice work, thanks everyone! Pushed to 22.11.x for the next release.
Backported to 22.05.x for upcoming 22.05.12
applied to 21.11.x for 21.11.20
Not backported to 21.05.x