fix(company): not requiring logo when creating a company

Logo is not required when updating a company.
GEWIS · Jan 2, 2025 · 9201dcc · 9201dcc
1 parent b1338d3
commit 9201dcc
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/module/Company/src/Service/Company.php b/module/Company/src/Service/Company.php
@@ -349,7 +349,7 @@ public function createCompany(array $data): CompanyModel|bool
         }
 
         // Upload the logo of the company.
-        if (!$this->uploadFile($company, $data['logo'])) {
+        if (!$this->uploadFile($company, $data['logo'], required: true)) {
             return false;
         }
 
@@ -420,9 +420,10 @@ private function uploadFile(
         CompanyModel|CompanyPackageModel|JobModel $entity,
         ?array $file,
         string $languageSuffix = '',
+        bool $required = false,
     ): bool {
         if (null === $file) {
-            return true;
+            return !$required;
         }
 
         // Check if there is an actual file and no errors occurred during the upload.
@@ -535,7 +536,7 @@ public function createPackage(
         $package->setCompany($company);
 
         if (CompanyPackageTypes::Banner === $type) {
-            if (!$this->uploadFile($package, $data['banner'])) {
+            if (!$this->uploadFile($package, $data['banner'], required: true)) {
                 return false;
             }
         }