Bug #25302

Broken links representation gets saved to the database

Added by Oliver Hader about 10 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Should have
Category:
RTE (rtehtmlarea + ckeditor)
Target version:
-
Start date:
2011-03-10
Due date:
% Done:

100%

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

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

0017917_screenshot.png (36.2 KB) 0017917_screenshot.png Administrator Admin, 2011-03-10 12:20
0017917.patch (770 Bytes) 0017917.patch Administrator Admin, 2011-03-11 17:49
#1

Updated by Oliver Hader about 10 years ago

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.

#2

Updated by Oliver Hader about 10 years ago

  • 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
#3

Updated by Oliver Hader almost 10 years ago

  • Target version changed from 1061 to 1264
#4

Updated by Stanislas Rolland over 9 years ago

  • Assignee set to Stanislas Rolland
  • PHP Version set to 5.3
#5

Updated by Mr. Hudson over 9 years ago

Patch set 1 of change I0f517b9f01deec05c9d31661a5fcf13f60ea0d37 has been pushed to the review server.
It is available at http://review.typo3.org/4351

#6

Updated by Mr. Hudson over 9 years ago

Patch set 1 of change I1f68a0f9a67934b9ac3288117f831219e8d30f99 has been pushed to the review server.
It is available at http://review.typo3.org/4352

#7

Updated by Mr. Hudson over 9 years ago

Patch set 1 of change I025670f54fd8e78e8d06b03edf926470f8dce76a has been pushed to the review server.
It is available at http://review.typo3.org/4353

#8

Updated by Oliver Hader over 9 years ago

  • Target version changed from 1264 to 1341
#9

Updated by Stanislas Rolland over 9 years ago

  • Status changed from New to Under Review
#10

Updated by Stanislas Rolland over 9 years ago

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

Updated by Ernesto Baschny almost 8 years ago

  • Target version deleted (1341)
#12

Updated by Benni Mack over 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF