April 2018 - Koha-bugs - lists.koha-community.org

[Bug 12310] Accept different monetary formats as input
by bugzilla-daemon＠bugs.koha-community.org 09 Apr '18

09 Apr '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12310 --- Comment #63 from Victor Grousset/tuxayo <victor.grousset(a)biblibre.com> --- Created attachment 73873 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=73873&action=ed… Bug 12310 : Decimal separator issues: fixes in fines The main goal is to prevent - x100 or /100 of the amounts - decimal part truncation Fixes Bug 19521 - Partial fine payment is not working with comma decimal separator Bonuses: - Gets rid of the spaces, currency symbols, names (EUR) There is nothing specific about currency symbols is this patch. It just removes everything that is not a digit a minus or a separator. It handles: - by default comma and dot as decimal separators on the same instance - dot or comma as thousands separators if choose to use only one of them as decimal separator. See DecimalSeparators syspref. - minuses, doesn't remove them - non-decimal values: it doesn't touch them == Test plan == 1. Use a translation of the staff interface if possible. 2. When you pay an existing fine (more details in the next steps if needed). Check that you can't enter any string except like "XX.XX" (examples in Annex 1) without having an issue after validation. (x100, truncation, etc) 3. Apply this patch 4. run ~/src/installer/data/mysql/updatedatabase.pl 5. In the manual invoice (cgi-bin/koha/members/maninvoice.pl) 6. Apply the annex 1 to the "Amount" field 7. When paying a fine (cgi-bin/koha/members/paycollect.pl) 8. Click on "Pay amount" to go to the page to pay all fines 9. Apply the annex 1 to the "Collect from patron" field 10. Click on "Pay" to go to the page to pay fine 11. Apply the annex 1 to the "Collect from patron" field 12. Go to "Create manual credit" (staff://cgi-bin/koha/members/mancredit.pl) 13. Apply the annex 1 to the "Amount" field 14. If you have ideas of stuff to try to find *regressions* from the previous versions, then try. You might spare librarian tears by finding something before it's released. (and prevent me from having bounty on my head) 15. Express your happiness. This is important, otherwise the sign off spell won't work! The details of expressing your happiness depend on your culture and habits. === Annex 1: Subpart for each concerned page === 1. Set DecimalSeparators syspref to ",." 2. Input "12,34" 3. Press the "Tab/Tabulation" key. It will unfocus the field and trigger the conversion. Validating the form does the same but it's a trick to test more quickly by not switching to another page. 4. The value should have been converted to 12.34 5. What follows is just a list of examples. They don't have to be tried on each field. 6. Do the same with "12,34 EUR" and ensure that it's converted to "12,34" 7. Same with "EUR 12.34" -> "12.34" 8. "EUR 12.34€hi lyonthree" -> "12.34" 9. "123 456,7" -> "123456.7" 10. "2" -> "2" 11. "2,0" -> "2.0" 12. "-1'345.95" -> "-1345.95" 13. ",1234" -> ".1234" 14. Set DecimalSeparators syspref to "," 15. Reload the page with the form 16. "1.000" -> "1000" 17. "1.000.000,10" -> "1000000.10" 18. Set DecimalSeparators syspref to "." 19. Reload the page with the form 20. "1,000" -> "1000" 21. "1,000,000.10" -> "1000000.10" 22. Validate the form and check that the final value with the dot was well handled. 23. Think about Koalas and smile. 🐨🐨🐨 -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 8643] Add important constraint to marc tags and subfields
by bugzilla-daemon＠bugs.koha-community.org 09 Apr '18

09 Apr '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8643 --- Comment #59 from Charles Farmer <charles.farmer(a)inlibro.com> --- (In reply to Katrin Fischer from comment #58) > One thing for next time :) Please take care of always writing good commit > messages and the format of the subject line spefically: > https://wiki.koha-community.org/wiki/Commit_messages > > These patches have a lot of history, being started in 2012. It works alright > now in my eyes. You're right, I'll keep it in mind next time I rebase/submit a bug :) -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14769] Authorities merge: Set correct indicators in biblio field
by bugzilla-daemon＠bugs.koha-community.org 09 Apr '18

09 Apr '18

1 0

[Bug 9701] Configure default indicators
by bugzilla-daemon＠bugs.koha-community.org 09 Apr '18

09 Apr '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9701 Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Text to go in the| |This report adds default release notes| |indicators to biblio | |frameworks. Table | |marc_tag_structure is | |adjusted. In order to make | |effective use of this | |enhancement, you may want | |to add values in MARC | |frameworks administration. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 9701] Configure default indicators
by bugzilla-daemon＠bugs.koha-community.org 09 Apr '18

09 Apr '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9701 Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://bugs.koha-community | |.org/bugzilla3/show_bug.cgi | |?id=20547 -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12571] Add ability to customize SIP2 screen messages
by bugzilla-daemon＠bugs.koha-community.org 09 Apr '18

09 Apr '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12571 Katrin Fischer <katrin.fischer(a)bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://bugs.koha-community | |.org/bugzilla3/show_bug.cgi | |?id=20545 -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 9701] Configure default indicators
by bugzilla-daemon＠bugs.koha-community.org 09 Apr '18

09 Apr '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9701 Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |release-notes-needed -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14769] Authorities merge: Set correct indicators in biblio field
by bugzilla-daemon＠bugs.koha-community.org 09 Apr '18

09 Apr '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14769 Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |release-notes-needed -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 9701] Configure default indicators
by bugzilla-daemon＠bugs.koha-community.org 09 Apr '18

09 Apr '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9701 --- Comment #48 from Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> --- (In reply to Katrin Fischer from comment #47) > (In reply to Marcel de Rooy from comment #42) > > Comment on attachment 73781 [details] [review] [review] > > Bug 9701: (QA follow-up) Fix indentation and some strings > > > > This patch continually generates malloc errors and other git problems with > > me. It seems that we do not need it too. I have no qa warnings about tabs. > > Strange, but I am not attached to it. Comment24 for instance -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14769] Authorities merge: Set correct indicators in biblio field
by bugzilla-daemon＠bugs.koha-community.org 09 Apr '18

09 Apr '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14769 --- Comment #71 from Katrin Fischer <katrin.fischer(a)bsz-bw.de> --- Pref description if the behaviour remains unchanged after update. Also "Text to go in the release notes" should be used, especially if there is a change in behaviour. -- You are receiving this mail because: You are watching all bug changes.

1 0