From 2e8a3d0f43d7b394873ce21396ae49feacb99a74 Mon Sep 17 00:00:00 2001 From: Aaron Patterson Date: Tue, 7 Sep 2010 11:28:01 -0700 Subject: removing useless code, cleaning variable names --- activerecord/lib/active_record/relation/query_methods.rb | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/activerecord/lib/active_record/relation/query_methods.rb b/activerecord/lib/active_record/relation/query_methods.rb index 6860da80d3..cee240e6a1 100644 --- a/activerecord/lib/active_record/relation/query_methods.rb +++ b/activerecord/lib/active_record/relation/query_methods.rb @@ -228,7 +228,6 @@ module ActiveRecord private def build_joins(relation, joins) - joined_associations = [] association_joins = [] joins = @joins_values.map {|j| j.respond_to?(:strip) ? j.strip : j}.uniq @@ -259,11 +258,8 @@ module ActiveRecord end end - to_join.each do |tj| - unless joined_associations.detect {|ja| ja[0] == tj[0] && ja[1] == tj[1] && ja[2] == tj[2] } - joined_associations << tj - relation = relation.join(tj[0], tj[1]).on(*tj[2]) - end + to_join.uniq.each do |left, join_class, right| + relation = relation.join(left, join_class).on(*right) end relation.join(custom_joins) -- cgit v1.2.3