TYPO3 Core

closed as duplicate. already solved.

Georg Ringer wrote:

closed as duplicate. already solved.

Hi Georg,

I don't see, that the actual type ot the date is taken in account in the latest version.

https://github.com/TYPO3/TYPO3.CMS/blob/0bd1746384980d9f6f7387d8638d280443ee843b/typo3/sysext/extbase/Classes/Persistence/Generic/Storage/Typo3DbQueryParser.php

In line 414 and following there is still no "columnMap" as second parameter provided to the createTypedNamedParameter method:

case QueryInterface::OPERATOR_EQUAL_TO:
                if ($value === null) {
                    $expr = $fieldName . ' IS NULL';
                } else {
                    $placeHolder = $this->createTypedNamedParameter($value);
                    $expr = $exprBuilder->comparison($fieldName, $exprBuilder::EQ, $placeHolder);
                }
                break;

If the value is of type DateTime it still gets converted to a unixttimestamp (as a fallback) by createTypedNamedParameter method.
So in my eyes, this bug is not resolved an due to this, DateTime Values cannot be used straigt forward in the database and the extbase repositories methods.

All best,
Uwe

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/58950

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/58950

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/58950

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/58950