View Issue Details

Jump to NotesJump to History
IDProjectCategoryView StatusDate SubmittedLast Update
0002214EresseaGeneralpublic2016-06-17 21:452025-01-17 05:22
ReporterSolthar Assigned ToEnno  
PrioritynormalSeverityminorReproducibilityhave not tried
Status closedResolutionfixed 
Product Version3.9.1 
Target Version3.15.0Fixed in Version3.15.0 
Summary0002214: Abtreibende Schiffe nehmen Schaden
Description

Sollten sie in E3 aber nicht.

Additional Information

Irgendwas ist da in Zusammenhang mit #2164 kaputt gegangen.

TagsNo tags attached.
Factionall
GameE3
Report360

Relationships

related to 0002164 closedEnno Schiffe nehmen beim Abtreiben durch Stürme keinen Schaden 
related to 0002212 closedSolthar Zusätzlicher Schiffschaden nach Kampf durch neuen Überladeschutz 

Activities

Enno

Enno

2017-09-04 15: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 15: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 17: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-15 23:29

administrator   ~0007673

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

Enno

2017-12-25 22:23

administrator   ~0007712

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

Issue History

Date Modified Username Field Change
2016-06-17 21:45 Solthar New Issue
2016-06-17 21:45 Solthar Relationship added related to 0002164
2016-06-17 21:46 Solthar Relationship added related to 0002212
2017-09-04 15:40 Enno Note Added: 0007465
2017-09-04 15:40 Enno Assigned To => Enno
2017-09-04 15:40 Enno Status new => assigned
2017-09-04 15:42 Enno Note Added: 0007466
2017-09-04 17:32 Enno Note Added: 0007467
2017-12-05 18:43 Enno Status assigned => confirmed
2017-12-15 23:29 Enno Assigned To Enno =>
2017-12-15 23:29 Enno Status confirmed => acknowledged
2017-12-15 23:29 Enno Product Version => 3.9.1
2017-12-15 23:29 Enno Target Version => 3.15.0
2017-12-15 23:29 Enno Note Added: 0007673
2017-12-25 22:23 Enno Assigned To => Enno
2017-12-25 22:23 Enno Status acknowledged => resolved
2017-12-25 22:23 Enno Resolution open => fixed
2017-12-25 22:23 Enno Fixed in Version => 3.15.0
2017-12-25 22:23 Enno Note Added: 0007712
2025-01-17 05:22 Xolgrim Status resolved => closed