[TYPO3-core] RFC #7984: Bug: stdWrap.crop now closes opened tags and counts chars correctly

[Steffen Kamper]

Hi Jochen,

as promised here my review:

I added missing spaces. Also i removed count($splittedContent) from 
for-loop and replaced by prefilled
$countSplittedContent = count($splittedContent);

Code looks good, but i'm not sure about the charset handling and 
html_entity_decode(). But this has to be improved, so it's no 
showstopper for me.

Now the Test: failed with current conditions:

Used bodytext with disabled RTE and following snippet:
tt_content.text.20.crop = 80| ... |0

I used a text from a webpage:

<h2>20 Jahre Mauerfall</h2>
<h3><a 
href="http://www.spiegel.de/wirtschaft/soziales/0,1518,645335,00.html" 
title="20 Jahre Mauerfall: Ost und West driften wieder auseinander">Ost 
und West driften wieder auseinander</a></h3>
		<div class="spArticleImageBox spAssetAligncenter" style="width: 522px;">
	<a 
href="http://www.spiegel.de/wirtschaft/soziales/0,1518,645335,00.html" 
title="20 Jahre Mauerfall: Ost und West driften wieder auseinander"><img 
src="http://www.spiegel.de/images/image-7729-panoV9free-sjxk.jpg" 
alt="20 Jahre Mauerfall: Ost und West driften wieder auseinander" 
title="20 Jahre Mauerfall: Ost und West driften wieder auseinander" 
align="center" border="0" height="248" 
width="520"></a></div><p>Jahrelang näherten sich die Einkommen in Ost 
und West an - jetzt kehrt sich die Entwicklung wieder um: Laut einer 
Studie des Wirtschaftsinstituts DIW ist Deutschland auch 20 Jahre nach 
dem Mauerfall ökonomisch geteilt. Die gefühlten Unterschiede sind sogar 
noch größer als die tatsächlichen. <span class="spAutorenzeile">Von 
Anselm Waldermann</span> <a 
href="/wirtschaft/soziales/0,1518,645335,00.html" title="20 Jahre 
Mauerfall: Ost und West driften wieder auseinander">mehr...</a> <span 
class="spInteractionMarks">[&nbsp;<a 
href="http://forum.spiegel.de/showthread.php?t=8409">Forum</a>&nbsp;]</span></p>

	<ul class="spLinkList">
	<li><img 
src="http://www.spiegel.de/static/sys/v9/icons/ic_listbullet.gif" alt="" 
height="4" width="4"><a href="/fotostrecke/fotostrecke-45888.html" 
title="Alle Grafiken im Überblick:  So denken die Ostdeutschen ">Alle 
Grafiken im Überblick:<span> So denken die Ostdeutschen</span></a></li>
	<li><img 
src="http://www.spiegel.de/static/sys/v9/icons/ic_listbullet.gif" alt="" 
height="4" width="4"><a 
href="/politik/deutschland/0,1518,645260,00.html" title="SPD-Kandidat 
Maas:  Die CDU spielt sich als Staatspartei auf ">SPD-Kandidat 
Maas:<span> "Die CDU spielt sich als Staatspartei auf"</span></a></li>
	<li><img 
src="http://www.spiegel.de/static/sys/v9/icons/ic_listbullet.gif" alt="" 
height="4" width="4"><a 
href="/politik/deutschland/0,1518,645059,00.html" 
title="Koalitionsspiele im Wahlkampf:  Warum Merkel die Große Koalition 
II meiden sollte ">Koalitionsspiele im Wahlkampf:<span> Warum Merkel die 
Große Koalition II meiden sollte</span></a></li>

	<li><img 
src="http://www.spiegel.de/static/sys/v9/icons/ic_listbullet.gif" alt="" 
height="4" width="4"><a 
href="http://www.manager-magazin.de/unternehmen/artikel/0,2828,645208,00.html" 
title="Tagesticker:  Längste Gewinnserie im Dow seit 2007 
">Tagesticker:<span> Längste Gewinnserie im Dow seit 2007</span> <span> 
(manager-magazin.de)</span>

PROBLEM
========

this is the gemerated HTML:

<!--  CONTENT ELEMENT, uid:15/text [begin] -->
		<div id="c15" class="csc-default" >
		<!--  Text: [begin] -->
			<h2>20 Jahre Mauerfall</h2>
<h3><a href="http://www.spiegel.de/wirtschaft/sozial...
		<!--  Text: [end] -->

			</div>
	<!--  CONTENT ELEMENT, uid:15/text [end] -->

The content is corrupt, so no +1 until this is fixed, sry.

vg Steffen
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: bug_7984_v6_typo3_src-trunk.diff
Url: http://lists.netfielders.de/pipermail/typo3-team-core/attachments/20090828/e65d1b5e/attachment.txt