Magento2_German_LocalePack_de_DE icon indicating copy to clipboard operation
Magento2_German_LocalePack_de_DE copied to clipboard

Übersetzungen von Magento 2.4 wurden fehlerhaft integriert.

Open skybreaker85 opened this issue 3 years ago • 5 comments

Im commit 135c6e92d71628485f2fbc012b67af4cfb9ceba6 wurden Übersetztungen für Magento2.4 hinzugefügt. Soweit, so löblich. Allerdings ist dies fehlerhaft geschehen.

  1. Die erste Spalte fehlt, d.h. der Ausgangstext fehlt, wodurch die komplette Übersetzung nicht angenommen wird.
  2. Die Übersetzungen enthalten Fehler in der Zeichenkodierung
  3. Die Übsetzungen sind auch dort enhalten, wo es nicht sein darf, z.b. in HTML Tags.

Zu 1.) Dies ist so ziemlich in jeder hinzugefügten Zeile zu sehen

Zu 2.) Beispiel: https://github.com/splendidinternet/Magento2_German_LocalePack_de_DE/commit/135c6e92d71628485f2fbc012b67af4cfb9ceba6#diff-e9b8c80c57f520ab67a7456235dbcfe94b0061198d47e8d115850f9308842b4eR325

zu 3.) Beispiel: https://github.com/splendidinternet/Magento2_German_LocalePack_de_DE/commit/135c6e92d71628485f2fbc012b67af4cfb9ceba6#diff-e9b8c80c57f520ab67a7456235dbcfe94b0061198d47e8d115850f9308842b4eR821

Warum wurde das in einem pull request nicht gesehen? (rhetorische Frage)

Ich hoffe ich konnte darauf aufmerksam machen.

skybreaker85 avatar Mar 04 '21 12:03 skybreaker85

@niclashoyer He's absolutely right about that. Any ideas on the easiest way to fix this?

t-heuser avatar Sep 23 '21 13:09 t-heuser

Hm, we could revert the pull request and redo the translations, in that way we could be sure that the integration is done correctly. I don't know how much manual work is required to find all occurrences of false integration.

niclashoyer avatar Sep 23 '21 13:09 niclashoyer

I just had a look, maybe it is easier to remove all lines where the first column is blank and then re add all phrases via the method described in the README

niclashoyer avatar Sep 23 '21 13:09 niclashoyer

@niclashoyer Sounds like the easiest method. Can you assign the issue to me or do you want to solve it?

t-heuser avatar Sep 23 '21 13:09 t-heuser

Yes, sure, we're glad if someone takes care of this 👍🏽

niclashoyer avatar Sep 23 '21 13:09 niclashoyer