Created attachment 83839 [details] [review] Bug 22119: Add price formatting in more places in circulation This adds some missing Price filters to some places in circulation. To check verify the amount are displayed according to setting of CurrencyFormat (use FR or CH for seeing a change) Use a patron account with no pending fines at the beginning for all tests. 1) - Switch RentalFeesCheckoutConfirmation to "Ask" - Check out an item with a rental charge - Verify message is formatted correctly: Rental charge for this item: ... 2) - Switch RentalsInNoissuesCharge to "Don't include" - Make sure patron has only Rental fines on their account - Check out another item - Verify message is formatted correctly: The patron has unpaid charges for holds, rentals etc ... 3) - Switch BatchCheckouts to "Allow" - Add the patron category of your test patron to BatchCheckoutsValidCategories - Checkout any item - Verify the message in the information column is formatted correctly: The patron has unpaid charges for holds, rentals etc of ...
Created attachment 83921 [details] [review] Bug 22119: Add price formatting in more places in circulation This adds some missing Price filters to some places in circulation. To check verify the amount are displayed according to setting of CurrencyFormat (use FR or CH for seeing a change) Use a patron account with no pending fines at the beginning for all tests. 1) - Switch RentalFeesCheckoutConfirmation to "Ask" - Check out an item with a rental charge - Verify message is formatted correctly: Rental charge for this item: ... 2) - Switch RentalsInNoissuesCharge to "Don't include" - Make sure patron has only Rental fines on their account - Check out another item - Verify message is formatted correctly: The patron has unpaid charges for holds, rentals etc ... 3) - Switch BatchCheckouts to "Allow" - Add the patron category of your test patron to BatchCheckoutsValidCategories - Checkout any item - Verify the message in the information column is formatted correctly: The patron has unpaid charges for holds, rentals etc of ... Signed-off-by: Jasmine Amohia <jasmineamohia.student@wegc.school.nz>
Thx, Jasmine!
Created attachment 84885 [details] [review] Bug 22119: Add price formatting in more places in circulation This adds some missing Price filters to some places in circulation. To check verify the amount are displayed according to setting of CurrencyFormat (use FR or CH for seeing a change) Use a patron account with no pending fines at the beginning for all tests. 1) - Switch RentalFeesCheckoutConfirmation to "Ask" - Check out an item with a rental charge - Verify message is formatted correctly: Rental charge for this item: ... 2) - Switch RentalsInNoissuesCharge to "Don't include" - Make sure patron has only Rental fines on their account - Check out another item - Verify message is formatted correctly: The patron has unpaid charges for holds, rentals etc ... 3) - Switch BatchCheckouts to "Allow" - Add the patron category of your test patron to BatchCheckoutsValidCategories - Checkout any item - Verify the message in the information column is formatted correctly: The patron has unpaid charges for holds, rentals etc of ... Signed-off-by: Jasmine Amohia <jasmineamohia.student@wegc.school.nz> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
$Price, $Price, $Price... Passing QA
Awesome work all! Pushed to master for 19.05
Pushed to 18.11.x for 18.11.03
backported to 18.05.x for 18.05.09
I prefer not to push formatting changes to 17.11.x