Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

unerwünschter Absatz // empty paragraph #45

Closed
IngoWinter opened this issue Jul 5, 2018 · 9 comments
Closed

unerwünschter Absatz // empty paragraph #45

IngoWinter opened this issue Jul 5, 2018 · 9 comments
Labels
bug Something isn't working help wanted Extra attention is needed vendor issue

Comments

@IngoWinter
Copy link
Member

Wenn ich ein Modul mit einer leeren cke eingabemaske speichere, haut er mir ein <p>&nbsp;<\/p> in die Datenbank.

@skerbis
Copy link
Member

skerbis commented Jul 5, 2018

ckeditor/ckeditor5#401

@IngoWinter
Copy link
Member Author

rex_extension::register('OUTPUT_FILTER', function ($ep) {
    if (!rex::isBackend())
    {
        $ep->setSubject(str_replace('<p>&nbsp;</p>', '', $ep->getSubject()));
    }
});

Dann sollte zumindest sowas in die boot.php, bis die ihren Arsch hochkriegen. Dieser sinnlose Absatz zerhaut einem komplett die Abstände.

@skerbis
Copy link
Member

skerbis commented Jul 5, 2018

na ja, das löst zwar das Problem in der Ausgabe, jedoch nicht in der db. Man könnte es in Modulen durch eine Aktion abfangen. Oder ein js entfernt es aus dem input. Aber wozu braucht man ein leeres Textfeld?

@skerbis skerbis added the bug Something isn't working label Jul 5, 2018
@skerbis skerbis changed the title unerwünschter Absatz unerwünschter Absatz // empty paragraph Jul 5, 2018
@IngoWinter
Copy link
Member Author

Ich habe ein recht umfangreiches "Universalmodul". Mehrspaltigkeit, Text, Bild, Galerie, Video, Formular, Downloads etc. Da habe ich sehr oft leere Textfelder. Ich könnte mir auch andere Fälle sehr gut vorstellen (optionale Bildunterschriften zb).
Natürlich wäre es besser, wenn der Rotz gar nicht erst in der Datenbank landet - aber für die JS Lösung bin ich nicht tief genug im Thema und mir fehlt jegliche Motivation (hab Ohrensausen und Flashbacks von deinem verlinkten Issue bekommen).
Die OPF Lösung ist halt bombensicher und schnell gemacht. Aktion im Modul wäre auch sauber, allerdings von jedem Entwicker für jedes Modul zu setzen.

@skerbis
Copy link
Member

skerbis commented Jul 5, 2018

@IngoWinter wenn die bei CK nix finden, werden wir es hier. Den Filter hier in die boot zu setzen ändert ja leider nichts am Problem. Aber es ist ein Trick oder ein Workaround, den wir erläutern können.

@joachimdoerr joachimdoerr added the help wanted Extra attention is needed label Feb 9, 2019
@joachimdoerr
Copy link
Member

Sollen wir hier wirklich etwas bauen? Ich denke das ist die Aufgabe des Moduls per preSaveAction so etwas zu bereinigen.

@skerbis
Copy link
Member

skerbis commented Feb 9, 2019

Na ja, da klappt aber nich im YForm oder Meta-Kontext. if($dings=='') funktioniert halt nicht

@joachimdoerr
Copy link
Member

es gibt ein pullrequest ckeditor/ckeditor5-engine#1656 ggf. löst sich das problem beim nächsten release.

@joachimdoerr
Copy link
Member

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed vendor issue
Projects
None yet
Development

No branches or pull requests

3 participants