Сделал удаление записи свойств из таблицы. Начал делать сохранение и получение id каталогов из таблицы

2024-11-22 05:16:09 +03:00 · 2024-11-14 19:37:35 +03:00 · 2024-11-14 19:37:35 +03:00 · b7c54305bc
commit b7c54305bc
parent e71c62b1ab
6 changed files with 114 additions and 91 deletions
--- a/intaro.retailcrm/export/export_run.php
+++ b/intaro.retailcrm/export/export_run.php
@ -46,7 +46,7 @@ if (file_exists($_SERVER['DOCUMENT_ROOT'] . '/bitrix/php_interface/retailcrm/exp
    $iblockPropertyProductHl = [];
    $iblockPropertyUnitProduct = [];

-    foreach (array_keys($settingService->actrualPropList) as $prop) {
+    foreach (array_keys($settingService->actualPropList) as $prop) {
        $skuUnitProps = ('iblockPropertyUnitSku_' . $prop);
        $skuUnitProps = $$skuUnitProps;

--- a/intaro.retailcrm/export/export_setup.php
+++ b/intaro.retailcrm/export/export_setup.php
@ -473,6 +473,7 @@ if ($STEP === 1) {
                            } ?>

                            <?php
+                            if (!empty($catalogCustomProps)) {
                                $catalogId = $arIBlock['ID'];
                                $catalogCustomProps = $settingsService->customPropList[$catalogId];

@ -555,7 +556,9 @@ if ($STEP === 1) {
                                            </td>
                                        <?php } ?>
                                    </tr>
-                            <?php } ?>
+                            <?php }
+                            }
+                            ?>
                            </tbody>
                        </table>
                        <button class="adm-btn-save add-custom-row" type="button">Добавить</button>
@ -664,7 +667,7 @@ if ($STEP === 1) {
            <input type="hidden" name="STEP" value="<?=$STEP + 1?>">
            <input type="hidden" name="SETUP_FIELDS_LIST" value="<?=
            $settingsService->getSetupFieldsString(
-                array_keys($settingsService->actrualPropList) ?? [],
+                array_keys($settingsService->actualPropList) ?? [],
                $hlblockModule === true,
                $hlBlockList ?? []
            )
@ -944,10 +947,10 @@ if ($STEP === 1) {
            if (promises.length > 0) {
                Promise.all(promises)
                    .finally(() => {
-                        // formElem.submit();
+                        formElem.submit();
                    });
            } else {
-                // formElem.submit();
+                formElem.submit();
            }
        });

--- a/intaro.retailcrm/lib/controller/customexportprops.php
+++ b/intaro.retailcrm/lib/controller/customexportprops.php
@ -48,7 +48,9 @@ class CustomExportProps extends Controller
                    'title' => $property['title']
                ];
            }
-            $settingsService->saveCustomProps($profileId, $catalogId, $catalogCustomProps);
+            $settingsService
+                ->setCatalogCustomPropsOptionName($profileId)
+                ->saveCustomProps($catalogId, $catalogCustomProps);
        }
    }

@ -71,7 +73,9 @@ class CustomExportProps extends Controller
                    'title' => $property['title']
                ];
            }
-            $settingsService->removeCustomProps($profileId, $catalogId, $catalogCustomProps);
+            $settingsService
+                ->setCatalogCustomPropsOptionName($catalogId)
+                ->removeCustomProps($catalogCustomProps, $catalogId);
        }
    }
 }
--- a/intaro.retailcrm/lib/icml/settingsservice.php
+++ b/intaro.retailcrm/lib/icml/settingsservice.php
@ -39,6 +39,12 @@ class SettingsService

    private const MODULE_ID = 'intaro.retailcrm';

+    private string $catalogCustomPropsOptionName;
+
+    private string $profileCatalogsOptionName;
+
+    private string $exportProfileId;
+
    /**
     * @var array
     */
@ -105,7 +111,7 @@ class SettingsService
    public $loadNonActivity;

    /** @var array */
-    public $actrualPropList = [];
+    public $actualPropList = [];

    /** @var array */
    public $customPropList = [];
@ -141,9 +147,12 @@ class SettingsService
        $this->getPriceTypes();
        $this->getVatRates();

-        $this->customPropList = $this->parseNewProps();
+        $this->exportProfileId = $this->getExportProfileId();
+        $this->profileCatalogsOptionName = $this->getProfileCatalogsOptionName();
+
+        $this->customPropList = $this->getNewProps();
        $this->defaultPropList = $this->getIblockPropsPreset();
-        $this->actrualPropList = $this->getActualPropList();
+        $this->actualPropList = $this->getActualPropList();
    }

    /**
@ -175,14 +184,6 @@ class SettingsService
        ];
    }

-    public function getFrontPropList(): array
-    {
-        $result = [...$this->defaultPropList];
-        $result['customProps'] = $this->customPropList;
-
-        return $result;
-    }
-
    public function getPriceTypes()
    {
        $dbPriceType = CCatalogGroup::GetList(['SORT' => 'ASC'], [], [], [], ['ID', 'NAME', 'BASE']);
@ -299,33 +300,21 @@ class SettingsService
        ];
    }

-    private function parseNewProps(): array
+    private function getNewProps(): array
    {
-        global $PROFILE_ID;
-        $currentProfile = $PROFILE_ID;
-        $currentProfileCatalogIds = $this->getProfileCatalogIds();
-
        $result = [];
+        $currentProfileCatalogIds = $this->getProfileCatalogs();
+
+        if (!is_null($currentProfileCatalogIds)) {
            foreach ($currentProfileCatalogIds as $catalogId) {
-            $catalogCustomProps = $this->getCustomProps($currentProfile, $catalogId);
+                $catalogCustomProps = $this
+                    ->setCatalogCustomPropsOptionName($catalogId)
+                    ->getCustomProps();

                foreach ($catalogCustomProps as $prop) {
                    $result[$catalogId][$prop['code']] = $prop['title'];
                }
            }
-
-        return $result;
-    }
-
-    private function getProfileCatalogIds(): array
-    {
-        $result = [];
-
-        foreach ($this->arOldSetupVars as $varName => $oldSetupVar) {
-            if (is_array($oldSetupVar) && strpos($varName, 'iblockPropertySku_') !== false) {
-                $result = array_keys($oldSetupVar);
-                break;
-            }
        }

        return $result;
@ -410,7 +399,7 @@ class SettingsService

    public function setProps(): void
    {
-        foreach (array_keys($this->actrualPropList) as $prop) {
+        foreach (array_keys($this->actualPropList) as $prop) {
           $this->setProperties($this->iblockPropertySku, 'iblockPropertySku_' . $prop);
           $this->setProperties($this->iblockPropertyUnitSku, 'iblockPropertyUnitSku_' . $prop);
           $this->setProperties($this->iblockPropertyProduct, 'iblockPropertyProduct_' . $prop);
@ -647,7 +636,7 @@ class SettingsService
        $props = [];

        if (isset($oldValues[$iblockId])) {
-            foreach (array_keys($this->actrualPropList) as $prop) {
+            foreach (array_keys($this->actualPropList) as $prop) {
                $fullKey = $keyGroup . '_' . $prop;
                $props[$prop] = $oldValues[$iblockId][$fullKey];
            }
@ -800,79 +789,106 @@ class SettingsService
        return [$arIBlockList, $intCountChecked, $intCountAvailIBlock, $arIBlockList['iblockExport'] ?? false];
    }

-    private function getCustomPropsOptionName(string $profileId, string $catalogId): string
+    public function setCatalogCustomPropsOptionName(string $catalogId): self
    {
-        return sprintf(
-            'customProps_profileId_%s_catalogId_%s',
-            $profileId,
+        $this->catalogCustomPropsOptionName = sprintf(
+            'customProps_exportProfileId_%s_catalogId_%s',
+            $this->exportProfileId,
            $catalogId
        );
+
+        return $this;
    }

-    public function getCustomProps(string $profileId, string $catalogId)
+    private function getExportProfileId(): string
    {
-        $optionName = $this->getCustomPropsOptionName($profileId, $catalogId);
+        global $PROFILE_ID;

-        return unserialize(COption::GetOptionString(self::MODULE_ID, $optionName));
+        return $PROFILE_ID;
    }

-    public function removeCustomProps(
-        string $profileId,
-        string $catalogId,
-        array $propsToDelete
-    ): void {
-        $currentCatalogProps = $this->getCustomProps($profileId, $catalogId);
+    private function getProfileCatalogsOptionName(): string
+    {
+        return sprintf('exportProfile_%s_catalogs', $this->exportProfileId);
+    }
+
+    public function getCustomProps(): ?array
+    {
+        $props = unserialize(COption::GetOptionString(self::MODULE_ID, $this->catalogCustomPropsOptionName));
+
+        if (!$props) {
+            return null;
+        }
+
+        return $props;
+    }
+
+    public function removeCustomProps(array $propsToDelete, string $catalogId): void
+    {
+        $currentCatalogProps = $this->getCustomProps();
        $updatedCatalogProps = array_values(array_filter(
            $currentCatalogProps,
            fn ($currentProp) => !in_array($currentProp, $propsToDelete)
        ));

        if (empty($updatedCatalogProps)) {
-            $this->removeCustomPropsOptionEntry($profileId, $catalogId);
+            COption::RemoveOption(self::MODULE_ID, $this->catalogCustomPropsOptionName);
+            COption::RemoveOption(self::MODULE_ID, $this->profileCatalogsOptionName);
        } else {
-            $this->updateCustomPropsOptionEntry($profileId, $catalogId, $updatedCatalogProps);
+            $this->updateProfileCatalogs($catalogId);
+            $this->updateCustomPropsOptionEntry($updatedCatalogProps);
        }
    }

-    public function updateCustomPropsOptionEntry(
-        string $profileId,
-        string $catalogId,
-        array $updatedProps
-    ) {
-        $this->removeCustomPropsOptionEntry($profileId, $catalogId);
-        $this->setCustomPropsOptionEntry($profileId, $catalogId, $updatedProps);
-    }
-
-    private function removeCustomPropsOptionEntry($profileId, $catalogId)
+    public function updateCustomPropsOptionEntry(array $updatedProps)
    {
-        $optionName = $this->getCustomPropsOptionName($profileId, $catalogId);
-        $delRes = COption::RemoveOption(self::MODULE_ID, $optionName);
+        COption::SetOptionString(self::MODULE_ID, $this->catalogCustomPropsOptionName, '');
+        $this->setCustomPropsOptionEntry($updatedProps);
    }

-    private function setCustomPropsOptionEntry(
-        string $profileId,
-        string $catalogId,
-        array $props
-    )
+    private function setCustomPropsOptionEntry(array $props)
    {
-        $optionName = $this->getCustomPropsOptionName($profileId, $catalogId);
        $propsString = serialize($props);
-
-        $setResult = COption::SetOptionString(self::MODULE_ID, $optionName, $propsString);
+        COption::SetOptionString(self::MODULE_ID, $this->catalogCustomPropsOptionName, $propsString);
    }

-    public function saveCustomProps(
-        string $profileId,
-        string $catalogId,
-        array $newProps
-    ): void {
-        $currentProps = $this->getCustomProps($profileId, $catalogId);
+    public function saveCustomProps(string $catalogId, array $newProps): void
+    {
+        $currentProps = $this->getCustomProps();

-        if (empty($currentProps)) {
-            $this->setCustomPropsOptionEntry($profileId, $catalogId, $newProps);
+        if (is_null($currentProps)) {
+            $this->setCustomPropsOptionEntry($newProps);
+            $this->setProfileCatalogs($catalogId);
        } else {
            $updatedProps = array_merge($currentProps, $newProps);
-            $this->updateCustomPropsOptionEntry($profileId, $catalogId, $updatedProps);
+            $this->updateCustomPropsOptionEntry($updatedProps);
+            $this->updateProfileCatalogs($catalogId);
        }
    }
+
+    private function getProfileCatalogs(): ?array
+    {
+        $catalogs = unserialize(COption::GetOptionString(self::MODULE_ID, $this->profileCatalogsOptionName));
+
+        if (!$catalogs) {
+            return null;
+        }
+
+        return $catalogs;
+    }
+
+    private function setProfileCatalogs(string $catalogId): void
+    {
+        $catalogs = serialize([$catalogId]);
+        COption::SetOptionString(self::MODULE_ID, $this->profileCatalogsOptionName, $catalogs);
+    }
+
+    private function updateProfileCatalogs(string $catalogId): void
+    {
+        $currentCatalogs = unserialize(COption::GetOptionString(self::MODULE_ID, $this->profileCatalogsOptionName));
+        $updatedCatalogs = serialize([...$currentCatalogs, $catalogId]);
+
+        COption::SetOptionString(self::MODULE_ID, $this->profileCatalogsOptionName, '');
+        COption::SetOptionString(self::MODULE_ID, $this->profileCatalogsOptionName, $updatedCatalogs);
+    }
 }
--- a/intaro.retailcrm/lib/icml/xmlofferbuilder.php
+++ b/intaro.retailcrm/lib/icml/xmlofferbuilder.php
@ -479,7 +479,7 @@ class XmlOfferBuilder
    private function createParamObject(array $params): array
    {
        $offerParams = [];
-        $names = $this->settingsService->actrualPropList;
+        $names = $this->settingsService->actualPropList;

        foreach ($params as $code => $value) {

--- a/tests/lib/icml/SettingServiceTest.php
+++ b/tests/lib/icml/SettingServiceTest.php
@ -30,8 +30,8 @@ class SettingServiceTest extends \BitrixTestCase
        $settingService = SettingsService::getInstance($this->getSetupVars(), "");

        $this->assertInstanceOf(SettingsService::class, $settingService);
-        $this->assertArrayHasKey('property1', $settingService->actrualPropList);
-        $this->assertArrayHasKey('property2', $settingService->actrualPropList);
+        $this->assertArrayHasKey('property1', $settingService->actualPropList);
+        $this->assertArrayHasKey('property2', $settingService->actualPropList);

        return $settingService;
    }