Opened 8 years ago

Last modified 5 years ago

#3601 new defect

Information Leak - I see rubble of destroyed buildings

Reported by: elexis Owned by:
Priority: Should Have Milestone: Backlog
Component: Simulation Keywords:
Cc: Patch: Phab:D635

Description (last modified by elexis)

You can see if enemy buildings are destroyed through the fog of war:

Reproduce:

  1. Open developer overlay
  2. Delete CC
  3. Change perspective to enemy

For both explored and unexplored buildings you can see the rubble.

#3179 claims to fix this, but it doesn't (completely?). It works for units but not for buildings.

The commit intends to work for buildings though, see the change in the structures template and IRC on the day it was committed:

(11:38:02) elexis: does it work for structures too?
(11:38:11) Itms: yes, it's the same

By hosting a game with a second player attached, you can even faster change to the enemy perspective. It becomes apparent that the building doesn't fall anymore, but still the rubble is spawned and visible in fog.

Change History (6)

comment:1 by elexis, 7 years ago

Description: modified (diff)
Milestone: BacklogAlpha 22
Patch: Phab:D635
Priority: Should HaveMust Have

This is an annoying bug (it can reveal your intention to build a CC near that one spot with resources to enemies) with a trivial fix in Phab:D635 (copy and paste of r16919)

comment:2 by elexis, 7 years ago

In 19782:

Hide rubble in the fog of war except to the owner, so that other players don't see deleted buildings, especially civic center foundations.
Should use a special filter once the AI supports it (#4611), so that decorative rubble can be placed by maps too.

Differential Revision: https://code.wildfiregames.com/D635
Reviewed By: fatherbushido
Refs #3601 #3910 #3179

comment:3 by elexis, 7 years ago

Milestone: Alpha 22Backlog
Priority: Must HaveShould Have

comment:4 by Imarok, 6 years ago

Isn't this fixed by that commit(r19782)?

Last edited 6 years ago by Imarok (previous) (diff)

comment:5 by elexis, 6 years ago

Not according to the commit message.

comment:6 by Imarok, 5 years ago

Component: UI & SimulationSimulation

Move tickets to Simulation as UI & Simulation got some sub components.

Note: See TracTickets for help on using tickets.