Add German validation, parser, and scalar errors by dondonz · Pull Request #2917 · graphql-java/graphql-java

dondonz · 2022-08-14T10:25:12Z

Edit: this PR has been merged, but the GitHub UI says "closed".

Now that we have i18n for validation errors, let's try one more language!

Major health warning: my German level is abysmal and this PR was mostly written by Herr Google. I sincerely apologise in advance for the miserable translations in this PR.

Forgive me, I thought it would be easier to get the ball rolling with an open PR. Hopefully, you're sufficiently annoyed to start commenting :)

dondonz · 2022-09-03T06:00:41Z

src/main/resources/i18n/Parsing_de.properties

+#InvalidSyntax.full=Ung\u00fcltige Syntax mit ANTLR-Fehler ''{0}'' in Zeile {1} Spalte {2}
+#
+#InvalidSyntaxBail.noToken=Ung\u00fcltige Syntax in Zeile {0} Spalte {1}
+#InvalidSyntaxBail.full=Ung\u00fcltige Syntax mit ung\u00fcltigem Token ''{0}'' in Zeile {1} Spalte {2}


In English, the phrase here is "offending token". I wasn't sure if the direct translation of "offensive" can be used here.

I went with the word ungültig/invalid instead.

dondonz · 2022-09-03T06:02:16Z

src/main/resources/i18n/Parsing_de.properties

+# Prior to Java 9, properties files are encoded in ISO-8859-1.
+# We have to use \u00fc instead of the German ue character, \u00e4 for ae, \u00f6 for oe, \u00df for ss
+#
+# TODO: uncomment these messages after the default (English) Parsing bundle is merged in


Uncomment these messages after #2921 is merged in.

#2921 contains the default parsing bundle

dondonz · 2022-09-03T06:03:16Z

src/main/resources/i18n/Scalars.properties

 # so use 2 '' characters to make it one ' on output.  This will take for the form ''{0}''
 #
-Scalar.unexpectedAstType=Expected an AST type of ''{0}'' but it was a ''{1}''.
+Scalar.unexpectedAstType=Expected an AST type of ''{0}'' but it was a ''{1}''


Nit: all the default validation messages did not end in a trailing full stop, so removing for consistency

dondonz · 2022-09-03T06:06:22Z

src/test/groovy/graphql/i18n/I18nTest.groovy

+        when:
+        def message = i18n.msg("ExecutableDefinitions.notExecutableType")
+        then:
+        message == "Validierungsfehler ({0}) : Die Typdefinition '{1}' ist nicht ausführbar"


This test is important to verify \u00fc was parsed correctly as ü

bbakerman · 2022-10-13T23:09:47Z

The Parser i18n code is now master - so there is more i18n keys to translate

dondonz · 2022-10-14T00:44:42Z

The Parser i18n code is now master - so there is more i18n keys to translate

I added them already (commented out) https://github.com/graphql-java/graphql-java/pull/2917/files#r962109513

andimarek · 2022-10-21T01:00:20Z

src/main/resources/i18n/Parsing_de.properties

+# We have to use \u00fc instead of the German ue character, \u00e4 for ae, \u00f6 for oe, \u00df for ss
+#
+InvalidSyntax.noMessage=Ung\u00fcltige Syntax in Zeile {0} Spalte {1}
+InvalidSyntax.full=Ung\u00fcltige Syntax mit ANTLR-Fehler ''{0}'' in Zeile {1} Spalte {2}


"Mit" sounds a bit strange, but not totally wrong. Maybe just a comma after "Syntax"?

andimarek · 2022-10-21T01:01:23Z

src/main/resources/i18n/Parsing_de.properties

+InvalidSyntax.full=Ung\u00fcltige Syntax mit ANTLR-Fehler ''{0}'' in Zeile {1} Spalte {2}
+
+InvalidSyntaxBail.noToken=Ung\u00fcltige Syntax in Zeile {0} Spalte {1}
+InvalidSyntaxBail.full=Ung\u00fcltige Syntax mit ung\u00fcltigem Token ''{0}'' in Zeile {1} Spalte {2}


