Whebon / dale-of-merchants

Board Game Arena adaptation of Dale Of Merchants
Other
2 stars 0 forks source link

Interaction: Whirligig and Quality Inspection #106

Open Whebon opened 1 month ago

Whebon commented 1 month ago

Whirligig moves all cards to the opponent hands under the hood). This might trigger the Quality Inspection where it is not allowed to.

Whebon commented 1 month ago

A similar problem applies to Cunning Neighbour and Quality Inspection:

For getAllDatas purposes, the opponent's hand is actually moved to the player's limbo (and returned to the opponent after the action). If the Quality Inspection hook would exist, this would cause a faulty trigger.

//cunningNeighbourWatch
$cards = $this->cards->getCardsInLocation(HAND.$opponent_id);
$this->cards->moveCards($this->toCardIds($cards), LIMBO.$player_id);

//cunningNeighbourReturn
$cards = $this->cards->getCardsInLocation(LIMBO.$player_id);
$this->cards->moveCards($this->toCardIds($cards), HAND.$opponent_id);