Merge pull request #476 from symfony-cmf/remove-legacy

remove legacy support code

Author: dbu

UPGRADE-3.0.md

@@ -4,3 +4,12 @@
 
 If you extend any of the classes in this repository, you will need to adjust
 overwritten methods to specify the parameter and return types.
+
+## Content Template Request Attribute
+
+The content template request attribute is only at `template` and no longer also at `contentTemplate`.
+Use the `DynamicRouter::CONTENT_TEMPLATE` constant when referencing to it in code.
+
+## RouteDefaultsTemplatingValidator removed
+
+Use the RouteDefaultsTwigValidator instead.

src/CmfRoutingBundle.php

@@ -20,7 +20,6 @@
 use Doctrine\Persistence\Mapping\Driver\DefaultFileLocator;
 use Symfony\Cmf\Bundle\RoutingBundle\DependencyInjection\Compiler\RedirectableMatcherPass;
 use Symfony\Cmf\Bundle\RoutingBundle\DependencyInjection\Compiler\SetRouterPass;
-use Symfony\Cmf\Bundle\RoutingBundle\DependencyInjection\Compiler\TemplatingValidatorPass;
 use Symfony\Cmf\Bundle\RoutingBundle\DependencyInjection\Compiler\ValidationPass;
 use Symfony\Cmf\Component\Routing\DependencyInjection\Compiler\RegisterRouteEnhancersPass;
 use Symfony\Cmf\Component\Routing\DependencyInjection\Compiler\RegisterRoutersPass;
@@ -38,7 +37,6 @@ public function build(ContainerBuilder $container): void
         $container->addCompilerPass(new RegisterRouteEnhancersPass());
         $container->addCompilerPass(new SetRouterPass());
         $container->addCompilerPass(new ValidationPass());
-        $container->addCompilerPass(new TemplatingValidatorPass());
         $container->addCompilerPass(new RedirectableMatcherPass());
 
         $this->buildPhpcrCompilerPass($container);

src/DependencyInjection/Compiler/TemplatingValidatorPass.php

@@ -83,7 +83,7 @@ public function getParentDocument(): ?object
      */
     public function setParent($parent)
     {
-        @trigger_error('The '.__METHOD__.'() method is deprecated and will be removed in version 3.0. Use setParentDocument() instead.', \E_USER_DEPRECATED);
+        @trigger_error('The '.__METHOD__.'() method is deprecated and will be removed in version 4.0. Use setParentDocument() instead.', \E_USER_DEPRECATED);
 
         return $this->setParentDocument($parent);
     }
@@ -94,7 +94,7 @@ public function setParent($parent)
      */
     public function getParent()
     {
-        @trigger_error('The '.__METHOD__.'() method is deprecated and will be removed in version 3.0. Use getParentDocument() instead.', \E_USER_DEPRECATED);
+        @trigger_error('The '.__METHOD__.'() method is deprecated and will be removed in version 4.0. Use getParentDocument() instead.', \E_USER_DEPRECATED);
 
         return $this->getParentDocument();
     }

src/Doctrine/Phpcr/RedirectRoute.php

@@ -95,7 +95,7 @@ public function getParentDocument(): object
      */
     public function setParent($parent)
     {
-        @trigger_error('The '.__METHOD__.'() method is deprecated and will be removed in version 3.0. Use setParentDocument() instead.', \E_USER_DEPRECATED);
+        @trigger_error('The '.__METHOD__.'() method is deprecated and will be removed in version 4.0. Use setParentDocument() instead.', \E_USER_DEPRECATED);
 
         return $this->setParentDocument($parent);
     }
@@ -106,7 +106,7 @@ public function setParent($parent)
      */
     public function getParent()
     {
-        @trigger_error('The '.__METHOD__.'() method is deprecated and will be removed in version 3.0. Use getParentDocument() instead.', \E_USER_DEPRECATED);
+        @trigger_error('The '.__METHOD__.'() method is deprecated and will be removed in version 4.0. Use getParentDocument() instead.', \E_USER_DEPRECATED);
 
         return $this->getParentDocument();
     }

src/Doctrine/Phpcr/Route.php

@@ -98,9 +98,6 @@ protected function cleanDefaults(array $defaults, Request $request = null): arra
         if (\array_key_exists(RouteObjectInterface::TEMPLATE_NAME, $defaults)) {
             $request->attributes->set(self::CONTENT_TEMPLATE, $defaults[RouteObjectInterface::TEMPLATE_NAME]);
 
-            // contentTemplate is deprecated as of version 2.0, to be removed in 3.0
-            $request->attributes->set('contentTemplate', $defaults[RouteObjectInterface::TEMPLATE_NAME]);
-
             unset($defaults[RouteObjectInterface::TEMPLATE_NAME]);
         }