Closed (fixed)
Project:
German translation
Version:
master
Component:
PO files
Priority:
Normal
Category:
Support request
Assigned:
Unassigned
Reporter:
Created:
21 Aug 2018 at 09:38 UTC
Updated:
11 Mar 2019 at 12:10 UTC
Jump to comment: Most recent
Comments
Comment #2
hass commentedDas liegt am Security-Filter für Übersetzungen.
Comment #3
fisherman90Zu 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.
Comment #4
m-schmitt commentedOk, 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.
Comment #5
c-logemannZu 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.
Comment #6
c-logemannIch 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.
Comment #7
sanduhrsIch stimme #5 vollständig zu. Weder inline Styles noch externes image sind in Ordnung.
Comment #8
joachim namysloIst 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
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.
Comment #9
sanduhrsIm ersten Code fehlt eine
}ganz am Ende.Im zweiten Code fehlt eine
)am Ende.Comment #10
joachim namysloIch habe den entsprechenden Issue ergänzt, da dieser schon bearbeitet wird.
https://www.drupal.org/project/bootstrap/issues/2994562#comment-12756087
Comment #11
joachim namysloDer von mir verlinkte Issue ist als Fixed markiert.
@Markus kann der Issue dann zu oder gibts noch Probleme?Comment #12
hass commentedBugfix: https://localize.drupal.org/translate/languages/de/translate?sid=2654015
Comment #13
joachim namyslo@hass fixed Danke!
Comment #14
c-logemannDie 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.
Comment #16
joachim namyslo