![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
Opened 19 years ago
Closed 19 years ago
#3677 closed enhancement (fixed)
i18n for number_format()
| Reported by: | markjaquith | Owned by: | rob1n |
|---|---|---|---|
| Priority: | low | Milestone: | 2.3 |
| Component: | I18N | Version: | |
| Severity: | normal | Keywords: | has-patch i18n commit |
| Cc: | Focuses: |
Description
We're using number_format() in a few places, which converts to American style formatting for thousands grouping. We might consider making a __number_format() function that localizers can use to format numbers appropriately.
Attachments (3)
Change History (12)
#7
@
19 years ago
- Cc added
- Resolution fixed
- Status closed → reopened
Let's move these variables from gettext system to $locale.php. It is created for such things.
Note:
See TracTickets
for help on using tickets.
I named the function
number_format_i18nafterdate_i18n.There are two patches - the first is the function implementation. The second one just substitutes
number_formatwithnumber_format_i18nwherever I found occurrences. I have skipped Akismet. It could fallback tonumber_formatif the new function does not exists, but may be some other time :)