View Issue Details

IDProjectCategoryView StatusLast Update
0002342EresseaGeneralpublic2017-12-05 18:49
ReporterCaranthir Assigned ToEnno  
PrioritynormalSeverityminorReproducibilityalways
Status closedResolutionfixed 
Product Version3.12.1 
Target Version3.13.0Fixed in Version3.13.0 
Summary0002342: PFLANZE Samen wird zu ZÜCHTE Samen / echeck meckert
Description

Durch die Auswertung wird der Befehl PFLANZE Samen zu ZÜCHTE Samen umgewandelt und als neuer Default gesetzt. Beim späteren EInsenden meckert echeck über ZÜCHTE Samen. Entweder ist das ein Auswertungsbug, oder eine fehlende Option in echeck. Beispielausgabe von echeck, anonymisiert:

Verarbeite /home/eressea/eressea/game-2/orders.dir/turn-xxxxxxxx@online.de,gruenbaer,0

Partei xxxx

ECheck (Version 4.3.6, Jun 18 2017), Zug-Checker für Eressea - Freeware!

Verarbeite Datei 'turn-xxxxxxxx@online.de,gruenbaer,0'. Befehle für Partei xxxx gefunden. Rekrutierungskosten auf xxx Silber gesetzt, Warning Level 0. Silberpool aktiviert.

Fehler in Zeile 148: ZÜCHTE PFERDE oder ZÜCHTE KRÄUTER. ZÜCHTE Samen' Fehler in Zeile 151: ZÜCHTE PFERDE oder ZÜCHTE KRÄUTER. ZÜCHTE Samen' Fehler in Zeile 153: ZÜCHTE PFERDE oder ZÜCHTE KRÄUTER. ZÜCHTE Samen' Fehler in Zeile 155: ZÜCHTE PFERDE oder ZÜCHTE KRÄUTER. ZÜCHTE Samen' Fehler in Zeile 157: ZÜCHTE PFERDE oder ZÜCHTE KRÄUTER. `ZÜCHTE Samen'

Es wurden Befehle für 1 Partei und 326 Einheiten gelesen. Es wurden 5 Fehler entdeckt.

Steps To Reproduce
  1. Befehle einsenden: EINHEIT xyz / PFLANZE Samen
  2. Auswertung abwarten
  3. Default-Befehl von Einheit xyz ist ZÜCHTE Samen
  4. Befehle einsenden: EINHEIT xyz / ZÜCHTE Samen
  5. echeck meldet Fehler
TagsNo tags attached.
Factioncara
GameE2
Report1030

Activities

Enno

Enno

2017-06-19 08:27

administrator   ~0007289

Ich bin nicht sicher, ob die beiden Befehle nicht evtl. intern als Synonym auf das selbe Kommando abgebildet werden. Es kann sein, dass PFLANZE PFERDE und ZUECHTE SAMEN erlaubt ist? Muss ich mal gucken. Es gibt zu viele Befehle...

Enno

Enno

2017-06-19 18:20

administrator   ~0007290

Ja, PFLANZE und ZUECHTE sind in der Tat Synonyme. ECheck weiss das nicht, und gewollt war das sicher auch nie. Warum hat das noch nie jemand gemerkt?

Enno

Enno

2017-06-19 18:25

administrator   ~0007291

PFLANZE und ZUECHTE voneinander trennen wäre mal eine gute Idee.

Enno

Enno

2017-06-24 11:38

administrator   ~0007297

Ich habe das gerade getrennt, so dass die Befehle weiterhin austauschbar sind, aber der vom Spieler gegebene erhalten bleibt. Fuer's Erste ist das die einfachste Loesung, glaube ich.

Enno

Enno

2017-06-24 11:39

administrator   ~0007298

gefixt in https://github.com/eressea/server/pull/702

Enno

Enno

2017-06-24 12:56

administrator   ~0007300

Du hast keine Einheit xyz. Aber ein paar andere, die den Befehl eingesetzt haben, und deren Report sieht in einer Neuauswertung von 1030 jetzt so aus:

  • Galindraés Schüler (7rrp), Heimstatt (cara), 5 Elfen, flieht, Talente: Reiten 1, Kräuterkunde 7, Tarnung 6, hat: 10 Pferde, 400 Silber, 19 Samen, "PFLANZE Samen".

  • Galindraés Schüler (ifsc), Heimstatt (cara), 5 Elfen, flieht, Talente: Kräuterkunde 7, Reiten 1, Tarnung 5, hat: 10 Pferde, 400 Silber, 35 Samen, "PFLANZE Samen".

Ich hoffe, das ist der gewünschte Zustand.

Issue History

Date Modified Username Field Change
2017-06-19 04:25 Caranthir New Issue
2017-06-19 08:27 Enno Note Added: 0007289
2017-06-19 18:20 Enno Note Added: 0007290
2017-06-19 18:25 Enno Assigned To => Enno
2017-06-19 18:25 Enno Status new => acknowledged
2017-06-19 18:25 Enno Note Added: 0007291
2017-06-19 18:25 Enno Target Version => 3.13.0
2017-06-24 11:38 Enno Note Added: 0007297
2017-06-24 11:39 Enno Status acknowledged => resolved
2017-06-24 11:39 Enno Resolution open => fixed
2017-06-24 11:39 Enno Fixed in Version => 3.13.0
2017-06-24 11:39 Enno Note Added: 0007298
2017-06-24 12:56 Enno Note Added: 0007300
2017-12-05 18:49 Enno Status resolved => closed