Bugs

[bnye88]

The AI is able to purchase Transmutes with an empty hand once the copper pile depletes in a 2p game.

[Elestan]

The AI continues to play Sea Hag, even after all the curses are gone.

[Elestan]

After buying Mandarin, the player is prompted whether they want to put their treasures on top of their deck. If they say 'No', the game crashes. They should not have a choice about it at all.

[jatillisawesome]

1. I'm sure this has been noted, so sorry for mentioning it:

Young Witch makes you discard your whole hand rather than just 2 cards after you pick up.

2. The other one that only comes up in a really specific scenario:

Game crashes when your opponent has zero cards in their deck (achievable through the Goons-King's Court-Masquerade combo, or some variant thereof).

3. Just a UI thing.. Border Village would be much better displayed the way Haggler is. Currently, Border Village goes one by one with a dialogue box asking 'do you want THIS card? how about this one,' whereas Haggler (and other cards like Ironworks) are simply 'click the card costing x or less to gain.' Seems like it'd be easy enough to make Border Village match up.

Thanks for making such an awesome game btw. I play it ALL the time!

[Glenfar]

I've been periodically getting a 'subscript out of range' error. I happen to be a programmer myself, and since you were kind enough to provide the code, I went ahead and debugged it. It's happening in the gain_card method on the pile_size check when card=-1.

The way I got there is Saboteur trashed a value 3 card, and all the Copper's were gone, so there was no valid card to replace it with.

[Glenfar]

Found another one, this one can be a bit nasty - again in the gain_card method, at this section:

Code: Select all

        If seal_found = 1 Or watchtower_found = 1 Then
            If actual_player = 0 And watchtower_found = 0 Then
                intResponse = MsgBox("Would you like to put " & names(discard(actual_player, discard_count(actual_player) - 1)) & " on top of your deck?", vbYesNo)
            End If
            If intResponse = vbYes Then
                Call put_on_top_of_deck(actual_player, card, 1)
                discard(actual_player, discard_count(actual_player) - 1) = -1
            End If
        End If


The problem is the code is assuming that the card you gained is on top of the discard pile ... if it isn't, then effectively (assuming response is yes) you trash the top card of your discard pile and gain a second copy of the card you had just gained. And if your discard pile was empty, you get a subscript out of range error.

I got there by playing Bureaucrat with Watchtower in my hand.

[Elestan]

If you play Warehouse as the last card in your hand with no cards in your deck or discard, the game hangs, insisting that you discard.

[Elestan]

Swindler and Saboteur appear to be able to affect cards in your hand, at least if your deck is empty. They should not.

[Elestan]

Haggler should be able to be used to gain a Curse; it currently cannot.

[Elestan]

Something isn't right with Inn. After buying one, and using its ability to move action cards from my discard to my deck, several of the cards previously in my deck were missing, as though they had been overwritten by the moved cards.