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

Deprecate QueryBuilder APIs exposing its internal state #9945

Merged
merged 1 commit into from Aug 1, 2022

Conversation

derrabus
Copy link
Member

@derrabus derrabus commented Jul 31, 2022

This PR is a shameless copy of doctrine/dbal#5551 by @morozov. I'd like to deprecate those methods for the same reasons as they've been deprecated in DBAL.

@derrabus derrabus added this to the 2.13.0 milestone Jul 31, 2022
Co-authored-by: Sergei Morozov <morozov@tut.by>
@derrabus derrabus force-pushed the deprecate/query-builder-state branch from 3b882bc to d3664d0 Compare July 31, 2022 14:24
@derrabus derrabus requested review from greg0ire and SenseException and removed request for greg0ire July 31, 2022 14:24
@derrabus derrabus merged commit 2ec2c58 into doctrine:2.13.x Aug 1, 2022
@derrabus derrabus deleted the deprecate/query-builder-state branch August 1, 2022 09:42
derrabus added a commit to derrabus/orm that referenced this pull request Aug 2, 2022
* 2.13.x:
  Deprecate QueryBuilder APIs exposing its internal state (doctrine#9945)
  Update branch info in README and .doctrine-project.json (doctrine#9943)
  Psalm 4.25.0, PHPStan 1.8.2 (doctrine#9941)
  Stop passing event manager to constructor (doctrine#9938)
  Use a more precise phpdoc for ClassMetadataInfo::versionField than mixed (doctrine#9937)
  Make EntityManager `@final` and its constructor public
derrabus added a commit to derrabus/orm that referenced this pull request Aug 3, 2022
* 2.13.x:
  Deprecate QueryBuilder APIs exposing its internal state (doctrine#9945)
  Update branch info in README and .doctrine-project.json (doctrine#9943)
  Psalm 4.25.0, PHPStan 1.8.2 (doctrine#9941)
  Stop passing event manager to constructor (doctrine#9938)
  Use a more precise phpdoc for ClassMetadataInfo::versionField than mixed (doctrine#9937)
  Make EntityManager `@final` and its constructor public
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.

None yet

2 participants