activeNextPlayer() includes eliminated players
Posted: 09 November 2018, 01:18
I suspect https://en.boardgamearena.com/#!bug?id=11368 is a BGA framework bug. It appears $this->activeNextPlayer() does not check for player elimination. In game https://en.boardgamearena.com/#!gamerev ... e=45466119 player adamjthompson00 is eliminated @ move 8. The system sets him to be the active player @ end of move 17, which triggers the bug. Why was he made active again?
If a player is eliminated, the framework should never make him active under any circumstance. All of these functions need to ignore eliminated players:
If a player is eliminated, the framework should never make him active under any circumstance. All of these functions need to ignore eliminated players:
- $this->activeNextPlayer()
- $this->activePrevPlayer()
- $this->gamestate->changeActivePlayer( $player_id )
- $this->gamestate->setAllPlayersMultiactive()
- $this->gamestate->setAllPlayersNonMultiactive( $next_state )
- $this->gamestate->setPlayersMultiactive( $players, $next_state, $bExclusive = false )
- $this->gamestate->setPlayerNonMultiactive( $player_id, $next_state )