Skip to content

Commit

Permalink
[#3319] Linking to the whole method
Browse files Browse the repository at this point in the history
  • Loading branch information
weaverryan committed Dec 23, 2013
1 parent 06f2f74 commit 2a10a46
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions cookbook/security/acl_advanced.rst
Original file line number Diff line number Diff line change
Expand Up @@ -48,9 +48,9 @@ your application. Each role, or user has its own security identity.
.. versionadded:: 2.5
For users, the security identity is based on the username. This means that,
if for any reason, a user's username was to change, you must ensure its
security identity is updated too. The method ``updateUserSecurityIdentity``
of the :class:`Symfony\\Component\\Security\\Acl\\Dbal\\MutableAclProvider`
class is there to handle the update.
security identity is updated too. The
:method:`MutableAclProvider::updateUserSecurityIdentity <Symfony\\Component\\Security\\Acl\\Dbal\\MutableAclProvider::updateUserSecurityIdentity>`
``updateUserSecurityIdentity`` method is there to handle the update.

This comment has been minimized.

Copy link
@xabbuh

xabbuh Dec 23, 2013

Member

should be

security identity is updated too. The
:method:`MutableAclProvider::updateUserSecurityIdentity() <Symfony\\Component\\Security\\Acl\\Dbal\\MutableAclProvider::updateUserSecurityIdentity>`
method is there to handle the update.

shouldn't it?

This comment has been minimized.

Copy link
@weaverryan

weaverryan Dec 26, 2013

Author Member

You're right - I have a few mistakes here - fixed at sha: 34551d2

Interestingly, there is not a line break in between the label and the class name inside the :method:, even though the GitHub diff is showing one...

Thanks!

This comment has been minimized.

Copy link
@stof

stof Dec 26, 2013

Member

@weaverryan it does not show one. Look on the left for the line numbers. Diffs are soft-wrapped for rST or markdown files now (they are not for code)

This comment has been minimized.

Copy link
@weaverryan

weaverryan Dec 26, 2013

Author Member

ah, thanks - I see it now :)

This comment has been minimized.


Database Table Structure
------------------------
Expand Down

0 comments on commit 2a10a46

Please sign in to comment.