Project

General

Profile

Actions

Bug #21485

closed

Multiple assigned classes in Text Style get lost in HTML cleaner.

Added by Schmidt Timo about 15 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Should have
Category:
RTE (rtehtmlarea + ckeditor)
Target version:
-
Start date:
2009-11-06
Due date:
% Done:

0%

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

Description

  • Open the RTE
  • Create a paragraph and apply multiple text styles to a part of the paragraph.
  • During the saving process the classes get lost even if they are allowed.
  • The same seems to work for a complete paragraph.

I've figured out that there are two problems:

In the Method HTMLparserConfig the list of allowed
tags will be overwritten with an expoded string.

Line 1256 ad 1257:

if (strcmp($keepTags[$key]['fixAttrib'][$atName]['range'],'')) $keepTags[$key]['fixAttrib'][$atName]['range'] = t3lib_div::trimExplode(',',$keepTags[$key]['fixAttrib'][$atName]['range']);
if (strcmp($keepTags[$key]['fixAttrib'][$atName]['list'],'')) $keepTags[$key]['fixAttrib'][$atName]['list'] = t3lib_div::trimExplode(',',$keepTags[$key]['fixAttrib'][$atName]['list']);

But at runtime $keepTags[$key]['fixAttrib'][$atName]['range'] contains an array, not a string. Therefor The Sting "Array" is afterwards content of $keepTags[$key]['fixAttrib'][$atName]['list'].

In my opinion we need to check first if the value is realy a string and if
not, we shouln't do anything:

Second Problem:

In the method HTMLcleaner the handling of a list need to be enhanced to allow the user to use MultipleListItems. To solve this i've introduced an configuration option "allowMultipleListItems"

(issue imported from #M12496)


Files

class.t3lib_parsehtml.php.patch (2.4 KB) class.t3lib_parsehtml.php.patch Administrator Admin, 2009-11-06 16:49

Related issues 1 (0 open1 closed)

Is duplicate of TYPO3 Core - Bug #22900: Span tags with multiple classes being removedClosedStanislas Rolland2010-06-16

Actions
Actions

Also available in: Atom PDF