Folgende Übersetzungen enthalten unerlaubtes HTML welches nicht importiert werden kann.
https://localize.drupal.org/translate/languages/de/translate?sid=2514073
https://localize.drupal.org/translate/languages/de/translate?sid=2514075

Bitte erstmal entfernen. Ich werde ein Follow-up im Slick Modul erstellen.

CommentFileSizeAuthor
#8 slick bug.png28.33 KBjoachim namyslo

Comments

volkerk created an issue. See original summary.

volkerk’s picture

joachim namyslo’s picture

Hab ich schon gesehen, ich weiß nur leider noch nicht so ganz, wie ich das ganze fixen soll und ob es sich lohnt, hier in Slick einen Bug auf zu machen, ohne eine Lösung parat zu haben. Betrifft ja schließlich auch die anderen 113 Sprachen. aber Danke fürs Eintragen.

Hab die Dinger erst mal entfernt. Den Slick-Bug hier zu referenzieren, wenn er erstellt ist, fände ich Klasse. Dann können wir ihn nämlich beobachten.

joachim namyslo’s picture

joachim namyslo’s picture

Oh Slick steuert 100 % Entgegen. Ich werde in den nächsten Tagen testen, ob es nach der Übersetzung noch zu Abbrechen beim Import kommt.

joachim namyslo’s picture

Hm Der Maintianer scheint den Bug für unwichtig zu halten. Bleibt die Schaltfläche halt englisch. Das geht uns nicht nur in diesem Modul so. Also sammeln und dann direkt irgendwie auf die nächste Con tragen.

joachim namyslo’s picture

Gut, dann wollen wir das ganze noch mal probieren. Wenn bug korrekt gefixed wurde, dann sollten die Strings auf l.d.o doch eigentlich gart nicht mehr vorhanden sein. Sollte der Fehler, nach dem einpflegen, der Übersetzung wieder auftauchen, bitte melden. Dann müssen wir den Bug da drüben noch mal aufmachen.

In ca. 30 - 60 Minuten weis jeder, der dieses Modul in Deutsch verwendet, ob der Bug korrekt behoben wurde oder nicht. Ich tippe ja darauf, dass er nicht korrekt behoben wurde. Wenn das sich nicht auf so viele Slider-Module auswirken würde wäre es mir fast schon egal, aber eben nur fast.

joachim namyslo’s picture

StatusFileSize
new28.33 KB

Slick bug
Der Bug ist immer noch da. Heißt, was auch immewr der Maintainer von slick gefixed hat, hat er nicht auf die betreffende Version zurückportiert. Kann man so machen, dann ist das aber kacke. Muss man die Funktion t() da nicht einfach so einbauen, dass keine HTML-Tags mehr drin sind und dann ist's gut?

Irgendwann lern ich auf meine Alten Tage noch so viel PHP, dass ich in der Lage bin, einen Patch zu schreiben, wenn jemand die Funktion t() so setzt, dass HTML-Tags drin sind, die da nicht rein gehören. So schwer kann das doch nicht sein.

Also ich lösch' die Strings selbstverständlich wieder raus und kommentier in den nächsten Tagen noch mal den Bug-Report, der hier referenziert wurde.

m-schmitt’s picture

In Slick Carousel sind die Strings gar nicht mehr drin in der aktuellen Version, wenn ich das richtig sehe. Von daher liegt das Problem eher bei den Distributionen, die noch alte Versionen des Moduls verwenden.

joachim namyslo’s picture

Ich hab das gestern nachgeprüft. Varbase verwendet in der aktuellen Version slick 8.2.1 und wenn du die beiden Strings übersetzt, schlägt der mallformed-Filter immer noch aus. Ergo ist der Bug noch da.

joachim namyslo’s picture

Status: Active » Postponed
joachim namyslo’s picture

Status: Postponed » Fixed
joachim namyslo’s picture

Status: Fixed » Closed (fixed)