Make shadow importing more resilient

Author: nickygerritsen

webapp/src/DataTransferObject/Shadowing/ContestEvent.php

@@ -9,7 +9,7 @@ public function __construct(
         public readonly string $name,
         public readonly string $duration,
         public readonly ?string $scoreboardType,
-        public readonly int $penaltyTime,
+        public readonly ?int $penaltyTime,
         public readonly ?string $formalName,
         public readonly ?string $startTime,
         public readonly ?string $countdownPauseTime,

webapp/src/Service/ExternalContestSourceService.php

@@ -766,7 +766,7 @@ protected function validateAndUpdateContest(Event $event, EventData $data): void
 
         // Also compare the penalty time
         $penaltyTime = $data->penaltyTime;
-        if ($this->config->get('penalty_time') != $penaltyTime) {
+        if ($penaltyTime !== null && $this->config->get('penalty_time') != $penaltyTime) {
             $this->logger->warning(
                 'Penalty time does not match between feed (%d) and local (%d)',
                 [$penaltyTime, $this->config->get('penalty_time')]
@@ -863,9 +863,11 @@ protected function validateLanguage(Event $event, EventData $data): void
         } else {
             $this->removeWarning($event->type, $data->id, ExternalSourceWarning::TYPE_DATA_MISMATCH);
 
-            $toCheck = ['extensions' => $data->extensions];
+            if ($data->extensions !== null) {
+                $toCheck = ['extensions' => $data->extensions];
 
-            $this->compareOrCreateValues($event, $data->id, $language, $toCheck);
+                $this->compareOrCreateValues($event, $data->id, $language, $toCheck);
+            }
         }
     }