Originally Posted by
eponymous_coward
It's a bug because Red isn't telling the "user paused song, so wait for the user to hit play again" scenario apart from the "FA paused song for announcement, start playing when announcement is done" scenario, and is automagically starting the song again for BOTH scenarios after the FA announcement. Red should be waiting for the user to hit play again in the first scenario.
Sounds like the second "PA" pause is disregarded:
Red gets a pause event from the user (state: paused)
Red gets a pause event from the PA (state: paused)
Red gets an unpause event from the PA (state: unpaused)
A very simple solution would be to store the current pause state before the PA announcement and then restore it after. It should have been picked up and fixed during testing (I guess that is what you have a 'beta' phase for).
Red gets a pause event from the user (state: paused)
Red gets a pause event from the PA (state: paused, previous state: paused)
Red gets an unpause event from the PA (state: <previous state> [paused])
or:
Red is playing (state: unpaused)
Red gets a pause event from the PA (state: paused, previous state: unpaused)
Red gets an unpause event from the PA (state: <previous state> [unpaused])