This project is archived and is in readonly mode.
Delegate Inflector.transliterate to i18n
Reported by Norman Clarke | April 30th, 2010 @ 02:24 PM
The patch I'll attach after posting adds support for the new transliteration API in i18n.
The patch makes some small ancillary changes as well:
-
Move ActiveSupport::Multibyte::Chars#normalize into a class method; the current method invokes
self.class
so many times it seems appropriate to follow the pattern of many other methods in that class, and have the instance method simply delegate to a (more concise) class method. -
Removes a now-unused UTF8_PAT constant; this is something I should have changed when I updated
tidy_bytes
a few weeks ago, but didn't notice at the time.
Comments and changes to this ticket
-
Norman Clarke April 30th, 2010 @ 03:13 PM
- Tag changed from activesupport, chars, multibyte to activesupport, chars, multibyte, patch
Patch attached.
In case it's useful for discussion, it's a rebase of the commits located here:
-
Repository April 30th, 2010 @ 03:22 PM
- State changed from new to resolved
(from [f0e754e7136f92398495590908162b2501de5869]) Delegate Inflector.transliterate to i18n. [#4508 state:resolved]
Ancillary changes: Moved Chars#normalize into a class method; removed
unused UTF_PAT constant.Signed-off-by: José Valim jose.valim@gmail.com
http://github.com/rails/rails/commit/f0e754e7136f92398495590908162b...
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile »
<h2 style="font-size: 14px">Tickets have moved to Github</h2>
The new ticket tracker is available at <a href="https://github.com/rails/rails/issues">https://github.com/rails/rails/issues</a>
People watching this ticket
Attachments
Tags
Referenced by
- 4508 Delegate Inflector.transliterate to i18n (from [f0e754e7136f92398495590908162b2501de5869]) Delegat...