Double Post Merge: Smilies deaktiviert

Beitrag von **Holger** » Mo 09.Jun, 2008 12:44

Hi,

ich benutze den Double Post Merge Mod.
Sobald damit aber ein Beitrag mit einem vorhergehenden zusammengefügt wird, sind die Smilies deaktiviert.
Warum?

Wenn ich den neuen, zusammengeschlagenen, Beitrag editiere, ist das Kästchen "Smilies deaktivieren" nicht markiert.

Gruss
Holger

Beitrag von **oxpus** » Mo 09.Jun, 2008 14:34

Ich hatte den MOD auch nur auf einem meiner Foren installiert, allerdings mich nicht wirklich um die Funktionsweise gekümmert.
Da müsste ich erst mal Zeit für finden, dem nachzugehen, was wann wie wo gemacht wird, damit alles wieder passt.

Könnte es sein, daß einer der beiden Posts, die zusammengefahren wurden, die Smilies deaktiviert wurden?
Vielleicht auch "nur" ein Einbaufehler...

Beitrag von **Holger** » Mo 09.Jun, 2008 14:44

Ne, ich behaupte mal selbstbewusst: beides nein.

Beitrag von **oxpus** » Mo 09.Jun, 2008 14:50

Wie gesagt: Ich kenne den MOD "intern" so gut wie garnicht, kann also hierzu aktuell weniger sagen...

Beitrag von **Holger** » Do 19.Jun, 2008 10:39

Ich suche mal weiter, dieser Bug nervt nämlich.

In den Instructions kommt folgendes vor:

Code: Alles auswählen

			$last_message_row = $db->sql_fetchrow($result);
			$bb_uid = ':' . $last_message_row['bbcode_uid'];
			
			// i don't know much about parsing the message, so it may well be that,
			// there are some wrongs, anyway all seems to work fine
			$last_message = str_replace($bb_uid, '', $last_message_row['post_text']);
			$last_message = preg_replace('/\:[0-9a-z\:]+\]/si', ']', $last_message);
			$last_message = undo_htmlspecialchars(addslashes($last_message));

Kann da was im Argen liegen?

/Holger

Beitrag von **oxpus** » Do 19.Jun, 2008 11:09

Anstelle der Zeilen

Code: Alles auswählen

			$last_message = str_replace($bb_uid, '', $last_message_row['post_text']);
			$last_message = preg_replace('/\:[0-9a-z\:]+\]/si', ']', $last_message);
			$last_message = undo_htmlspecialchars(addslashes($last_message));

würde ich eher diese hier verwenden:

Code: Alles auswählen

			$unhtml_specialchars_match = array('#>#', '#<#', '#"#', '#&#');
			$unhtml_specialchars_replace = array('>', '<', '"', '&');
			
			$last_message = preg_replace($unhtml_specialchars_match, $unhtml_specialchars_replace, last_message);
			
			if ($bb_uid != '')
			{
				$last_message = preg_replace("/\:(([a-z0-9]:)?)$bb_uid/si", '', $last_message);
			}

Beitrag von **Holger** » Do 19.Jun, 2008 11:19

Ok, aber hat das was mit dem Smilies zu tun?

Zusätzliche Info:
Wenn ich einen Beitrag schreibe, werden die Smilies angezeigt.
Schiebe ich einen Beitrag nach, auf den der Mod anspringt ist das Ergebnis ein Beitrag, in dem der erste Teil keine Smilies parst, der zweite Teil jedoch normal die Smilies anzeigt.

Beitrag von **oxpus** » Do 19.Jun, 2008 11:24

Ok, aber hat das was mit dem Smilies zu tun?

Richtig, war aber dennoch nicht korrekt.

Und nochmal:

Ich kenne den MOD "intern" so gut wie garnicht, kann also hierzu aktuell weniger sagen...

Beitrag von **Holger** » Do 19.Jun, 2008 11:29

