Skip to content
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.

Sign up for GitHub

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

Jump to bottom

CsrfExtension namespace wrong in the documentation #6602

Closed
yamiko-ninja opened this issue May 23, 2016 · 1 comment
Closed

CsrfExtension namespace wrong in the documentation #6602

yamiko-ninja opened this issue May 23, 2016 · 1 comment
Labels
actionable Clear and specific issues ready for anyone to take them. bug Form good first issue Ideal for your first contribution! (some Symfony experience may be required) hasPR A Pull Request has already been submitted for this issue.

Comments

@yamiko-ninja
Copy link
Contributor

The documentation for the form component uses Symfony\Component\Security\Extension\Csrf\CsrfExtension which is not defined.

I believe the correct class is Symfony\Component\Form\Extension\Csrf\CsrfExtension.

Am I right? If so I can do a pull request to update the doc.
@xabbuh
Copy link
Member

xabbuh commented May 23, 2016

Good catch and welcome your pull request. :)

@xabbuh xabbuh added bug good first issue Ideal for your first contribution! (some Symfony experience may be required) actionable Clear and specific issues ready for anyone to take them. Form labels May 23, 2016
yamiko-ninja added a commit to roboparker/symfony-docs that referenced this issue May 23, 2016
@yamiko-ninja
fix symfony#6602
29f005d
@xabbuh xabbuh added the hasPR A Pull Request has already been submitted for this issue. label May 23, 2016
wouterj added a commit that referenced this issue May 27, 2016
@wouterj
bug #6606 fix #6602 (yamiko-ninja)
b29d2a3 
This PR was submitted for the master branch but it was merged into the 2.7 branch instead (closes #6606).

Discussion
----------

fix #6602

Fixes namespace for Extension in the docs

Commits
-------

1a00342 fix #6602
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
actionable Clear and specific issues ready for anyone to take them. bug Form good first issue Ideal for your first contribution! (some Symfony experience may be required) hasPR A Pull Request has already been submitted for this issue.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@xabbuh @yamiko-ninja