Check is removed as the DTO checks this

We had this before the DTO, PHPstan correctly flags this as a variable which can never be . So we either had to add this null to keep PHPstan wondering or accept that when someone alters the database they get problems.

Author: vmcj

webapp/src/Entity/Contest.php

@@ -1340,14 +1340,6 @@ public function validate(ExecutionContextInterface $context): void
         }
 
         if ($this->medalsEnabled) {
-            foreach (['goldMedals', 'silverMedals', 'bronzeMedals'] as $field) {
-                if ($this->$field === null) {
-                    $context
-                        ->buildViolation('This field is required when \'Enable medals\' is set.')
-                        ->atPath($field)
-                        ->addViolation();
-                }
-            }
             if ($this->medal_categories->isEmpty()) {
                 $context
                     ->buildViolation('This field is required when \'Process medals\' is set.')