Drupal will zwei Strings nicht importieren, wegen ungültigem HTML.

String Nummer 1 von Bootstrap: https://localize.drupal.org/translate/languages/de/translate?sid=2644878
Ich dachte es liegt an "//" vor "www.jsdelivr.com/img/logo.png", aber das will er auch nicht importieren. Und scheint auch egal zu sein, siehe https://stackoverflow.com/questions/10764460/html-javascript-remote-img-....
Irgendjemand eine Idee?

String Nummer 2 von EU Cookie Compliance: https://localize.drupal.org/translate/languages/de/translate?sid=1752228
Hier ist "<front>" glaube ich nicht erlaubt, Drupal Core verwendet immer "%front". Das muss der Maintainer des Moduls ändern, korrekt?

Comments

M-Schmitt created an issue. See original summary.

hass’s picture

Das liegt am Security-Filter für Übersetzungen.

fisherman90’s picture

Zu String Nummer 2: Es macht gar keinen Sinn, dass dieser als Übersetzbar markiert ist, schließlich handelt es sich nur um einen Token für die Startseite. Da wurde wohl die t()-Funktion vom Modul-Maintainer an einer Stelle eingesetzt, wo sie nicht benötigt wird, also das muss auf jeden Fall der Maintainer ändern.

m-schmitt’s picture

Ok, beim 2. String mache ich ein Issue für das Modul auf.
Irgendeine Ahnung, wo das ungültige HTML beim 1. String ist? Selbst in VS Code habe ich keine Probleme gefunden.

c-logemann’s picture

Zu String Nr. 1: Ich weiß zwar gerade nicht woher dieser stammt, aber ad hoc würde finde ich daran zwar Dinge problematisch: Erstens wird ein Logo von Bild von einer externen Seite eingebunden. Zweitens werden Inline Styling Anweisungen verwendet, die damit nur ganz übel anpassbar wären. Wenn beides nicht ohnehin verboten sein sollte in Modulen und Themes, sollte man die Coding-Standards diesbezüglich ergänzen. Wenn der Security-Filter das Images-Tag verdächtig finden sollte, wäre das übrigens völlig logisch. Das heißt hier sollte auch dringend eine Issue beim entsprechenden Projekt eröffnet werden und wegen dem Link zur jsdeliver auch mit #GDPR getaggt werden.

c-logemann’s picture

Ich kenne mich nicht so genau mit dem Translation-Management aus, aber ich kann mir vorstellen, daß eine Überesetzung nicht akzeptiert wird, wenn die Quelle schon problematisch ist. In dem Fall bliebe nichts anders übrig, als das Prbleme erst mal an der Quelle d.h. in den Modulen zu lösen.

sanduhrs’s picture

Ich stimme #5 vollständig zu. Weder inline Styles noch externes image sind in Ordnung.

joachim namyslo’s picture

Ist die Funktion hier überhaupt richtig geschlossen?

Ich hab ja nur wenig Ahnung von PHP aber ich finde, die Klammer am Ende der t-Funktion seltsam

Beanstandeter String

public function processApi(array $json, array &$definition) {
    $definition['description'] = t('<p style="background:#EB4C36"><a href=":jsdelivr" target="_blank"><img src="//www.jsdelivr.com/img/logo.png" alt="jsDelivr Logo"/></a></p><p><a href=":jsdelivr" target="_blank">jsDelivr</a> is a free multi-CDN infrastructure that uses <a href=":maxcdn" target="_blank">MaxCDN</a>, <a href=":cloudflare" target="_blank">Cloudflare</a> and many others to combine their powers for the good of the open source community... <a href=":jsdelivr_about" target="_blank">read more</a></p>', [
      ':jsdelivr' => 'https://www.jsdelivr.com',
      ':jsdelivr_about' => 'https://www.jsdelivr.com/about',
      ':maxcdn' => 'https://www.maxcdn.com',
      ':cloudflare' => 'https://www.cloudflare.com',
    ]);

Anderer Sting aus Bootstrap

->setAttribute('title', t('Opens in new window')

Faalls der Corde oben mit der falschen Klamm,er endert, ist es kein Wunder, dass der String nicht in der UI auftaucht und dass tut er nicht, obwohl die übersetzung seit über einer Woche zugelassen ist.

sanduhrs’s picture

Im ersten Code fehlt eine } ganz am Ende.
Im zweiten Code fehlt eine ) am Ende.

joachim namyslo’s picture

Ich habe den entsprechenden Issue ergänzt, da dieser schon bearbeitet wird.

https://www.drupal.org/project/bootstrap/issues/2994562#comment-12756087

joachim namyslo’s picture

Der von mir verlinkte Issue ist als Fixed markiert. @Markus kann der Issue dann zu oder gibts noch Probleme?

hass’s picture

joachim namyslo’s picture

@hass fixed Danke!

c-logemann’s picture

Die Issue ist eine Frage, die als Support Request inzwischen hinreichend beantwortet wurde. Und da die beiden Probleme nicht genügend gemeinsam haben, um aus dieser Issue eine "Meta-Issue" zu machen, schließe ich diese nun.

Es wäre aber hilfreich für andere Nutzer.innen die hier evtl. landen, daß zum String 2 ("front") evtl. auch noch eine Issue verlinkt werden könnte – am besten auch strukturiert als "Related" Issue.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

joachim namyslo’s picture