Eintragsdetails ansehen

Zu Notizen springenZu Historie springen
IDProjektKategorieSichtbarkeitMeldungsdatumZuletzt aktualisiert
0002214EresseaGeneralöffentlich2016-06-17 23:452017-12-25 23:23
ReporterSolthar Bearbeitung durchEnno  
PrioritätnormalSchweregradkleinerer FehlerReproduzierbarnicht getestet
Status erledigtLösungerledigt 
Produktversion3.9.1 
Zielversion3.15.0Behoben in Version3.15.0 
Zusammenfassung0002214: Abtreibende Schiffe nehmen Schaden
Beschreibung

Sollten sie in E3 aber nicht.

Zusätzliche Informationen

Irgendwas ist da in Zusammenhang mit #2164 kaputt gegangen.

TagsKeine Tags zugeordnet.
Parteiall
SpielE3
Report360

Eintrags-Beziehungen

verwandt mit 0002164 geschlossenEnno Schiffe nehmen beim Abtreiben durch Stürme keinen Schaden 
verwandt mit 0002212 erledigtSolthar Zusätzlicher Schiffschaden nach Kampf durch neuen Überladeschutz 

Notizen / Dateien

Enno

Enno

2017-09-04 17:40

Administrator   ~0007465

Der Schaden (Default 2%) ist abhängig von rules.ship.damage_drift. Im Code gibt es sonst keine weiteren Hinweise auf einen Untershcied von Spiel zu Spiel. Steht das in den Regeln, dass E3 anders ist?
Enno

Enno

2017-09-04 17:42

Administrator   ~0007466

In 2164 gibt es eine Änderung an config.json für alle drei Spiele, wo der Eintrag entfernt wird.

z.B: https://github.com/eressea/server/pull/407/files#diff-f19559f823387f6727699cef18da222aL64
Enno

Enno

2017-09-04 19:32

Administrator   ~0007467

An dieser Stelle im Code sind wir uns gegenseitig auf die Füße getreten, glaube ich. Ich hatte Schaden durch Abtreiben erst ganz entfernt, dann hat sich @Solthar beschwert, dass er das Feature eigentlich mag, und es wieder eingebaut, aber das war wohl nur für überladene Schiffe geplant? Ich verstehe das nicht mehr komplett, was hier vor sich ging: https://github.com/eressea/server/pull/408

Die Regeln im Wiki sagen zu E3 jedenfalls: "Abtreibende Schiffe nehmen keinen Schaden mehr", ohne dort Unterschiede zu machen, aus welchem Grund das Schiff abtreibt. So gesehen sollte rules.ship.damage_drift dort wohl 0 sein?
Enno

Enno

2017-12-16 00:29

Administrator   ~0007673

Um diesen Bug könnten wir uns echt Mal kümmern, glaube ich.
Enno

Enno

2017-12-25 23:23

Administrator   ~0007712

E3 config gefixt und einen Test für die Variable nachgereicht.

Eintrags-Historie

Änderungsdatum Benutzername Feld Änderung
2016-06-17 23:45 Solthar Neuer Eintrag
2016-06-17 23:45 Solthar Beziehung hinzugefügt verwandt mit 0002164
2016-06-17 23:46 Solthar Beziehung hinzugefügt verwandt mit 0002212
2017-09-04 17:40 Enno Notiz hinzugefügt: 0007465
2017-09-04 17:40 Enno Bearbeitung durch => Enno
2017-09-04 17:40 Enno Status neu => zugewiesen
2017-09-04 17:42 Enno Notiz hinzugefügt: 0007466
2017-09-04 19:32 Enno Notiz hinzugefügt: 0007467
2017-12-05 19:43 Enno Status zugewiesen => bestätigt
2017-12-16 00:29 Enno Bearbeitung durch Enno =>
2017-12-16 00:29 Enno Status bestätigt => anerkannt
2017-12-16 00:29 Enno Produktversion => 3.9.1
2017-12-16 00:29 Enno Zielversion => 3.15.0
2017-12-16 00:29 Enno Notiz hinzugefügt: 0007673
2017-12-25 23:23 Enno Bearbeitung durch => Enno
2017-12-25 23:23 Enno Status anerkannt => erledigt
2017-12-25 23:23 Enno Lösung offen => erledigt
2017-12-25 23:23 Enno Behoben in Version => 3.15.0
2017-12-25 23:23 Enno Notiz hinzugefügt: 0007712