Eintragsdetails ansehen
ID | Projekt | Kategorie | Sichtbarkeit | Meldungsdatum | Zuletzt aktualisiert |
---|---|---|---|---|---|
0002170 | ECheck | General | öffentlich | 2015-12-04 21:49 | 2017-03-17 20:35 |
Reporter | Julian | Bearbeitung durch | MrTurner | ||
Priorität | niedrig | Schweregrad | kleinerer Fehler | Reproduzierbar | immer |
Status | erledigt | Lösung | doppelt | ||
Zusammenfassung | 0002170: 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. | ||||
Tags | Keine Tags zugeordnet. | ||||
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? |
|
Gleiches bei "VERGISS" (In der Auflistung steht VERGESSEN) |
|
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. |
|
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. |
|
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! |
|
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. |
|
Ä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 |