Skip to content
Permalink
Browse files
Don't have a cow, man! 
Remove cow => kine from default inflections but leave the test case
in place to prevent regression of the [old Trac ticket 4929][1].

[1]: http://web.archive.org/web/20090314050915/http://dev.rubyonrails.org/ticket/4929
  • Loading branch information
@pixeltrix
pixeltrix committed Jul 30, 2013
1 parent 42f01e9 commit c300dca9963bda78b8f358dbcb59cabcdc5e1dc9
Showing 1 changed file with 0 additions and 1 deletion.
1 activesupport/lib/active_support/inflections.rb
View file
@@ -57,7 +57,6 @@ module ActiveSupport
inflect.irregular('child', 'children')
inflect.irregular('sex', 'sexes')
inflect.irregular('move', 'moves')
inflect.irregular('cow', 'kine')
inflect.irregular('zombie', 'zombies')

inflect.uncountable(%w(equipment information rice money species series fish sheep jeans police))

5 comments on commit c300dca

@jeremy
Copy link
Member

@jeremy jeremy commented on c300dca Dec 18, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

An inflection too far! Don't tip my kine.

@rdpoor
Copy link

@rdpoor rdpoor commented on c300dca Apr 19, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree that the cow<=>kine inflection was too cute, but there's still a problem in the implementation, e.g. "human".pluralize => "humen" and "abdomen".singularize => "abdoman". See #14092 (comment)

@rsim
Copy link
Contributor

@rsim rsim commented on c300dca Dec 10, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In Rails 3.2 due to "cow" => "kine" got an issue that "Moscow".pluralize => "Moskine" :)

@cantino
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well, what is the plural of Moscow?

@sergiotapia
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cantino Moskii

Please sign in to comment.