Urls umschreiben vBulletin

[ollimagic]

Vielleicht findet jemand ja den Fehler in dem Code:

 if(THIS_SCRIPT == 'forumdisplay') {
$found = preg_match_all('/<a href="forumdisplay\.php\?f=([0-9]+)&amp;page=([0-9]+)&amp;order=desc#i', 
$GLOBALS['navbits']['breadcrumb'], $matches);
if($found)
{
$somesize = sizeof($matches[1]);
                       for($i = 0; $i <= $somesize; $i++)
                       {
                               $tid = $matches[1][$i];
$output = str_replace("forumdisplay.php?f=' . $matches[1][$i] . '&amp;page=' . $matches[2][$i] . '&amp;order=desc", 
"testlink.html", $output);

                               unset($tid, $somematch);
                       }
            }
} 

[sagg]

preg_match_all('/<a href="forumdisplay.php?f=([0-9]+)&page=([0-9]+)&order=desc#i', 

Da sehe ich einen Fehler.

[ollimagic]

und welchen

[sagg]

Die Delimiter...

[ollimagic]

meinst du die \ vor .php usw?
habe gerade gesehen, das diese in der Darstellung hier nicht da sind.
also ohne php
$found = preg_match_all('/<a href="forumdisplay\.php\?f=([0-9]+)
&amp;page=([0-9]+)&amp;order=desc#i', $GLOBALS['navbits']['breadcrumb'], $matches);
und & = & amp ; ohne Leerzeichen

[wahsaga]

Original geschrieben von ollimagic
meinst du die \ vor .php usw?

Nein, er meint das, was er schrieb: Delimiter.

[ollimagic]

das komma?
ich habe die Zeile umgebrochen wegen der darstellung.
also ganz:

  preg_match_all
('/<a href="forumdisplay\.php\?f=([0-9]+)&amp;page=([0-9]+)
   &amp;order=desc#i', $GLOBALS['navbits']['breadcrumb'], $matches); 

[sagg]

Pattern-Syntax!

[ollimagic]

Lustiges Raten.
Habe nun mal aus
&order=desc#i',
&order=desc/i',
gemacht.

[sagg]

Und?
Funktioniert es jetzt?

[ollimagic]

Ne, leider nicht.

[wahsaga]

Original geschrieben von ollimagic
Lustiges Raten.

Nicht unser Problem, wenn dir nichts anderes bleibt - weil du die Grundbegrifflichkeiten nicht kennst.

[ollimagic]

Ne ist ja in Ordnung, aber das funktioniert eben noch nicht.
Der falsche Delimiter war es eben nicht alleine

[sagg]

Dann solltest du nun das tun, was du vorhin schon von selbst erkannt hast, noch ein paar Slashes rein werfen.
Um genau zu sein sind es nur 2.

Und dann mach bitte noch das "+" raus, dann müsste es gehen.