before(nil) and after(nil)

[zacharywelch]

Similar to #60 and #63, wondering if there's a way for .before and .after to return scope (do nothing) when the argument is nil, similar to .between_times.

between_times(start_time, nil) - same as after(start_time)
between_times(nil, end_time) - same as before(end_time)

Current behavior is to default base and time range scopes to Time.zone.now, but would prefer that only apply to the time range scopes 😅

Current behavior

Event.after(nil) #=> returns all events after the current time 

Desired behavior

Event.after(nil) #=> returns all events

[radar]

I agree that both of these should return the default scope. Please submit a patch to that effect.

[johnnyshields]

Hmmm... it's ok but please make sure .after_now continues to work.