Lost and defeated militarysites don't match up

Bug #1817764 reported by GunChleoc
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
widelands
Won't Fix
Undecided
Unassigned

Bug Description

In the attached 2-player replay, the count of militarysites lost and defeated in the general statistics don't match each other

Tags: statistics
Revision history for this message
GunChleoc (gunchleoc) wrote :
Revision history for this message
GunChleoc (gunchleoc) wrote :

Screenshots

Changed in widelands:
milestone: none → build20-rc1
Revision history for this message
Toni Förster (stonerl) wrote :

I checked the replay, not entirely but I found the issue.

35:50 Blue attacks red and conquers the sentinel. 1 defeat for blue, 1 lost for red. At 37:21 blue destroys the conquered sentinel. Stats are 2 defeats for blue, 1 lost for red and 1 lost for blue.

Revision history for this message
Toni Förster (stonerl) wrote :

This one can be easily reproduced:

Destroy a military building while it is under attack.

- Blue conquers a building from red.
- Red attacks the building blue just conquered
- Blue destroys the building while under attack.
- Blue gains 1 point in defeated and 1 point in destroyed
  military buildings
- Red gains no point.

This only happens when destroying a conquered building, that's under attack. If red had destroyed the building, while blue was attacking, none of the players would have received any points.

Revision history for this message
hessenfarmer (stephan-lutz) wrote :

From my point of view the question is how hard is it to fix this and is it worth having it in b20.

Revision history for this message
kaputtnik (franku) wrote :

I wonder why this is targeted to build 20 anyhow...

Revision history for this message
GunChleoc (gunchleoc) wrote :

I did that to make sure that we look at it. Retargetet to Build 21.

Changed in widelands:
milestone: build20-rc1 → build21-rc1
Revision history for this message
GunChleoc (gunchleoc) wrote :
Changed in widelands:
status: New → Won't Fix
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.