Compact order values
What does this MR do and why?
- Compact
order_values
of a scope inSimpleOrderBuilder
class.
Because of default_scope
used in some models, SimpleOrderBuilder
cannot properly pick up order values.
Example: when a model has a default scope reorder(nil)
does not actually remove nil
.
> Event.reorder(nil).order_values
=> [nil]
Related to #323730 (closed).
MR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
-
I have evaluated the MR acceptance checklist for this MR.
Edited by euko