refactor to use faster empty?

2026-04-26 03:00:59 -04:00 · 2010-09-08 16:06:48 -07:00
parent 4ccf491ec2
commit cb1f80591e
1 changed files with 3 additions and 1 deletions
--- a/activerecord/lib/active_record/relation/query_methods.rb
+++ b/activerecord/lib/active_record/relation/query_methods.rb
@@ -12,8 +12,10 @@ module ActiveRecord
    def includes(*args)
      args.reject! {|a| a.blank? }

+      return clone if args.empty?
+
      relation = clone
-      relation.includes_values = (relation.includes_values + args).flatten.uniq if args.present?
+      relation.includes_values = (relation.includes_values + args).flatten.uniq
      relation
    end