I would recommend "wegen" which means "because of" instead of "mit"

Nur als Hinweis: Das Wort "wegen" hat den Genitiv zur Folge

-> Ungültige Syntax wegen des ungültigen Tokens X .......

andimarek · 2022-10-21T01:04:14Z

src/main/resources/i18n/Parsing_de.properties

+#
+ParseCancelled.full=Es wurden mehr als {0} ''{1}'' Token pr\u00e4sentiert. Um Denial-of-Service-Angriffe zu verhindern, wurde das Parsing abgebrochen
+#
+InvalidUnicode.trailingLeadingSurrogate=Ung\u00fcltiger Unicode gefunden. Nachfolgendes Surrogat muss ein f\u00fchrendes Surrogat vorangestellt werden. Ung\u00fcltiges Token ''{0}'' in Zeile {1} Spalte {2}


Honestly I am struggling here: not really wrong, but I would not translate it into "nachfolgend" . Leading and traling surrogate should not be translated.

andimarek · 2022-10-21T01:04:37Z

src/main/resources/i18n/Parsing_de.properties

+#
+InvalidUnicode.trailingLeadingSurrogate=Ung\u00fcltiger Unicode gefunden. Nachfolgendes Surrogat muss ein f\u00fchrendes Surrogat vorangestellt werden. Ung\u00fcltiges Token ''{0}'' in Zeile {1} Spalte {2}
+InvalidUnicode.leadingTrailingSurrogate=Ung\u00fcltiger Unicode gefunden. Auf ein f\u00fchrendes Surrogat muss ein abschlie\u00dfendes Surrogat folgen. Ung\u00fcltiges Token ''{0}'' in Zeile {1} Spalte {2}
+InvalidUnicode.invalidCodePoint=Ung\u00fcltiger Unicode gefunden. Kein g\u00fcltiger Codepunkt. Ung\u00fcltiges Token ''{0}'' in Zeile {1} Spalte {2}


I would not translate codepoint.

andimarek · 2022-10-21T01:05:10Z

src/main/resources/i18n/Scalars_de.properties

+# Prior to Java 9, properties files are encoded in ISO-8859-1.
+# We have to use \u00fc instead of the German ue character, \u00e4 for ae, \u00f6 for oe, \u00df for ss
+#
+Scalar.unexpectedAstType=Erwartet wurde ein AST-Typ von ''{0}'', aber es war ein ''{1}''


AST Type should not be translated imho

Sofern man den AST-Type doch übersetzen wollen würde, wären folgende Optionen denkbar.

Erwartet wurde ein AST-Typ vom Typ X, jedoch war dieser Typ ein Y

Erwartet wurde DER AST-Typ X, es kam aber ein AST-Typ Y.

Sofern man den - wie von @andimarek vorgeschlagen - AST-Typ weglassen möchte, könnte man das ganze so schreiben:

Erwartet wurde X, gekommen ist aber Y

Was will ich damit sagen? Dass das "es war ein....." nicht passt, da das Wort "erwarten" in diesem Zusammenhang ein "kommen" fordert.

Ich erwartete @dondonz , es kam aber @andimarek
Ich erwartete @dondonz , es war aber @andimarek (, der gekommen ist)
Ohne das, was in Klammern steht, würde der Satz so nicht funktionieren.

Etwas unkritischer ist folgendes zu sehen, wenn ich vorher noch einen abstrakten Klassifizierer einbaue.

Ich erwartete die Person @dondonz , aber die Person war @andimarek (,die dafür sorgte, dass meine Erwartungshaltung nicht eingehalten worden ist^^)
In dem Fall bezieht sich das "aber (die Person)" nur auf das Unterobjekt von Person aus dem vorhergehenden Hauptsatz.

Soviel zu dem Versuch, mein Sprachgefühl etwas zu Papier zu bekommen.

Hi @knotenpunkt, thanks for your comments on this PR, I really do value having experts review this PR.

