Eintragsdetails ansehen

IDProjektKategorieSichtbarkeitZuletzt aktualisiert
0002170ECheckGeneralöffentlich2017-03-17 20:35
ReporterJulian Bearbeitung durchMrTurner  
PrioritätniedrigSchweregradkleinerer FehlerReproduzierbarimmer
Status erledigtLösungdoppelt 
Zusammenfassung0002170: ECHECK kennt BEFÖRDERE nicht
Beschreibung

Befehle für Partei ftkf gefunden. Rekrutierungskosten auf 130 Silber gesetzt, Warning Level 0. Silberpool aktiviert.

Fehler in Zeile 660: Befehl nicht erkannt. `BEFÖRDERE'

Es wurden Befehle für 1 Partei und 324 Einheiten gelesen. Es wurde ein Fehler entdeckt.

Schritte zur Reproduktion

Befehl 'BEFÖRDERE' an den Server schicken.

TagsKeine Tags zugeordnet.

Eintrags-Beziehungen

Duplikat von 0002280 erledigtEnno Serverseitiges ECheck mahnt Fehler an wo keiner ist (befördere) 

Notizen / Dateien

MrTurner

MrTurner

2015-12-29 10:33

Tester   ~0006433

Das scheint an der Auflistung in https://github.com/eressea/echeck/blob/master/e2/de/commands.txt und dem Äquivalent für e3 zu liegen. Dort ist lediglich "Beförderung" hinterlegt.

Sind da Listen von Befehls-Schlüsselwörtern möglich?

chriull

chriull

2016-02-04 11:32

Reporter   ~0006476

Gleiches bei "VERGISS" (In der Auflistung steht VERGESSEN)

Enno

Enno

2016-04-03 13:53

Administrator   ~0006528

Das Problem ist, dass in ECheck die Implementierung nur eine Schreibweise pro Befehl zulässt, und bereits BEFÖRDERUNG kennt. Listen zu erlauben, erfordert mehr als nur eine Änderung an einer Textdatei, glaube ich.

MrTurner

MrTurner

2016-04-03 14:13

Tester   ~0006531

Das dachte ich mir fast, deshalb meine Frage.

Die nächste Frage wäre jetzt, ob es sich lohnt die Implementierung dahingehend zu erweitern. Bin zur Zeit wieder auf der Suche nach Gelegenheiten was beizutragen und würde mir das ankucken, falls die Antwort auf diese Frage "ja" lauten sollte.

Enno

Enno

2016-04-03 21:06

Administrator   ~0006532

Vor dem Echeck Code kann ich nur warnen, der ist nicht toll. Wenn ich da einen PR kriege, der diesen Bug fixt, freut mich das aber natürlich!

MrTurner

MrTurner

2016-04-09 00:45

Tester   ~0006533

Ich habe mir jetzt mal den Code angesehen... Die Implementierung ist zwar nicht explizit auf Doppeleinträge ausgerichtet, aber vertragen tut sie diese schon einigermaßen. Die eingelesenen Einträge werden in einer verketteten Liste gehalten. Bei der Befehlsprüfung wird diese dann nach dem vorgefundenen Token durchsucht und die ID des dazugehörigen Schlüsselworts entnommen. An der Stelle interessiert echeck nicht, ob mehrere Einträge zu dieser ID in der Liste hängen. Für die Prüfung an sich könnte man in der commands.txt also mehrere Einträge zu einem Schlüsselwort haben. Allerdings gibt es noch eine Funktion printkeyword(), die die Liste anhand der ID (des keys) durchsucht, wobei die Duplikate dann doch problematisch wären.

Soviel als Zwischenbericht.

Eintrags-Historie

Änderungsdatum Benutzername Feld Änderung
2015-12-04 21:49 Julian Neuer Eintrag
2015-12-05 10:24 Enno Projekt Eressea => ECheck
2015-12-29 10:33 MrTurner Notiz hinzugefügt: 0006433
2016-02-04 11:32 chriull Notiz hinzugefügt: 0006476
2016-04-03 13:53 Enno Notiz hinzugefügt: 0006528
2016-04-03 14:13 MrTurner Notiz hinzugefügt: 0006531
2016-04-03 21:06 Enno Notiz hinzugefügt: 0006532
2016-04-03 22:10 MrTurner Bearbeitung durch => MrTurner
2016-04-03 22:10 MrTurner Status neu => zugewiesen
2016-04-09 00:45 MrTurner Notiz hinzugefügt: 0006533
2017-03-17 20:35 Enno Status zugewiesen => erledigt
2017-03-17 20:35 Enno Lösung offen => doppelt
2017-03-17 20:35 Enno Beziehung hinzugefügt Duplikat von 0002280