Skeen / HearthstoneBot

A (DISCONTINUED) bot for the Blizzard Entertainment game: Hearthstone
62 stars 28 forks source link

Mulligan errors and replace stray messages #16

Open ghost opened 10 years ago

ghost commented 10 years ago

Currently the mulligan phase, causes errors, and this results in stray "Replaced" messages, that do not disappear. It seems to be a timing issue, namely that mulligan is triggered before the game is able to handle mulligan. A hacky fix would be a pre-delay.

ghost commented 10 years ago

The issue is an exception from within MulliganManager.ToggleHoldState

helios- commented 10 years ago

I think it's m_UIbuttons that's null. Waiting for that seems to fix it: https://github.com/helios-/HearthstoneBot/commit/c892be0671769dcd465914c2fec1d88eddcd7e71