Skip to content

Split and remove AbstractQuery#useResultCache()#7214

Closed
someniatko wants to merge 7 commits intodoctrine:old-prototype-3.xfrom
someniatko:someniatko-patch-2
Closed

Split and remove AbstractQuery#useResultCache()#7214
someniatko wants to merge 7 commits intodoctrine:old-prototype-3.xfrom
someniatko:someniatko-patch-2

Conversation

@someniatko
Copy link
Copy Markdown
Contributor

Ported changes from #7213, but also removed the deprecated useResultCache() method and added an entry in UPGRADE.md.

@someniatko
Copy link
Copy Markdown
Contributor Author

Scrutinizer says that code coverage dropped, but I do not see any reasons for that.

@someniatko
Copy link
Copy Markdown
Contributor Author

@Ocramius any chance for this to go forward?

@Majkl578 Majkl578 added this to the 3.0 milestone Sep 21, 2018
Base automatically changed from master to old-prototype-3.x February 23, 2021 08:18
@derrabus
Copy link
Copy Markdown
Member

Closing as obsolete because of #7701.

@derrabus derrabus closed this May 11, 2022
@derrabus derrabus removed this from the 3.0.0 milestone May 11, 2022
@someniatko
Copy link
Copy Markdown
Contributor Author

someniatko commented Apr 26, 2023

@derrabus Only noticed this now. #7701 was about deprecating in 2.x, and this PR was about removing the functionality in 3.x. The former does not obsolete this one.

But I see the related functionality is already removed in 3.x anyway.

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.

3 participants