Bug #101286
closed
Epic #99669: CKEditor5 Collection
RTE validation broken when field is required
0%
Description
Hey guys,
when declaring a field in TCA as RTE and adding the option required => true, the validation is broken:
TCA:
'content' => [
'exclude' => true,
'label' => 'LLL:EXT:my_ext/Resources/Private/Language/locallang_db_event.xlf:content.label',
'config' => [
'type' => 'text',
'enableRichtext' => true,
'required' => true,
],
],
SQL schema:
...
content text NULL DEFAULT NULL
...
When creating a new record, the field is not marked as invalid, even as it should be (because of required => true):
After putting some content in, it gets marked as invalid. Thus the record cannot be saved:
If you need feedback, just reach out. Thanks for looking into this and fixing/support.
Files
Updated by Christoph Erdmann over 1 year ago
Just noticed, that the record gets saved, when saving is triggered via onChange => 'reload' on another property.
Updated by Christoph Erdmann over 1 year ago
Update after 12.4.3:
Main issue is fixed. Thanks.
Didn't notice, there was already an issue for that as it is closed (https://forge.typo3.org/issues/100838). Sorry.
Another issue remains:
When the RTE field is 'required' => true and a new record is created, it is possible to save the record without filling out the required field. The exclamation mark indicator for failed validation is missing in this case to.
When editing an existing record and emptying the field, the validation works as expected.
Updated by Benjamin Franzke about 1 year ago
- Related to Bug #100838: RTE field not marked as required added
Updated by Benjamin Franzke about 1 year ago
- Related to Bug #98538: FormEngine state not updated anymore with CKEditor5 added
Updated by Georg Ringer 8 months ago
- Has duplicate Bug #103533: TCA type=text + enableRichtext=true cannot be required added
Updated by
Updated by Garvin Hicking 5 months ago
Since the field is nullable via its SQL definition, have you tried specifying default => '' in TCA? Or could changing the SQL type to "not null default ''" change this?
Updated by Christoph Erdmann 5 months ago
I checked with TYPO3 12.4.16 right now and it seems that all issues are fixed.
Updated by Garvin Hicking 5 months ago
- Status changed from New to Closed
Great, closing the issue then, thanks for reporting back! :)
Updated by cosmoblonde GmbH 26 days ago
This issue needs to be re-opened as it started breaking again with 12.4.17!
I have tested and can confirm it was working with 12.4.16 and before.
Updated by Garvin Hicking 26 days ago
Please open a follow-up/fresh issue. Please tell a minimal reproducible example, especially what your TCA 'default' option is set to.
Updated by cosmoblonde GmbH 25 days ago
· Edited
Meanwhile, I have found the cause for this issue due to commit https://github.com/typo3/typo3/commit/e9904c0e8d (which is was part of release 12.4.17) to avoid race conditions.
I have added a comment to that commit and created a new ticket #105505
Updated by Garvin Hicking 25 days ago
- Related to Bug #105505: Required field validation broken on RTE fields (12.4.17) added