novendraap / androminion

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

No matter how many Dark Ages cards are in setup, possibility of Shetlers is alawys 0% #639

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. Start a new game.  Select Random, and then "Random Dark Ages"
2. Look at the first box of the game log along the bottom...

What is the expected output? What do you see instead?
In the first box of the game log, part of that will indicate the chances of 
Shelters being used as per the Dominion: Dark Ages rulebook.  This figure is 
ALWAYS 0%.  In this example, all cards are from Dark Ages, so Shelters 
should've been guaranteed.

What version of the product are you using? On what operating system?
4.02 (with Guilds).  Android 4.2.2

Please provide any additional information below.
Platinum-Colony does work correctly.  If you play a game with say, 4 Prosperity 
cards, then there will be a 40% chance of a P/C game.  I've played enough games 
on this version to see them come out at times.  Again, whenever there are 1 or 
more DA cards, it's always 0% for Shelters.

Original issue reported on code.google.com by aya...@gmail.com on 4 May 2014 at 6:05

GoogleCodeExporter commented 8 years ago
Oh, just to be clear, the odds printed isn't the only thing incorrect, as it's 
impossible to have Shelters in play unless you force that option from the 
settings.

Original comment by aya...@gmail.com on 4 May 2014 at 6:15

GoogleCodeExporter commented 8 years ago
Is there any version available with a fix of this issue. It's working in older 
versions.

Original comment by fabian.s...@googlemail.com on 8 Nov 2014 at 10:52

GoogleCodeExporter commented 8 years ago
Issue 642 has been merged into this issue.

Original comment by andromin...@gmail.com on 7 Mar 2015 at 6:20

GoogleCodeExporter commented 8 years ago
chanceForShelters was incorrect because of misspelling
Fixed in r916 (multiplayer).

Original comment by andromin...@gmail.com on 15 Mar 2015 at 11:04

GoogleCodeExporter commented 8 years ago
So to get these changes, do we need to find an updated version with r916 
implemented?  Would that be in the Source tab?

Original comment by aya...@gmail.com on 16 Mar 2015 at 6:01

GoogleCodeExporter commented 8 years ago
To get r917 see issue 672

Original comment by andromin...@gmail.com on 23 May 2015 at 9:44