php problem :/

[cart]

ich hab folgendes replace:

$news = preg_replace( "/\[li\](.*?)\[\/li\]/", "<li>$1</li>", $news);


das problem ist, dass es nicht funktioniert, wenn zwishen
[li] und [/li] ein zeilenumbruch ist.
plz help

 

[gim~li]

Dann musst du vorher aus $news alle Zeilenumbrüche entfernen ...

 

[cart]

lol? und wie bekomme ich sie dann wieder?

das hier kommt übrigens ganz am anfang von dem kompletten replace script:

$news = nl2br(stripslashes($daten['news']));

danach kommen div replaces u.a. auch das [li] ding.

 

[Wiseguy3]

Du brauchst den Switch "s" - der matcht über mehrere Zeilen.

So also:

$news = preg_replace( "/\[li\](.*?)\[\/li\]/s", "<li>$1</li>", $news);

Übrigens würde ich auch noch "i" und "U" als Switch nehmen. Also besser so:

$news = preg_replace( "/\[li\](.*?)\[\/li\]/isU", "<li>$1</li>", $news);

Dann hast du keine Probleme mehr mit Gross und Kleinschreibung der Tags, Zeilenumbrüchen und zu "gierigem" Suchen...

 

[cart]

ich hatte es jetzt zwar so gelöst
$news = preg_replace('/'.preg_quote('[li]','/').'(.*?)'.preg_quote('[/li]','/').'/is', '<li>$1</li>', $news);
aber deine lösung ist doch deutlich weniger text

 

[Wiseguy3]

Deine Lösung ist auch problematisch, wenn du

[li]erster listeneintrag[/li]
[li]zweiter listeneintrag[/li]

hast.

Dein Regexp mach daraus nämlich:

<li>erster listeneintrag[/li]
[li]zweiter listeneintrag</li>

weil es greedy matcht.

 

[cart]

eigentlich machte deine lösung das, wenn ich das U mit drin hatte... wenn das raus war gings. aber mit preg_quote() gings einwandfrei. war halt nur fast doppelt soviel text^^ ich habs jetzt durchgängig für den kompletten code so:
$news = preg_replace( "/\[li\](.*?)\[\/li\]/si", "<li>$1</li>", $news);

 

[BiBaButzemann]

machs doch einfach mit str_replace() ...

 

[Dari]

Original geschrieben von BiBaButzemann
machs doch einfach mit str_replace() ...

bei str_replace kannst nur string teile komplett ersetzen nicht aber geschlossende code tags wie es in foren, news oder kommentar scripts geused wird

 

[BiBaButzemann]

er will doch nur sein [li] durch <li> ersetzen lassen und das geht doch ganz prima mit str_replace().

 

[Wiseguy3]

Nein, will er nicht.

 

[Dari]

er will [li] durch <li> nur dann ersetzen wenns ebenfalls durch [/li] abgeschlossen wird, sonst kann man gleich html dem user zur verfügung stellen damit er alles schrottet

 

[The_Company]

Was macht die Regexp aus "[li] 1 [li] 2 [/li] 3 [/li]" ?

Vermutlich "<li> 1 [li] 2 </li> 3 [/li]", oder?

 

[Wiseguy3]

Original geschrieben von The_Company
Was macht die Regexp aus "[li] 1 [li] 2 [/li] 3 [/li]" ?

Vermutlich "<li> 1 [li] 2 </li> 3 [/li]", oder?

Ja, könnte so sein. Ist aber immer das Problem bei Regexp, weil sie den Status des aktuellen Strings nicht speichern (schon geöffnet, noch nicht geöffnet).

Da hilft nur n Parser...

 

[cart]

die einzige lösung, ausser meiner preg_quote()-lösung, die funktioniert ist diese:
$news = preg_replace( "/\[li\](.*?)\[\/li\]/si", "<li>$1</li>", $news);

da macht er auch net aus

[li] 1 [li] 2 [/li] 3 [/li] -> <li> 1 [li] 2 </li> 3 [/li]

biba: str_replace gibt mir aber keine überprüfungsmöglichkeit für [/li].

 

[BiBaButzemann]

jojo, hab nur halbherzig geschaut...

nehme alles zurück und behaupte das gegenteil.

 

[cart]

akzeptiert