Do not output text type when product is simple

Setono · Dec 12, 2024 · 8fc2fcc · 8fc2fcc
1 parent 7d8beb2
commit 8fc2fcc
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/src/Form/Type/WishlistItemType.php b/src/Form/Type/WishlistItemType.php
@@ -36,11 +36,7 @@ public function buildForm(FormBuilderInterface $builder, array $options): void
 
                 $form = $event->getForm();
 
-                if ($product->isSimple()) {
-                    $form->add('variant', TextType::class, [
-                        'disabled' => true,
-                    ]);
-                } else {
+                if (!$product->isSimple()) {
                     $form->add('variant', ProductVariantChoiceType::class, [
                         'choice_label' => fn (ProductVariantInterface $variant) => implode(', ', array_map(static fn ($optionValue) => sprintf('%s: %s', (string) $optionValue->getOption()?->getName(), (string) $optionValue->getValue()), $variant->getOptionValues()->toArray())),
                         'placeholder' => 'setono_sylius_wishlist.ui.select_variant',

diff --git a/src/Resources/views/shop/wishlist/show.html.twig b/src/Resources/views/shop/wishlist/show.html.twig
@@ -68,7 +68,9 @@
                         </td>
                         <td>
                             <div class="ui form">
-                                {{ form_widget(form.items[key].variant) }}
+                                {% if form.items[key].variant is defined %}
+                                    {{ form_widget(form.items[key].variant) }}
+                                {% endif %}
                             </div>
                         </td>
                         <td>