-
Notifications
You must be signed in to change notification settings - Fork 179
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
Symfony 4.2 depreciation fixes #263
Conversation
@romantymoshyk thanks a lot for the PR, can you remove anything related with Travis configurations? No worries if the tests fail at the moment. The most important is to support the latest versions of symfony. |
@@ -70,6 +75,10 @@ public function registerContainerConfiguration(LoaderInterface $loader) | |||
if (self::VERSION_ID > 30200) { | |||
$loader->load(__DIR__.'/config/disable_annotations.yml'); | |||
} | |||
|
|||
if (self::VERSION_ID < 40200 && file_exists(__DIR__.'/Resources/translations') === false) { | |||
self::recurseCopy(__DIR__.'/../translations', __DIR__.'/Resources/translations'); |
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.
@stof want ask your help. Is this the correct way to do it? Doing a recursive copy? Thanks for the help!
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.
@monteiro, that was written as in SF4.2 was changed default directory structure. If SF will not found translations in new expected place it will throw deprecation message in SF4.2 and fail in SF5.
… 4.2" This reverts commit 32a4628.
@monteiro , .travis.yml changes were reverted |
This pull request apply fixes to: