Skip to content

Commit ecfb252

Browse files
committed
fewer method calls, using faster methods when possible
1 parent d1c53a9 commit ecfb252

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

activerecord/lib/active_record/relation/query_methods.rb

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -143,18 +143,18 @@ def build_arel
143143
end
144144
end
145145

146-
arel = arel.having(*@having_values.uniq.select{|h| h.present?}) if @having_values.present?
146+
arel = arel.having(*@having_values.uniq.select{|h| h.present?}) unless @having_values.empty?
147147

148148
arel = arel.take(@limit_value) if @limit_value
149-
arel = arel.skip(@offset_value) if @offset_value.present?
149+
arel = arel.skip(@offset_value) if @offset_value
150150

151-
arel = arel.group(*@group_values.uniq.select{|g| g.present?}) if @group_values.present?
151+
arel = arel.group(*@group_values.uniq.select{|g| g.present?}) unless @group_values.empty?
152152

153-
arel = arel.order(*@order_values.uniq.select{|o| o.present?}) if @order_values.present?
153+
arel = arel.order(*@order_values.uniq.select{|o| o.present?}) unless @order_values.empty?
154154

155155
arel = build_select(arel, @select_values.uniq)
156156

157-
arel = arel.from(@from_value) if @from_value.present?
157+
arel = arel.from(@from_value) if @from_value
158158
arel = arel.lock(@lock_value) if @lock_value
159159

160160
arel

0 commit comments

Comments
 (0)