WBCE CMS Forum

webbird

Administrator

Re: JQuery Meldung im Modul Forum

Hm ich wüßte nicht dass ich an Umbrüchen was geändert hätte... Ich schau mal.

---

Ich habe eine Amazon-Wishlist. Oder spende an das Projekt.
Ich kann, wenn ich will, aber wer will, dass ich muss, kann mich mal

webbird

Administrator

Re: JQuery Meldung im Modul Forum

Ich kann's zwar reproduzieren, momentan aber noch nicht erklären. Auf den ersten Blick verwenden beide Funktionen $database->escapeString() und tun das auch nicht doppelt. Ich installier' nochmal die alte Version zum Vergleich.

Anbei der Fix für das "Jupp". Das ist halt so meine Debugging-Methode...

Beitrag geändert von webbird (24.08.2020 11:27:08)

---

Ich habe eine Amazon-Wishlist. Oder spende an das Projekt.
Ich kann, wenn ich will, aber wer will, dass ich muss, kann mich mal

webbird

Administrator

Re: JQuery Meldung im Modul Forum

Also die thread_reply.php ruft die content.php auf.

[== PHP ==]
define('FORUM_DISPLAY_CONTENT', 'reply_thread');
define('PAGE_CONTENT', WB_PATH . '/modules/forum/content.php');

require WB_PATH . '/index.php';

In der thread_reply.php ist $_POST['text'] noch okay, wenn's in der content.php ankommt, ist es schon einmal escape't. Also scheint irgendeine interne WBCE-Komponente den Inhalt von $_POST zu verwurschteln. Da dann innterhalb der content.php nochmal escape't wird, kommen die \r\n dabei raus.

---

Ich habe eine Amazon-Wishlist. Oder spende an das Projekt.
Ich kann, wenn ich will, aber wer will, dass ich muss, kann mich mal

florian

Administrator

Re: JQuery Meldung im Modul Forum

Hm. Die Outputfilter sind's nicht. Gibt wohl irgendwo einen Konflikt zwischen WBCE-Corefunktionen und class.forum_parser.php oder class.subway.php.

Ich muss aber ganz ehrlich sagen, dass das für mich gerade nur Prio 3 hat.  Wenn ich mir was wünschen darf, würde ich mich freuen, wenn Du Dich erstmal dem NWI-Problem mit den Bildern/Tabellen annehmen könntest, das ist momentan der Showstopper für die 1.4.5.

---

Code allein macht nicht glücklich. Jetzt spenden!

florian

Administrator

Re: JQuery Meldung im Modul Forum

Etwas inkonsequent, ich habe mir das jetzt doch angeschaut - ich blicke da mindestens genau so wenig durch wie bei NWI.
Was ich herausgefunden habe:
wenn in der content.php die Zeilen 635/636 und 779/780 auskommentiert werden, title und post also nicht escaped werden, also so

        //$_POST['title'] = $database->escapeString($_POST['title']);
        //$_POST['text'] = $database->escapeString($_POST['text']);

tritt das \r\n-Problem nicht mehr auf und es passiert augenscheinlich auch sonst nichts Schlimmes.

Die Posts werden sonst escaped in die DB geschrieben und dann - vermute ich - beim Laden wohl nochmal escaped.

Beitrag geändert von florian (25.08.2020 10:57:25)

---

Code allein macht nicht glücklich. Jetzt spenden!

webbird

Administrator

Re: JQuery Meldung im Modul Forum

Dann ist es aber vielleicht nicht mehr kompatibel mit WB Classic. Mir ist das ja egal, aber dann wird das sozusagen ein "offizieller Fork", der dementsprechend dann auch hier gepflegt werden muss.

NWI ist eine größere Baustelle, dafür habe ich definitiv nicht genug Zeit.

---

Ich habe eine Amazon-Wishlist. Oder spende an das Projekt.
Ich kann, wenn ich will, aber wer will, dass ich muss, kann mich mal

florian

Administrator

Re: JQuery Meldung im Modul Forum

Das \r\n-Problem tritt mit WB 2.12.2 r379 auch auf. 

Edit: Und auch mit WBCE 1.3.3.

Beitrag geändert von florian (26.08.2020 12:39:24)

---

Code allein macht nicht glücklich. Jetzt spenden!

webbird

Administrator

Re: JQuery Meldung im Modul Forum

Mich würde aber schon interessieren wo das schon mal "escape-t" wird. Ist ja eventuell auch für andere Module interessant.

---

Ich habe eine Amazon-Wishlist. Oder spende an das Projekt.
Ich kann, wenn ich will, aber wer will, dass ich muss, kann mich mal

florian

Administrator

Re: JQuery Meldung im Modul Forum

Ich glaube, es liegt wirklich nur am Forenmodul. Escaped wird sowohl in der class.subway als auch der content.php.
https://github.com/WebsiteBaker-modules … ay.php#L32
https://github.com/WebsiteBaker-modules … t.php#L680

---

Code allein macht nicht glücklich. Jetzt spenden!

webbird

Administrator

Re: JQuery Meldung im Modul Forum

Dann bin ich beruhigt.

---

Ich habe eine Amazon-Wishlist. Oder spende an das Projekt.
Ich kann, wenn ich will, aber wer will, dass ich muss, kann mich mal

webbird

Administrator

Re: JQuery Meldung im Modul Forum

Ungetestet.

---

Ich habe eine Amazon-Wishlist. Oder spende an das Projekt.
Ich kann, wenn ich will, aber wer will, dass ich muss, kann mich mal

florian

Administrator

Re: JQuery Meldung im Modul Forum

gerade getestet (updated; edit: auch bei Neuinstallation), sehr puristischer output

Beitrag geändert von florian (27.08.2020 10:18:40)

---

Code allein macht nicht glücklich. Jetzt spenden!

florian

Administrator

Re: JQuery Meldung im Modul Forum

hier dann ohne den Debug-Output.
Scheint zu funktionieren. 

---

Code allein macht nicht glücklich. Jetzt spenden!

chap

Mitglied

Re: JQuery Meldung im Modul Forum

danke auch mir hat das Update was gebracht.

---

Liebe Grüße,
cHAp

florian

Administrator

Re: JQuery Meldung im Modul Forum

Danke für die Rückmeldung. Ich habe den Eintrag im AOR aktualisiert.
https://addons.wbce.org/pages/addons.ph … em&item=57

---

Code allein macht nicht glücklich. Jetzt spenden!