Skip to content

fix compare begin and end dates

setop requested to merge setop/mobilizon:fix-compare-begin-end-dates into master

this bug prevent from creating an event with the day of month of the begin date greater than the day of month of the end date, event if end date is effectively greater than begin date.

for example, if the begin date is '2020-04-28' and end date is '2020-05-13', these dates are valid but because 28 > 13, the validation fails.

this is better explained in this article "Never compare dates in Elixir using < or >".

using Date.compare, as proposed in this PR fix the issue.

Merge request reports