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.

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

doesn't play nice with JMSSecurityExtraBundle? #12

Closed
bclozel opened this issue Feb 6, 2012 · 4 comments
Closed

doesn't play nice with JMSSecurityExtraBundle? #12

bclozel opened this issue Feb 6, 2012 · 4 comments

Comments

@bclozel
Copy link

bclozel commented Feb 6, 2012

When checking for the method addSecurityListenerFactory, this line fails with JMSSecurityExtraBundle since its SecurityExtension class does not extend sf2's SecurityExtension class.

Does something like this work?

is_callable(array($extension,'addSecurityListenerFactory'))

Maybe there's something wrong with my setup?

@stof
Copy link
Member

stof commented Feb 6, 2012

this is an issue in JMSSecurityExtraBundle as it will occur for all bundles tryign to maintain the BC. So please report it there.

And btw, this bundle is not yet ready to be used

@bclozel
Copy link
Author

bclozel commented Feb 10, 2012

Created an issue in JMSSecurityExtraBundle.

@bclozel bclozel closed this as completed Feb 10, 2012
@bclozel
Copy link
Author

bclozel commented Feb 11, 2012

I think @schmittjoh concerns are valid - maybe there's a better way than if (method_exists($extension, 'addSecurityListenerFactory')) { ?

@bclozel
Copy link
Author

bclozel commented Feb 24, 2012

Fixed with 2e3facc

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants