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 deprecations when running against ODM 1.3 #586

Merged
merged 1 commit into from
Oct 1, 2019

Conversation

alcaeus
Copy link
Member

@alcaeus alcaeus commented Sep 30, 2019

Apparently, some deprecations slipped through the cracks.

  • Indirect notices are skipped - we generally can't fix these (think vendor code triggering deprecation notices)
  • A single usage of a deprecated document repository class is ignored as fixing it requires bumping the minimum ODM version (which can't be done in a patch release)
  • All other deprecations were fixed in the test suite.

@alcaeus alcaeus self-assigned this Sep 30, 2019
@alcaeus alcaeus requested a review from malarzm September 30, 2019 09:53
@@ -5,6 +5,9 @@

use Doctrine\Bundle\MongoDBBundle\Mapping\Driver\YamlDriver;

/**
* @group legacy
Copy link
Member

Choose a reason for hiding this comment

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

this seems unrelated, is it some magic group name that allows deprecations to be present?

Copy link
Member Author

Choose a reason for hiding this comment

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

The group tells the Symfony deprecation handler to ignore all deprecation notices that come up during the execution of this test. This can be done to specifically test legacy behaviour without having to define an exception for the deprecation in configuration.

@malarzm malarzm merged commit 39e621c into doctrine:3.6 Oct 1, 2019
@alcaeus alcaeus deleted the fix-deprecations branch October 1, 2019 18:34
@alcaeus alcaeus added this to the 3.6.2 milestone Oct 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants