From: marina <m.zozirova@gmail.com>
Date: Tue, 27 May 2025 09:17:23 +0000 (+0300)
Subject: ERP-417
X-Git-Url: https://gitweb.erp-flowers.ru/?a=commitdiff_plain;h=90d4b52e10699a156dbbf94c8303190743ad93de;p=erp24_rep%2Fyii-erp24%2F.git

ERP-417
---

diff --git a/erp24/controllers/WriteOffsErpController.php b/erp24/controllers/WriteOffsErpController.php
index bcbed2cd..25735df9 100644
--- a/erp24/controllers/WriteOffsErpController.php
+++ b/erp24/controllers/WriteOffsErpController.php
@@ -535,7 +535,7 @@ class WriteOffsErpController extends Controller
                         if ($flag = $model->save(false)) {
                             $writeOffsErpId = $model->getId();
 
-                            foreach ($modelsProducts as $modelProduct) {
+                            foreach ($modelsProducts as $keyModelProduct => $modelProduct) {
                                 /** @var WriteOffsProductsErp $modelProduct */
                                 $modelProduct->setWriteOffsErpId($writeOffsErpId);
                                 $modelProduct->setCreatedAt();
@@ -553,19 +553,15 @@ class WriteOffsErpController extends Controller
 
                                 $modelProductId = $modelProduct->getId();
 
-                                if (!empty($modelsProductsFiles) && array_key_exists($modelProductId, $modelsProductsFiles)) {
-                                    $filesForProduct = $modelsProductsFiles[$modelProductId];
+                                if (!empty($modelsProductsFiles) && array_key_exists($keyModelProduct, $modelsProductsFiles)) {
+                                    $filesForProduct = $modelsProductsFiles[$keyModelProduct];
 
                                     $imageFiles = ArrayHelper::getValue($filesForProduct, 'imageFiles', []);
                                     foreach ($imageFiles as $imageFile) {
                                         if (Images::isImageFile($imageFile)) {
                                             $image = new Images();
                                             $imageId = $image->loadImage($imageFile);
-                                            if ($imageId === false) {
-                                                // Ошибка загрузки изображения
-                                                var_dump("Ошибка загрузки изображения:", $imageFile);
-                                                continue;
-                                            }
+
                                             $imageDocumentLink = new ImageDocumentLink();
                                             $imageDocumentLink->setActive()
                                                 ->setCreatedAt()
@@ -574,22 +570,18 @@ class WriteOffsErpController extends Controller
                                                 ->setDocumentId($writeOffsErpId)
                                                 ->setDocumentItemId($modelProductId)
                                                 ->setImageId($imageId);
-                                            if (!$imageDocumentLink->validate()) {
-                                                var_dump("Ошибка валидации ImageDocumentLink:", $imageDocumentLink->getErrors());
-                                                continue;
+
+                                            if ($imageDocumentLink->validate()) {
+                                                $imageDocumentLink->save();
+                                            } else {
+                                                var_dump($imageDocumentLink->getErrors());
                                             }
-                                            $imageDocumentLink->save();
-                                        } else {
-                                            var_dump("Файл не является изображением:", $imageFile);
                                         }
                                     }
 
-                                    $videoFile = ArrayHelper::getValue($filesForProduct, 'videoFiles', []);
+                                    $videoFile = ArrayHelper::getValue($filesForProduct, 'videoFiles', null);
                                     if ($videoFile) {
-                                        $saved =  FileService::saveUploadedFile($videoFile, WriteOffsProductsErp::WRITE_OFFS_VIDEO, $modelProductId);
-                                        if (!$saved) {
-                                            var_dump("Ошибка сохранения видео файла:", $videoFile);
-                                        }
+                                        FileService::saveUploadedFile($videoFile, WriteOffsProductsErp::WRITE_OFFS_VIDEO, $modelProductId);
                                     }
                                 }
                             }