Eintragsdetails ansehen

IDProjektKategorieSichtbarkeitZuletzt aktualisiert
0002379Eressea[Alle Projekte] Generalöffentlich2018-03-20 11:51
ReporterSoltharBearbeitung durchEnno 
PrioritätnormalSchweregradkleinerer FehlerReproduzierbarnicht getestet
Status erledigtLösungerledigt 
Produktversion3.14.1 
Zielversion3.15.0Behoben in Version3.15.0 
Zusammenfassung0002379: Insekten dürfen mit beliebigen Gebäuden handeln?
Beschreibung

Ich stolperte über folgenden Code:

economy.c:1598 in buy()
if (u_race(u) == get_race(RC_INSECT)) {
/ entweder man ist insekt, oder... /
if (r->terrain != newterrain(T_SWAMP) && r->terrain != newterrain(T_DESERT)
&& !rbuildings(r)) {
cmistake(u, ord, 119, MSG_COMMERCE);
return;
}
}

Da steht, Insekten dürfen nicht handeln, wenn sie nicht in Sumpf oder Wüste sind und es kein Gebäude ist. Das bedeutet, sie dürfen handeln, wenn es ein Gebäude gibt, egal welches. Alle anderen brauchen eine Burg. Das klingt falsch. Außerdem ist da jede Menge duplizierter Code. Das muss ich mir noch mal ansehen.

Schritte zur Reproduktion

1

Partei1wpy
SpielE2
Report1040

Notizen / Dateien

Enno

Enno

2017-11-04 19:05

Administrator   ~0007599

Wir haben scheinbar keinerlei Tests für KAUFE oder VERKAUFE, ich werde mal ein paar bauen, und schauen ob ich dabei diesen Bug reproduzieren kann.

Enno

Enno

2017-11-05 15:16

Administrator   ~0007600

Die Regel, um die es hier geht, lautet konkret: Insekten können in Wüsten und Sümpfen auch ohne Burgen handeln.

Einfachster Test: Region ohne Burg, Insekt handelt, bekommt ein Handelsgut. Failed.

Enno

Enno

2017-11-05 19:17

Administrator   ~0007601

Test für Insekten im Sumpf geschrieben, Bug gefixt.

Eintrags-Historie

Änderungsdatum Benutzername Feld Änderung
2017-10-22 21:47 Solthar Neuer Eintrag
2017-11-04 19:04 Enno Bearbeitung durch => Enno
2017-11-04 19:04 Enno Status neu => zugewiesen
2017-11-04 19:05 Enno Notiz hinzugefügt: 0007599
2017-11-05 15:16 Enno Notiz hinzugefügt: 0007600
2017-11-05 19:17 Enno Status zugewiesen => erledigt
2017-11-05 19:17 Enno Lösung offen => erledigt
2017-11-05 19:17 Enno Behoben in Version => 3.15.0
2017-11-05 19:17 Enno Notiz hinzugefügt: 0007601
2017-12-05 19:51 Enno Produktversion => 3.14.1
2017-12-05 19:51 Enno Zielversion => 3.15.0
2018-03-19 18:22 Xolgrim Beziehung hinzugefügt verwandt mit 0002429
2018-03-20 11:50 Enno Beziehung ersetzt hat Duplikat 0002429
2018-03-20 11:51 Enno Beziehung gelöscht hat Duplikat 0002429