Turret Health and Placement Bug Fix (Issue #848)

Maescool / Catacomb-Snatch

Mojang's Humble bundle source

Other

295 stars 99 forks source link

Turret Health and Placement Bug Fix (Issue #848) #878

Closed EvanCahill closed 12 years ago

EvanCahill commented 12 years ago

Fix for Issue #848: When a turret was placed the makeUpgradeableWithCosts() and thus the upgradeComplete() methods were called unnecessarily, causing the turrets maxHealth to be incremented by 10 and its heath to reset. This was fixed by moving the makeUpgradable call from init to the constructor similar to the harvester.

master-lincoln commented 12 years ago

Nice description and short solution. That's how pull requests should always be ;)