Merge pull request #81 from Florian-Merle/fix/search-exception

fix(search): exception when only 1 result is found

Author: maximehuran

src/Controller/SearchController.php

@@ -16,11 +16,11 @@
 use MonsieurBiz\SyliusSearchPlugin\Context\TaxonContextInterface;
 use MonsieurBiz\SyliusSearchPlugin\Exception\MissingLocaleException;
 use MonsieurBiz\SyliusSearchPlugin\Exception\NotSupportedTypeException;
+use MonsieurBiz\SyliusSearchPlugin\Helper\RenderDocumentUrlHelper;
 use MonsieurBiz\SyliusSearchPlugin\Model\Config\GridConfig;
 use MonsieurBiz\SyliusSearchPlugin\Model\Document\Index\Search;
 use MonsieurBiz\SyliusSearchPlugin\Model\Document\Result;
 use MonsieurBiz\SyliusSearchPlugin\Model\Document\ResultSet;
-use MonsieurBiz\SyliusSearchPlugin\Twig\Extension\RenderDocumentUrl;
 use Sylius\Component\Channel\Context\ChannelContextInterface;
 use Sylius\Component\Currency\Context\CurrencyContextInterface;
 use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
@@ -52,6 +52,9 @@ class SearchController extends AbstractController
     /** @var GridConfig */
     private $gridConfig;
 
+    /** @var RenderDocumentUrlHelper */
+    private $renderDocumentUrlHelper;
+
     /**
      * SearchController constructor.
      *
@@ -60,22 +63,25 @@ class SearchController extends AbstractController
      * @param ChannelContextInterface $channelContext
      * @param CurrencyContextInterface $currencyContext
      * @param TaxonContextInterface $taxonContext
-     * @param array $gridConfig
+     * @param GridConfig $gridConfig
+     * @param RenderDocumentUrlHelper $renderDocumentUrlHelper
      */
     public function __construct(
         EngineInterface $templatingEngine,
         Search $documentSearch,
         ChannelContextInterface $channelContext,
         CurrencyContextInterface $currencyContext,
         TaxonContextInterface $taxonContext,
-        GridConfig $gridConfig
+        GridConfig $gridConfig,
+        RenderDocumentUrlHelper $renderDocumentUrlHelper
     ) {
         $this->templatingEngine = $templatingEngine;
         $this->documentSearch = $documentSearch;
         $this->channelContext = $channelContext;
         $this->currencyContext = $currencyContext;
         $this->taxonContext = $taxonContext;
         $this->gridConfig = $gridConfig;
+        $this->renderDocumentUrlHelper = $renderDocumentUrlHelper;
     }
 
     /**
@@ -117,8 +123,7 @@ public function searchAction(Request $request): Response
             /** @var Result $document */
             $document = current($resultSet->getResults());
             try {
-                $renderDocumentUrl = new RenderDocumentUrl();
-                $urlParams = $renderDocumentUrl->getUrlParams($document);
+                $urlParams = $this->renderDocumentUrlHelper->getUrlParams($document);
 
                 return new RedirectResponse($this->generateUrl($urlParams->getPath(), $urlParams->getParams()));
             } catch (NotSupportedTypeException $e) {