Multiple problems with Knights

ll2585 / androminion

Automatically exported from code.google.com/p/androminion

0 stars 0 forks source link

Multiple problems with Knights #530

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago

What steps will reproduce the problem?
1. When an AI player plays a Knight and it trashes another Knight, the played 
Knight does not get trashed despite showing in the trash pile.
2. When the AI player buys a Knight, it sometimes buys a Knight that is not on 
top of the Knight pile.
3. Dame Josephine is the worst offender of the above, though other knights 
might have the same problem. She is the most noticeable of the group.

What is the expected output? What do you see instead?
When a Knight trashes another Knight, the played Knight must be trashed.
When a Knight is bought, only the top Knight showing of the pile is allowed to 
be bought.

What version of the product are you using? On what operating system?
Androminion 4.01

Please provide any additional information below.

Original issue reported on code.google.com by pagemast...@gmail.com on 2 Jan 2013 at 4:22

GoogleCodeExporter commented 9 years ago

I have seen some AI to have knights when knight pile had 10 cards.

Original comment by ras.na...@gmail.com on 3 Jan 2013 at 11:41

GoogleCodeExporter commented 9 years ago

Another strange thing: why does this knight have cost 4?

Notice also that there is a knight in my hand, one in Sarah's hand and 9 in the 
pile.

Log also reveals that Sarah gained it with an Armory.

What version of the product are you using? On what operating system?
Androminion 4.01 from google play

Original comment by ras.na...@gmail.com on 3 Jan 2013 at 11:44

Attachments:

log_2013-01-03_17-21-24.txt
[Knight cost 4.png](https://storage.googleapis.com/google-code-attachments/androminion/issue-530/comment-2/Knight cost 4.png)

GoogleCodeExporter commented 9 years ago

It had cost 4 in teh beginning it looks like...

Cards in play
---------------
[...]
SirMartin (4) +2 Buys (...)

Original comment by ras.na...@gmail.com on 3 Jan 2013 at 11:48

GoogleCodeExporter commented 9 years ago

This issue was closed by revision r778.

Original comment by tkden...@gmail.com on 9 Jan 2013 at 2:19

Changed state: Fixed

GoogleCodeExporter commented 9 years ago

Sir Martin actual cost is 4 and can be gained with Armory: 
http://wiki.dominionstrategy.com/index.php/Knight

Original comment by tkden...@gmail.com on 9 Jan 2013 at 2:21

GoogleCodeExporter commented 9 years ago

I couldn't reproduce having more than 10 knights in hands and pile. If it's 
still happening in the latest build, can you please attach a log file?

Original comment by tkden...@gmail.com on 9 Jan 2013 at 2:23