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

$csrfProvider definition is missed #6338

Closed
sensorario opened this issue Mar 7, 2016 · 2 comments
Closed

$csrfProvider definition is missed #6338

sensorario opened this issue Mar 7, 2016 · 2 comments
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) Hack Day hasPR A Pull Request has already been submitted for this issue.
Milestone
2.7

Comments

@sensorario
Copy link
Contributor

In this file, $csrfProvider definition is missed: https://github.com/symfony/symfony-docs/blob/master/components/form/introduction.rst
@wouterj
Copy link
Member

wouterj commented May 20, 2016

Thanks for reporting this!

This should be replaced with $csrfManager from the previous section. This was changed in the 2.7 version of the documentation.

@wouterj wouterj added 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. Hack Day labels May 20, 2016
@snoek09
Copy link

snoek09 commented May 21, 2016

I'm fixing this.

@xabbuh xabbuh added this to the 2.7 milestone May 21, 2016
@snoek09 snoek09 mentioned this issue May 21, 2016
Closed
@xabbuh xabbuh added the hasPR A Pull Request has already been submitted for this issue. label May 21, 2016
xabbuh added a commit that referenced this issue May 21, 2016
@xabbuh
minor #6586 6338 use csrfManager instead of csrfProvider (snoek09)
f9edfd5 
This PR was squashed before being merged into the 2.7 branch (closes #6586).

Discussion
----------

6338 use csrfManager instead of csrfProvider

| Q             | A
| ------------- | ---
| Doc fix?      | yes
| New docs?     | no
| Applies to    | 2.7
| Fixed tickets | #6338

Commits
-------

500dce5 6338 use csrfManager instead of csrfProvider
@xabbuh xabbuh closed this as completed May 21, 2016
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) Hack Day hasPR A Pull Request has already been submitted for this issue.
Projects
None yet
Milestone
2.7
Development

No branches or pull requests
4 participants
@snoek09 @wouterj @sensorario @xabbuh