Grammar problem in German translation

[Chamaeleon-]

We have a problem with the grammar of the translation of for example "Every 3 months the third Wednesday":

Correct in German is "Alle 3 Monate der dritte Mittwoch des Monats" but it translates to "Alle 3 Monate der dritte Mittwochs des Monats" ("mittwochs" is the adverb of "Mittwoch" / thursday).

A possible "workaround" would be to change line 73 to "am" (German for "on") and all the weekdays from adverb form to noun to keep byweekday and nth_weekday sound.

Do you have preferences or suggestions on this topic?

[rlanvin]

Hey, thanks for the report. Are you sure we shouldn't also change the nth_weekday and -nth_weekday too? Does "am der erste Montag" work, or should it be "am ersten Montag"?

Also would you be happy to submit a pull request for this change?

[Chamaeleon-]

you're right, I didn't realize byweekday is used in front of nth_weekday too... "am der erste Montag" doesn't make sense...

I will have a look and play around a bit with the combinations to submit a PR 👍

[rlanvin]

Great thanks! Looking forward to your PR. You can use the script in the bin folder to test the result when you play around, for example ./bin/review_translations.php --locale de

[Chamaeleon-]

Made some changes to the German translation and this is the output of my proposed changes. I think it's okay like this.

Jährlich, ab dem 01.06.15, 6 Mal insgesamt
Jährlich am Dienstag, ab dem 02.09.97, 2 Mal insgesamt
Jährlich im Januar, Mai und August, ab dem 02.09.97, 2 Mal insgesamt
Jährlich am ersten, 42. und letzten Tag des Jahres, ab dem 02.09.97, für immer
Jährlich in Kalenderwoche 10 und 42, ab dem 02.09.97, für immer
Monatlich, ab dem 02.09.97, für immer
Alle 3 Monate am 5. Tag und 10. Tag des Monats und am 5. letzten Tag und letzten Tag des Monats, ab dem 01.01.97, bis zum 31.12.97
Monatlich am ersten Montag und letzten Dienstag des Monats, ab dem 06.09.22, für immer
Wöchentlich, ab dem 02.09.97, 2 Mal insgesamt
Alle 2 Wochen, ab dem 06.09.22, für immer
Alle 3 Wochen am Montag, Dienstag und Freitag, nur das erste Auftreten, ab dem 06.09.22, für immer
Täglich am Dienstag und Freitag, ab dem 02.09.97, 2 Mal insgesamt
Stündlich zur 21 und 22 Stunde und 10, 20 und 30 Minute, ab dem 02.09.97, 09:00, einmalig
Minütlich zur 21 Stunde und 1 Minute, ab dem 02.09.97, 09:00, einmalig
Minütlich zur 21 Stunde und 0, 10, 20, 30, 40 und 50 Minute, ab dem 06.09.22, 11:32, für immer
Sekündlich zur 21 Stunde und 10 Minute und 10 Sekunde, ab dem 06.09.22, 11:32:56 UTC, für immer