janisozaur / androminion

Automatically exported from code.google.com/p/androminion
1 stars 0 forks source link

Update Ambassador logic with non-supply cards #429

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Use Ambassador to reveal a non-supply card from your hand
2. Card is returned to pile (even if not in the supply) and handed out to 
opponents (even if the card names don't match, e.g. Ruins)

What is the expected output? What do you see instead?
Ambassador only interacts with cards in the supply, so Madman, Mercenary, and 
Spoils (Dark Ages) shouldn't be returned to their piles and distributed to 
opponents. This is similar to the interaction between Ambassador and Prizes. 
However, it should still be legal to reveal them (I think).

Knights and Ruins are slightly different, in that they are part of the Supply, 
but the different names might prevent cards from being passed to all opponents. 
For example, if a player ambassadors a Ruined Market and returns it to the 
Supply, the first opponent to the left will gain that card. Subsequent 
opponents will only receive Ruined Markets from the Supply if the card on top 
of the Ruins pile is a Ruined Market.

Original issue reported on code.google.com by Thomas.S...@gmail.com on 14 Sep 2012 at 6:13

GoogleCodeExporter commented 9 years ago
Oh, and this goes for Shelters as well...

Original comment by Thomas.S...@gmail.com on 14 Sep 2012 at 6:17

GoogleCodeExporter commented 9 years ago

Original comment by buralien on 15 Sep 2012 at 5:29

GoogleCodeExporter commented 9 years ago
Some more things:
When Ambassador returns a Ruins, each player gains one, instead of just the 
first one
When Death Cart is returned by Ambassador, other players don't gain 2 Ruins 
when they gain a Death Cart

Original comment by buralien on 15 Sep 2012 at 5:31

GoogleCodeExporter commented 9 years ago
This issue was closed by revision r668.

Original comment by buralien on 15 Sep 2012 at 12:35

GoogleCodeExporter commented 9 years ago
Ambassador still doesn't work correctly with ruins and knights.  I've opened 
issue 485 with details.

Original comment by August.D...@gmail.com on 7 Nov 2012 at 7:07