Tickets :: [#8780] number_format() doesn't work with multi-byte characters

6.0.0-alpha14

7/3/25

Summary	number_format() doesn't work with multi-byte characters
Queue	Horde Base
Queue Version	3.3.5
Type	Bug
State	Resolved
Priority	1. Low
Owners	jan (at) horde (dot) org
Requester	e.cerny (at) seznam (dot) cz
Created	12/16/2009 (5678 days ago)
Due
Updated	12/17/2009 (5677 days ago)
Assigned	12/16/2009 (5678 days ago)
Resolved	12/16/2009 (5678 days ago)
Github Issue Link
Github Pull Request
Milestone	3.3.7
Patch	No

12/17/2009 12:35:05 AM	e (dot) cerny (at) seznam (dot) cz	Comment #7	Reply to this comment
Great, than kyou.

12/16/2009 10:12:26 PM	Jan Schneider	Comment #6 Assigned to Jan Schneider Milestone ⇒ 3.3.7 State ⇒ Resolved Summary ⇒ number_format() doesn't work with multi-byte characters	Reply to this comment
It actually turned out that not localconv() was the culprit, it worked just as expected. But number_format() doesn't work with multi-byte characters.

12/16/2009 09:56:16 PM	CVS Commit	Comment #5	Reply to this comment
Changes have been made in CVS for this ticket: Workaround broken PHP number formatting with some locales (~~Bug #8780~~). http://cvs.horde.org/diff.php/ansel/lib/Widget/OwnerFaces.php?rt=horde&r1=1.6.2.4&r2=1.6.2.5&ty=u http://cvs.horde.org/diff.php/ansel/templates/faces/faces.inc?rt=horde&r1=1.2&r2=1.2.2.1&ty=u http://cvs.horde.org/diff.php/ansel/templates/tile/face.inc?rt=horde&r1=1.9&r2=1.9.2.1&ty=u http://cvs.horde.org/diff.php/framework/MIME/MIME/Attic/Contents.php?rt=horde&r1=1.129.4.45&r2=1.129.4.46&ty=u http://cvs.horde.org/diff.php/framework/MIME/MIME/Attic/Part.php?rt=horde&r1=1.177.4.27&r2=1.177.4.28&ty=u http://cvs.horde.org/diff.php/framework/NLS/Attic/NLS.php?rt=horde&r1=1.82.4.26&r2=1.82.4.27&ty=u http://cvs.horde.org/diff.php/imp/compose.php?rt=horde&r1=2.800.2.129&r2=2.800.2.130&ty=u http://cvs.horde.org/diff.php/imp/docs/CHANGES?rt=horde&r1=1.699.2.417&r2=1.699.2.418&ty=u http://cvs.horde.org/diff.php/imp/lib/IMP.php?rt=horde&r1=1.449.4.128&r2=1.449.4.129&ty=u http://cvs.horde.org/diff.php/imp/lib/UI/Mailbox.php?rt=horde&r1=1.9.2.7&r2=1.9.2.8&ty=u http://cvs.horde.org/diff.php/whups/lib/Reports.php?rt=horde&r1=1.35.2.1&r2=1.35.2.2&ty=u

12/16/2009 07:52:29 PM	e (dot) cerny (at) seznam (dot) cz	Comment #4	Reply to this comment
Maybe it should be configurable at admin or even user level. That would be great. I presonaly hate , as a decimal_point which is default for my locale

12/16/2009 10:33:37 AM	Jan Schneider	Comment #3 State ⇒ Feedback	Reply to this comment
This sucks, because there is no way to find out which charset is actually being used. I would expect localeconv() to use the charset of the current locale, but that's obviously not the case. I don't see a clean solution at the moment. For Horde 4 we could use intl's NumberFormatter instead, hoping that it is locale-aware.

12/16/2009 05:42:16 AM	e (dot) cerny (at) seznam (dot) cz	Comment #2	Reply to this comment
in lib/Horde/NLS.php on line 452 should be something like $lc_info = iconv($charset, 'utf-8', localeconv()); instead of $lc_info = localeconv();

12/16/2009 05:21:21 AM	e (dot) cerny (at) seznam (dot) cz	Comment #1 Priority ⇒ 1. Low Type ⇒ Bug Summary ⇒ output from localeconv() must be converted from $nls['charsets']['xx_XX'] encoding to utf-8 Queue ⇒ Horde Base Milestone ⇒ Patch ⇒ No State ⇒ Unconfirmed	Reply to this comment
output from localeconv() is in the default encoding for the choosed locale localeconv() for cs_CZ outputs in ISO-8859-2 but is displayed in utf-8 In IMP::New Mail window down in Attachment section (Maximum Attachment Size: 10?000?000 bytes)