Merge pull request #33 from delyriand/fix/route-condition

Routes condition: we only check if nocommerce is active because we have api in the list

Author: maximehuran

src/Collector/NoCommerceSyliusCollector.php

@@ -107,7 +107,7 @@ public function getDefaultLocaleCode(): ?string
      * @SuppressWarnings(PHPMD.EmptyCatchBlock)
      * @SuppressWarnings(PHPMD.CyclomaticComplexity)
      */
-    public function collect(Request $request, Response $response, Throwable $exception = null): void
+    public function collect(Request $request, Response $response, ?Throwable $exception = null): void
     {
         try {
             /** @var ChannelInterface $channel */

src/Kernel/SyliusNoCommerceKernelTrait.php

@@ -216,7 +216,7 @@ public function loadRoutes(LoaderInterface $loader): RouteCollection
         foreach ($collection as $name => $route) {
             foreach ($routesToRemove as $routeToRemove) {
                 if (false !== strpos($name, $routeToRemove)) {
-                    $route->setCondition("not(context.getPathInfo() matches '`^%sylius.security.new_api_route%`') and not context.checkNoCommerce()");
+                    $route->setCondition('not context.checkNoCommerce()');
                 }
             }
         }

src/Provider/FeaturesProvider.php

@@ -32,7 +32,7 @@ public function __construct(
         $this->nocommerceSettings = $nocommerceSettings;
     }
 
-    public function isNoCommerceEnabledForChannel(ChannelInterface $channel = null): bool
+    public function isNoCommerceEnabledForChannel(?ChannelInterface $channel = null): bool
     {
         try {
             if (null === $channel) {

src/Provider/FeaturesProviderInterface.php

@@ -17,5 +17,5 @@
 
 interface FeaturesProviderInterface
 {
-    public function isNoCommerceEnabledForChannel(ChannelInterface $channel = null): bool;
+    public function isNoCommerceEnabledForChannel(?ChannelInterface $channel = null): bool;
 }