[BUGFIX] Fix EXT:solr route enhancer

SolrRoutingMiddleware was limited to indexing processs, but as the SolrFacetMaskAndCombineEnhancer requires this middleware to add the required parameters from the route path, this limitation causes that the route enhancer is ineffective. This commit fixes this issue by reenabling the middleware for frontend requests and also ensures that route paths can be resolved correctly if PageTypeSuffix enhancer is active. Relates: #3202 Resolves: #3741
TYPO3-Solr · Aug 2, 2023 · 88d6a1a · 88d6a1a
1 parent 4dd5d41
commit 88d6a1a
Show file tree

Hide file tree

Showing 4 changed files with 37 additions and 40 deletions.
diff --git a/Classes/EventListener/EnhancedRouting/CachedPathVariableModifier.php b/Classes/EventListener/EnhancedRouting/CachedPathVariableModifier.php
@@ -51,7 +51,7 @@ public function __invoke(BeforeCachedVariablesAreProcessedEvent $event): void
 
         $this->routingService = GeneralUtility::makeInstance(
             RoutingService::class,
-            $enhancerConfiguration['solr'],
+            $enhancerConfiguration['solr'] ?? [],
             (string)$enhancerConfiguration['extensionKey']
         );
 

diff --git a/Classes/EventListener/EnhancedRouting/PostEnhancedUriProcessor.php b/Classes/EventListener/EnhancedRouting/PostEnhancedUriProcessor.php
@@ -38,7 +38,7 @@ public function __invoke(AfterUriIsProcessedEvent $event): void
         /** @var RoutingService $routingService */
         $routingService = GeneralUtility::makeInstance(
             RoutingService::class,
-            $configuration['solr'],
+            $configuration['solr'] ?? [],
             (string)$configuration['extensionKey']
         );
 

diff --git a/Classes/Middleware/SolrRoutingMiddleware.php b/Classes/Middleware/SolrRoutingMiddleware.php
@@ -84,7 +84,7 @@ public function injectRoutingService(RoutingService $routingService): void
      */
     public function process(ServerRequestInterface $request, RequestHandlerInterface $handler): ResponseInterface
     {
-        if (!$request->hasHeader(PageIndexerRequest::SOLR_INDEX_HEADER)) {
+        if ($request->hasHeader(PageIndexerRequest::SOLR_INDEX_HEADER)) {
             return $handler->handle($request);
         }
 
@@ -128,7 +128,7 @@ public function process(ServerRequestInterface $request, RequestHandlerInterface
         /*
          * Take slug path segments and argument from incoming URI
          */
-        [$slug, $parameters] = $this->extractParametersFromUriPath(
+        [, $parameters] = $this->extractParametersFromUriPath(
             $request->getUri(),
             $enhancerConfiguration['routePath'],
             $page['slug']
@@ -175,7 +175,7 @@ public function process(ServerRequestInterface $request, RequestHandlerInterface
      */
     protected function configure(array $enhancerConfiguration): void
     {
-        $this->settings = $enhancerConfiguration['solr'];
+        $this->settings = $enhancerConfiguration['solr'] ?? [];
         $this->namespace = $enhancerConfiguration['extensionKey'] ?? $this->namespace;
         $this->routingService = null;
     }
@@ -287,45 +287,42 @@ protected function retrievePageInformation(UriInterface $uri, Site $site): array
                 $path,
                 $this->language
             );
+
             if (empty($items)) {
-                $this->logger
-                    ->
-                    error(
-                        vsprintf(
-                            'Could not determine page for slug "%1$s" and language "%2$s". Given path "%3$s"',
-                            [
-                                $path,
-                                $this->language->getLocale()->getLanguageCode(),
-                                $uri->getPath(),
-                            ]
-                        )
-                    );
+                if (empty($path)) {
+                    $message = 'Could not resolve page by path "%3$s" and language "%2$s".';
+                } else {
+                    $message = 'Could not determine page for slug "%1$s" and language "%2$s". Given path "%3$s"';
+                }
+
+                $this->logger->error(
+                    sprintf(
+                        $message,
+                        $path,
+                        $this->language->getTwoLetterIsoCode(),
+                        $uri->getPath()
+                    )
+                );
                 $scan = false;
-            } elseif (empty($path)) {
-                $this->logger
-                    ->
-                    error(
-                        vsprintf(
-                            'Could not resolve page by path "%1$s" and language "%2$s".',
-                            [
-                                $uri->getPath(),
-                                $this->language->getLocale()->getLanguageCode(),
-                            ]
-                        )
-                    );
+            } elseif (empty($path) && count($items) === 1) {
+                $page = $items[0];
+                $this->logger->info(
+                    sprintf(
+                        'Path "%1$s" -> slug "%2$s"',
+                        $uri->getPath(),
+                        $page['slug']
+                    )
+                );
                 $scan = false;
             } else {
                 foreach ($items as $item) {
-                    $this->logger
-                        ->info(
-                            vsprintf(
-                                'Path "%1$s" -> slug "%2$s"',
-                                [
-                                    $path,
-                                    $item['slug'],
-                                ]
-                            )
-                        );
+                    $this->logger->info(
+                        sprintf(
+                            'Path "%1$s" -> slug "%2$s"',
+                            $path,
+                            $item['slug']
+                        )
+                    );
 
                     if ($item['slug'] === $path) {
                         $page = $item;

diff --git a/Classes/Routing/Enhancer/SolrFacetMaskAndCombineEnhancer.php b/Classes/Routing/Enhancer/SolrFacetMaskAndCombineEnhancer.php
@@ -298,7 +298,7 @@ protected function getRoutingService(): RoutingService
         /** @var RoutingService $routingService */
         $routingService = GeneralUtility::makeInstance(
             RoutingService::class,
-            $this->configuration['solr'],
+            $this->configuration['solr'] ?? [],
             (string)$this->configuration['extensionKey']
         );
         return $routingService->withPathArguments($this->configuration['_arguments']);