Problem mit BBCode in search.php für post-results gelöst?

[schmidtedv]

Ich habe mir mal erlaubt ein bißchen zu basteln und bin nun soweit, das auch die Suche nach Beiträgen diese meinen Wünschen entsprechend anzeigt und habe sogar da noch den Acronym-Mod mit reingepackt...meine Frage zu der Stelle wäre nur, ob ich damit irgendein Sicherheitsloch aufmache oder so, denn ich wüßte nicht, warum das phpBB sonst gekürzte Beiträge so verstümmelt darstellt...

Code: Alles auswählen

if ( $show_results == 'posts' )
			{
				if ( isset($return_chars) )
				{
					$bbcode_uid = $searchset[$i]['bbcode_uid'];

					if ( $return_chars != -1 )
					{
					if ( $bbcode_uid != '' )
						{
						//$message = strip_tags($message);
						//$message = preg_replace("/\[.*?:$bbcode_uid:?.*?\]/si", '', $message);
						//$message = preg_replace('/\[url\]|\[\/url\]/si', '', $message);
						$message = ( $board_config['allow_bbcode'] ) ? bbencode_second_pass($message, $bbcode_uid) : preg_replace("/\:$bbcode_uid/si", '', $message);
						}
						$message = ( strlen($message) > $return_chars ) ? substr($message, 0, $return_chars) . ' ...' : $message;
						$message = acronym_pass( $message );
					}
					else
					{
						if ( !$board_config['allow_html'] )
						{
							if ( $postrow[$i]['enable_html'] )
							{
								$message = preg_replace('#(<)([\/]?.*?)(>)#is', '<\2>', $message);
							}
						}

						if ( $bbcode_uid != '' )
						{
							$message = ( $board_config['allow_bbcode'] ) ? bbencode_second_pass($message, $bbcode_uid) : preg_replace("/\:$bbcode_uid/si", '', $message);
						}

						$message = acronym_pass( $message );

						$message = make_clickable($message);

[oxpus]

Nö, der Code ist i.O...

[AmigaLink]

Ich vermute mal das die phpBB-group dies nicht so gemacht hat um das Board ein wenig zu entlasten. Denn die Suchfunktion ist schon recht heftig.

[oxpus]

Das stimmt. Und wenn man sich den Speed eines vanilla phpBB im Vergleich zu anderen Board anschaut, ist das mittlerweile leicht verstaubte Konstrukt doch immer noch sehr fix unterwegs Eben genau aus diesem Grunde wurden solche Dinge nicht programmiert. Kompromiss zur Geschwindigkeit eben.
Aber wir haben ja findige User hier, die sowas entdecken

[schmidtedv]

nun, für mich stellt sich das problem mit einem dicken Board ja nicht direkt...da macht es, wenn ich auch nachvollziehen kann, das die korrekte BBCode-Formatierung mehr Belastung erzeugt, mehr Sinn, wenn der Benutzer sich nicht wundert, was ihm da so angezeigt wird. Bzw. Sinn machen würde jetzt mal wieder, aber die Schleife da einbauen kommt als nächstes, eine ACP-Option die suche betreffend, ob Suchresultate als Topic oder Post angezeigt werden sollen...und, im Sinne von Amigas Post, die Ansicht als Topics wäre dann weniger belastend, gelle?

[oxpus]

Ja, als Topic ist es ja auch okay, denn dort wird ja kein Post-Text angezeigt.
Bei der Ergebnisdarstellung als Post ist es aber schon kniffliger, da die Angaben ja auch noch abgeschnitten werden.
Dabei kann schon einmal eine Formatierung zerstört werden (sofern man diese nicht wieder umständlich erkennen und "umschiffen" will), was dann auch wieder zu unschönen Darstellungen führt.
Und das kann dann auch bei kleineren Boards ein Stück bremsen...

[schmidtedv]

...mit der kürzung sieht's zumindest bei mir klasse aus...selbst Code-Felder sehen gut aus, also zwar mit Rahmen, aber in der Vertikalen einfach abgeschnitten und da im Code dann das ... am Ende, kann mich vom Ästhetischen her nicht beklagen

Jetzt muß ich da nur noch die show_post.php mit einbauen für Zitate bei denen [Beitrag anzeigen] beisteht, aber das hab ich in der topic_review.php hinbekommen, sollte da auch gehen...