From 07b36edb9e9e1b1d6fbe13e0cf3a35d17effe696 Mon Sep 17 00:00:00 2001 From: Nicky Gerritsen Date: Sat, 24 May 2025 14:43:13 -0400 Subject: [PATCH] Fix setting external judgement types This used to reset some other config variables. By merging with all existing ones it doesn't do this anymore. --- webapp/src/Service/ExternalContestSourceService.php | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/webapp/src/Service/ExternalContestSourceService.php b/webapp/src/Service/ExternalContestSourceService.php index 73372eb6fd..0438ccaf32 100644 --- a/webapp/src/Service/ExternalContestSourceService.php +++ b/webapp/src/Service/ExternalContestSourceService.php @@ -772,7 +772,11 @@ protected function importJudgementType(Event $event, EventData $data): void // Verdict not found, import it as a custom verdict; assume it has a penalty. $customVerdicts = $this->config->get('external_judgement_types'); $customVerdicts[$verdict] = str_replace(' ', '-', $data->name); - $this->config->saveChanges(['external_judgement_types' => $customVerdicts], $this->eventLog, $this->dj); + $this->config->saveChanges( + array_merge($this->config->all(), ['external_judgement_types' => $customVerdicts]), + $this->eventLog, + $this->dj + ); $this->verdicts = $this->config->getVerdicts(['final', 'external']); $penalty = true; $solved = false;