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

Fix Symfony 4.2 deprecations, and Travis build #876

Merged
merged 5 commits into from Nov 16, 2018

Conversation

dunglas
Copy link
Contributor

@dunglas dunglas commented Nov 15, 2018

Fix tests, and improve Symfony 4.2 compat

@kimhemsoe kimhemsoe self-assigned this Nov 15, 2018
@kimhemsoe kimhemsoe added this to the 1.9.2 milestone Nov 15, 2018
@kimhemsoe kimhemsoe added the Bug label Nov 15, 2018
@dunglas
Copy link
Contributor Author

dunglas commented Nov 15, 2018

todo:

  • Moving commands from ContainerAwareCommand to proper DI

Unfortunately I'm not sure that I'll be able to work on this soon, maybe can we merge this PR as a first step, and I hope somebody will fix the remaining deprecations before me :p

See #877

dunglas added a commit to dunglas/DoctrineBundle that referenced this pull request Nov 15, 2018
dunglas added a commit to dunglas/DoctrineBundle that referenced this pull request Nov 15, 2018
Copy link
Member

@stof stof left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The 2 failing jobs are OK:

  • the alpha stability job will be fixed when migrating commands away from the deprecated parent class, in the next PR.
  • the PHP nightly job will be fixed once the ORM releases a version supporting PHP 7.3 without deprecation.

@kimhemsoe kimhemsoe merged commit b98183a into doctrine:master Nov 16, 2018
@kimhemsoe
Copy link
Member

@dunglas Thanks

@@ -48,7 +49,9 @@ public function testContainer()
$this->assertFalse($container->has('doctrine.dbal.default_connection.events.mysqlsessioninit'));

if (interface_exists('Symfony\Component\PropertyInfo\PropertyInfoExtractorInterface') && class_exists('Symfony\Bridge\Doctrine\PropertyInfo\DoctrineExtractor')) {
$this->assertInstanceOf('Doctrine\Common\Persistence\Mapping\ClassMetadataFactory', $container->get('doctrine.orm.default_entity_manager.metadata_factory'));
if (!interface_exists(PropertyInitializableExtractorInterface::class)) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change violates CS.

chalasr pushed a commit to chalasr/DoctrineBundle that referenced this pull request Nov 30, 2018
kimhemsoe added a commit that referenced this pull request Nov 30, 2018
 Port commands for Symfony 4.2 (follows #876)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants