Update to symfony 6.0 and drop support for symfony 4 and php 7.Update to symfony 6.0 and drop support for symfony 4

Author: mfadul24

DependencyInjection/Compiler/SetRouterPass.php

@@ -28,7 +28,7 @@
  */
 class SetRouterPass implements CompilerPassInterface
 {
-    public function process(ContainerBuilder $container)
+    public function process(ContainerBuilder $container): void
     {
         $container->setAlias('router', 'jms_i18n_routing.router')->setPublic(true);
 

DependencyInjection/Configuration.php

@@ -24,7 +24,7 @@
 
 final class Configuration implements ConfigurationInterface
 {
-    public function getConfigTreeBuilder()
+    public function getConfigTreeBuilder(): \Symfony\Component\Config\Definition\Builder\TreeBuilder
     {
         $tb = new TreeBuilder('jms_i18n_routing');
 

DependencyInjection/JMSI18nRoutingExtension.php

@@ -30,7 +30,7 @@
  */
 class JMSI18nRoutingExtension extends Extension
 {
-    public function load(array $configs, ContainerBuilder $container)
+    public function load(array $configs, ContainerBuilder $container): void
     {
         $config = $this->processConfiguration(new Configuration, $configs);
 
@@ -84,7 +84,7 @@ public function load(array $configs, ContainerBuilder $container)
         }
     }
 
-    public function getAlias()
+    public function getAlias(): string
     {
         return 'jms_i18n_routing';
     }

EventListener/CookieSettingListener.php

@@ -33,18 +33,18 @@ public function __construct($cookieName, $cookieLifetime, $cookiePath, $cookieDo
         $this->cookieHttponly = $cookieHttponly;
     }
 
-    public function onKernelResponse(ResponseEvent $event)
+    public function onKernelResponse(ResponseEvent $event): void
     {
         //Check if the current response contains an error. If it does, do not set the cookie as the Locale may not be properly set
-        if (HttpKernelInterface::MASTER_REQUEST !== $event->getRequestType() || !($event->getResponse()->isSuccessful() || $event->getResponse()->isRedirection())) {
+        if (HttpKernelInterface::MAIN_REQUEST !== $event->getRequestType() || !($event->getResponse()->isSuccessful() || $event->getResponse()->isRedirection())) {
             return;
         }
 
         $request = $event->getRequest();
 
         if (!$request->cookies->has($this->cookieName)
                 || $request->cookies->get($this->cookieName) !== $request->getLocale()) {
-            $event->getResponse()->headers->setCookie(new Cookie($this->cookieName, $request->getLocale(), time() + $this->cookieLifetime, $this->cookiePath, $this->cookieDomain, $this->cookieSecure, $this->cookieHttponly));
+            $event->getResponse()->headers->setCookie(\Symfony\Component\HttpFoundation\Cookie::create($this->cookieName, $request->getLocale(), time() + $this->cookieLifetime, $this->cookiePath, $this->cookieDomain, $this->cookieSecure, $this->cookieHttponly));
         }
     }
 }

EventListener/LocaleChoosingListener.php

@@ -48,9 +48,9 @@ public function __construct($defaultLocale, array $locales, LocaleResolverInterf
         $this->localeResolver = $localeResolver;
     }
 
-    public function onKernelException(ExceptionEvent $event)
+    public function onKernelException(ExceptionEvent $event): void
     {
-        if (HttpKernelInterface::MASTER_REQUEST !== $event->getRequestType()) {
+        if (HttpKernelInterface::MAIN_REQUEST !== $event->getRequestType()) {
             return;
         }
 

Exception/NotAcceptableLanguageException.php

@@ -1,21 +1,21 @@
 <?php
-
-/*
- * Copyright 2012 Johannes M. Schmitt <schmittjoh@gmail.com>
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
+
+/*
+ * Copyright 2012 Johannes M. Schmitt <schmittjoh@gmail.com>
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
 namespace JMS\I18nRoutingBundle\Exception;
 
 class NotAcceptableLanguageException extends NotAcceptableException
@@ -36,8 +36,8 @@ public function getRequestedLanguage()
         return $this->requestedLanguage;
     }
 
-    public function getAvailableLanguages()
+    public function getAvailableLanguages(): array
     {
         return $this->availableLanguages;
     }
-}
+}

JMSI18nRoutingBundle.php

@@ -22,6 +22,7 @@
 use Symfony\Component\DependencyInjection\Compiler\ResolveDefinitionTemplatesPass;
 use JMS\I18nRoutingBundle\DependencyInjection\Compiler\SetRouterPass;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
+use Symfony\Component\DependencyInjection\Extension\ExtensionInterface;
 use Symfony\Component\HttpKernel\Bundle\Bundle;
 
 /**
@@ -31,13 +32,13 @@
  */
 class JMSI18nRoutingBundle extends Bundle
 {
-    public function build(ContainerBuilder $container)
+    public function build(ContainerBuilder $container): void
     {
         $container->addCompilerPass(new SetRouterPass());
     }
 
-    public function getContainerExtension()
+    public function getContainerExtension(): ?ExtensionInterface
     {
         return new JMSI18nRoutingExtension();
     }
-}
+}

Router/DefaultLocaleResolver.php

@@ -31,7 +31,7 @@ public function __construct($cookieName, array $hostMap = array())
     /**
      * {@inheritDoc}
      */
-    public function resolveLocale(Request $request, array $availableLocales)
+    public function resolveLocale(Request $request, array $availableLocales): ?string
     {
         if ($this->hostMap && isset($this->hostMap[$host = $request->getHost()])) {
             return $this->hostMap[$host];
@@ -74,4 +74,4 @@ public function resolveLocale(Request $request, array $availableLocales)
 
         return null;
     }
-}
+}

Router/DefaultPatternGenerationStrategy.php

@@ -16,9 +16,9 @@
  */
 class DefaultPatternGenerationStrategy implements PatternGenerationStrategyInterface
 {
-    const STRATEGY_PREFIX = 'prefix';
-    const STRATEGY_PREFIX_EXCEPT_DEFAULT = 'prefix_except_default';
-    const STRATEGY_CUSTOM = 'custom';
+    public const STRATEGY_PREFIX = 'prefix';
+    public const STRATEGY_PREFIX_EXCEPT_DEFAULT = 'prefix_except_default';
+    public const STRATEGY_CUSTOM = 'custom';
 
     private $strategy;
     private $translator;
@@ -40,7 +40,7 @@ public function __construct($strategy, TranslatorInterface $translator, array $l
     /**
      * {@inheritDoc}
      */
-    public function generateI18nPatterns($routeName, Route $route)
+    public function generateI18nPatterns($routeName, Route $route): array
     {
         $patterns = array();
         foreach ($route->getOption('i18n_locales') ?: $this->locales as $locale) {
@@ -81,12 +81,12 @@ public function generateI18nPatterns($routeName, Route $route)
     /**
      * {@inheritDoc}
      */
-    public function addResources(RouteCollection $i18nCollection)
+    public function addResources(RouteCollection $i18nRouteCollection): void
     {
         foreach ($this->locales as $locale) {
             if (file_exists($metadata = $this->cacheDir.'/translations/catalogue.'.$locale.'.php.meta')) {
                 foreach (unserialize(file_get_contents($metadata)) as $resource) {
-                    $i18nCollection->addResource($resource);
+                    $i18nRouteCollection->addResource($resource);
                 }
             }
         }

Router/DefaultRouteExclusionStrategy.php

@@ -32,7 +32,7 @@
  */
 class DefaultRouteExclusionStrategy implements RouteExclusionStrategyInterface
 {
-    public function shouldExcludeRoute($routeName, Route $route)
+    public function shouldExcludeRoute($routeName, Route $route): bool
     {
         if ('_' === $routeName[0]) {
             return true;