Bug #18535
closedsupport html-tags in crop function of stdWrap
0%
Description
current text crop function doesn't take html tags in account.
example:
temp.mytext = TEXT
temp.mytext.field = text
temp.mytext.crop = 150|...|1
(possible) html output:
lorem ipsum dolor <b>sit amet...
needs a logical parsingfunction which closes open html-tags
(issue imported from #M7984)
Files
Updated by Jochen Rau over 16 years ago
The patch solves the decribed problem and ensures the correct char count (chars of tags ar not counted; entities are decoded).
Updated by Jochen Rau over 16 years ago
The new patch (bug_7984_v2_typo3_4-1-6.diff) takes the "link" tag into account. Thanks to Boris for testing.
Updated by Jochen Rau over 16 years ago
The new patch (bug_7984_v3_typo3_4-1-6.diff) solves a problem (no content was shown, if the number of chars is larger than the content length).
Updated by Jochen Rau over 16 years ago
The new patch (bug_7984_typo3_4-2-0RC1.diff) closes cropped tags, respects entities and is unit-tested (CropTest.php). For plain text the patched version of crop behaves exactly as the original version.
Updated by Michiel Roos over 16 years ago
Boris!
Please post this as a RFC to the core list!
Updated by Jochen Rau over 16 years ago
I posted an RFC on April 4th 2008 on typo3-team-core@lists.netfielders.de .
Updated by Michiel Roos over 16 years ago
Well Jochen,
That's a nice discussion! Good work.
For posterity: http://lists.netfielders.de/pipermail/typo3-team-core/2008-April/015278.html
So, now we wait. Dmitry is already Monitoring, Stucki will look at it this evening, Patrick has just escaped to the US . . . .
Updated by Dmitry Dulepov over 16 years ago
Re-posted the patch to the core list. Now it uses cropHTML property.
Updated by Michiel Roos about 15 years ago
So here we are one year later, and we have a 'relationship'.
Nice!
:-)
Updated by Jochen Rau about 15 years ago
This was mainly to bubble the thing out again. I was a little bit exhausted after the lengthy discussion on the core list. But I am currently preparing a post to sum up the discussion. I really want this feature get into 4.3.