Skip to content

[WIP] create voters_data_permission.rst article #3138

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

Closed
wants to merge 17 commits into from
Closed

[WIP] create voters_data_permission.rst article #3138

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
e14eee5
create voters_data_permission.rst article
Oct 31, 2013
f1217d7
a couple of changes according to the comments, not finished now
Oct 31, 2013
b1923b7
improvements according to the reviews
Nov 5, 2013
b72bd67
updated page with suggestion from the review
Nov 12, 2013
ed86973
Merge branch 'master' of https://github.com/symfony/symfony-docs into…
Nov 12, 2013
6817c53
improved tip box with additional link to /cookbook/security
Nov 12, 2013
a528e05
updated docs according to the reviews
Nov 26, 2013
d56730d
updated with missing fixes
Nov 26, 2013
f6db8c8
updated docs according to the review
Jan 22, 2014
3957b3a
pulled latest version from master
Jan 22, 2014
76a9db0
updated the link from ACL to the data permission voters
Jan 22, 2014
54b9877
added newline to the end of file
Jan 22, 2014
c410082
Update voter_interface.rst.inc
Jan 22, 2014
0159097
updated the docs according to the last review
Jan 22, 2014
639b188
missed one comment
Jan 22, 2014
aa8f2cc
updated according to the review
Feb 18, 2014
8540498
simplified the example
Feb 19, 2014
File filter

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
updated the link from ACL to the data permission voters
  • Loading branch information
Michael Klein committed Jan 22, 2014
commit 76a9db0971b80e090de2f84e712b7247f0c2d551
2 changes: 1 addition & 1 deletion cookbook/security/acl.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ the ACL system comes in.
Using ACL's isn't trivial, and for simpler use cases, it may be overkill.
If your permission logic could be described by just writing some code (e.g.
to check if a Blog is owned by the current User), then consider using
:doc:`voters </cookbook/security/voters>`. A voter is passed the object
:doc:`voters </cookbook/security/dataPermissionVoters>`. A voter is passed the object
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This document doesn't exist.
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

how is the link then for

.. index:: single: Security; Data Permission Voters

with dashes? sorry I could not figure out.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The doc directive references documents (e.g. filenames). So, if you want to reference your newly created voter chapter, this should like this:

:doc:`voters </cookbook/security/voters_data_permission>`
being voted on, which you can use to make complex decisions and effectively
implement your own ACL. Enforcing authorization (e.g. the ``isGranted``
part) will look similar to what you see in this entry, but your voter
Expand Down