8000 Merge branch '2.5' into 2.6 · symfony/symfony@e49b325 · GitHub
[go: up one dir, main page]

Skip to content

Commit e49b325

Browse files
committed
Merge branch '2.5' into 2.6
* 2.5: [Form] fixed the CSRF extension to allow using only the new interfaces
2 parents 1d3a5c6 + 04ec942 commit e49b325

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

src/Symfony/Component/Form/Extension/Csrf/Type/FormTypeCsrfExtension.php

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -128,6 +128,10 @@ public function setDefaultOptions(OptionsResolverInterface $resolver)
128128

129129
// BC clause for the "csrf_provider" option
130130
$csrfTokenManager = function (Options $options) {
131+
if ($options['csrf_provider'] instanceof CsrfTokenManagerInterface) {
132+
return $options['csrf_provider'];
133+
}
134+
131135
return $options['csrf_provider'] instanceof CsrfTokenManagerAdapter
132136
? $options['csrf_provider']->getTokenManager()
133137
: new CsrfProviderAdapter($options['csrf_provider']);
@@ -139,7 +143,7 @@ public function setDefaultOptions(OptionsResolverInterface $resolver)
139143
'csrf_message' => 'The CSRF token is invalid. Please try to resubmit the form.',
140144
'csrf_token_manager' => $csrfTokenManager,
141145
'csrf_token_id' => $csrfTokenId,
142-
'csrf_provider' => new CsrfTokenManagerAdapter($this->defaultTokenManager),
146+
'csrf_provider' => $this->defaultTokenManager,
143147
'intention' => null,
144148
));
145149
}

0 commit comments

Comments
 (0)
0