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

Übersetzungen von Magento 2.4 wurden fehlerhaft integriert. #173

Open
skybreaker85 opened this issue Mar 4, 2021 · 5 comments
Open

Übersetzungen von Magento 2.4 wurden fehlerhaft integriert. #173

skybreaker85 opened this issue Mar 4, 2021 · 5 comments
Assignees

Comments

@skybreaker85
Copy link

skybreaker85 commented Mar 4, 2021

Im commit 135c6e9 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:
135c6e9#diff-e9b8c80c57f520ab67a7456235dbcfe94b0061198d47e8d115850f9308842b4eR325

zu 3.)
Beispiel:
135c6e9#diff-e9b8c80c57f520ab67a7456235dbcfe94b0061198d47e8d115850f9308842b4eR821

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

Ich hoffe ich konnte darauf aufmerksam machen.

@oneserv-heuser
Copy link
Contributor

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

@niclashoyer
Copy link
Collaborator

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
Copy link
Collaborator

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

@oneserv-heuser
Copy link
Contributor

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

@niclashoyer
Copy link
Collaborator

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants