Project

General

Profile

Actions

Bug #23334

closed

htmlArea RTE: W3C Markup Validation issues

Added by Frank Frewer over 14 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Should have
Category:
-
Target version:
-
Start date:
2010-08-02
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

In conjunction with extensions based an extbase / fluid the RTE produces tags with ID's which are not valid:
<div id="pleasewaittx_meyxt_pi1[myobject][myfield] ...
<div id="editorWraptx_meyxt_pi1[myobject][myfield] ...
<textarea id="RTEareatx_meyxt_pi1[myobject][myfield] ...

The W3C Markup Validator says:
character "[" is not allowed in the value of attribute "id"

(issue imported from #M15326)


Files

Actions #1

Updated by Stanislas Rolland about 14 years ago

The attached patch fixes the issue. Please confirm.

Note that square brackets are allowed in id in HTML 5.

Actions #2

Updated by Frank Frewer about 14 years ago

The patch is okay. No validator error at this point anymore.
Thank you very much.

But there are two other validator errors which have nothing to do with extbase / fluid:
- required attribute "rows" not specified
- required attribute "cols" not specified

The html output:
<textarea id="RTEareatx_myext_pi1_myobject__myfield__1" name="tx_myext_pi1[myobject][myfield]" style="position:relative; left:0px; top:0px; height:380px; width:460px; border: 1px solid black;">

Should I open a new issue for this?

Actions #3

Updated by Stanislas Rolland about 14 years ago

The attributes are added with zero values by second version of the patch.

Actions #4

Updated by Stanislas Rolland about 14 years ago

Please confirm that the second version of the patch fixes both issues.

Actions #5

Updated by Frank Frewer about 14 years ago

The second version of the patch fixes both issues.
Thank you very much.

Actions #6

Updated by Stanislas Rolland about 14 years ago

Thanks for testing. Committed to SVN TYP3core trunk (revision 8758) and branch TYPO3_4-4 (revision 8759).

Actions #7

Updated by Stanislas Rolland about 14 years ago

Attached follow-up patch committed to ensure that the id of the textarea is always valid:

trunk (revision 8778)
branch TYPO3_4-4 (revision 8779)

Thanks to Jigal van Hemert.

Actions #8

Updated by Benni Mack about 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF