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

Undefined method #68

Open
houdmont opened this issue Mar 28, 2014 · 7 comments
Open

Undefined method #68

houdmont opened this issue Mar 28, 2014 · 7 comments

Comments

@houdmont
Copy link

Call to undefined method Symfony\Component\Config\Definition\Builder\ArrayNodeDefinition::normalizeKeys() in DependencyInjection\Configuration.php on line 54.

->normalizeKeys(false) was added to the configuration recently, however it causes the above exception to be thrown. (added here)

Which version of ArrayNodeDefinition supports normalizeKeys as I'm unable to find it?

@ihortymoshenko
Copy link
Member

@houdmont
Copy link
Author

Thank you,

Is it possible to state that fact in the dependencies?

The project I am currently working on is using v2.1 (which explains why I ended up with issues). I'm not sure if it's possible, but should composer.json state that symfony >= 2.2 is necessary?

@ihortymoshenko
Copy link
Member

@houdmont, I'm not sure that's a good idea as I don't know how many people are using the old version.

@houdmont
Copy link
Author

@IgorTimoshenko I can't understand why this would be a bad idea?

If someone is using an old version of Symfony, adding this requirement would prevent them from breaking projects when including the RequireJSBundle.

If someone is using an older version of the RequireJSBundle, then they won't be affected by the new dependency requirement added to the latest version of the bundle.

I'm probably misunderstanding something here though, so please explain if that's the case.

@ihortymoshenko
Copy link
Member

@houdmont, yeah, you're right. I'll fix it.

@ihortymoshenko
Copy link
Member

@houdmont, moreover, it makes sense to require "symfony/framework-bundle": ">=2.2.0".

@houdmont
Copy link
Author

@IgorTimoshenko that's great! Thanks for helping, sorry I wasn't able to submit a pull request myself with a fix.

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