From 10cb28ce41b5b11dad093cd260098b70017395ea Mon Sep 17 00:00:00 2001 From: prandla Date: Sat, 26 Sep 2026 17:17:17 +0300 Subject: [PATCH] italy_yaml fix loading datetimes parse_datetime is already called in make_group, so we mustn't use it with main_group options. (otherwise we will call parse_datetime or make_timedelta twice, which is incorrect. this caused me a very infuriating debug session because i thought my fix for #1565 was somehow buggy.) --- cmscontrib/loaders/italy_yaml.py | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/cmscontrib/loaders/italy_yaml.py b/cmscontrib/loaders/italy_yaml.py index 08f04ef25b..0c4fdce5d6 100644 --- a/cmscontrib/loaders/italy_yaml.py +++ b/cmscontrib/loaders/italy_yaml.py @@ -230,10 +230,10 @@ def get_contest(self): # Times main_group = {} - load(conf, main_group, ["start", "inizio"], conv=parse_datetime) - load(conf, main_group, ["stop", "fine"], conv=parse_datetime) + load(conf, main_group, ["start", "inizio"]) + load(conf, main_group, ["stop", "fine"]) load(conf, args, ["timezone"]) - load(conf, main_group, ["per_user_time"], conv=make_timedelta) + load(conf, main_group, ["per_user_time"]) # Limits load(conf, args, "max_submission_number") @@ -243,8 +243,8 @@ def get_contest(self): # Analysis mode load(conf, main_group, "analysis_enabled") - load(conf, main_group, "analysis_start", conv=parse_datetime) - load(conf, main_group, "analysis_stop", conv=parse_datetime) + load(conf, main_group, "analysis_start") + load(conf, main_group, "analysis_stop") # Groups main_group_name: str | None = load(conf, None, "main_group")