MArcomage

Free multiplayer on-line fantasy card game

Please log in

Spoon on 16:25, 1. Oct, 2012
This card currently seems to give no more than 1 extra resource. Will post replay link once I have one.
Fithz Hood on 21:38, 1. Oct, 2012
I never noticed it but you are probably right.
In the code the $tmp variable is used two times in a conflictual way making the number of resources gained random.
Wow, I used it so many times and I didn't saw this bug.
DPsycho on 22:40, 1. Oct, 2012
Aha, you're right! I read the code earlier and didn't spot it, but it looks like if the first randomly assigned resource is a brick or a gem, it will stop counting there, and if it assigns a recruit, it will grant one more resource since it now thinks the total number of Aria is 2.
Mojko on 06:35, 2. Oct, 2012
@_@
Mojko on 07:35, 2. Oct, 2012
Thanks for reporting. U-mage applied the fix a while ago, you can check the code for yourself :) Anyway, me and Fithz both tested the cards before the update, but it seems we got the correct card effect results by chance. I know I played the card at least twice with only one Aria card in hand (the Windmill itself) and I got 1 random resource.
dindon on 09:21, 8. Oct, 2012
Fithz wrote:
In the code the $tmp variable is used two times in a conflictual way

Truly the best variable name.
Mojko on 12:00, 8. Oct, 2012
I've seen worse - like $t1. Anyway, such variable names are acceptable when used in very small context (like 4 lines of code). I agree that a better name could be used, though.