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

made sure this bundle is registered after security bundle #89

Open
wants to merge 1 commit into
base: 1.2
Choose a base branch
from

Conversation

bamarni
Copy link
Contributor

@bamarni bamarni commented Nov 11, 2012

At this step all extensions are already registered, this looks like a hack but it is the only fix I could find.

@bamarni
Copy link
Contributor Author

bamarni commented Nov 11, 2012

hum this breaks your functional tests, I will check this when I'll get some time

@bamarni bamarni closed this Dec 3, 2012
@bamarni bamarni reopened this Dec 3, 2012
@schmittjoh
Copy link
Owner

Could you explain the problem that you have experienced?

@bamarni
Copy link
Contributor Author

bamarni commented Dec 3, 2012

Hi @schmittjoh ,

Well I had a strange security.access.decision_manager definition which was causing an error, the security bundle was registered after this one, and it was messing up with some compiler passes.

Then I noticed this exception message, but no exception is thrown if it's not the case, this patch tries to fix it, but I don't understand why tests are failing on travis, it just passes locally :\

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

Successfully merging this pull request may close these issues.

2 participants