REDAXO Forum

Hallo,

vielleicht kann mir einer von euch bei folgendem RegEx helfen?

Aus dem String sollen folgendes enfernt werden: Leere Paragraphen-Tags, leere Paragraphen-Tags mit   sowie BRs die mehrfach hintereinander vorkommen (mit oder ohne Zeilenumbruch dazwischen). Also das hier muss alles raus (nur die 3 Ps foo/bar/batz müssen bleiben):

Code: Alles auswählen

<p></p>

<p></p>

<p>foo</p>

<p>&nbsp;</p>
<p>&nbsp;</p>

<p>bar<br />
bar</p>

<br />

<br />

<p>batz</p>

<br>
<br>

Das ganze ist für das CKEditor-Modul und sieht gerade unschönerweise und ohne die Berücksichtigung der BRs so aus: https://github.com/RexDude/ckeditor/blo ... ut.php#L19

Das RegEx sollte also mit JS laufen, aber ich glaube das ist ja eh sprachunabhängig, oder?

Danke

Hi Rudi,

versuch´s mal damit:

Code: Alles auswählen

// match all tags except lonely <br />s outside of any elements ;-(
data = data.match(/(<(?!br)(\w)[^>]*>(.*?<\/\2>)?)/igm).join("\n");
// replace empty paragraphs
data = data.replace(/(<p>(\s|&nbsp;)*<\/p>)/igm, '');

Hi Robert,

vielen Dank für deine Hilfe

...sogar mit lustigen Kommentaren versehen

Was ich gerade jetzt erste merke ist, dass der CKEditor die BRs innerhalb der Ps setzt. Deshalb scheinen deine RegExe noch nicht die richtige Wirkung zu entfalten.

Momentan ergbit diese Originalsausgabe aus dem Editor:

Code: Alles auswählen

<p>foo</p>

<p>bar</p>

<p></p>

<p><br />
<br />
batz</p>

<p>foo</p>

<p><br />
<br />
<br />
<br />
&nbsp;</p>

<p></p>

<p></p>

<p>bar</p>

<p></p>

<p>batz</p>

<p>foo</p>

<p><br />
bar<br />
batz</p>

<p></p>

<p></p>

das hier mit deinen RegExen:

Code: Alles auswählen

<p>foo</p>
<p>bar</p>
<p>
<p>foo</p>
<p>
<p>bar</p>
<p>batz</p>
<p>foo</p>
<p>
<p>foo</p>
<p>bar</p>
<p>
<p>foo</p>
<p>
<p>bar</p>
<p>batz</p>
<p>foo</p>
<p>
<p>

Das 1. batz wurde z.B. verschluckt oder auch das letzte bar und batz. Und es sind jetzt ungeschlossene Ps da

Hast noch mal ein Update?

Hi Rudi,

dein Problem kann ich nicht nachvollziehen.

Mit dieser leicht modifizierten Variante, in der ich noch die leeren s wegschneide

Code: Alles auswählen

// match all tags except lonely <br />s outside of any elements ;-(
data = data.match(/(<(?!br)(\w)[^>]*>(.*?<\/\2>)?)/igm).join("\n");
// replace empty paragraphs
data = data.replace(/(<p>(\s|&nbsp;|<br\s*\/ )*<\/p>)/igm, '');

bekomme ich folgendes Ergebnis:

Code: Alles auswählen

<p>foo</p>
<p>bar</p>

<p><br />    <br />    batz</p>
<p>foo</p>



<p>bar</p>

<p>batz</p>
<p>foo</p>
<p><br />    bar<br />    batz</p>

Siehe auch http://jsfiddle.net/tDdRc/.

Aber hast du hier nicht noch 2 BRs in deinem Ergebnis hintereinander?

Code: Alles auswählen

<p><br />    <br />    batz</p>

RexDude hat geschrieben:Aber hast du hier nicht noch 2 BRs in deinem Ergebnis hintereinander?
Code: Alles auswählen
 batz

Ja klar. Die können ja auch beabsichtigt sein, oder nicht?
Man könnte natürlich aufeinanderfolgende s noch durch ein einziges ersetzen.

Ja klar. Die können ja auch beabsichtigt sein, oder nicht?

Genau das möchte ich ja vermeiden. Hatte erst kürzlich einen Redakteur, der sich hier ganz lustige Abstände zusammengebaut hat. Dabei hab ich doch ein Abstandsmodul für diesen Fall vorgesehen

Mit Textile ist sowas ja auch nicht möglich für den normal User.

Also prinzipiell kann man folgendes sagen: alle Paragraphen die keinen normalen Text enthalten, sonder leer sind,   enthalten oder min. 1 BR enthalten sollen herausgefiltert werden. Dafür müsste eigentlich ein RexEx ausreichen, oder?

RexDude hat geschrieben:Also prinzipiell kann man folgendes sagen: alle Paragraphen die keinen normalen Text enthalten, sonder leer sind,   enthalten oder min. 1 BR enthalten sollen herausgefiltert werden. Dafür müsste eigentlich ein RexEx ausreichen, oder?

Wie gesagt, ich würde Massen- s mit einzelnen austauschen. Schließlich kommt es ziemlich oft vor, dass man einzelne Zeilenumbrüche braucht (Gedichte, zweizeilige Überschriften, Fließtexte, etc.).

Ich denke das hier, wird dir gefallen:

Code: Alles auswählen

// match all tags except lonely <br />s outside of any elements ;-(
data = data.match(/(<(?!br)(\w)[^>]*>(.*?<\/\2>)?)/igm).join("\n");
// replace leading <br>s
data = data.replace(/(<(?!br)(\w)[^>]*>)(\s*<br\s*\/?>\s*)+/igm, '$1');
// replace trailing <br>s
data = data.replace(/(\s*<br\s*\/?>\s*)+(<\/(?!br)(\w)>)/igm, '$2');
// replace empty paragraphs
data = data.replace(/(<p>(\s|&nbsp;|<br\s*\/?>)*<\/p>)/igm, '');
// replace multiple <br>s with a single one
data = data.replace(/(<br\s*\/?>\s*)+/igm, '<br />');

http://jsfiddle.net/tDdRc/5/

Edit: Hab grad gemerkt, dass das Forum meinen Code verändert hat. Deshalb haben meine ersten Regexe, nicht das gewünschte Ergebnis gebracht. Hol dir den Code am besten von jsFiddle.

Oh ja Baby, das gefällt mir

Ich schau später mal wie ich das in das CKEditor-Modul packe.

Danke Robert! ...der nächste Burger geht definitiv auf mich

Hast du hierzu auch noch ne Idee, Robert?
https://github.com/RexDude/email_obfuscator/issues/7

REDAXO Forum

[solved] RegEx Hilfe benötigt

[solved] RegEx Hilfe benötigt

Re: RegEx Hilfe benötigt

Re: RegEx Hilfe benötigt

Re: RegEx Hilfe benötigt

Re: RegEx Hilfe benötigt

Re: RegEx Hilfe benötigt

Re: RegEx Hilfe benötigt

Re: RegEx Hilfe benötigt

Re: RegEx Hilfe benötigt

Re: RegEx Hilfe benötigt

Re: RegEx Hilfe benötigt