I thought German grammar was hard already but Denglisch grammar is beyond me! All these English words really mess with the grammar 😅. While I will be merging this PR as is soon, wanted to say I'm going to ask for more reviews of these strings and I will take your comments into account. I didn't want to block the PR while I ask for more reviews.

Forgive me for replying in English!

andimarek · 2022-10-21T01:07:55Z

src/main/resources/i18n/Scalars_de.properties

+Int.notInt=Erwartet wurde ein Wert, der in den Typ ''Int'' konvertiert werden kann, aber es war ein ''{0}''
+Int.outsideRange=Erwarteter Wert im Integer-Bereich, aber es war ein ''{0}''
+#
+ID.notId=Erwartet wurde ein Wert, der in den Typ ''ID'' umgewandelt werden kann, aber es war ein ''{0}''


I recommend above not to translate AST type, but this would probably also mean not to translate type. I am on the fence about that.

I'll retain Typ here as it's a standalone word.

Also mir gefällt das mit dem "aber es war" nicht wirklich gut, aus den Gründen, die ich bereits zu einem anderen Reviewpunkt geschrieben habe.
Außerdem bezieht sich das "aber" in der Regel auch meistens auf den kompletten vorherigen Satz und nicht unbedingt nur auf einzelne Objekte. Der ganze vorherige Satz soll damit "relativiert" werden. Möchte man sich nur einem bestimmten Objekt aus dem vorherigen Satz beziehen und diesem widersprechen bzw. "relativieren", dann sollte man es in Kombination mit dem aber referenzieren.
Das heißt:

Erwartet wurde ein Wert, der in Typ Int konveritert werde kann, aber dieser Typ war ein String
vllt. noch etwas schöner: ......, aber dieser Typ war vom Typ String (naja gut ist glaube ich Geschmackssache^^)

Und was ich persönlich noch etwas schöner finde, ist das "aber" nicht direkt an den Anfang des zweiten Hauptsatzes zu stellen, sondern etwas weiter hinein, so wird diese Referenzierung außerdem och etwas klarer erkennbar.

-> Erwartet wurde ein Wert, der in den Typ int konvertiert werde kann, DIESER war ABER vom Typ String

Und wenn man es noch etwas poetischer haben möchte, dann wäre auch folgendes eine Option:

Erwartet wurde......, DIESER ABER war vom Typ String
Gramatikalisch gesehen ist diese Variante im deutschen vermutlich gar nicht erlaubt; was aber nicht heißt, dass man die Regeln nicht doch vllt. an der ein oder anderen Stelle etwas beugen darf oder vllt. sollte^^

andimarek · 2022-10-21T01:08:56Z

src/main/resources/i18n/Validation_de.properties

+# Prior to Java 9, properties files are encoded in ISO-8859-1.
+# We have to use \u00fc instead of the German ue character, \u00e4 for ae, \u00f6 for oe, \u00df for ss
+#
+ExecutableDefinitions.notExecutableType=Validierungsfehler ({0}) : Die Typdefinition ''{1}'' ist nicht ausf\u00fchrbar


I would not translate any GraphQL specific terms like type definition, schema definition etc.

@andimarek warum eigentlich nicht?^^ Ich hatte damals einen Prof an der UNI, bei dem kam kein einziges englisches Wort mehr vor. So wurden bspw. auch Threads zu Fäden übersetzt, Scheduler zu Ablaufplaner usw....
Aber davon mal abgesehen, warum meinst du, dass wir hier die englischen Begriffe beibehalten sollten

-> Validierungsfehler X: Die "type definition" Y ist nicht ausführtbar?

The choice between German and English terms for GraphQL specific words is a tough one.

I'm going to retain English terms for GraphQL specific words, as that's what I've seen elsewhere such as in Nils' Spring for GraphQL course, and more often I have seen English terms in German university computer science courses.

andimarek · 2022-10-21T01:09:19Z

src/main/resources/i18n/Validation_de.properties

