core.cs: output.replace funktioniert das eigentlich?

2 views
Skip to first unread message

Thomas

unread,
Aug 16, 2007, 9:36:00 AM8/16/07
to SharpCms (DE)
In der Datei core.cs gibt es eine Funktion, die "Bad-Tags" löschen
soll. Unter Bad-Tags versteht man dabei wohl z.B. <ul />

ur leider habe ich so das gefühl, dass das nicht wirklich
funktioniert. Ich konnte bisher auch nicht herausfinden, wieso es
nciht funktioniert.

Jürgen Gutsch

unread,
Aug 16, 2007, 9:48:39 AM8/16/07
to sharp...@googlegroups.com
Hi Thomas,
 
hast du das mal durchdebuggt?

 
Am 16.08.07 schrieb Thomas <thoma...@gmx.net>:

Thomas

unread,
Aug 16, 2007, 10:43:48 AM8/16/07
to SharpCms (DE)
Ja, habe ich. Das ist ja das seltsame. Es kommt dabei keine
Fehlermeldung.

Die Funktion heisst "Parse" (Zeile 53). Wenn ich nun absichtlich ein
<ul /> setzte, müsste diese Funktion dies löschen "output.replace()".
Tut es aber nicht. Die Funktion wird ausgeführt aber das Replace
übersieht das </ul>. Echt seltsam.

On 16 Aug., 15:48, "Jürgen Gutsch" <juergen.gut...@googlemail.com>
wrote:


> Hi Thomas,
>
> hast du das mal durchdebuggt?
>


> Am 16.08.07 schrieb Thomas <thomas3...@gmx.net>:


>
>
>
>
>
> > In der Datei core.cs gibt es eine Funktion, die "Bad-Tags" löschen
> > soll. Unter Bad-Tags versteht man dabei wohl z.B. <ul />
>
> > ur leider habe ich so das gefühl, dass das nicht wirklich
> > funktioniert. Ich konnte bisher auch nicht herausfinden, wieso es

> > nciht funktioniert.- Zitierten Text ausblenden -
>
> - Zitierten Text anzeigen -

Thomas

unread,
Aug 16, 2007, 1:38:47 PM8/16/07
to SharpCms (DE)
Ok, peinlich, dass ich das nicht selber bemerkt habe :-)

Da steht in der zeile 68 doch glatt das hier:

foreach (string a in badtags)
{
output.Replace(a, "");
}


Richtig wäre hier wohl

foreach (string a in badtags)
{
output = output.Replace(a, "");
}

> > - Zitierten Text anzeigen -- Zitierten Text ausblenden -

Jürgen Gutsch

unread,
Aug 16, 2007, 2:19:58 PM8/16/07
to SharpCms (DE)
Hi Thomas,

ich hab den Bug mal weitergegeben.
Du kannst es ja bei dir korrigieren, bis eine neue Version draussen
ist.

> > - Zitierten Text anzeigen -- Hide quoted text -
>
> - Show quoted text -

Reply all
Reply to author
Forward
0 new messages