Forum

[Markus.Staab]

Hi zusammen,

kann noch jemand bestätigen, dass mit der Anpassung von Nicole das Problem nicht mehr auftritt?

Meiner Meinung nach müsste diese Lösung aber auch noch weiterhin fehlerhaft sein, da auf (.) ja auch eine Zahl trifft.. oder?

Müsste es nicht statt (.){1}, dann [^0-9]{1} heissen?

Viele Grüße,
Markus

[Markus.Staab]

Hi nochmal zusammen,

ich hab das ganze mal ein bisschen getestet und meiner Meinung nach, müsste das ganze so richtig sein:

Code: Alles auswählen

  function replaceLinks($content)
  {
    // -- preg match redaxo://[ARTICLEID]-[CLANG] --
    preg_match_all('@redaxo://([0-9]*)\-([0-9]*)([^0-9]|$)+/?@imU',$content,$matches,PREG_SET_ORDER);
    foreach($matches as $match)
    {
      if(empty($match)) continue;

      $url = rex_getURL($match[1], $match[2]);
      $content = str_replace($match[0],$url.$match[3],$content);
    }

    // -- preg match redaxo://[ARTICLEID] --
    preg_match_all('@redaxo://([0-9]*)([^0-9]|$)+/?@imU',$content,$matches,PREG_SET_ORDER);
    foreach($matches as $match)
    {
      if(empty($match)) continue;

      $url = rex_getURL($match[1], $this->clang);
      $content = str_replace($match[0],$url.$match[2],$content);
    }

    return $content;
  }

Bitte mal testen bei euch...

Wer sich mit RegEx auskennt, bitte mal drüber schauen.

Viele Grüße,
Markus

[n.ruediger]

Was gefällt dir an meiner Lösung nicht?

[Markus.Staab]

Hi Nicole,

schau mal hier

Meiner Meinung nach müsste diese Lösung aber auch noch weiterhin fehlerhaft sein, da auf (.) ja auch eine Zahl trifft.. oder?

Bist du anderer Meinung?

Ein Fehler ist auf jeden Fall noch drinn:
Das erste

Code: Alles auswählen

$content = str_replace($match[0],$url.$match[2],$content);

muss

Code: Alles auswählen

$content = str_replace($match[0],$url.$match[3],$content);

heissen, da $match[2] schon die clang ist.

Gruß,
Markus

[n.ruediger]

Hallo Markus,

nein, der Ausdruck wird nicht auf eine Zahl treffen, da alle Zahlen vom vorhergehenden Ausdruck geparst werden. Man sollte dazu vielleicht auch betrachten, dass das Zeilenende als ein Zeichen angesehen wird...

Liebe Grüße,
Nicole

[Markus.Staab]

Hi Nicole,

alles klar, mit dem vorher genannten fix, gehts dann auch..

Ich check das jetzt mal ein.

Fix bitte testen:
[1]

Gruß,
Markus

[Michael83]

Hallo,

ich wollte nur kurz anmerken, dass das Problem in der aktuellen Version 4.1.0 weiterhin auftritt. Ich habe auch hier den Code von Nicole eingesetzt und es funktioniert wieder.

[Ingo]

Ich bin gerade über diesen Fehler gestolpert. Interne Links in Textile nach dem Schema "Link":redaxo://9 ergeben <a href="domain.tld/artikeslug/9">Link</a>. In einer aktuellen 4.3.2er Installation.
Nachdem ich den regulären Ausdruck von Nicole eingesetzt habe funktioniert es. Wie kann denn das nach 4 Jahren sein? Setzt denn sonst keiner interne Links?

[jeandeluxe]

Setzt denn sonst keiner interne Links? :mrgreen:

Zuhauf.., aber ohne probs ;-)

lg,
Jan