Skip to content

Fix method link to isCsrfTokenValid #15738

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Aug 30, 2021
Merged

Fix method link to isCsrfTokenValid #15738

merged 1 commit into from
Aug 30, 2021

Conversation

mathieu-ducrot
Copy link
Contributor

Changelog

Fix the method link from AbstractController.php to ControllerTrait.php because the method comes from the Trait.

## Changelog

Fix the method link from AbstractController.php to ControllerTrait.php because the method comes from the Trait.
@carsonbot carsonbot added this to the 4.4 milestone Aug 21, 2021
@mathieu-ducrot
Copy link
Contributor Author

I don't know why but the CI / Code Blocks is failing :

Uncaught Error: Failed opening required '/home/runner/work/symfony-docs/symfony-docs/_checker/vendor/composer/../phpunit/phpunit/src/Framework/Assert/Functions.php'

Do i need to do something for this ?

@javiereguiluz
Copy link
Member

Good catch, thanks Mathieu.

@javiereguiluz javiereguiluz merged commit 52cb0a7 into symfony:4.4 Aug 30, 2021
javiereguiluz added a commit that referenced this pull request Oct 12, 2021
This PR was merged into the 5.3 branch.

Discussion
----------

Fix link to isCsrfTokenValid() for 5.3+

effectively reverting #15738 on 5.3+ as the trait is not present there

Commits
-------

b4b78eb Fix link to isCsrfTokenValid() for 5.3+
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants