Class Drupal\Core\Entity\Query\QueryFactory does not implement
Drupal\Core\Entity\Query\QueryFactoryInterface interface.

#2019651: Add a QueryFactoryInterface for QueryFactory classes seems related.

Comments

EdgarPE created an issue. See original summary.

EdgarPE’s picture

EdgarPE CreditAttribution: EdgarPE commented
Assigned: EdgarPE » Unassigned
Issue summary: View changes

The method implementations are incompatible, so I'm not sure how to fix this.

markdorison’s picture

markdorison
he/him
CreditAttribution: markdorison at Chromatic commented
Issue summary: View changes
Status: Needs work » Closed (works as designed)

Based on the discussion in #2019651: Add a QueryFactoryInterface for QueryFactory classes, it seems like this is intentional:

We don't usually add interfaces for factory classes. Our one exception is Drupal\Component\Plugin\Factory\FactoryInterface, but that's because plugins can have swappable factories.

You are totally right, Let's remove any changes to the factory factory class (Drupal\Core\Entity\Query\QueryFactory).