Skip to content

Vérification d'événement pas trop long #3112

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
SebouChu wants to merge 10 commits into
base: main
Choose a base branch
from
Draft

Vérification d'événement pas trop long #3112

Changes from all commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
c9e7765
bring back the not_too_long validator
SebouChu Jun 2, 2025
dfd5303
Merge branch 'main' into event-not-too-long-validator
arnaudlevy Jun 25, 2025
c46e92f
Merge branch 'main' into event-not-too-long-validator
arnaudlevy Jul 7, 2025
0ba2de5
Merge branch 'main' into event-not-too-long-validator
arnaudlevy Aug 25, 2025
9a126b3
Merge branch 'main' into event-not-too-long-validator
arnaudlevy Sep 8, 2025
500c840
Merge branch 'main' into event-not-too-long-validator
arnaudlevy Sep 9, 2025
92e834a
Merge branch 'main' into event-not-too-long-validator
arnaudlevy Sep 10, 2025
4f052f6
Merge branch 'main' into event-not-too-long-validator
arnaudlevy Oct 17, 2025
7b936d6
Merge branch 'main' into event-not-too-long-validator
arnaudlevy Nov 5, 2025
8147376
Merge branch 'main' into event-not-too-long-validator
arnaudlevy Nov 12, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 8 additions & 8 deletions app/models/communication/website/agenda/event/with_kinds.rb
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ module Communication::Website::Agenda::Event::WithKinds

validate :no_child_before?, if: :kind_parent?
validate :no_child_after?, if: :kind_parent?
# validate :not_too_long # Uncomment when Rennes is ready
validate :not_too_long

before_validation :set_to_day
after_save :manage_time_slots
Expand Down Expand Up @@ -70,13 +70,13 @@ def no_child_after?
end
end

# def not_too_long
# max_duration_in_days = MAX_DURATION / 1.day
# if duration_in_days > max_duration_in_days
# max_end_date = from_day + max_duration_in_days
# errors.add(:to_day, :too_long, max_end_date: max_end_date)
# end
# end
def not_too_long
max_duration_in_days = MAX_DURATION / 1.day
if duration_in_days > max_duration_in_days
max_end_date = from_day + max_duration_in_days
errors.add(:to_day, :too_long, max_end_date: max_end_date)
end
end

def set_to_day
if kind_child?
Expand Down
Loading