Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Merge pull request #29097 from EilisHamilton/fix_uncountable_pluraliz…
…ation_locale

Fix pluralization of uncountables when given a locale
  • Loading branch information
matthewd committed May 28, 2017
2 parents 58a5aa4 + 7c45146 commit 352865d
Show file tree
Hide file tree
Showing 3 changed files with 20 additions and 6 deletions.
4 changes: 4 additions & 0 deletions activesupport/CHANGELOG.md
@@ -1,3 +1,7 @@
* `#singularize` and `#pluralize` now respect uncountables for the specified locale.

*Eilis Hamilton*

* Add ActiveSupport::CurrentAttributes to provide a thread-isolated attributes singleton.
Primary use case is keeping all the per-request attributes easily available to the whole system.

Expand Down
15 changes: 9 additions & 6 deletions activesupport/lib/active_support/inflector/methods.rb
Expand Up @@ -28,7 +28,7 @@ module Inflector
# pluralize('CamelOctopus') # => "CamelOctopi"
# pluralize('ley', :es) # => "leyes"
def pluralize(word, locale = :en)
apply_inflections(word, inflections(locale).plurals)
apply_inflections(word, inflections(locale).plurals, locale)
end

# The reverse of #pluralize, returns the singular form of a word in a
Expand All @@ -45,7 +45,7 @@ def pluralize(word, locale = :en)
# singularize('CamelOctopi') # => "CamelOctopus"
# singularize('leyes', :es) # => "ley"
def singularize(word, locale = :en)
apply_inflections(word, inflections(locale).singulars)
apply_inflections(word, inflections(locale).singulars, locale)
end

# Converts strings to UpperCamelCase.
Expand Down Expand Up @@ -387,12 +387,15 @@ def const_regexp(camel_cased_word)

# Applies inflection rules for +singularize+ and +pluralize+.
#
# apply_inflections('post', inflections.plurals) # => "posts"
# apply_inflections('posts', inflections.singulars) # => "post"
def apply_inflections(word, rules)
# If passed an optional +locale+ parameter, the uncountables will be
# found for that locale.
#
# apply_inflections('post', inflections.plurals, :en) # => "posts"
# apply_inflections('posts', inflections.singulars, :en) # => "post"
def apply_inflections(word, rules, locale = :en)
result = word.to_s.dup

if word.empty? || inflections.uncountables.uncountable?(result)
if word.empty? || inflections(locale).uncountables.uncountable?(result)
result
else
rules.each { |(rule, replacement)| break if result.sub!(rule, replacement) }
Expand Down
7 changes: 7 additions & 0 deletions activesupport/test/inflector_test.rb
Expand Up @@ -420,6 +420,8 @@ def test_inflector_locality
inflect.singular(/es$/, "")

inflect.irregular("el", "los")

inflect.uncountable("agua")
end

assert_equal("hijos", "hijo".pluralize(:es))
Expand All @@ -432,12 +434,17 @@ def test_inflector_locality
assert_equal("los", "el".pluralize(:es))
assert_equal("els", "el".pluralize)

assert_equal("agua", "agua".pluralize(:es))
assert_equal("aguas", "agua".pluralize)

ActiveSupport::Inflector.inflections(:es) { |inflect| inflect.clear }

assert ActiveSupport::Inflector.inflections(:es).plurals.empty?
assert ActiveSupport::Inflector.inflections(:es).singulars.empty?
assert ActiveSupport::Inflector.inflections(:es).uncountables.empty?
assert !ActiveSupport::Inflector.inflections.plurals.empty?
assert !ActiveSupport::Inflector.inflections.singulars.empty?
assert !ActiveSupport::Inflector.inflections.uncountables.empty?
end

def test_clear_all
Expand Down

0 comments on commit 352865d

Please sign in to comment.