NullPointerException from Trader after Silver runs out

[GoogleCodeExporter]

What steps will reproduce the problem?
1. Buy a Trader.
2. Empty the Silver supply.
3. Draw a hand with Trader.
4. Buy a card, or have an opponent play a card-granting action, triggering the 
Trader as a reaction.
5. Choose to gain a Silver instead of the other card.
6. The game crashes with java.lang.NullPointerException.

What is the expected output? What do you see instead?
I'm not sure what is correct--with Torturer, I can choose a Curse even after 
the Curses run out, and then I gain nothing, so maybe the same applies here.  
Or maybe I should not have the choice.  I don't have the physical game to check 
the book.

What version of the product are you using? On what operating system?
1.9.b16

Original issue reported on code.google.com by paulj...@gmail.com on 17 Dec 2011 at 3:44

[GoogleCodeExporter]

Trader works the same as Torturer.  From the Hinterlands rule book:  "If there 
are no Silvers left in the Supply, you can still reveal Trader when you gain a 
card; you gain nothing instead of the card you would have gained."

On a related note, the Rio Grande Games website has all of the rulebooks 
posted.  This is Hinterlands:  
http://www.riograndegames.com/uploads/Game/Game_396_gameRules.pdf

Original comment by August.D...@gmail.com on 17 Dec 2011 at 3:37

• Changed state: Accepted

[GoogleCodeExporter]

I fixed the buy portion of this problem, and couldn't find the problem with the 
other case, let me know if you still see this.

Original comment by fatmonkeysmile@gmail.com on 19 Dec 2011 at 9:00

• Changed state: Fixed

[GoogleCodeExporter]

Forget to note that it is fixed in 1.9.b18

Original comment by fatmonkeysmile@gmail.com on 19 Dec 2011 at 9:14

[GoogleCodeExporter]

I was unable to replicate the problem when an opponent played a card-granting 
action (Jester), so this fix should fix everything.

Original comment by August.D...@gmail.com on 19 Dec 2011 at 9:41