Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Updated: Comment #0
Problem/Motivation
After implementing #1941672: Remove Drupal\views\Plugin\query\QueryInterface there is still @var \Drupal\views\Plugin\query\QueryInterface
referenced in ViewExecutable
.
Proposed resolution
Replace \Drupal\views\Plugin\query\QueryInterface
with \Drupal\views\Plugin\views\query\QueryPluginBase
Remaining tasks
-
User interface changes
-
API changes
-
Comment | File | Size | Author |
---|---|---|---|
#1 | vardoc-views-queryinterface-2143111-1.patch | 523 bytes | derhasi |
Comments
Comment #1
derhasi CreditAttribution: derhasi commentedAnd there is the small patch.
Comment #2
longwaveComment #3
webchickNice find!
Committed and pushed to 8.x. Thanks!
Comment #4
jhodgdonIs this actually right? We normally want to have @var reference an interface if possible, not a base class?
Comment #5
longwave\Drupal\views\Plugin\query\QueryInterface was removed in #1941672: Remove Drupal\views\Plugin\query\QueryInterface. In that issue it is pointed out that Views generally does not provide interfaces, and in practice you will always extend the base class, so I think it is safe to reference that base class here.
Comment #6
jhodgdonOK. Thanks.