AI's Envoy logic could be easily improved

[GoogleCodeExporter]

What steps will reproduce the problem?
1. Play Envoy, drawing at least one Province and one Gold.

What is the expected output? What do you see instead?
The AI chooses to discard Province, instead of a Gold. It looks to me like the 
logic is "discard the card with the highest cost", which could be easily 
improved by simply changing it to "discard the card with the highest cost, 
unless it is a Victory card and has no other types". 

This appears to be the logic when the Patrick AI plays Pillage.

What version of the product are you using? On what operating system?
Latest unstable build as of 2/19, Android 4.1, Galaxy S3.

Original issue reported on code.google.com by wesc...@gmail.com on 19 Feb 2013 at 6:55

[GoogleCodeExporter]

Original comment by tkden...@gmail.com on 18 Mar 2013 at 5:03

• Changed state: Accepted
• Added labels: Type-Enhancement, Component-AI
• Removed labels: Type-Defect

[GoogleCodeExporter]

This issue was closed by revision r790.

Original comment by tkden...@gmail.com on 21 Mar 2013 at 2:55

• Changed state: Fixed