WBCE CMS Forum

WBCE CMS – Way Better Content Editing.

You are not logged in.

#1 14.08.2020 20:40:32

mave
Member

Import von Topics: \r\n und Beitragsbild(er)

... ist das richtig und "normal", dass bei Importen von Topics nach NWI

1.  Absatzmarkierungen als '\r\n' rüber kommen und
2. die Beitragsbilder nicht übernommen werden?

Ich frage nicht in der Erwartung, dass die Importfunktion derartige Priorität haben müßte, sondern , um den eigenen Aufwand zu klären. Danke!

Gruss
Markus

Offline

#2 15.08.2020 19:59:51

colinax
Developer

Re: Import von Topics: \r\n und Beitragsbild(er)

Hallo mave,

kannst du einen Link posten wo das mit r\n\ steht und welche Beitragsbilder meinst du?

In der Readme steht nur folgendes:

https://github.com/WBCE/News-with-Images wrote:

Importfunktion
So lange noch kein Beitrag im jeweiligen NWI-Abschnitt erstellt wurde, können Beiträge aus anderen NWI-Abschnitten, News 3.x sowie Topics automatisch importiert werden. Die Seiteneinstellungen werden mit übernommen. Beim Import von Topics-Beiträgen sind aber noch manuelle Nacharbeiten erforderlich, sofern bei Topics die "Additional Images"-Funktion genutzt wurde.

Offline

#3 15.08.2020 20:26:58

mrbaseman
Core-Team

Re: Import von Topics: \r\n und Beitragsbild(er)

Das mit den Absatzmarkierungen ist mir auch schon aufgefallen und das ist so nicht beabsichtigt gewesen.
Ich dachte das hätte mal korrekt funktioniert, aber vielleicht ist das jetzt ein Nebeneffekt einer anderen Korrektur...
Ich bin leider noch nicht dazu gekommen mir das genauer anzusehen.
Zu den Beitrsgsbildern: Additional Images nutzt glsube ich auch eine Art integrierte Gallerie, aber eine andere als die bei NWI mitgelieferten.
PS: Hab gerade keinen PC dabei und muss mir das daher für später aufheben;-)

Last edited by mrbaseman (15.08.2020 20:33:15)

Offline

#4 17.08.2020 10:00:19

webbird
Administrator

Re: Import von Topics: \r\n und Beitragsbild(er)

Liegt vielleicht an der Funktion mod_nwi_escapeString(), die wiederum $database->escapeString() kapselt, das wiederum mysqli_real_escape_string() verwendet. Glaub ich zwar nicht, könnte aber. Das ist, soweit ich das sehe, das einzige, was beim Import mit den Werten aus der Topics-Tabelle passiert.

Andere Möglichkeit: Wenn Topics als Tabellentyp myISAM benutzt und NWI InnoDB bzw. Topics als Collation was anderes als UTF-8, kann das Probleme machen.


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

Offline

#5 17.08.2020 10:52:36

webbird
Administrator

Re: Import von Topics: \r\n und Beitragsbild(er)

Jo. scheint wohl mysqli_real_escape_string zu sein:

Characters encoded are NUL (ASCII 0), \n, \r, \, ', ", and Control-Z.

-> https://www.php.net/manual/en/mysqli.re … string.php


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

Offline

#6 17.08.2020 11:02:28

webbird
Administrator

Re: Import von Topics: \r\n und Beitragsbild(er)

Also quick&dirty: import.php Zeilen 263-265:

[== PHP ==]
            $short = mod_nwi_escapeString($fetch_content['content_short']);
            $long = mod_nwi_escapeString($fetch_content['content_long']);
            $block2 = mod_nwi_escapeString($fetch_content['content_block2']);

...jeweils das mod_nwi_escapeString rausnehmen. Da der Importer aber von mrbaseman ist, wäre es mir lieb, wenn er noch was dazu sagt. big_smile

[== PHP ==]
            $short = $fetch_content['content_short'];
            $long = $fetch_content['content_long'];
            $block2 = $fetch_content['content_block2'];

Edit: Hoppla, die Zeilennummern oben sind für ein altes NWI, die für Topics wären 718 und 719. Ich glaube ich hab jetzt auch den Grund entdeckt, da wird doppelt escape-t...

Last edited by webbird (17.08.2020 11:26:02)


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

Offline

#7 17.08.2020 11:28:09

webbird
Administrator

Re: Import von Topics: \r\n und Beitragsbild(er)

So, hier mal die import.php zum Austausch.

