Bug #25302
closed
Broken links representation gets saved to the database
Added by Oliver Hader over 13 years ago.
Updated about 6 years ago.
Category:
RTE (rtehtmlarea + ckeditor)
Description
Imagine that you have a content element with a link to a internal page that does not exist anymore. On editing this content element, RTEhtmlarea marks this link with yellow background and a red border.
On saving the form without touching the content in the RTE at all, the styles used for representing this broken link are saved to the database and then get wrapped by an addition span tag.
So, on saving again one will end out in something like this:
< p >< em >
< span style="background-color: yellow; border:2px red solid; color: black;" >
< a style="background-color: yellow; border:2px red solid; color: black;" rteerror="No page found: 2913" title="Opens internal link in current window" class="internal-link" href="http://dev45.local/?id=99999" >TEST< /a >
< /span >
< /em >< /p >
(issue imported from #M17917)
Files
This is a tricky one. The call to TS_transform_db() in t3lib_parsehtml_proc will add the span tag around the anchor. The call to TS_links_db() would have removed the rteerror and style attributes of that broken anchor tag. However, since TS_links_db() is called after TS_transform_db() the span tag is already there.
So my patch just changes the order of processing the modes, to have ts_links before ts_transform/css_transform. I'm not an expert on the ParseHtml class and thus also don't know about negative side-effects.
- Subject changed from Broken links representation gets save to the database to Broken links representation gets saved to the database
- Category set to RTE (rtehtmlarea + ckeditor)
- Target version changed from 0 to 1061
- TYPO3 Version set to 4.4
- Target version changed from 1061 to 1264
- Assignee set to Stanislas Rolland
- PHP Version set to 5.3
Patch set 1 of change I0f517b9f01deec05c9d31661a5fcf13f60ea0d37 has been pushed to the review server.
It is available at http://review.typo3.org/4351
Patch set 1 of change I1f68a0f9a67934b9ac3288117f831219e8d30f99 has been pushed to the review server.
It is available at http://review.typo3.org/4352
Patch set 1 of change I025670f54fd8e78e8d06b03edf926470f8dce76a has been pushed to the review server.
It is available at http://review.typo3.org/4353
- Target version changed from 1264 to 1341
- Status changed from New to Under Review
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
- Target version deleted (
1341)
- Status changed from Resolved to Closed
Also available in: Atom
PDF