Task #90794
closed
Add between() method to QueryInterface
0%
Description
The extbase Query objects supports between() for some time now. But it doesn't appear in code completion when writing code in an extbase repository, because QueryInterface itself does not define between(). I see no reason why it shouldn't be there.
Adding the method to the QueryInterface could possibly break classes, that implement the interface. In TYPO3 core only the Query object implements QueryInterface.
There might be third party code using that interface, so this should probably done for TYPO3 11?
Updated by Christian Eßl over 4 years ago
- Related to Feature #47812: Query support for BETWEEN added
Updated by Alexander Schnitzler over 4 years ago
- Assignee set to Stefan Froemken
- Tags set to persistence
Updated by Stefan Froemken over 4 years ago
Hello Christian,
yeah...that's correct. Some years ago I have created my own CsvQuery (QueryInterface) and CsvBackend (BackendInterface) and I'm happy to not implement an additional BETWEEN method. Further there are DBs out there which does not support BETWEEN. For me it is wrong to have BETWEEN in Query object, too. IMO BETWEEN will only work for MySQL and maybe PostgreSQL, but not for MSSQL. So BETWEEN should by part of a MySqlQuery, but not for Query-object.
I would prefer to close this ticket.
Please add an Annotation like this:
/** @var Query $query */ $query = $this->createQuery();
Stefan
Updated by Alexander Schnitzler over 4 years ago
- Status changed from New to Rejected
I would like to add that to achieve a native between, first doctrine dbal would need to support that. Then, our QOM would need to support that as well. Then we would need to parse the QOM between depending on the current DBMS. The current approach is quite alright for all DBMS'.
If you feel the need to object, please do so.