Last edited by webbird (17.08.2020 11:29:16)


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

Offline

#8 17.08.2020 12:19:38

florian
Administrator

Re: Import von Topics: \r\n und Beitragsbild(er)

Die \r\ns kommen damit nicht mehr. Die Beitragsbilder werden allerdings noch nach der NWI4-Logik abgelegt /media/.news_img/$post_id/dateiname.jpg statt /media/.news_img/dateiname.jpg). Deshalb fehlen die dann.

Der Import aus anderen NWI-Abschnitten schlägt übrigens fehl mit der Meldung "Unknown column 'post_id' in 'field list'"

Last edited by florian (18.08.2020 12:12:20)


Wir Benötigen: Cents, Euros... jetzt spenden!

Offline

#9 17.08.2020 14:59:41

webbird
Administrator

Re: Import von Topics: \r\n und Beitragsbild(er)

Von welcher NWI-Version reden wir jetzt? Ich habe mit der mitgelieferten getestet, also 4.x.

Last edited by webbird (17.08.2020 15:00:14)


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

Offline

#10 17.08.2020 15:07:48

florian
Administrator

Re: Import von Topics: \r\n und Beitragsbild(er)

Der aus der "Branch for 5.0.0". Die Änderungen in 5.0.5 / 5.0.6 hatten ja nur das Stylesheet betroffen, und meine TNL-Integration in 5.0.7 ist vom Import / Bilderhandhabung  ja unabhängig.

In WBCE 1.4.4 ist NWI 5.0.4.

Last edited by florian (18.08.2020 12:11:52)


Wir Benötigen: Cents, Euros... jetzt spenden!

Offline

#11 19.08.2020 09:32:31

mave
Member

Re: Import von Topics: \r\n und Beitragsbild(er)

Hallo Leute,

sorry, Beispiel(link) hätte ich geliefert, war aber ein paar Tage urlaubsmäßig abseits ... Das ist aber ja nicht mehr nötig, denn hier ging's ja flott zu Werke und am original Code hätte ich ja leider nichts beizutragen gehabt. Super aber, dass Uhr da dran seid!

Offline

#12 20.08.2020 09:50:10

mave
Member

Re: Import von Topics: \r\n und Beitragsbild(er)

... ich muss nochmal nachfragen:

Wenn ich einen neuen NWI-Abschnitt anlege, werden die einzelnen Posts standardmäßig mit einer Linie getrennt. Nach einem Topics-Import (in meinem Kontext) fehlt diese.

Die import.php habe ich nun mal angeschaut, erkenne aber nicht ob/inwieweit hier auch css o.ä. angerührt wird. Stutzig werde ich darin, dass allein durch den Import die Trennlinie abhanden kommt. Könnte mir hier jemand auf die Sprünge helfen?

Offline

#13 20.08.2020 10:00:46

florian
Administrator

Re: Import von Topics: \r\n und Beitragsbild(er)

Schau mal bei den Seiteneinstellungen, beim Import wird versucht, die vorherige Darstellung halbwegs zu übernehmen. Das HTML mit dem Trennstrich
    <div class="mod_nwi_spacer"><hr /></div>
usw. wird nur bei "natürlichen" NWI-Seiten erzeugt.


Wir Benötigen: Cents, Euros... jetzt spenden!

Offline

#14 20.08.2020 10:13:15

webbird
Administrator

Re: Import von Topics: \r\n und Beitragsbild(er)

Da werden auch die Settings übernommen, guck da mal nach. Also Einstelllungen -> Beitrag Schleife und so.

Edit: Da war Florian wieder schneller. big_smile

Last edited by webbird (20.08.2020 10:14:08)


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

Offline

#15 08.09.2020 22:10:07

mrbaseman
Core-Team

Re: Import von Topics: \r\n und Beitragsbild(er)

webbird wrote:

Also quick&dirty: [...] Da der Importer aber von mrbaseman ist, wäre es mir lieb, wenn er noch was dazu sagt. big_smile

ähm, ich war in Urlaub und hab das etwas aus den Augen verloren. Florian hat's ja getestet und das doppelte Escaping war wohl der Grund. Dann muss man für NWI 5.x wohl noch Pfade anpassen. Ist das noch offen?

Offline

#16 08.09.2020 22:16:12

florian
Administrator

Re: Import von Topics: \r\n und Beitragsbild(er)

Pfade & DB-Tabellen für Bilder ist noch offen, ja,


Wir Benötigen: Cents, Euros... jetzt spenden!

Offline

Board footer

up