Einbauproblem

[pervamon]

Beim Einbau des CPS ist beim viewtopic.php folgende Anweisung

Code: Alles auswählen

      if ( $bbcode_uid != '' )
      {
         $message = ( $board_config['allow_bbcode'] ) ? bbencode_second_pass($message, $bbcode_uid) : preg_replace('/\:[0-9a-z\:]+\]/si', ']', $message);
      }

Nur gibt es diese Stelle bei mir nicht. Eine ähnliche Stelle geht so:

Code: Alles auswählen

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

Ich habe ja schon ein paar Mods eingebaut, aber meiner Erinnerung nach habe ich da nichts rumgeschraubt.

[oxpus]

Das wäre aber genau die Stelle, die Du auch hierfür ersetzen müsstest.

[pervamon]

Das habe ich angenommen. Beim ersetzen wird ja dann das andere genommen. Muss ich da bedingt ersetzen oder was bringt dieser Unterschied?

[oxpus]

Wenn Du das nicht ersetzt, wird der BBCode auch nach dem Cachen des Posts weiterhin ausgeführt. Und damit wäre an dieser Stelle nichts gewonnen...

[pervamon]

Du hast meine Frage falsch verstanden. Mit bedingt ersetzen habe ich gemeint, den Teil, der (weiter innen in der neuen if-Abrage) unverändert wieder vorkommt, nach 'meinem' Muster zu schreiben.
Was ist eigentlich genau der Unterschied zwischen dem Codestück das gesucht wird und dem, das ich gefunden habe? / Welches ist besser?

[oxpus]

Ach so!
Ja, sorry...

Öhm, beides erreicht das gleicht, nämlich die BBCODE-UID zu finden und entsprechend zu handeln.
Es wäre also egal, welche der beiden Varianten Du an dieser Stelle nimmst.

[pervamon]

Ist eine Variante besser / weniger rechenintensiv?

[oxpus]

Nein. preg_match ist in diesen beiden Fällen gleich schnell.

[Son]

Hallo,
ich bin gerade bei das CPS System einzubauen und komme in der viewtopic.php nicht weiter :/

laut anleitung soll das gemacht werden

Code: Alles auswählen

#
#-----[ FIND ]----------------------------------------------------------------
#

//
// Go ahead and pull all data for this topic
//

#
#-----[ FIND ]----------------------------------------------------------------
#

//
// Okay, let's do the loop, yeah come on baby let's do the loop
// and it goes like this ...
//

#
#-----[ FIND ]----------------------------------------------------------------
#

   $message = $postrow[$i]['post_text'];

#
#-----[ REPLACE WITH ]--------------------------------------------------------
#

// Cache Posts
   $parsed = ($board_config['enable_post_cache']) ? $postrow[$i]['parsed'] : 0;
   if (!$parsed || !$board_config['enable_post_cache'])
   {
      $message = $postrow[$i]['post_text'];
   } else {
      $cstyle = ($userdata['session_logged_in'] && $userdata['user_style'] != $board_config['default_style'] && $board_config['cache_each_template']) ? $userdata['user_style'].'_' : '';
      $cachefile = 'cache/posts/'.$cstyle.'post_'.$postrow[$i]['post_id'].'.gz';
      if (file_exists($cachefile) && filemtime($cachefile) >= $postrow[$i]['post_edit_time'] && $postrow[$i]['cache_file_md5'] == md5_file($cachefile))
      {
         $handle = @fopen($phpbb_root_path.$cachefile, 'r');
         $message = gzinflate(@fread($handle, @filesize($cachefile)+1));
         @fclose($handle);
      } else {
         @unlink($cachefile);
         $message = $postrow[$i]['post_text'];
         $parsed = 0;
      }
   }
// Cache Posts
//   $message = $postrow[$i]['post_text'];

ist da einmal ein FIND zu viel oder fehlt da was ?

[AmigaLink]

Warum haben eigentlich so viele leute Probleme mit dieser Stelle

Die ersten beiden FINDs sind eine Suchhilfe!
Wenn du die Anweisung genau befolgst (also der reihe nach die 3 angaben suchst und dann die letzte Fundstelle gegen den neuen Code austauschst), dann ist es (selbst bei stark veränderten Boards) nahezu Unmöglich das die falsche stelle bearbeitet wird.

Das ist eine vollkommen normale Vorgehensweise bei MODs die schwer einzubauen sind, weil die zu bearbeitende stelle evtl. mehrmals im Code vorkommt (aber nur einmal bearbeitet werden darf) oder schwer zu finden ist (weil sie anderen stellen sehr ähnlich ist).
Ich habe mich für diese art der Anweisung entschieden, weil letzteres der Fall sein kann.