No need to branch after return

ahawkins · Apr 30, 2012 · f654bd7 · f654bd7
1 parent 8f9c0ea
commit f654bd7
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/activerecord/lib/active_record/relation/calculations.rb b/activerecord/lib/active_record/relation/calculations.rb
@@ -127,10 +127,10 @@ def pluck(column_name)
 
       if eager_loading? || (includes_values.present? && references_eager_loaded_tables?)
         return construct_relation_for_association_calculations.pluck(column_name)
-      else
-        result = klass.connection.select_all(select(column_name).arel, nil, bind_values)
       end
 
+      result = klass.connection.select_all(select(column_name).arel, nil, bind_values)
+
       types  = result.column_types.merge klass.column_types
       column = types[key]