feat: related PrestaShop#35591 - bug3 : dynamic form fields based on …

…attribute type for creating new values
mattgoud · Apr 25, 2024 · 0ff5e93 · 0ff5e93
1 parent 0b383ea
commit 0ff5e93
Show file tree

Hide file tree

Showing 2 changed files with 47 additions and 4 deletions.
diff --git a/admin-dev/themes/new-theme/js/pages/attribute/form/index.ts b/admin-dev/themes/new-theme/js/pages/attribute/form/index.ts
@@ -37,3 +37,34 @@ $(() => {
 
   new window.prestashop.component.ChoiceTree(AttributeFormMap.attributeShopAssociationInput).enableAutoCheckChildren();
 });
+
+document.addEventListener('DOMContentLoaded', () => {
+  const attributeGroupSelect: HTMLElement | null = document.querySelector('#attribute_attribute_group');
+  const attributeColorRow: HTMLElement | null = document.querySelector('.js-attribute-type-color-form-row');
+  const attributeTextureRow: HTMLElement | null = document.querySelector('.js-attribute-type-texture-form-row');
+
+  const attributeGroupSelectValue = (attributeGroupSelect as HTMLInputElement | null)?.value;
+
+  if (attributeColorRow && attributeTextureRow) {
+    if (attributeGroupSelectValue === '2') {
+      attributeColorRow.style.display = 'flex';
+      attributeTextureRow.style.display = 'flex';
+    } else {
+      attributeColorRow.style.display = 'none';
+      attributeTextureRow.style.display = 'none';
+    }
+  }
+  attributeGroupSelect?.addEventListener('change', () => {
+    const NewattributeGroupSelectValue = (attributeGroupSelect as HTMLInputElement | null)?.value;
+
+    if (attributeColorRow && attributeTextureRow) {
+      if (NewattributeGroupSelectValue === '2') {
+        attributeColorRow.style.display = 'flex';
+        attributeTextureRow.style.display = 'flex';
+      } else {
+        attributeColorRow.style.display = 'none';
+        attributeTextureRow.style.display = 'none';
+      }
+    }
+  });
+});
diff --git a/src/PrestaShopBundle/Form/Admin/Sell/Catalog/AttributeType.php b/src/PrestaShopBundle/Form/Admin/Sell/Catalog/AttributeType.php
@@ -106,17 +106,29 @@ public function buildForm(FormBuilderInterface $builder, array $options)
                 'help' => $this->trans('Your internal name for this attribute.', 'Admin.Catalog.Help')
                     . '&nbsp;' . $this->trans('Invalid characters:', 'Admin.Notifications.Info')
                     . ' ' . TypedRegexValidator::CATALOG_CHARS,
-            ]);
+            ])
 
-        if ($hasAttributeGroupId === true && $attributeGroup->group_type === AttributeGroupType::ATTRIBUTE_GROUP_TYPE_COLOR) {
-            $builder->add('color', ColorType::class, [
+            ->add('color', ColorType::class, [
                 'label' => $this->trans('Color', 'Admin.Global'),
+                'row_attr'   =>  array(
+                    'class'   => 'js-attribute-type-color-form-row'),
                 'required' => false,
             ])->add('texture', FileType::class, [
                 'label' => $this->trans('Texture', 'Admin.Global'),
+                'row_attr'   =>  array(
+                    'class'   => 'js-attribute-type-texture-form-row'),
                 'required' => false,
             ]);
-        }
+
+        // if ($hasAttributeGroupId === true && $attributeGroup->group_type === AttributeGroupType::ATTRIBUTE_GROUP_TYPE_COLOR) {
+        //     $builder->add('color', ColorType::class, [
+        //         'label' => $this->trans('Color', 'Admin.Global'),
+        //         'required' => false,
+        //     ])->add('texture', FileType::class, [
+        //         'label' => $this->trans('Texture', 'Admin.Global'),
+        //         'required' => false,
+        //     ]);
+        // }
 
         if ($this->multistoreFeature->isUsed()) {
             $builder->add('shop_association', ShopChoiceTreeType::class, [