diff options
| author | Teddy Wing | 2017-07-20 16:10:22 +0200 |
|---|---|---|
| committer | Teddy Wing | 2017-07-21 15:59:53 +0200 |
| commit | 5187bc61081a06949039730d0e991394bf62ccba (patch) | |
| tree | 89a0b2b26510b6fb034a241b4279bd06691e6141 | |
| parent | 766e6a2a1617d5dee1e31a566421ac74087297b9 (diff) | |
| download | chouette-core-5187bc61081a06949039730d0e991394bf62ccba.tar.bz2 | |
VehicleJourneyAtStop: Remove `error_message` from day offset validator
Robert wasn't a fan of the lambda in code review, so I'm getting rid of
it and just duplicating the I18n call.
Refs #3597
| -rw-r--r-- | app/models/chouette/vehicle_journey_at_stop.rb | 26 |
1 files changed, 16 insertions, 10 deletions
diff --git a/app/models/chouette/vehicle_journey_at_stop.rb b/app/models/chouette/vehicle_journey_at_stop.rb index 3e83eb41a..3c350b2ae 100644 --- a/app/models/chouette/vehicle_journey_at_stop.rb +++ b/app/models/chouette/vehicle_journey_at_stop.rb @@ -35,20 +35,26 @@ module Chouette end def day_offset_must_be_within_range - error_message = lambda do - I18n.t( - 'vehicle_journey_at_stops.errors.day_offset_must_not_exceed_max', - local_id: vehicle_journey.objectid.local_id, - max: DAY_OFFSET_MAX + 1 - ) - end - if day_offset_outside_range?(arrival_day_offset) - errors.add(:arrival_day_offset, error_message) + errors.add( + :arrival_day_offset, + I18n.t( + 'vehicle_journey_at_stops.errors.day_offset_must_not_exceed_max', + local_id: vehicle_journey.objectid.local_id, + max: DAY_OFFSET_MAX + 1 + ) + ) end if day_offset_outside_range?(departure_day_offset) - errors.add(:departure_day_offset, error_message) + errors.add( + :departure_day_offset, + I18n.t( + 'vehicle_journey_at_stops.errors.day_offset_must_not_exceed_max', + local_id: vehicle_journey.objectid.local_id, + max: DAY_OFFSET_MAX + 1 + ) + ) end end |
