Bug 12668

Summary: Stray dollar ($) -sign in opac-reserve.pl
Product: Koha Reporter: Olli-Antti Kivilahti <olli-antti.kivilahti>
Component: OPACAssignee: Olli-Antti Kivilahti <olli-antti.kivilahti>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: gmcharlt, katrin.fischer, tomascohen, veron
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: String patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 12668 - Stray dollar ($) -sign in opac-reserve.pl
[Signed-off] Bug 12668 - Stray dollar ($) -sign in opac-reserve.pl
[PASSED QA] Bug 12668 - Stray dollar ($) -sign in opac-reserve.pl

Description Olli-Antti Kivilahti 2014-07-29 09:02:29 UTC
A dollar sign is hard-coded in opac-reserve.pl and becomes apparent when trying to place a reservation when one has "too_much_oweing" or too much fines.

Removing the dollar sign so we just get

<"Käyttömaksujen katto ylitetty. Et voi tehdä varauksia. Sinulla on maksamattomia maksuja 9.50.">

instead of 

<"Käyttömaksujen katto ylitetty. Et voi tehdä varauksia. Sinulla on maksamattomia maksuja $9.50.">
Comment 1 Olli-Antti Kivilahti 2014-07-29 09:03:01 UTC
*** Bug 12667 has been marked as a duplicate of this bug. ***
Comment 2 Olli-Antti Kivilahti 2014-07-29 09:26:32 UTC Comment hidden (obsolete)
Comment 3 Marc Véron 2014-07-30 22:01:32 UTC Comment hidden (obsolete)
Comment 4 Katrin Fischer 2014-07-31 06:10:33 UTC
Created attachment 30372 [details] [review]
[PASSED QA] Bug 12668 - Stray dollar ($) -sign in opac-reserve.pl

A dollar sign is hard-coded in opac-reserve.pl and becomes apparent when trying to place a reservation
 when one has "too_much_oweing" or too much fines.

Removing the dollar sign so we just get

<"Käyttömaksujen katto ylitetty. Et voi tehdä varauksia. Sinulla on maksamattomia maksuja 9.50.">

instead of

<"Käyttömaksujen katto ylitetty. Et voi tehdä varauksia. Sinulla on maksamattomia maksuja $9.50.">

Patch removes hard coded $ sign.
Signed-off-by: Marc Véron <veron@veron.ch>

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
The dollar sign is gone and the message still displays correctly.
Comment 5 Tomás Cohen Arazi 2014-08-03 20:36:01 UTC
Patch pushed to master.

Thanks Olli!
Comment 6 Olli-Antti Kivilahti 2014-08-04 13:28:40 UTC
No, thank you for testing this!
And thank my boss who found it ;)
Comment 7 Galen Charlton 2014-08-26 22:46:53 UTC
Pushed to 3.16.x for inclusion in 3.16.3.