diff options
author | Rafael Mendonça França <rafaelmfranca@gmail.com> | 2014-04-18 10:51:21 -0300 |
---|---|---|
committer | Rafael Mendonça França <rafaelmfranca@gmail.com> | 2014-04-18 10:51:21 -0300 |
commit | d1756ba91b7e5ff80322f434f8348e6e471f0511 (patch) | |
tree | c03099883cc58cf3d870e56695d4f1646d61ab83 | |
parent | a3fd6e7fa9be80e7f53fec99d0c1bfb5e2a52c5e (diff) | |
parent | c0b6e164ee6bbc7941d280ea629d70d400561668 (diff) | |
download | rails-d1756ba91b7e5ff80322f434f8348e6e471f0511.tar.gz rails-d1756ba91b7e5ff80322f434f8348e6e471f0511.tar.bz2 rails-d1756ba91b7e5ff80322f434f8348e6e471f0511.zip |
Merge pull request #14793 from arthurnn/fixes_8928
Fixed has_many association to make it support irregular inflections.
-rw-r--r-- | activerecord/CHANGELOG.md | 6 | ||||
-rw-r--r-- | activerecord/lib/active_record/reflection.rb | 4 | ||||
-rw-r--r-- | activerecord/test/cases/reflection_test.rb | 8 |
3 files changed, 16 insertions, 2 deletions
diff --git a/activerecord/CHANGELOG.md b/activerecord/CHANGELOG.md index 57a083c1df..0cba1009b6 100644 --- a/activerecord/CHANGELOG.md +++ b/activerecord/CHANGELOG.md @@ -1,3 +1,9 @@ +* Fixed has_many association to make it support irregular inflections. + + Fixes #8928. + + *arthurnn*, *Javier Goizueta* + * Fixed a problem where count used with a grouping was not returning a Hash. Fixes #14721. diff --git a/activerecord/lib/active_record/reflection.rb b/activerecord/lib/active_record/reflection.rb index 1724ea95b0..4fde6677be 100644 --- a/activerecord/lib/active_record/reflection.rb +++ b/activerecord/lib/active_record/reflection.rb @@ -449,9 +449,9 @@ module ActiveRecord end def derive_class_name - class_name = name.to_s.camelize + class_name = name.to_s class_name = class_name.singularize if collection? - class_name + class_name.camelize end def derive_foreign_key diff --git a/activerecord/test/cases/reflection_test.rb b/activerecord/test/cases/reflection_test.rb index fed199f6e9..c085fcf161 100644 --- a/activerecord/test/cases/reflection_test.rb +++ b/activerecord/test/cases/reflection_test.rb @@ -87,6 +87,14 @@ class ReflectionTest < ActiveRecord::TestCase end end + def test_irregular_reflection_class_name + ActiveSupport::Inflector.inflections do |inflect| + inflect.irregular 'plural_irregular', 'plurales_irregulares' + end + reflection = AssociationReflection.new(:has_many, 'plurales_irregulares', nil, {}, ActiveRecord::Base) + assert_equal 'PluralIrregular', reflection.class_name + end + def test_aggregation_reflection reflection_for_address = AggregateReflection.new( :composed_of, :address, nil, { :mapping => [ %w(address_street street), %w(address_city city), %w(address_country country) ] }, Customer |