search

snollygolly / sourceio-automation

:video_game: A bot to automatically play s0urce.io for you
MIT License
28 stars 27 forks source link

Fixed some gui and loop issues #30

Closed LeeConnelly12 closed 7 years ago

LeeConnelly12 commented 7 years ago

Each time app.start() was ran it would create more of the same loops causing the bot to hack too quickly.

If you closed the GUI and then entered app.start() it would append a another GUI as well as show the original GUI, and you would be unable to stop the bot.

I have added null checks to the intervals so that the intervals are only ran if they're not already running. app.start() now checks to see if GUI has been appended as apposed to being visible or not. If the GUI was not visible it would run the GUI function and create duplicate events attached to the GUI controls that made the bot unable to stop.

snollygolly commented 7 years ago

Looks good to me, thanks for the PR! :shipit: