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
Fix/php 7.2 deprecation warnings #163
Fix/php 7.2 deprecation warnings #163
Conversation
(it's okay as minimal PHP version we support is 5.3)
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.
Interesting.
You missed some create_function
inside Doctrine plugin (like https://github.com/LExpress/symfony1/blob/master/lib/plugins/sfDoctrinePlugin/lib/task/sfDoctrineDropDbTask.class.php#L76)
Hm... It means that part is not covered with tests. Alright. I'll fix that as well. |
Yeah, the test suite doesn't cover 100% of the code. |
@j0k3r done! |
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.
This look good, thanks.
This pull replaces
create_function
usages to regular lambda functions (introduced in PHP 5.3) to improve PHP 7.2 compatibility.It will fix PHP nightly build compatibility at your Travis-CI output: