String fixes #918

stm2 · 2020-07-16T19:36:02Z

Viele Fehler in strings.de/en.po verbessert.
Die letzten beiden Commits ändern Dinge, die nicht nur kosmetisch sind, sondern Befehle ungültig machen können. Habe viele fragwürdige Zusammenschreibweisen im Englischen auseinandergerissen (stonecircle, dragonblood). Die sind im Englischen zwar grundsätzlich möglich (JRR Martin schreibt zum Beispiel direwolves, D&D kennt den hellhound), ich empfinde sie aber vielfach als komisch. Auch im Sinne von unfreiwillig komisch.

This closes bug 2669.

The logic for "insects do not need castles in deserts or swamps" was broken.

bug 2678: fix trade_needs_castle

Bug 2679

Add a message and don't pay for the failed spell.

Bug 2680: shapeshift spell cannot be cast twice.

Fixing the symptoms, not the cause, becasue I don't know how this happened.

Zauber sollen force benutzen, nicht level.

stm2 · 2020-07-16T20:38:05Z

Aha, ich Trottel such ich ewig einen Bug, dabei ist der build schon in develop kaputt.

ennorehling · 2020-07-18T14:08:11Z

res/translations/strings.de.po


 msgid "rustychainmail"
 msgstr "Rostiges Kettenhemd"

 msgctxt "spellinfo"
 msgid "destroy_magic"
-msgstr ""
-"Dieser Zauber ermöglicht dem Magier, Verzauberungen einer Einheit, eines Schiffes, Gebäudes oder auch der Region aufzulösen."
+msgstr "Dieser Zauber ermöglicht der Zauberin, Verzauberungen einer Einheit, eines Schiffes, Gebäudes oder auch der Region aufzulösen."


Gibt es einen Grund, hier von der Zauberin statt vom Magier zu sprechen?

ennorehling · 2020-07-18T14:08:58Z

res/translations/strings.de.po

@@ -36,7 +36,7 @@ msgstr "Schattendämonen"

 msgctxt "race"
 msgid "sphinx_x"
-msgstr "Spinx"


Kann weg. Die Rasse kommt weder im Code noch in den Daten vor, und auch nicht unter den Rassen.

ennorehling · 2020-07-18T14:10:02Z

res/translations/strings.de.po

@@ -126,7 +126,7 @@ msgid "eternal_walls"
 msgstr "Mauern der Ewigkeit"

 msgid "spinx03"


Ditto: Auch keine Nachrichten zu einer Spinx finden sich im Code. Kann weg.

ennorehling · 2020-07-18T14:10:18Z

res/translations/strings.de.po

@@ -151,7 +151,7 @@ msgid "fortress_generic"
 msgstr "Burg"

 msgid "spinx07"
-msgstr "Auf dem gelben Segel prankt ein Kreuz als Wappen"
+msgstr "Auf dem gelben Segel prangt ein Kreuz als Wappen"


Weg mit Spinxen!

ennorehling · 2020-07-18T14:12:21Z

res/translations/strings.de.po

@@ -260,7 +260,7 @@ msgstr "Rindenhaut"

 msgctxt "spellinfo"
 msgid "auraleak"
-msgstr "Der Schwarzmagier kann mit diesem dunklen Ritual einen Riss in das Gefüge der Magie bewirken, der alle magische Kraft aus der Region reißen wird. Alle magisch begabten in der Region werden einen Großteil ihrer Aura verlieren."
+msgstr "Die Schwarzmagierin kann mit diesem dunklen Ritual einen Riss in das Gefüge der Magie bewirken, der alle magische Kraft aus der Region reißen wird. Alle magisch Begabten in der Region werden einen Großteil ihrer Aura verlieren.a"

 msgctxt "race"
 msgid "apophis_p"


Auch der/die/das Apophis gibt es nicht.

ennorehling · 2020-07-18T14:14:31Z

res/translations/strings.de.po


 msgctxt "border"
 msgid "gate_open"
 msgstr "gewaltiges offenes Tor"

 msgctxt "spellinfo"
 msgid "magicboost"
-msgstr "Der Magier öffnet seinen Geist den Sphären des Chaos und wird so für einige Zeit über mehr magische Kraft verfügen. Doch die Hilfe der Herren der Sphären hat seinen Preis, und so wird die Phase der Macht abgelöst von einer Phase der Schwäche."
+msgstr "Der Hexer öffnet seinen Geist den Sphären des Chaos und wird so für einige Zeit über mehr magische Kraft verfügen. Doch die Hilfe der Herren der Sphären hat ihren Preis, und so wird die Phase der Macht abgelöst von einer Phase der Schwäche."


Hier ist es ein Hexer, weiter oben eine Hexerin. Scheint mir uneinheitlich, und ausserdem haben wir schon ein Wort dafür, die heisst Hexe, und ist eben gerade nicht die weinliche Form eines männlichen Berufes.

ennorehling · 2020-07-18T14:39:08Z

res/translations/strings.de.po

+
+msgctxt "keyword"
+msgid "bid"
+msgstr "BID"


Was ist das für ein keyword? Das kenne ich nicht.

ennorehling · 2020-07-18T14:39:32Z

res/translations/strings.de.po

+msgstr "Eine giftige und grüne Flüssigkeit in einer kleinen Phiole; keine humanoide Rasse außer Dämonen wagt damit in Kontakt zu kommen."
+
+msgid "INFO"
+msgstr "INFO"


Das ist neu. Wofür wird INFO gebraucht?

ennorehling · 2020-07-18T14:41:09Z

res/translations/strings.de.po

+
+msgctxt "keyword"
+msgid "locale"
+msgstr "LOCALE"


LOCALE ist kein keyword (wie PARTEI) aus Gründen, die sich mir nicht mehr erschliessen, aber damit irgendwas zu tun haben. Warum ist das hier zugefügt worden?

ennorehling · 2020-07-18T14:41:54Z

res/translations/strings.de.po

+
+msgctxt "keyword"
+msgid "report"
+msgstr "REPORT"


Ein Befehl REPORT? Wo kommt der her?

ennorehling · 2020-07-18T14:43:37Z

res/translations/strings.de.po

+msgstr "REPORT"
+
+msgid "unknownunit"
+msgstr "an unknown unit"


Was macht denn ein englischer Text hier? Und obwohl unknownunit scheinbar im strings.en.po existiert, sehe ich keinen Bedarf dafür. Kann das weg?

Die neuen Strings waren alle aus strings.en.po. Keine Ahnung warum die da sind. Ich wollte das einheitlich in beiden Dateien haben. Habe erstmal nichts gelöscht, denn zu viele Strings tun ja erstmal nicht weh.

ennorehling · 2020-07-18T16:47:08Z

res/translations/strings.en.po


 msgctxt "spellinfo"
 msgid "destroy_magic"
 msgstr "This spell lets a magician destroy spells on a ship, a building, or a region."

+#, fuzzy
 msgctxt "spell"
 msgid "shadowknights"
 msgstr "Shadow Knights"


Warum sind eigentlich die shadow knights in title case (Großbuchstabe), aber die shadow demons nicht?

Oh. Zaubersprüche in title case, Rassen nicht? Das scheint in der Tat anderswo auch so zu sein. Habe ich mir bestimmt was bei gedacht.

Rassen müssen ja auch im Fließtext auftauchen, während Zauber mehr oder weniger immer Titel sind. Das passt schon so.

ennorehling · 2020-07-18T16:50:44Z

res/translations/strings.en.po

 msgid "seashell"
-msgstr "seashell"
+msgstr "sea shell"


Hier glaube ich, dass seashell nicht nur in Ordnung ist, sondern auch gebräuchlicher.
https://trends.google.com/trends/explore?geo=US&q=%22sea%20shell%22,seashell

ennorehling · 2020-07-18T16:51:19Z

res/translations/strings.en.po

@@ -785,7 +796,7 @@ msgstr "Astral Disruption"
 #, fuzzy
 msgctxt "spell"
 msgid "migration"
-msgstr "Rit of Acceptance"
+msgstr "Rite of Acceptance"


Das wird bereits im Zuge der Umbenennung der Zauber geändert.

ennorehling · 2020-07-18T16:52:10Z

res/translations/strings.en.po

@@ -1127,7 +1139,7 @@ msgstr "an illusionary wall"

 msgctxt "race"
 msgid "apepsnake"
-msgstr "apepsnake"
+msgstr "apep snake"


ennorehling · 2020-07-18T16:57:58Z

res/translations/strings.en.po

@@ -1933,7 +1949,7 @@ msgstr "Chaos Curse"

 msgctxt "spell"
 msgid "wdwpyramid_illaun"
-msgstr "Dream of the gods"
+msgstr "Dream of the Gods"


Da es die Pyramide nur in WDW gab, und die Zauber nicht mehr im Code sind, könnten diese Strings auch einfach verschwinden.

ennorehling · 2020-07-18T16:58:52Z

res/translations/strings.en.po

@@ -2023,7 +2039,7 @@ msgstr "Air Shield"
 #, fuzzy
 msgctxt "race"
 msgid "museumghost"
-msgstr "museumghost"
+msgstr "museum ghost"


Gibt es im Spiel nicht mehr. Die Rasse existiert noch in races.xml, könnte aber auch weg.

ennorehling · 2020-07-18T17:00:56Z

res/translations/strings.en.po

@@ -2695,9 +2712,10 @@ msgctxt "spellinfo"
 msgid "chaossuction"
 msgstr "By sacrificing the lives of 200 peasants, the chaos sorcerer is able to open a planar gate. This gate can be used during the following week to transfer units to the astral plane. It dissipates at the end of the following week."

+#, fuzzy


Oh, bitte kein fuzzy matching, das macht nur Ärger (obwohl der Server das ignorieren sollte). Ich erinner mich nicht mehr, warum, aber da habe ich auf der Arbeit mal einen Tag dran verloren.

ennorehling · 2020-07-18T17:04:14Z

res/translations/strings.en.po

@@ -4474,9 +4496,10 @@ msgctxt "spell"
 msgid "oldrace"
 msgstr "Unknown Effect"


Davon gibt es mehrere, warum?

Nicht die geringste verdammte Ahnung. Ich lösche keine Strings, denn ich kenne keine automatische Möglichkeit festzustellen, welche noch gebraucht werden und welche nicht.

ennorehling · 2020-07-18T17:06:29Z

res/translations/strings.de.po

@@ -1746,7 +1747,7 @@ msgstr "DEBUG"

 msgctxt "spell"
 msgid "goodwinds"
-msgstr "Beschwörung eines Wasserelementares"
+msgstr "Beschwöre einen Wasserelementar"


Das habe ich schon angefasst, und bin am Ende bei "Wasserlementar" etc. gelandet. Commit ist bereits in devlop, mit Code, der die alten Namen als Alternativen versteht.

Fixing the E3 rules.lua test We no longer need pyramid, scarab, sphinx, pharao.

ennorehling · 2020-07-18T17:51:09Z

Ich habe gerade aus dem develop branch die WDW Strings (Sphinx & Co) entfernt, und die Umbenennung der Zauber vollzogen. Ein Rebase wäre also mindestens angebracht.

stm2 · 2020-07-18T18:23:26Z

Bin dabei ...

-msgid "fortress_generic"
-msgstr "Burg"

Kann das wirklich weg oder hast du das nur übersehen?

ennorehling · 2020-07-18T19:36:19Z

Bin dabei ...
-msgid "fortress_generic"
-msgstr "Burg"
Kann das wirklich weg oder hast du das nur übersehen?

Ja, fortress_generic gibt es nicht.

stm2 · 2020-07-18T20:45:18Z

Rebased.
Ich brauch noch feedback: Habe im Englischen viele Strings verändert. Das betrifft ein paar konsequente Zauberumbenennungen, aber auch viele Zusammenschreibungen. Ist in den commits "edited strings that affect", "edited a few spells+amend"

stm2 and others added 26 commits June 27, 2020 21:43

fix NOTLOST items lost in battle

a6f00c6

fix tests for parsing single-letter keywords.

889b111

remove legacy filereader code

bc66965

Bug 2670 trim   and others from strings.

11c489e

bug 2673 ignore commands that have fewer than 2 characters.

8666ab9

update default version number

ceb4560

install lunitx only once

d271efc

Durch GIB "leer" gewordene Schiffe können nicht sinken.

00f2182

bug 2674 send message to player, not monsters

ec5dcda

bug 2674 fix null-ptr deref

90837b3

Lighthouse shows roads in both CR and NR.

656248c

This closes bug 2669.

accept some selected changes from stm2's script rewrite.

c2823e1

delete obsolete scripts, shellcheck the rest

0c87bc9

make coverity understand that there is no buffer overflow here.

8b1ecf9

stop showing unhandled keycodes in status

0037a26

bug 2678: fix trade_needs_castle

cf84a95

The logic for "insects do not need castles in deserts or swamps" was broken.

Merge pull request eressea#915 from eressea/bug-2678

4ea08e5

bug 2678: fix trade_needs_castle

Merge branch 'develop' of github.com:eressea/server into develop

aecbe41

Bug 2679: Nur Bergwächter bewachen Eisen.

9a72403

Merge pull request eressea#916 from eressea/bug-2679

4052487

