fixer

Oksydan · Sep 12, 2023 · 2b6034f · 2b6034f
1 parent 2afc8a4
commit 2b6034f
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/src/Entity/ImageSliderLang.php b/src/Entity/ImageSliderLang.php
@@ -179,9 +179,9 @@ public function setDescription(string $description): ImageSliderLang
     }
 
     /**
-     * @return string
+     * @return string|null
      */
-    public function getImage(): string
+    public function getImage(): ?string
     {
         return $this->image;
     }
@@ -199,9 +199,9 @@ public function setImage(string $image): ImageSliderLang
     }
 
     /**
-     * @return string
+     * @return string|null
      */
-    public function getImageMobile(): string
+    public function getImageMobile(): ?string
     {
         return $this->imageMobile;
     }

diff --git a/src/Form/DataHandler/ImageSliderFormDataHandler.php b/src/Form/DataHandler/ImageSliderFormDataHandler.php
@@ -141,14 +141,14 @@ public function update($id, array $data): int
                 ->setDescription($data['description'][$langId] ?? '');
 
             if (!empty($data['image'][$langId])) {
-                if (!$newImageSliderLang) {
+                if ($imageSliderLang->getImage() !== null) {
                     $this->eraseFile($imageSliderLang->getImage());
                 }
                 $imageSliderLang->setImage($this->uploadFile($data['image'][$langId]));
             }
 
             if (!empty($data['image_mobile'][$langId])) {
-                if (!$newImageSliderLang) {
+                if ($imageSliderLang->getImage() !== null) {
                     $this->eraseFile($imageSliderLang->getImageMobile());
                 }
                 $imageSliderLang->setImageMobile($this->uploadFile($data['image_mobile'][$langId]));