[FrontendBundle] remove container injection and use router directly

coreshop · May 4, 2023 · e1ee76a · e1ee76a
1 parent 7f966ab
commit e1ee76a
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/src/CoreShop/Bundle/FrontendBundle/Resources/config/services.yml b/src/CoreShop/Bundle/FrontendBundle/Resources/config/services.yml
@@ -21,7 +21,7 @@ services:
  - '@Pimcore\Model\Document\Service'
  - '@CoreShop\Component\Core\Context\ShopperContextInterface'
  - '@Symfony\Component\HttpFoundation\RequestStack'
- - '@service_container'
+ - '@router'
  tags:
  - { name: twig.extension }
 

diff --git a/src/CoreShop/Bundle/FrontendBundle/Twig/LocaleSwitcherExtension.php b/src/CoreShop/Bundle/FrontendBundle/Twig/LocaleSwitcherExtension.php
@@ -28,6 +28,7 @@
 use Symfony\Component\DependencyInjection\ContainerInterface;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\RequestStack;
+use Symfony\Component\Routing\RouterInterface;
 use Twig\Extension\AbstractExtension;
 use Twig\TwigFunction;
 
@@ -37,7 +38,7 @@ public function __construct(
  private Document\Service $documentService,
  private ShopperContextInterface $shopperContext,
  private RequestStack $requestStack,
- protected ContainerInterface $container,
+ private RouterInterface $router,
  ) {
  }
 
@@ -95,7 +96,7 @@ public function getLocalizedLinks(Document $document): array
  if ( str_contains($staticRoute->getVariables(), '_locale') ) {
  $params = ['_locale' => $language];
  }
- $link = $this->container->get('router')->generate($route, $params);
+ $link = $this->router->generate($route, $params);
  } else {
  if ( isset($translations[$language]) ) {
  $localizedDocument = Document::getById($translations[$language]);