bug #25502 Fixing wrong class_exists on interface (weaverryan)

fabpot · fabpot · commit 5fd5f19eaf3c · 2017-12-14T11:42:48.000-08:00
This PR was merged into the 3.4 branch. Discussion ---------- Fixing wrong class_exists on interface | Q | A | ------------- | --- | Branch? | 3.4 | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | none | License | MIT | Doc PR | symfony/symfony-docs#8873 already does not mention changing anything in the config This was a bug introduced in #25151 on the 3.4 branch. It's... pretty self-explanatory I hope :). Cheers! Commits ------- be75bd9 Fixing wrong class_exists on interface
diff --git a/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/Configuration.php b/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/Configuration.php
@@ -143,7 +143,7 @@ private function addCsrfSection(ArrayNodeDefinition $rootNode)
         $rootNode
             ->children()
                 ->arrayNode('csrf_protection')
-                    ->{!class_exists(FullStack::class) && class_exists(CsrfTokenManagerInterface::class) ? 'canBeDisabled' : 'canBeEnabled'}()
+                    ->{!class_exists(FullStack::class) && interface_exists(CsrfTokenManagerInterface::class) ? 'canBeDisabled' : 'canBeEnabled'}()
                 ->end()
             ->end()
         ;

Original file line number	Diff line number	Diff line change
`@@ -143,7 +143,7 @@ private function addCsrfSection(ArrayNodeDefinition $rootNode)`
`143`	`143`	`$rootNode`
`144`	`144`	`->children()`
`145`	`145`	`->arrayNode('csrf_protection')`
`146`		`- ->{!class_exists(FullStack::class) && class_exists(CsrfTokenManagerInterface::class) ? 'canBeDisabled' : 'canBeEnabled'}()`
	`146`	`+ ->{!class_exists(FullStack::class) && interface_exists(CsrfTokenManagerInterface::class) ? 'canBeDisabled' : 'canBeEnabled'}()`
`147`	`147`	`->end()`
`148`	`148`	`->end()`
`149`	`149`	`;`