Eintragsdetails ansehen

IDProjektKategorieSichtbarkeitZuletzt aktualisiert
0002225Eressea[Alle Projekte] Generalöffentlich2017-12-05 19:50
ReporterEnnoBearbeitung durchEnno 
PrioritätnormalAuswirkungkleinerer FehlerReproduzierbarnicht getestet
Status geschlossenLösungerledigt 
Produktversion 
Zielversion3.9.1Behoben in Version3.9.1 
Zusammenfassung0002225: 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.

Partei0
SpielE3
Report0

Eintrags-Beziehungen

verwandt mit 0002229 geschlossenEnno Markt ohne Unterhalt produziert trotzdem Güter 

Notizen / Dateien

Enno

Enno

2016-08-06 17:22

Administrator   ~0006681

Märkte zahlen in der Tat doppelt aus, einmal in Lua, einmal in C. Schitt.

Enno

Enno

2016-08-06 21:33

Administrator   ~0006684

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.

Enno

Enno

2016-08-06 21:48

Administrator   ~0006685

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.

Enno

Enno

2016-08-06 21:55

Administrator   ~0006686

PR: https://github.com/eressea/server/pull/533

Eintrags-Historie

Ä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