Extrem komisch.
Ich werde Deine Code-Änderung mal testen.
So sieht das aus, wenn ich mehrere Beiträge nacheinander schreibe.
Komischerweise klappt es manchmal, manchmal nicht, manchmal nur teilweis

Sehr irritierend, hänge noch ein Beispiel an.
Im Zitat ist alles ok, im ersten Post (mit Zitat) wird der Smily nicht mehr richtig angezeigt.

EDIT: habe jetzt festgestellt, dass es nur passiert, wenn der Beitrag mit einem Smiley endet! Und wenn dann der Double Post Merge zur Anwendung kommt.

Beitrag von **oxpus** » Mo 15.Sep, 2008 10:37

Hm, dann müsste dem Text vor dem Speichern noch ein Leerzeichen angefügt werden, damit beim Formatieren zur Anzeige genau das Smiley wieder angezeigt werde kann...

Beitrag von **Holger** » Mo 15.Sep, 2008 10:44

Ah! Ich kann ja die Stelle mit der Grafik und "Tillägg efter x minut(er)" erweitern.
Das wird jetzt im Moment mit einem /n<img erledigt
Ich könnte da ja {blank}/n<img draus machen ...

EDIT: hmmmmm ... andere Stellen haben auch keinen Blank nach dem Smiley, dort wird er richtig angezeigt ...

Wäre es ev. sinnvoll diesen zu verwenden?
http://phpbbguru.net/mods/?download&id=89
Scheint auch mit dem Mod Autor (Mercuree) zu tun zu haben.

Beitrag von **oxpus** » Mo 15.Sep, 2008 13:09

Wäre es ev. sinnvoll diesen zu verwenden?

Kann ich nicht beurteilen, da ich beide nicht, bzw. nicht besonders gut kenne.

Es ist immer die Frage, welches Ende-Zeichen nach einem Smiley am Post-Ende gespeichert wird.
Wenn die Double Post Merge MOD hier ein anderes Zeichen angibt, als durch einen normalen Post, kann die Formatierung der Smileys eben in die Hose gehen.
Dann muss man eben ein Blank einfügen, damit alles wie gewollt klappt.
Daran wird vermutlich die andere Modifikation auch nichts ändern...

Beitrag von **Holger** » Mo 15.Sep, 2008 13:24

Komisch ist doch nur, dass der "zerstörte" Smiley mitten drin liegt, oder? Der Smiley, der mit dem 2. Post geschrieben wurde, ist ja korrekt!

Beitrag von **oxpus** » Mo 15.Sep, 2008 18:33

Wenn der Smiley am Ende des 1. Textes liegt, würde die MOD stur den Text direkt dranhängen und damit das Smiley zerstören.
Besser eben erst ein Leerzeichen und dann den 2. Post anfügen

Beitrag von **Holger** » Mo 15.Sep, 2008 20:12

Ich bin so ein Idiot!
Beim Einbau und modifizieren hatte ich

Code: Alles auswählen

		$separator = "\n\n[img]http://www.maskinisten.net/images/plus.gif[/img] [size=9][color=#FF0000]Tillägg efter";

Anstatt

Code: Alles auswählen

		$separator = " \n\n[img]http://www.maskinisten.net/images/plus.gif[/img] [size=9][color=#FF0000]Tillägg efter";

Man bemerke das Leerzeichen vor \n\n!!!

Jetzt geht es!!!!!!!!

ÜBRIGENS!
Dein Hinweis auf den fehlerhaften Code in Beitrag 6 hat dazu geführt, dass der erste Beitrag nur noch aus dem Text "last_message" bestand ...

Beitrag von **oxpus** » Mo 15.Sep, 2008 23:04

Man bemerke das Leerzeichen vor \n\n!!!

Sag ich doch

Dein Hinweis auf den fehlerhaften Code in Beitrag 6 hat dazu geführt, dass der erste Beitrag nur noch aus dem Text "last_message" bestand ...

Sorry, war aber auch ungetestet...