+#
+FieldsOnCorrectType.unknownField=Validierungsfehler ({0}) : Feld ''{1}'' vom Typ ''{2}'' ist nicht definiert
+#
+FragmentsOnCompositeType.invalidInlineTypeCondition=Validierungsfehler ({0}) : Die Bedingung f\u00fcr den Inline-Fragmenttyp ist ung\u00fcltig, muss auf Objekt/Schnittstelle/Union stehen


same here: lets not translate inline fragment, object/interface/untion.

andimarek · 2022-10-21T01:10:55Z

src/main/resources/i18n/Validation_de.properties

+LoneAnonymousOperation.withOthers=Validierungsfehler ({0}) : Anonyme Operation mit anderen Operationen
+LoneAnonymousOperation.namedOperation=Validierungsfehler ({0}) : Operation ''{1}'' folgt der anonymen Operation
+#
+NoFragmentCycles.cyclesNotAllowed=Validierungsfehler ({0}) : Fragmentzyklen nicht erlaubt


Fragmentzyklen looks really odd for me ... If you want to translate it, I would probably make two words out of it.

andimarek · 2022-10-21T01:11:40Z

src/main/resources/i18n/Validation_de.properties

+#
+OverlappingFieldsCanBeMerged.differentFields=Validierungsfehler ({0}) : ''{1}'' : ''{2}'' und ''{3}'' sind unterschiedliche Felder
+OverlappingFieldsCanBeMerged.differentArgs=Validierungsfehler ({0}) : ''{1}'' : Felder haben unterschiedliche Argumente
+OverlappingFieldsCanBeMerged.differentNullability=Validierungsfehler ({0}) : ''{1}'' : Felder haben unterschiedliche Nullbarkeitsformen


Lets not translate Nullbarkeitsformen or Listenformen ... I believe this is more confusing than the english word

jord1e

See below.

jord1e

My German is crap (around A2) but I was able to read everything without a translator.
I don't know if there is any native German speakers around here (Andi I think🥲 ?).

src/main/resources/i18n/Validation_de.properties

Haben! Co-authored-by: Jordie <30464310+jord1e@users.noreply.github.com>

* Add all Float and Double coercion exception cases * Clean up tests, remove no longer allowed NaN value * Add early return for Doubles * Reorder NaN/Infinity check and add string representation to test * Remove unnecessary NaN check * Tidy package name * Tidy up

…on-main-repo Only run GCP tests on main graphql-java repo (skip on forks).

* use ast builders and stub test * remove main and add test * restore jvm check * revert test spacing * fix test * revers build.gradle * use spec specifiedByURL capitalization * update another specifiedByURL test * refactor introspection query builder, restore and deprecate __Type.specifiedByUrl * test that AST printed introspection query is equivalent to original string

Update aQute builder version

* Avoid an allocation of a chained context in the most common case * Avoid calling back to methods that create state parameters

* ImmutableKit cleanups - adding @SafeVarargs to ImmutableKit - dropping unused methods * Migrating links to the spec to https://spec.graphql.org/October2021

…3003)

* LambdaMetafactory support Another PR on improving getter access * LambdaMetafactory support Fixed test where parameters are present * LambdaMetafactory support Benchmark fix up * final fields * Tweaked code from PR feedback

* SchemaGeneratorPostProcessing should be deprecated * SchemaGeneratorPostProcessing should be deprecated - right date

* Remove unused imports and warnings from TwitterBenchmark * Configure ParserOptions for TwitterBenchmark * Fix javadoc of PersistedQueryCache

…nterface is implemented correctly

…plementations Bugfix for SDL check if an Interface is implemented correctly

* An error builder on GraphQLError * An error builder on GraphQLError - wat * ErrorClassification builder support * ErrorClassification builder support - readme example * ErrorClassification builder support - renamed method

Add German validation, parser, and scalar errors

dondonz · 2022-11-13T01:15:33Z

Maybe I fat fingered it, the PR was merged but looks "closed"

dondonz added 5 commits August 14, 2022 20:05

Add German validation errors

327559f

Tidy up IDE error

09fdf22

Tidy up

e5f8367

Fix word

7abed8c

Better words

3d8c552

