Skip to content

Revised security user class examples #181

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
Merged
merged 1 commit into from
Mar 24, 2011
Merged

Revised security user class examples #181

merged 1 commit into from
Mar 24, 2011

Conversation

@jmikola
Copy link
Contributor

@jmikola jmikola commented Mar 22, 2011

No description provided.

@jmikola
[security] Revise user entity/document class names
3cc43a8 
Added Acme vendor (also used in Validation chapters) and removed "bundle:object" syntax, which doesn't seem to be supported anymore.
weaverryan
weaverryan reviewed Mar 23, 2011
View reviewed changes
book/security/users.rst
Copy link
Member

@weaverryan weaverryan Mar 23, 2011

Choose a reason for hiding this comment

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

The short syntax doesn't work here? (e.g. AcmeMyBundle:UserRepository or on any of the providers? Just double-checking, if not - do we know if that's by design or a bug?
Copy link
Contributor Author

@jmikola jmikola Mar 23, 2011

Choose a reason for hiding this comment

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

I briefly tried the short syntax in our OpenSky codebase and received an exception that the class "bundle_name:repository_name" couldn't be found. And that bundle name was in the ODM config's "mapping" block. If this did work in the past, perhaps it's a regression?

FWIW: We use the short syntax frequently in Doctrine methods, and those all still work, but not in annotations.
Copy link
Member

@weaverryan weaverryan Mar 24, 2011

Choose a reason for hiding this comment

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

Cool - just wondering - if it's a bug, it'll get worked out and we'll iron it out in the docs later.
@weaverryan weaverryan merged commit 3cc43a8 into symfony:master Mar 24, 2011
@weaverryan
Copy link
Member

weaverryan commented Mar 24, 2011

Thanks, you rock!
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants

@jmikola @weaverryan