Bug #50144
closedHTMLParser does not remove end tag when applying rmTagIfNoAttrib
100%
Description
When I enter the following HTML in the html-mode of the RTE with a default configuration, the opening span is removed, but the closing span will stay as-is:
<table><tbody><tr><td><p><b><span>Test</span></b></p></td></tr></tbody></table>
This will save the following HTML to the database:
<table><tbody><tr><td><p><b>Test</span></b></p></td></tr></tbody></table>
Disabling rmTagIfNoAttrib for spans is a workaround, but this will leave all empty spans in the HTML code instead of cleaning them out:
RTE.default.proc.entryHTMLparser_db.tags.span.rmTagIfNoAttrib = 0
As a side note:
When editing a text content element with this incorrect HTML in IE8, the situation gets worse with every time I hit save (or just toggle between modes):
<table><tbody><tr><td><p><b>Test</span><//span></ span=""><//></ span=""><//></><//></b></p></td></tr></tbody></table>
Every click on the <> to toggle between wysiwyg and text-mode will add more junk code.
Updated by Philipp Gampe over 11 years ago
- Assignee set to Stanislas Rolland
@Stanislas please have a look
Updated by Philipp Gampe over 11 years ago
- Category set to RTE (rtehtmlarea + ckeditor)
Updated by Stanislas Rolland over 11 years ago
- Status changed from New to Accepted
This is a very nasty bug of the html parser. When all attributes are removed, the parser removes the opening tag, but it has no way to match the corresponding closing tag.
Most browser will ignore the superfluous closing tag, except IE, of course.
Updated by Stanislas Rolland over 11 years ago
- Subject changed from HTMLParser incorrectly cleans spans in tables of RTE to HTMLParser does not remove end tag when applying mTagIfNoAttrib
- TYPO3 Version changed from 6.1 to 6.0
Updated by Stanislas Rolland over 11 years ago
- Subject changed from HTMLParser does not remove end tag when applying mTagIfNoAttrib to HTMLParser does not remove end tag when applying rmTagIfNoAttrib
Updated by Gerrit Code Review over 11 years ago
- Status changed from Accepted to Under Review
Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/22681
Updated by Stanislas Rolland over 11 years ago
Please test this change: https://review.typo3.org/#/c/22681/
Updated by Gerrit Code Review about 10 years ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/22681
Updated by Gerrit Code Review almost 10 years ago
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/22681
Updated by Gerrit Code Review almost 10 years ago
Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/22681
Updated by Gerrit Code Review over 9 years ago
Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/22681
Updated by Gerrit Code Review over 9 years ago
Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/22681
Updated by Alexander Stehlik over 9 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 5e3bd676659f736da9e52967052a8db4da964a64.
Updated by Gerrit Code Review over 9 years ago
- Status changed from Resolved to Under Review
Patch set 1 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/41596
Updated by Alexander Stehlik over 9 years ago
- Status changed from Under Review to Resolved
Applied in changeset a8d0135f86a2a57ae753aa3bc5021b3654800a52.
Updated by Georg Ringer 5 months ago
- Related to Bug #95392: HTMLparser don't remove closing tag added