Changes between Initial Version and Version 1 of Ticket #5816


Ignore:
Timestamp:
Aug 31, 2020, 9:32:12 PM (4 years ago)
Author:
bb
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #5816 – Description

    initial v1  
    1 Arguably observers shouldn't send commands. However they can. This (except in some rare cases #5140) does not cause the sim-state to change (D2810 is there to fix everything sim-related). Nevertheless, the commands should be disabled from clientside too, to avoid useless network traffic and unneeded replay entries.
     1Arguably observers shouldn't send commands. However they can. This (except in some rare cases #5140) does not cause the sim-state to change (phab:D2810 is there to fix everything sim-related). Nevertheless, the commands should be disabled from clientside too, to avoid useless network traffic and unneeded replay entries.
    22
    33One can send a command as observer by:
    4 Start a game as observer
    5 Switch to a player
    6 Select a unit
     4Start a game as observer.
     5Switch to a player.
     6Select a unit.
    77Press "h" (stop hotkey, works with more hotkeys, back-to-work probably)
    8 Now look at the replay and see `cmd -1`
     8Now look at the replay and see `cmd -1`.
    99
    1010This could be fixed on the JS side, but that would be checking and fixing every individual command. Instead, we could disallow commands for observers to be processed in PostNetworkCommand (or wherever down the line seems suitable) in the engine.