-
Notifications
You must be signed in to change notification settings - Fork 76
Made PHPCR initializer independent, creating all route_basepaths #318
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
Conversation
|
Failed only on PHP7, may be connected with enabled XDebug on Travis. |
composer.json
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please do not depend on phpunit
115453e to
bb0e113
Compare
|
thanks! |
75fc700 to
656aef1
Compare
|
I disabled XDebug and it passes :) |
|
👍 |
|
So there is now a bC break in this PR: While route basepaths were not initialized by default before, they are now. I think we the RoutingBundle should keep using the old configuration and not initialize the basepaths unless configured. |
|
agreed. lets say that initializer defaults to `auto` and auto means if
sonata is active, we do the initializer, otherwise not. and if its
true/false we just do or not do the initializer.
|
|
@pamil can you adjust the PR to use |
|
@dbu Sure, thanks for reminding me this PR, I almost forgot about it. I'm on my way! :) |
|
@dbu should be up and running now. |
1762e06 to
56e2d92
Compare
|
Only StyleCI is failing and it's (mostly) not related to my changes. |
|
@pamil you should be able to rebase: try (on your branch): If it encounters any conflicts you will need to fix them and then: If you have any difficulties let us know |
db7db13 to
3f8bc5d
Compare
|
@dantleech thanks, I must have missed something, now the rebase went all easy :) Style CI is passing, tests should be passing. |
|
great, this looks good now! |
…ing all defined route_basepaths if enable_initializer=true
3f8bc5d to
49e6cb1
Compare
|
@dbu done :) |
|
thanks. one last thing: can you please do a pull request on the symfony-cmf-docs repository to update the configuration references of the routing bundle? i think its enough to mention the feature there. but if you want you can look over the routing documentation and see if there is a place where it would make sense to mention this feature. please use the |
Made PHPCR initializer independent, creating all route_basepaths
|
thanks a lot. created symfony-cmf/symfony-cmf-docs#724 for the documentation. |
|
👍 |
Related PR Sylius/Sylius#3668.