Bug #83240
closedTCA field with type=input, eval=date with range is broken, always lower range value is applied because of int cast of incoming value before ISO-8601->unix timestamp conversion.
100%
Description
Here an example TCA column from tt_products:
'endtime' => [
'exclude' => 1,
'label' => 'LLL:EXT:lang/Resources/Private/Language/locallang_general.xlf:LGL.endtime',
'config' => [
'type' => 'input',
'size' => '8',
'max' => '20',
'eval' => 'date',
'checkbox' => '0',
'default' => '0',
'range' => [
'upper' => mktime(0, 0, 0, 12, 31, 2020),
'lower' => mktime(0, 0, 0, date('m') - 1, date('d'), date('Y'))
]
]
],
This is correct according to the documentation: https://docs.typo3.org/typo3cms/TCAReference/ColumnsConfig/Type/Input.html#range
However, the core handles ranges (as integer) before it converts the date. Therefore, ranges are now broken for eval=date input columns.
This is caused by: https://docs.typo3.org/typo3cms/extensions/core/Changelog/8.5/Important-77702-CustomRenderTypesForDateAndDatetimeFieldsMustUseISO-8601.html?highlight=date
which also caused the distantly related bug https://forge.typo3.org/issues/82981
What happens is the following:
\TYPO3\CMS\Core\DataHandling\DataHandler::checkValueForInput()
checks for the TCA range and compares the $value
and $tcaFieldConf['range']['lower'] / ['upper']
both cast to "(int)".
As the incoming date is in ISO-8601, it will look like "2018-01-01T00:00:00+00:00"
. Casting it to integer to compare with the upper/lower range, the value becomes simply "2018". At this point, the lower end of the range ($tcaFieldConf['range']['lower']
) is applied to $value
.
Only afterwards, \TYPO3\CMS\Core\DataHandling\DataHandler::checkValue_input_Eval()
is called, which then attempts to convert the ISO-8601 string to a unix timestamp - but at this point we already lost our original incoming ISO-8601 string, since the lower range value was already applied.
Updated by Oliver Hader almost 7 years ago
- Subject changed from Regression: TCA field with type=input, eval=date with range is broken, always lower range value is applied because of int cast of incoming value before ISO-8601->unix timestamp conversion. to TCA field with type=input, eval=date with range is broken, always lower range value is applied because of int cast of incoming value before ISO-8601->unix timestamp conversion.
- Is Regression set to Yes
Updated by Oliver Hader almost 7 years ago
- Related to Task #77702: Use moment.js and ISO-8601 dates in the backend added
Updated by Oliver Hader over 6 years ago
- Target version changed from next-patchlevel to 8.7.11
Updated by Gerrit Code Review over 6 years ago
- Status changed from Accepted to Under Review
Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56020
Updated by Gerrit Code Review over 6 years ago
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/56020
Updated by Gerrit Code Review over 6 years ago
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/56020
Updated by Gerrit Code Review over 6 years ago
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/56020
Updated by Gerrit Code Review over 6 years ago
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/56020
Updated by Gerrit Code Review over 6 years ago
Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56020
Updated by Gerrit Code Review over 6 years ago
Patch set 1 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56060
Updated by Jigal van Hemert over 6 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 834f936808659053650574a51a398b6efb0c0e18.
Updated by Oliver Hader over 6 years ago
- Related to Bug #82981: FormEngineValidation: range-checks not working for datetime-fields added
Updated by Nikita Hovratov over 3 years ago
- Related to Bug #94621: startime/endtime are forced to "today" after saving added
Updated by Nikita Hovratov over 3 years ago
- Related to deleted (Bug #94621: startime/endtime are forced to "today" after saving)