add redirect to locale-prefix for partial match requests

UPGRADE.md

@@ -1,5 +1,34 @@
 # Upgrade
 
+## dev-release/1.5
+
+### RouteProvider
+
+The method `Sulu\Bundle\RouteBundle\Routing\RouteProvider::__construct` has removed the second parameter.
+
+__Before:__
+
+```
+public function __construct(
+    RouteRepositoryInterface $routeRepository,
+    RequestAnalyzerInterface $requestAnalyzer,
+    RouteDefaultsProviderInterface $routeDefaultsProvider,
+    RequestStack $requestStack,
+    LazyLoadingValueHolderFactory $proxyFactory = null
+)
+```
+
+__After:__
+
+```
+public function __construct(
+    RouteRepositoryInterface $routeRepository,
+    RouteDefaultsProviderInterface $routeDefaultsProvider,
+    RequestStack $requestStack,
+    LazyLoadingValueHolderFactory $proxyFactory = null
+)
+```
+
 ## 1.5.21
 
 ### User API performance improvement

src/Sulu/Bundle/RouteBundle/Resources/config/routing.xml

@@ -26,7 +26,6 @@
 
         <service id="sulu_route.routing.provider" class="Sulu\Bundle\RouteBundle\Routing\RouteProvider">
             <argument type="service" id="sulu.repository.route"/>
-            <argument type="service" id="sulu_core.webspace.request_analyzer"/>
             <argument type="service" id="sulu_route.routing.defaults_provider"/>
             <argument type="service" id="request_stack"/>
             <argument type="service" id="sulu_route.routing.proxy_factory"/>

src/Sulu/Bundle/RouteBundle/Routing/RouteProvider.php

@@ -18,6 +18,7 @@
 use Sulu\Bundle\RouteBundle\Entity\RouteRepositoryInterface;
 use Sulu\Bundle\RouteBundle\Model\RouteInterface;
 use Sulu\Bundle\RouteBundle\Routing\Defaults\RouteDefaultsProviderInterface;
+use Sulu\Component\Webspace\Analyzer\Attributes\RequestAttributes;
 use Sulu\Component\Webspace\Analyzer\RequestAnalyzerInterface;
 use Symfony\Cmf\Component\Routing\RouteProviderInterface;
 use Symfony\Component\HttpFoundation\Request;
@@ -68,22 +69,13 @@ class RouteProvider implements RouteProviderInterface
      */
     private $routeCache = [];
 
-    /**
-     * @param RouteRepositoryInterface $routeRepository
-     * @param RequestAnalyzerInterface $requestAnalyzer
-     * @param RouteDefaultsProviderInterface $routeDefaultsProvider
-     * @param RequestStack $requestStack
-     * @param LazyLoadingValueHolderFactory $proxyFactory
-     */
     public function __construct(
         RouteRepositoryInterface $routeRepository,
-        RequestAnalyzerInterface $requestAnalyzer,
         RouteDefaultsProviderInterface $routeDefaultsProvider,
         RequestStack $requestStack,
         LazyLoadingValueHolderFactory $proxyFactory = null
     ) {
         $this->routeRepository = $routeRepository;
-        $this->requestAnalyzer = $requestAnalyzer;
         $this->routeDefaultsProvider = $routeDefaultsProvider;
         $this->requestStack = $requestStack;
 
@@ -97,7 +89,21 @@ public function getRouteCollectionForRequest(Request $request)
     {
         $collection = new RouteCollection();
         $path = $request->getPathInfo();
-        $prefix = $this->requestAnalyzer->getResourceLocatorPrefix();
+
+        /** @var RequestAttributes|null $attributes */
+        $attributes = $request->attributes->get('_sulu');
+        if (!$attributes) {
+            return $collection;
+        }
+
+        $matchType = $attributes->getAttribute('matchType');
+        if (RequestAnalyzerInterface::MATCH_TYPE_REDIRECT == $matchType
+            || RequestAnalyzerInterface::MATCH_TYPE_PARTIAL == $matchType
+        ) {
+            return $collection;
+        }
+
+        $prefix = $attributes->getAttribute('resourceLocatorPrefix');
 
         if (!empty($prefix) && 0 === strpos($path, $prefix)) {
             $path = PathHelper::relativizePath($path, $prefix);
@@ -124,10 +130,7 @@ public function getRouteCollectionForRequest(Request $request)
             return $collection;
         }
 
-        $collection->add(
-            self::ROUTE_PREFIX . $route->getId(),
-            $this->createRoute($route, $request)
-        );
+        $collection->add(self::ROUTE_PREFIX . $route->getId(), $this->createRoute($route, $request, $attributes));
 
         return $collection;
     }
@@ -159,6 +162,14 @@ public function getRouteByName($name)
             throw new RouteNotFoundException();
         }
 
+        $request = $this->requestStack->getCurrentRequest();
+
+        /** @var RequestAttributes $attributes */
+        $attributes = $request->attributes->get('_sulu');
+        if (!$attributes) {
+            throw new RouteNotFoundException();
+        }
+
         $routeId = substr($name, strlen(self::ROUTE_PREFIX));
         if (array_key_exists($routeId, $this->symfonyRouteCache)) {
             return $this->symfonyRouteCache[$routeId];
@@ -178,7 +189,7 @@ public function getRouteByName($name)
             throw new RouteNotFoundException();
         }
 
-        return $this->createRoute($route, $this->requestStack->getCurrentRequest());
+        return $this->createRoute($route, $request, $attributes);
     }
 
     /**
@@ -197,7 +208,7 @@ public function getRoutesByNames($names)
      *
      * @return Route
      */
-    protected function createRoute(RouteInterface $route, Request $request)
+    protected function createRoute(RouteInterface $route, Request $request, RequestAttributes $attributes)
     {
         $routePath = $this->requestAnalyzer->getResourceLocatorPrefix() . $route->getPath();
 
@@ -207,7 +218,7 @@ protected function createRoute(RouteInterface $route, Request $request)
                 [
                     '_controller' => 'SuluWebsiteBundle:Redirect:redirect',
                     'url' => $request->getSchemeAndHttpHost()
-                        . $this->requestAnalyzer->getResourceLocatorPrefix()
+                        . $attributes->getAttribute('resourceLocatorPrefix')
                         . $route->getTarget()->getPath()
                         . ($request->getQueryString() ? ('?' . $request->getQueryString()) : ''),
                 ]