diff options
author | David Heinemeier Hansson <david@loudthinking.com> | 2008-03-28 16:13:57 +0000 |
---|---|---|
committer | David Heinemeier Hansson <david@loudthinking.com> | 2008-03-28 16:13:57 +0000 |
commit | bdd88810c1edf8289801f88170eb5c9659b2f774 (patch) | |
tree | 5db49fa022d20885310a4e86bcac0496f601396e /activerecord/lib | |
parent | 533da24bbce5c6eeb23ff372fa0b65911fc138d0 (diff) | |
download | rails-bdd88810c1edf8289801f88170eb5c9659b2f774.tar.gz rails-bdd88810c1edf8289801f88170eb5c9659b2f774.tar.bz2 rails-bdd88810c1edf8289801f88170eb5c9659b2f774.zip |
Fixed that has_many :through would ignore the hash conditions (closes #11447) [miloops]
git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@9110 5ecf4fe2-1ee6-0310-87b1-e25e094e27de
Diffstat (limited to 'activerecord/lib')
-rw-r--r-- | activerecord/lib/active_record/associations/has_many_through_association.rb | 16 |
1 files changed, 14 insertions, 2 deletions
diff --git a/activerecord/lib/active_record/associations/has_many_through_association.rb b/activerecord/lib/active_record/associations/has_many_through_association.rb index 083646f793..83c9865205 100644 --- a/activerecord/lib/active_record/associations/has_many_through_association.rb +++ b/activerecord/lib/active_record/associations/has_many_through_association.rb @@ -286,23 +286,35 @@ module ActiveRecord def build_conditions association_conditions = @reflection.options[:conditions] - through_conditions = @reflection.through_reflection.options[:conditions] + through_conditions = build_through_conditions source_conditions = @reflection.source_reflection.options[:conditions] uses_sti = !@reflection.through_reflection.klass.descends_from_active_record? if association_conditions || through_conditions || source_conditions || uses_sti all = [] - [association_conditions, through_conditions, source_conditions].each do |conditions| + [association_conditions, source_conditions].each do |conditions| all << interpolate_sql(sanitize_sql(conditions)) if conditions end + all << through_conditions if through_conditions all << build_sti_condition if uses_sti all.map { |sql| "(#{sql})" } * ' AND ' end end + def build_through_conditions + conditions = @reflection.through_reflection.options[:conditions] + if conditions.is_a?(Hash) + interpolate_sql(sanitize_sql(conditions)).gsub( + @reflection.quoted_table_name, + @reflection.through_reflection.quoted_table_name) + elsif conditions + interpolate_sql(sanitize_sql(conditions)) + end + end + def build_sti_condition "#{@reflection.through_reflection.quoted_table_name}.#{@reflection.through_reflection.klass.inheritance_column} = #{@reflection.klass.quote_value(@reflection.through_reflection.klass.name.demodulize)}" end |