-
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. |
@@ -25,6 +25,7 @@ | |||
"matthiasnoback/symfony-dependency-injection-test": "~0.6", | |||
"matthiasnoback/symfony-config-test": "0.*", | |||
"phpunit/php-code-coverage": "@stable", | |||
"phpunit/phpunit": "^4.0", |
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.