Bug 2679

Bug 2680: shapeshift spell cannot be cast twice.

300f46a

Add a message and don't pay for the failed spell.

Merge pull request eressea#917 from eressea/bug-2680

d173d48

Bug 2680: shapeshift spell cannot be cast twice.

empty convoys cannot be built on.

885037d

Merge branch 'develop' of github.com:eressea/server into develop

a946037

Bug 2683: fix buildings with size 0

25e9498

Fixing the symptoms, not the cause, becasue I don't know how this happened.

Bug 2668 Spruchstärke Ritual der Aufnahme

69d6e27

Zauber sollen force benutzen, nicht level.

ennorehling added 3 commits July 17, 2020 18:27

make it impossible to create buildings with size 0

23e5738

fake wormhole in test had negative size

a58831d

Allow alternative names for some spells.

ed2d83c

ennorehling reviewed Jul 18, 2020

View reviewed changes

ennorehling requested changes Jul 18, 2020

View reviewed changes

ennorehling reviewed Jul 18, 2020

View reviewed changes

Finish spell renaming, remove wdw leftovers.

b8738ed

Fixing the E3 rules.lua test We no longer need pyramid, scarab, sphinx, pharao.

stm2 added 2 commits July 18, 2020 21:34

reviewed all strings

e5e9a9d

edited strings that affect the user

1af5456

stm2 added 4 commits July 18, 2020 21:40

edited a few spells

dd49c05

amend edit spell

6250eb9

reviewed all messages

c1b1205

odds and ends

0e62db8

stm2 force-pushed the string-fixes branch from 572b5a1 to 0e62db8 Compare July 18, 2020 20:41

stm2 requested a review from ennorehling July 18, 2020 20:42

ennorehling force-pushed the develop branch from 6e88dfb to a5f457a Compare August 3, 2020 09:26

ennorehling force-pushed the develop branch 3 times, most recently from 62cc513 to 11ce5a2 Compare August 16, 2020 14:01

ennorehling force-pushed the develop branch 3 times, most recently from 68db444 to e61b8ad Compare September 13, 2020 12:33

ennorehling force-pushed the develop branch from 82abe1d to d4f972f Compare November 7, 2020 20:45

ennorehling force-pushed the develop branch from ad75eea to 0cfdee0 Compare December 6, 2020 14:49

ennorehling force-pushed the develop branch from 0c48a1d to 74f6076 Compare February 16, 2021 09:12

ennorehling force-pushed the develop branch 2 times, most recently from 3dfa0c4 to 5730779 Compare June 9, 2021 18:58

ennorehling force-pushed the develop branch from 50f5eb9 to afebb27 Compare March 9, 2024 12:42

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

String fixes #918

String fixes #918

stm2 commented Jul 16, 2020

stm2 commented Jul 16, 2020

ennorehling Jul 18, 2020

ennorehling Jul 18, 2020

ennorehling Jul 18, 2020

ennorehling Jul 18, 2020

ennorehling Jul 18, 2020

ennorehling Jul 18, 2020

ennorehling Jul 18, 2020

ennorehling Jul 18, 2020

ennorehling Jul 18, 2020

ennorehling Jul 18, 2020

ennorehling Jul 18, 2020

stm2 Jul 18, 2020

ennorehling Jul 18, 2020

ennorehling Jul 18, 2020

stm2 Jul 18, 2020

ennorehling Jul 18, 2020

stm2 Jul 18, 2020

ennorehling Jul 18, 2020

ennorehling Jul 18, 2020

ennorehling Jul 18, 2020

ennorehling Jul 18, 2020

ennorehling Jul 18, 2020

ennorehling Jul 18, 2020

stm2 Jul 18, 2020

ennorehling Jul 18, 2020

ennorehling commented Jul 18, 2020

stm2 commented Jul 18, 2020

ennorehling commented Jul 18, 2020

stm2 commented Jul 18, 2020

		@@ -126,7 +126,7 @@ msgid "eternal_walls"
		msgstr "Mauern der Ewigkeit"

		msgid "spinx03"

		@@ -4474,9 +4496,10 @@ msgctxt "spell"
		msgid "oldrace"
		msgstr "Unknown Effect"

String fixes #918

Are you sure you want to change the base?

String fixes #918

Conversation

stm2 commented Jul 16, 2020

stm2 commented Jul 16, 2020

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

ennorehling commented Jul 18, 2020

stm2 commented Jul 18, 2020

ennorehling commented Jul 18, 2020

stm2 commented Jul 18, 2020