Opened 5 years ago

Last modified 15 months 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 Changed 3 years ago by elexis

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 Changed 3 years ago by elexis

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 Changed 3 years ago by elexis

Milestone: Alpha 22Backlog
Priority: Must HaveShould Have

comment:4 Changed 2 years ago by Imarok

Isn't this fixed now?

Version 0, edited 2 years ago by Imarok (next)

comment:5 Changed 2 years ago by elexis

Not according to the commit message.

comment:6 Changed 15 months ago by Imarok

Component: UI & SimulationSimulation

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

Note: See TracTickets for help on using tickets.