From 7159558231e7cf3304ddc3e7924a72e7ae5fd9b5 Mon Sep 17 00:00:00 2001
From: Ivan Chaplygin <chaplygin@retailcrm.ru>
Date: Tue, 3 Dec 2024 17:44:56 +0300
Subject: [PATCH] =?UTF-8?q?=D0=A3=D0=B4=D0=B0=D0=BB=D0=B5=D0=BD=D0=B8?=
 =?UTF-8?q?=D0=B5=20=D0=BA=D0=BE=D0=BC=D0=BC=D0=B5=D0=BD=D1=82=D0=B0=D1=80?=
 =?UTF-8?q?=D0=B8=D0=B5=D0=B2?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 intaro.retailcrm/lib/icml/settingsservice.php | 31 ++++---------------
 1 file changed, 6 insertions(+), 25 deletions(-)

diff --git a/intaro.retailcrm/lib/icml/settingsservice.php b/intaro.retailcrm/lib/icml/settingsservice.php
index 245f8c40..dce182f5 100644
--- a/intaro.retailcrm/lib/icml/settingsservice.php
+++ b/intaro.retailcrm/lib/icml/settingsservice.php
@@ -149,14 +149,14 @@ class SettingsService
 
         $this->exportProfileId = $profileId ?? '0';
         $this->profileCatalogsOptionName = sprintf('exportProfileId_%s_catalogs', $this->exportProfileId);
-        //$this->setProfileCatalogsOptionName();//Установка названия каталога смысла не вижу от метода
+
         $this->linkNewProfile();
         $this->deleteEmptyProfileCatalogs();
         $this->deleteUnusedOptions();
 
-        $this->customPropList = $this->getNewProps();//получаеют свойства с разделениекм на каталоги. Но не понял при чем custom в названии
-        $this->defaultPropList = $this->getIblockPropsPreset(); //названия базовых свойств (код свойства для icml - перевод)
-        $this->actualPropList = $this->getActualPropList();//не пойму смысла объединения что намеренно было разделено - нигде не переиспользуется - изменить???
+        $this->customPropList = $this->getNewProps();
+        $this->defaultPropList = $this->getIblockPropsPreset();
+        $this->actualPropList = $this->getActualPropList();
     }
 
     /**
@@ -830,11 +830,6 @@ class SettingsService
         return $this;
     }
 
-    private function setProfileCatalogsOptionName(): void
-    {
-        $this->profileCatalogsOptionName = sprintf('exportProfileId_%s_catalogs', $this->exportProfileId);
-    }
-
     private function getCustomProps(): ?array
     {
         $props = unserialize(COption::GetOptionString(self::MODULE_ID, $this->catalogCustomPropsOptionName));
@@ -876,13 +871,12 @@ class SettingsService
 
     public function saveCustomProps(array $newProps): void
     {
-        $currentProps = $this->getCustomProps();//проверка сущестования комбинации профиля и категории
+        $currentProps = $this->getCustomProps();
 
         if (is_null($currentProps)) {
             $this->setCustomProps($newProps);
         } else {
-            $updatedProps = array_merge($currentProps, $newProps);//TODO также только добавляет но не удаляет, проверить как работает
-            //$this->updateProfileCatalogs($catalogId);
+            $updatedProps = array_merge($currentProps, $newProps);
             $this->updateCustomProps($updatedProps);
         }
     }
@@ -904,16 +898,6 @@ class SettingsService
         $this->setOptionEntry($this->profileCatalogsOptionName, $catalogs);
     }
 
-    private function updateProfileCatalogs(string $catalogId): void //TODO Возможно метод для удаления, проверить
-    {
-        $currentCatalogs = $this->getProfileCatalogs();
-
-        if (!in_array($catalogId, $currentCatalogs)) {//TODO: только добавляет но не удаляет исправить
-            $updatedCatalogs = serialize(array_merge($currentCatalogs, [$catalogId])); // только здесь объединение каталогов в профиль
-            $this->updateOptionEntry($this->profileCatalogsOptionName, $updatedCatalogs);
-        }
-    }
-
     private function deleteProfileCatalog(string $catalogId): void
     {
         $currentCatalogs = $this->getProfileCatalogs();
@@ -954,9 +938,6 @@ class SettingsService
 
     private function linkNewProfile(): void
     {
-        $this->profileCatalogsOptionName; // профиль со списокм каталогов
-        $this->exportProfileId; // id профиля
-
         $currentProfileCatalogs = unserialize(COption::GetOptionString(self::MODULE_ID, $this->profileCatalogsOptionName));
 
         if (!$currentProfileCatalogs) {