Eintragsdetails ansehen
ID | Projekt | Kategorie | Sichtbarkeit | Meldungsdatum | Zuletzt aktualisiert |
---|---|---|---|---|---|
0002225 | Eressea | General | öffentlich | 2016-08-06 12:41 | 2017-12-05 19:50 |
Reporter | Enno | Bearbeitung durch | Enno | ||
Priorität | normal | Schweregrad | kleinerer Fehler | Reproduzierbar | nicht getestet |
Status | geschlossen | Lösung | erledigt | ||
Zielversion | 3.9.1 | Behoben in Version | 3.9.1 | ||
Zusammenfassung | 0002225: Doppelte Marktwirkung? | ||||
Beschreibung | Ich fürchte, die Märkte in E3 und E4 haben eine doppelte Wirkung. Sie sind sowohl in Lua (markets.lua) als auch in C (market.c) implementiert. Ursprünglich dachte ich, die C-Implementierung sei inaktiv, aber in config.json steht "modules.markets": true, was do_markets ausführt. | ||||
Schritte zur Reproduktion | Keine Ahnung, ich werde mir mal einen Markt Live ansehen müssen, um das zu bestätigen. | ||||
Tags | Keine Tags zugeordnet. | ||||
Partei | 0 | ||||
Spiel | E3 | ||||
Report | 0 | ||||
Märkte zahlen in der Tat doppelt aus, einmal in Lua, einmal in C. Schitt. |
|
Zur Lösung: Ich würde ja am liebsten den Lua-Code behalten, rein aus Prinzip (was als Skript gemacht werden kann, sollte Skript sein). Allerdings sind die tests in tests/e3/rules.lua alle für den C Code geschrieben, sie rufen eressea.process.markets() auf, und benutzen nicht das eressea/markets.lua Modul. Es wäre also wohl ein guter Anfang, diese Tests auf den Lua-Code anzuwenden, und mal zu gucken, ob der eigentlich das Richtige tut. |
|
Wie erwartet ist der Lua-Code nicht kompatibel zu den Tests, mehrere Fehler treten auf, wenn ich das austausche (was sehr einfach ging, Lob an den Autor). Dann ist es evtl. am einfachsten, den Lua-Code zu löschen, oder zumindest zu deaktivieren. Das löst das unmittelbare Problem (doppelte Wirkung), auch wenn es weniger schön ist. |
|
Änderungsdatum | Benutzername | Feld | Änderung |
---|---|---|---|
2016-08-06 12:41 | Enno | Neuer Eintrag | |
2016-08-06 17:22 | Enno | Notiz hinzugefügt: 0006681 | |
2016-08-06 21:33 | Enno | Notiz hinzugefügt: 0006684 | |
2016-08-06 21:33 | Enno | Bearbeitung durch | => Enno |
2016-08-06 21:33 | Enno | Status | neu => zugewiesen |
2016-08-06 21:48 | Enno | Notiz hinzugefügt: 0006685 | |
2016-08-06 21:55 | Enno | Status | zugewiesen => erledigt |
2016-08-06 21:55 | Enno | Lösung | offen => erledigt |
2016-08-06 21:55 | Enno | Behoben in Version | => 3.9.1 |
2016-08-06 21:55 | Enno | Notiz hinzugefügt: 0006686 | |
2016-08-08 12:30 | Enno | Zielversion | => 3.9.1 |
2016-08-20 15:09 | Enno | Beziehung hinzugefügt | verwandt mit 0002229 |
2017-12-05 19:50 | Enno | Status | erledigt => geschlossen |