diff options
author | Pratik Naik <pratiknaik@gmail.com> | 2010-01-19 20:43:03 +0530 |
---|---|---|
committer | Pratik Naik <pratiknaik@gmail.com> | 2010-01-19 20:43:03 +0530 |
commit | ec63fdcff35d9ee195e11043ba3219b1e341a3f2 (patch) | |
tree | de7bbcac06efcb46464775bef5c5e425fabe7fb9 /activerecord/lib/active_record/associations/association_collection.rb | |
parent | 73b179eb689611ac0518584b21f2304756a7e981 (diff) | |
download | rails-ec63fdcff35d9ee195e11043ba3219b1e341a3f2.tar.gz rails-ec63fdcff35d9ee195e11043ba3219b1e341a3f2.tar.bz2 rails-ec63fdcff35d9ee195e11043ba3219b1e341a3f2.zip |
Get rid of construct_count_options_from_args
Diffstat (limited to 'activerecord/lib/active_record/associations/association_collection.rb')
-rw-r--r-- | activerecord/lib/active_record/associations/association_collection.rb | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/activerecord/lib/active_record/associations/association_collection.rb b/activerecord/lib/active_record/associations/association_collection.rb index e9402d3547..9487d16123 100644 --- a/activerecord/lib/active_record/associations/association_collection.rb +++ b/activerecord/lib/active_record/associations/association_collection.rb @@ -176,14 +176,15 @@ module ActiveRecord # be used for the query. If no +:counter_sql+ was supplied, but +:finder_sql+ was set, the # descendant's +construct_sql+ method will have set :counter_sql automatically. # Otherwise, construct options and pass them with scope to the target class's +count+. - def count(*args) + def count(column_name = nil, options = {}) if @reflection.options[:counter_sql] @reflection.klass.count_by_sql(@counter_sql) else - column_name, options = @reflection.klass.scoped.send(:construct_count_options_from_args, *args) + column_name, options = nil, column_name if column_name.is_a?(Hash) + if @reflection.options[:uniq] # This is needed because 'SELECT count(DISTINCT *)..' is not valid SQL. - column_name = "#{@reflection.quoted_table_name}.#{@reflection.klass.primary_key}" if column_name == :all + column_name = "#{@reflection.quoted_table_name}.#{@reflection.klass.primary_key}" unless column_name options.merge!(:distinct => true) end |