Merge pull request #52 from lanfisis/1.x

Add enabled / disabled capability for menu items

Author: lanfisis

.github/workflows/recipe.yaml

@@ -62,6 +62,7 @@ jobs:
         working-directory: ./sylius
         run: |
             composer require --no-install --no-scripts --no-progress sylius/sylius="${{ matrix.sylius }}"
+            composer require --no-install --no-scripts --no-progress symfony/serializer "<6.4.23" # Temporary fix for Sylius 1.x
 
       - name: Setup some requirements
         working-directory: ./sylius

.github/workflows/tests.yaml

@@ -15,6 +15,7 @@ jobs:
       fail-fast: false
       matrix:
         php: ['8.1', '8.2', '8.3']
+        node: ['20']
 
     env:
       SYMFONY_ARGS: --no-tls
@@ -25,7 +26,7 @@ jobs:
       - uses: actions/checkout@v2
       - uses: actions/setup-node@v2
         with:
-          node-version: '14'
+          node-version: '20'
       - name: Setup PHP
         uses: shivammathur/setup-php@v2
         with:

.gitignore

@@ -16,3 +16,4 @@
 /.phpunit.result.cache
 /node_modules
 /yarn.lock
+/.php-cs-fixer.cache

Makefile

@@ -72,6 +72,9 @@ setup_application:
 	(cd ${APP_DIR} && ${COMPOSER} config --no-plugins allow-plugins true)
 	(cd ${APP_DIR} && ${COMPOSER} config --no-plugins --json extra.symfony.endpoint '["https://api.github.com/repos/monsieurbiz/symfony-recipes/contents/index.json?ref=flex/master","flex://defaults"]')
 	(cd ${APP_DIR} && ${COMPOSER} require --no-install --no-scripts --no-progress sylius/sylius="~${SYLIUS_VERSION}") # Make sure to install the required version of sylius because the sylius-standard has a soft constraint
+	# Temporary fix for Sylius 1.x
+	(cd ${APP_DIR} && ${COMPOSER} require --no-install --no-scripts --no-progress symfony/serializer "<6.4.23")
+	# End Temporary fix for Sylius 1.x
 	$(MAKE) ${APP_DIR}/.php-version
 	$(MAKE) ${APP_DIR}/php.ini
 	(cd ${APP_DIR} && ${COMPOSER} install --no-interaction)

dist/src/Context/Channel/RequestBased/HostnameAndPortBasedRequestResolver.php

@@ -0,0 +1,33 @@
+<?php
+
+/*
+ * This file is part of Monsieur Biz' Menu plugin for Sylius.
+ *
+ * (c) Monsieur Biz <sylius@monsieurbiz.com>
+ *
+ * For the full copyright and license information, please view the LICENSE.txt
+ * file that was distributed with this source code.
+ */
+
+declare(strict_types=1);
+
+namespace App\Context\Channel\RequestBased;
+
+use Sylius\Component\Channel\Context\RequestBased\RequestResolverInterface;
+use Sylius\Component\Channel\Model\ChannelInterface;
+use Sylius\Component\Channel\Repository\ChannelRepositoryInterface;
+use Symfony\Component\DependencyInjection\Attribute\AutoconfigureTag;
+use Symfony\Component\HttpFoundation\Request;
+
+#[AutoconfigureTag('sylius.context.channel.request_based.resolver')]
+final class HostnameAndPortBasedRequestResolver implements RequestResolverInterface
+{
+    public function __construct(private ChannelRepositoryInterface $channelRepository)
+    {
+    }
+
+    public function findChannel(Request $request): ?ChannelInterface
+    {
+        return $this->channelRepository->findOneEnabledByHostname($request->getHost() . ':' . $request->getPort());
+    }
+}

src/Entity/MenuItem.php

@@ -49,6 +49,8 @@ class MenuItem implements MenuItemInterface
 
     protected bool $nofollow = false;
 
+    protected bool $enabled = true;
+
     /**
      * MenuItem constructor.
      */
@@ -228,4 +230,20 @@ protected function createTranslation(): MenuItemTranslationInterface
     {
         return new MenuItemTranslation();
     }
+
+    /**
+     * @inheritdoc
+     */
+    public function isEnabled(): bool
+    {
+        return $this->enabled;
+    }
+
+    /**
+     * @inheritdoc
+     */
+    public function setEnabled(bool $enabled): void
+    {
+        $this->enabled = $enabled;
+    }
 }

src/Entity/MenuItemInterface.php

@@ -66,4 +66,8 @@ public function setNofollow(bool $nofollow): void;
     public function getLabel(): ?string;
 
     public function getUrl(): ?string;
+
+    public function isEnabled(): bool;
+
+    public function setEnabled(bool $enabled): void;
 }

src/Fixture/Factory/MenuFixtureFactory.php

@@ -83,6 +83,7 @@ private function createMenuItem(array $item, int $position, MenuInterface $menu,
         $menuItem->setPosition($position);
         $menuItem->setMenu($menu);
 
+        $menuItem->setEnabled($item['enabled'] ?? true);
         $menuItem->setTargetBlank($item['targetBlank'] ?? false);
         $menuItem->setNoreferrer($item['noreferrer'] ?? false);
         $menuItem->setNoopener($item['noopener'] ?? false);

src/Fixture/MenuFixture.php

@@ -44,6 +44,7 @@ public function configureResourceNode(ArrayNodeDefinition $resourceNode): void
                                         ->end()
                                     ->end()
                                 ->end()
+                                ->booleanNode('enabled')->defaultTrue()->end()
                                 ->booleanNode('targetBlank')->defaultFalse()->end()
                                 ->booleanNode('noreferrer')->defaultFalse()->end()
                                 ->booleanNode('noopener')->defaultFalse()->end()

src/Form/Type/MenuItemType.php

@@ -39,6 +39,10 @@ public function buildForm(FormBuilderInterface $builder, array $options): void
                 /** @var MenuItemInterface $item */
                 Assert::isInstanceOf($item, MenuItemInterface::class);
                 $event->getForm()
+                    ->add('enabled', CheckboxType::class, [
+                        'label' => 'monsieurbiz_menu.ui.enabled',
+                        'required' => false,
+                    ])
                     ->add('parent', EntityType::class, [
                         'class' => $this->dataClass,
                         'required' => false,