From 03377c755fdc7619fb62b82f1ea823519b1b78d7 Mon Sep 17 00:00:00 2001 From: Aaron Patterson Date: Wed, 8 Sep 2010 16:56:13 -0700 Subject: eliminate present? for fewer method calls --- activerecord/lib/active_record/relation/query_methods.rb | 12 ++---------- 1 file changed, 2 insertions(+), 10 deletions(-) (limited to 'activerecord/lib/active_record') diff --git a/activerecord/lib/active_record/relation/query_methods.rb b/activerecord/lib/active_record/relation/query_methods.rb index b337606f84..6d6ae5e5c6 100644 --- a/activerecord/lib/active_record/relation/query_methods.rb +++ b/activerecord/lib/active_record/relation/query_methods.rb @@ -73,21 +73,13 @@ module ActiveRecord def where(opts, *rest) relation = clone - - if opts.present? - relation.where_values += build_where(opts, rest) - end - + relation.where_values += build_where(opts, rest) unless opts.blank? relation end def having(*args) relation = clone - - if args.present? - relation.having_values += build_where(*args) - end - + relation.having_values += build_where(*args) unless args.blank? relation end -- cgit v1.2.3