-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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 ORM related functions on ProxyQueryInterface #6386
Conversation
4e94e40
to
9fdeded
Compare
src/Datagrid/ProxyQueryInterface.php
Outdated
@@ -94,11 +94,19 @@ public function setMaxResults($maxResults); | |||
public function getMaxResults(); | |||
|
|||
/** | |||
* NEXT_MAJOR: Remove this method. |
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 method from the interface.
?
Otherwise it could be a bit misleading IMO
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.
I've changed it. I think if we usually put NEXT_MAJOR
comments next to the every file affected, so I usually understand that if there is no more instructions than Remove this method
, it's to remove the method where the NEXT_MAJOR
comment is.
BTW I had forgotten to add the comment to the implementation in tests 😬
9fdeded
to
93f94a5
Compare
These functions are only implemented in SonataDoctrineORMAdminBundle
93f94a5
to
c86f2b1
Compare
Thank you @franmomu! |
Subject
Apparently
getUniqueParameterId
andentityJoin
are only used in the ORM Bundle, in the other persistence bundles (also not maintained ones) the implementation is empty because they are not useful.I am targeting this branch, because these changes are BC.
Changelog