Bug 22111

Summary: Correctly format fines when placing holds (maxoutstanding warning)
Product: Koha Reporter: Katrin Fischer <katrin.fischer>
Component: CirculationAssignee: Katrin Fischer <katrin.fischer>
Status: CLOSED FIXED QA Contact: Josef Moravec <josef.moravec>
Severity: trivial    
Priority: P5 - low CC: ethan.amohia, fridolin.somers, gmcharlt, josef.moravec, kyle.m.hall, lucas, martin.renvoize, nick
Version: master   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on:    
Bug Blocks: 22112    
Attachments: Bug 22111: Format price on place holds place when maxoutstanding is reached
Bug 22111: Format amount shown on place holds when maxoutstanding is reached (staff)
Bug 22111: Format amount shown on place holds when maxoutstanding is reached (staff)
Bug 22111: Format amount shown on place holds when maxoutstanding is reached (staff)
Bug 22111: Format amount shown on place holds when maxoutstanding is reached (staff)

Description Katrin Fischer 2019-01-11 22:35:11 UTC
When a user has more fines than maxoutstanding holds are no longer possible. The fine amount displayed should be formatted according to the CurrencyFormat system preference.
Comment 1 Katrin Fischer 2019-01-11 22:47:24 UTC Comment hidden (obsolete)
Comment 2 Katrin Fischer 2019-01-12 08:17:59 UTC
Created attachment 83837 [details] [review]
Bug 22111: Format amount shown on place holds when maxoutstanding is reached (staff)

Displays amount outstanding correctly when holds are blocked
by maxoutstanding.

To test:
- Check maxoutstanding system preference value
- Add a fine to a patron account exceeding maxoutstanding
- Try to place a hold for this patron from staff
- Verify this message is shown:

     Patron has outstanding fines: 6.00

- Toggle CurrencyFormat system preference and verify display
  changes according to setting
Comment 3 Ethan 2019-01-15 01:06:31 UTC
I believe these changes have already been made, this is the message I got when I tried to test. Setting to resolved

$ git bz apply 22111
Bug 22111 - Correctly format fines when placing holds (maxoutstanding warning)

83832 - Bug 22111: Format price on place holds place when maxoutstanding is reached
83837 - Bug 22111: Format amount shown on place holds when maxoutstanding is reached (staff)

Apply? [(y)es, (n)o, (i)nteractive] y
Applying: Bug 22111: Format price on place holds place when maxoutstanding is reached
Applying: Bug 22111: Format amount shown on place holds when maxoutstanding is reached (staff)
Using index info to reconstruct a base tree...
M	koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt
Falling back to patching base and 3-way merge...
No changes -- Patch already applied.
Comment 4 Katrin Fischer 2019-01-15 06:14:07 UTC
Hi Ethan,

thx for testing! This was my fault. When I attached a second version of the patch with a changed commit message, I forgot to obsolete the first one. So when you tried to test it applied both and the second caused the warning as the change was the same in both.

Could you please try again?
Comment 5 Owen Leonard 2019-01-15 15:50:33 UTC
Created attachment 83987 [details] [review]
Bug 22111: Format amount shown on place holds when maxoutstanding is reached (staff)

Displays amount outstanding correctly when holds are blocked
by maxoutstanding.

To test:
- Check maxoutstanding system preference value
- Add a fine to a patron account exceeding maxoutstanding
- Try to place a hold for this patron from staff
- Verify this message is shown:

     Patron has outstanding fines: 6.00

- Toggle CurrencyFormat system preference and verify display
  changes according to setting

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 6 Mikaƫl Olangcay Brisebois 2019-01-18 21:59:07 UTC
Created attachment 84228 [details] [review]
Bug 22111: Format amount shown on place holds when maxoutstanding is reached (staff)

Displays amount outstanding correctly when holds are blocked
by maxoutstanding.

To test:
- Check maxoutstanding system preference value
- Add a fine to a patron account exceeding maxoutstanding
- Try to place a hold for this patron from staff
- Verify this message is shown:

     Patron has outstanding fines: 6.00

- Toggle CurrencyFormat system preference and verify display
  changes according to setting

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Mikael Olangcay Brisebois <mikael.olangcay-brisebois@inlibro.com>
Comment 7 Josef Moravec 2019-01-21 15:53:53 UTC
Created attachment 84269 [details] [review]
Bug 22111: Format amount shown on place holds when maxoutstanding is reached (staff)

Displays amount outstanding correctly when holds are blocked
by maxoutstanding.

To test:
- Check maxoutstanding system preference value
- Add a fine to a patron account exceeding maxoutstanding
- Try to place a hold for this patron from staff
- Verify this message is shown:

     Patron has outstanding fines: 6.00

- Toggle CurrencyFormat system preference and verify display
  changes according to setting

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Mikael Olangcay Brisebois <mikael.olangcay-brisebois@inlibro.com>

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Comment 8 Nick Clemens 2019-01-25 17:58:01 UTC
Awesome work all!

Pushed to master for 19.05
Comment 9 Martin Renvoize 2019-01-31 16:43:56 UTC
Pushed to 18.11.x for 18.11.03
Comment 10 Lucas Gass 2019-02-01 22:47:18 UTC
pushed to 18.05.x for 18.05.09
Comment 11 Fridolin Somers 2019-02-04 12:56:03 UTC
Pushed to 17.11.x for 17.11.15