Bug #70246

Story #69617: FormEngine bugs

RTE fields do not handle eval validation like required

Added by Andreas Allacher over 4 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
FormEngine aka TCEforms
Target version:
Start date:
2015-10-01
Due date:
% Done:

100%

TYPO3 Version:
7
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

With current 7.5 the RichTextElement does not add the validation rules and therefore no FormEngineValidation like required is done.


Related issues

Related to TYPO3 Core - Bug #21650: Cannot set an RTE field as required Closed 2009-11-23
Related to TYPO3 Core - Bug #78918: Pasting long text into the RTE leads to browser crashes Closed 2016-12-08

Associated revisions

Revision 848b2fc0 (diff)
Added by Frank Naegler over 4 years ago

[BUGFIX] RTE fields do not handle eval validation like required

Resolves: #70246
Releases: master
Change-Id: I3e536ab85740b58e07f1b262692bf3a9773edd62
Reviewed-on: https://review.typo3.org/44810
Reviewed-by: Andreas Fernandez <>
Tested-by: Andreas Fernandez <>
Reviewed-by: Christian Kuhn <>
Tested-by: Christian Kuhn <>

History

#1 Updated by Andreas Allacher over 4 years ago

However, it is a bit more complex as the Editor currently doesn't update the input textarea on the fly but only on save, so FormEngineValidation would the validate the rte field directly or the editor also needs to update the input field

#2 Updated by Christian Kuhn over 4 years ago

  • Parent task set to #69617

yes. pretty nasty js thing, frank didn't had a simple solution for this when validation was rewritten. maybe we could have another look now. iirc, there is a prepared but commented example in styleguide for this case already in the required test record.

#3 Updated by Mathias Schreiber over 4 years ago

  • Category changed from Backend User Interface to FormEngine aka TCEforms

#4 Updated by Christian Kuhn over 4 years ago

  • Assignee set to Frank Naegler

#5 Updated by Gerrit Code Review over 4 years ago

  • Status changed from New 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/44810

#6 Updated by Gerrit Code Review over 4 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/44810

#7 Updated by Gerrit Code Review over 4 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/44810

#8 Updated by Frank Naegler over 4 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#9 Updated by Riccardo De Contardi over 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF