Extended Quote Tag MOD und modcp.php

[schmidtedv]

Deine phpBB Version: Vanilla phpBB 2.0.18
MODs: Ja
Dein Wissensstand: Fortgeschritten
Link zu Deinem Forum: [url]http://[/url]

PHP Version:
MySQL Version:


Was hast Du gemacht, bevor das Problem aufgetreten ist?
-


Was hast Du bereits versucht um das Problem zu lösen?
-



Fehlerbeschreibung und Nachricht

Ist jemandem mal aufgefallen, das der MOD so nicht in der modcp.php funktionieren kann, da er versucht, das popup mit der modcp.php zu erzeugen...kurz gesagt, link ist fehlerhaft in diesem fall...?

gibt's da eventuell einen Lösungsansatz zu? :-)

[oxpus]

Öh, der funkt doch hier bereits wunderprächtig...
Oder reden wir von unterschiedlichen MODs???

[schmidtedv]

Ich denke es ist der gleiche...über die viewtopic.php geht der bei mir auch wunderbar...nur in der Moderatoren-Ansicht hab ich das ja auch automatisch mit drin und da geht der Link dann nicht mehr....

[oxpus]

Moderatorenansicht? Dort sollte doch eigentlich kein BBCode umgesetzt werden

[schmidtedv]

:-) wir kommen der sache näher...bei mir hab' ich den BBCode drin (möchte den ja auch noch in die search.php für results drin haben), also ansicht wie in der viewtopic...nur eben der extended mod verliert hier seine wirksamkeit, da er als popup-link einen falschen verweis bildet...

[oxpus]

Wieso eigentlich falsch? Er sollte doch irgend eine Datei öffnen. Und das modcp liegt doch auch im selben Verzeichnis wie die viewtopic.php...

[schmidtedv]

Ja, aber der Link den er verwendet ist so etwas: .../aphrodite/modcp.php?mode=split&t=5&sid=2eaa62bc0d5b3b1d27d1be65d8e4ed80# (also mit mode=split)

[oxpus]

Öhm, soll der Link nicht die show_...php irgendwas aufrufen?

[schmidtedv]

Jain, er macht vielmehr das # :-) mit jvascript...also ...dell4550/aphrodite/viewtopic.php?t=5# z.B. oder im Original #_somewhat....aber in der modcp klappt das irgendwie net...hast Du das bei dir ganz raus genommen mangels bbcode? oder sagen wir mal so, vielleicht liegts daran...der aufruf in der viwtopic verwendet kein sid........ im link, wie man sieht

[oxpus]

AAAAAAAAAAAAAAAAAAAAAlso:
Ich habe in der modcp.php kein BBCode drinnen, weil das per Default auch nicht der Fall ist.

Das Problem liegt dann auch hier:

Code: Alles auswählen

	$bbcode_tpl['quote_post_open'] = str_replace('{U_VIEW_POST}', '<a href="#_somewhat" onClick="javascript:open_postreview( \'' . $temp_url . '\' );" class="genmed">' . $lang['View_post'] . '</a>', $bbcode_tpl['quote_post_open']);

Das ist die Aufbereitung des Links in der bbcode.php.
Vielleicht sollte das mal so erstellt werden:

Code: Alles auswählen

	$bbcode_tpl['quote_post_open'] = str_replace('{U_VIEW_POST}', '<a href="javascript:open_postreview( \'' . $temp_url . '\' );" class="genmed">' . $lang['View_post'] . '</a>', $bbcode_tpl['quote_post_open']);

Dann ist zwar der Link zu sehen, aber es würde eher funktionieren...

[schmidtedv]

...leider nicht, da er zwar auch die show_post.php mit der richtigen beitragsnummer nimmt aber es scheinbar keinen (hm, wie drück ich das aus) direkten verweis gibt....mit anderen worten kein popup und Fehler im IE...ich denke mal, das show_post.php nicht weiß, sofern über die modcp aufgerufen, wo sie diesen beitrag suchen/finden soll oder so ähnlich...?

EDIT: gehen wir's mal anders an, ich schau den MOD grad' nochmal durch und denke ich muss was verbauen wie in der topic_review.php...bin grad' dabei...allerdings, kann es sein, das hier ein kleiner fehler drin ist?:

Code: Alles auswählen

#-----[ AFTER, ADD ]--------------------------------------- 
# 
			$plain_message = $row['post_text'];
			$plain_message = preg_replace('/\:(([a-z0-9]:)?)' . $bbcode_uid . '/s', '', $plain_message);
			$plain_message = str_replace('<', '<', $plain_message);
			$plain_message = str_replace('>', '>', $plain_message);
			$plain_message = str_replace('<br />', "\n", $plain_message);

			$orig_word = array();
			$replacement_word = array();
			obtain_word_list($orig_word, $replace_word);

			if ( !empty($orig_word) )
			{
				$plain_message = ( !empty($plain_message) ) ? preg_replace($orig_word, $replace_word, $plain_message) : '';
			}
			$plain_message = addslashes($plain_message);
			$plain_message = str_replace("\n", "\\n", $plain_message);

er setzt hier $replace_word ein...aber müßte es nicht $replacement_word sein ?

[oxpus]

Nein. Ersetz doch einfach

Code: Alles auswählen

$replacement_word = array(); 

durch

Code: Alles auswählen

$replace_word = array(); 

Und zum Link:
Ist diese Zeile etwa bei Dir in der bbcode.php nicht enthalten?
Oder klappt der Link weiterhin nicht?
Dann wäre vielleicht hier die viewtopic.php mit einzutragen...

[schmidtedv]

Hm, nein? an anderer stelle wird immer replacement_word genommen und nicht replace_word...ich dachte, das bezieht sich darauf...also z.B. in der modcp.php gibts ne ähnliche stelle und da steht auch replacement_word....wenn ich hier replacement_word nehme, scheint zumindest nicht's falsches zu passieren...oder kannst du mir sagen, WARUM hier replace_word stehen MUSS?

ansonsten hab' ich den fehler gefunden (extreme styles nutzer, lol)

ich mußte nur in die tpl noch oben

Code: Alles auswählen

<script language="Javascript" type="text/javascript">
<!--
function open_postreview(ref)
{
	height = 450;
	width = 650;
	window.open(ref, '_phpbbpostreview', 'HEIGHT=' + height + ',WIDTH=' + width + ',resizable=yes,scrollbars=yes');
	return;
}
//-->
</script>

einsetzen, seufz...hatte bei mir ausnahmsweise mit dem original-mod wenig zu tun....


NUN aber an dieser stelle doch mal was bbcode-mäßiges...wenn ich in einem normalen beitrag auf QUOTE gehe und darin ist "code" enthalten, setzt er mir folgendes ein:

Code: Alles auswählen

[quote="Florian Schmidt";p="13"][code]<td class="row-post-date">{postrow.POST_DATE} ({postrow.POST_NUMBER}/{postrow.POST_REPLIES})</td>
	<td class="row-post-buttons post-buttons"><span class="img-main">{postrow.PROFILE_IMG}</span>&nbsp;{postrow.PM_IMG} {postrow.EMAIL_IMG} {postrow.WWW_IMG}</td>
</tr>

[/quote][/code]

mache ich das aus der topic_review heraus ergibt das:

Code: Alles auswählen

[quote="Florian Schmidt";p="13"]
[code]<td class="row-post-date">{postrow.POST_DATE} ({postrow.POST_NUMBER}/{postrow.POST_REPLIES})</td>
	<td class="row-post-buttons post-buttons"><span class="img-main">{postrow.PROFILE_IMG}</span>&nbsp;{postrow.PM_IMG} {postrow.EMAIL_IMG} {postrow.WWW_IMG}</td>
</tr>

[/quote][/code]

also zwei ganz unterschiedliche bbcode-auflösungen und letztere ist nicht wirklich das, was ich möchte...da ich bbcode nicht wirklich verstehe, wäre es dir möglich da mal drauf zu schauen...?

[oxpus]

Nein, was Du nimmst, ob $replacement_word oder $replace_word, ist egal. Hauptsache es wird definiert, erstellt und dann verwendet. Und das durchgängig. Es könnte auch komplett $ersetzungen heissen.

Zum Javascript:
Egal, wie es aussieht, hier wird das Popup aufgerufen.
ref beinhaltet dazu den Link, den es aus der Seite bekommt.
Und eben in der bbcode.php wird (zumindest bei mir) der Link zusammengebaut.
Halt bei Dir mit dem modcp.php-Link-Teil, aber man kann in der bbcode.php ja auch viewtopic.php... fest vorgeben

[schmidtedv]

....siehe mein erweitertes EDIT oben :-))

[oxpus]

Nochmal:
Gibt es in Deiner bbcode.php diese Zeile:

Code: Alles auswählen

$bbcode_tpl['quote_post_open'] = str_replace('{U_VIEW_POST}', '<a href="#_somewhat" onClick="javascript:open_postreview( \'' . $temp_url . '\' );" class="genmed">' . $lang['View_post'] . '</a>', $bbcode_tpl['quote_post_open']);

?????????????????????

[schmidtedv]

Ja, ich mußte da bezüglich des popup's ja doch nichts ändern....bei mir wird das popup über die zusätzliche javascript-zeile in der tpl aufgerufen, da lag der fehler....was mir nun nur noch fehlt...warum der mod das auch immer so umsetzt mit der sog. $plain_message, ist bei einem zitat die korrekte übernahme des posts...das

Code: Alles auswählen

$plain_message = $row['post_text'];
			$plain_message = preg_replace('/\:(([a-z0-9]:)?)' . $bbcode_uid . '/s', '', $plain_message);
			$plain_message = str_replace('<', '<', $plain_message);
			$plain_message = str_replace('>', '>', $plain_message);
			$plain_message = str_replace('<br />', "\n", $plain_message);

			$orig_word = array();
			$replacement_word = array();
			obtain_word_list($orig_word, $replacement_word);

			if ( !empty($orig_word) )
			{
				$plain_message = ( !empty($plain_message) ) ? preg_replace($orig_word, $replacement_word, $plain_message) : '';
			}
			$plain_message = addslashes($plain_message);
			$plain_message = str_replace("\n", "\\n", $plain_message);

Code: Alles auswählen

'PLAIN_MESSAGE' => str_replace(chr(13), '', $plain_message),

zerbröselt ja das ganze zitat (zumindest wenn auch "code" drin ist...


EDIT: Mal so nebenbei...an sich bin ich ja recht stolz, das das kleine phpBB-Projekt so langsam ziemlich gute Formen annimmt, bzw. nicht mehr viel fehlt das auch alle kleinen fehler etc. hoffentlich weg sind (z.B. unseren alten feind, die mass_mail, muss ich noch hinkriegen, aber da komme ich nochmal die tage drauf, lol)

Zur Zeit sind's halt hier und da die BBcode umformatierungen, die mich nerven...und zu guter letzt will ich noch versuchen mehr sachen über das acp aus/ein-schalten zu können...

[oxpus]

Von wo nach wo willst Du denn $plain_message übergeben?

[schmidtedv]

Naja, in der topic_review wird das ganze ja für's zitieren verwendet...also antworten-->unten review-->beitrag z.B. mit code-->zitieren, und plopp wird in das antwortformular oben nur blödsinn eingesetzt...

[oxpus]

Die muss dann aber doch nicht aus der modcp.php übergeben werden. Das übernimmt doch dafür die show_post.php...

[schmidtedv]

ähm...auf welchen post bezieht sich deine antwort? die sache mit topic_review und dem bbcode oder das popup-fenster?

[oxpus]

Auf DEINEN letzten Beitrag.
Die topic_review muss selbstverständlich ja den Text übergeben, damit er angezeigt wird.
Ansonsten macht alles die show_post.php.
Also wäre der Text doch der show_post.php aus der modcp.php garnicht erst zu übergeben...