diff options
| author | Sean Griffin <sean@seantheprogrammer.com> | 2015-12-01 16:41:50 -0700 |
|---|---|---|
| committer | Sean Griffin <sean@seantheprogrammer.com> | 2015-12-01 16:41:50 -0700 |
| commit | 0d70078f11e61bd6cd1f67527b9622e569b989b5 (patch) | |
| tree | 78ba68d2984072e8a838e54e744dc3537e1b84c0 | |
| parent | c403edf9c150693df2477ddcf23dcd52e9bef9ff (diff) | |
| download | rails-0d70078f11e61bd6cd1f67527b9622e569b989b5.tar.gz rails-0d70078f11e61bd6cd1f67527b9622e569b989b5.tar.bz2 rails-0d70078f11e61bd6cd1f67527b9622e569b989b5.zip | |
Ensure `Relation` responds to `shuffle`
It appears that I missed this one when I delegated all the non-mutation
array methods that were not on Enumerable
| -rw-r--r-- | activerecord/lib/active_record/relation/delegation.rb | 2 | ||||
| -rw-r--r-- | activerecord/test/cases/relation/delegation_test.rb | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/activerecord/lib/active_record/relation/delegation.rb b/activerecord/lib/active_record/relation/delegation.rb index b1333f110c..e4e5d63006 100644 --- a/activerecord/lib/active_record/relation/delegation.rb +++ b/activerecord/lib/active_record/relation/delegation.rb @@ -37,7 +37,7 @@ module ActiveRecord # for each different klass, and the delegations are compiled into that subclass only. delegate :to_xml, :to_yaml, :length, :collect, :map, :each, :all?, :include?, :to_ary, :join, - :[], :&, :|, :+, :-, :sample, :reverse, :compact, to: :to_a + :[], :&, :|, :+, :-, :sample, :shuffle, :reverse, :compact, to: :to_a delegate :table_name, :quoted_table_name, :primary_key, :quoted_primary_key, :connection, :columns_hash, :to => :klass diff --git a/activerecord/test/cases/relation/delegation_test.rb b/activerecord/test/cases/relation/delegation_test.rb index b4269bd56d..f0e07e0731 100644 --- a/activerecord/test/cases/relation/delegation_test.rb +++ b/activerecord/test/cases/relation/delegation_test.rb @@ -27,7 +27,7 @@ module ActiveRecord module DelegationWhitelistBlacklistTests ARRAY_DELEGATES = [ - :+, :-, :|, :&, :[], + :+, :-, :|, :&, :[], :shuffle, :all?, :collect, :compact, :detect, :each, :each_cons, :each_with_index, :exclude?, :find_all, :flat_map, :group_by, :include?, :length, :map, :none?, :one?, :partition, :reject, :reverse, |