dondonz added this to the 20.0 milestone Aug 19, 2022

dondonz added 3 commits September 3, 2022 15:15

Merge branch 'master' into auf-deutsch

74e4440

Make English messages consistent, no full stop at end of message

c01fc27

Add German translations for parsing and scalar messages

453255f

dondonz changed the title ~~Add German validation errors~~ Add German validation, parser, and scalar errors Sep 3, 2022

dondonz commented Sep 3, 2022

View reviewed changes

dondonz added 2 commits October 16, 2022 12:13

Merge branch 'master' into auf-deutsch

70e9327

Uncomment Parsing translations

448c965

andimarek requested changes Oct 21, 2022

View reviewed changes

dondonz added 4 commits October 21, 2022 16:08

Merge branch 'master' into auf-deutsch

304db9a

Update after comments

744ebdd

Update test

2e8e1c4

Fix typo

e5287d6

dondonz requested a review from andimarek October 24, 2022 23:35

jord1e suggested changes Oct 26, 2022

View reviewed changes

src/main/resources/i18n/Validation_de.properties Outdated Show resolved Hide resolved

dondonz and others added 4 commits October 27, 2022 15:31

Update src/main/resources/i18n/Validation_de.properties

f8d9ade

Haben! Co-authored-by: Jordie <30464310+jord1e@users.noreply.github.com>

Only run GCP tests on main graphql-java repo (skip on forks).

9c359c5

Merge pull request #2997 from folone/gleontiev/only-run-gcp-workflow-…

be851d7

…on-main-repo Only run GCP tests on main graphql-java repo (skip on forks).

MayCXC and others added 15 commits October 29, 2022 14:38

Update version

2de42f4

Merge pull request #3002 from graphql-java/update-aqute-builder

db79724

Update aQute builder version

Avoid an allocation of a chained context in the most common case (#2979)

322ba6b

* Avoid an allocation of a chained context in the most common case * Avoid calling back to methods that create state parameters

Polishing (#3001)

c665248

* ImmutableKit cleanups - adding @SafeVarargs to ImmutableKit - dropping unused methods * Migrating links to the spec to https://spec.graphql.org/October2021

Document the new IntrospectionQueryBuilder and tweaked it a little (#…

71eccd4

…3003)

LambdaMetafactory support for property fetches (#2985)

9787497

* LambdaMetafactory support Another PR on improving getter access * LambdaMetafactory support Fixed test where parameters are present * LambdaMetafactory support Benchmark fix up * final fields * Tweaked code from PR feedback

SchemaGeneratorPostProcessing should be deprecated (#2999)

13ab000

* SchemaGeneratorPostProcessing should be deprecated * SchemaGeneratorPostProcessing should be deprecated - right date

Fixes in TwitterBenchmark (#3006)

5c0c68b

* Remove unused imports and warnings from TwitterBenchmark * Configure ParserOptions for TwitterBenchmark * Fix javadoc of PersistedQueryCache

applied directives on arguments are not relevant when checked if an i…

ae51c2d

…nterface is implemented correctly

Merge pull request #3014 from graphql-java/fix-argument-directives-im…

d194761

…plementations Bugfix for SDL check if an Interface is implemented correctly

improve test case by adding a new valid location example

04df386

Adds an error builder on GraphQLError (#3011)

df2a717

* An error builder on GraphQLError * An error builder on GraphQLError - wat * ErrorClassification builder support * ErrorClassification builder support - readme example * ErrorClassification builder support - renamed method

Fix grammar

04215b5

Merge pull request #2917 from graphql-java/auf-deutsch

e16db8b

Add German validation, parser, and scalar errors

dondonz closed this Nov 13, 2022

Conversation

dondonz commented Aug 14, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

bbakerman commented Oct 13, 2022

Uh oh!

dondonz commented Oct 14, 2022

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

jord1e left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

jord1e left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

dondonz commented Nov 13, 2022

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

dondonz commented Aug 14, 2022 •

edited

Loading

jord1e left a comment •

edited

Loading

jord1e left a comment •

edited

Loading