Bug - player is not defined in creature.enactcontagion

scrom / Experiments

a NodeJS text adventure - deliberately coded badly to re-experience the pain of legacy code

http://mvta.herokuapp.com/

2 stars 0 forks source link

Bug - player is not defined in creature.enactcontagion #407

Closed scrom closed 9 years ago

scrom commented 9 years ago

Issue from #400 again. The contagion part of creature.tick was moved into a standalone function without any tests. Playtesting has revealed a variable neds to be passed in for player object. Oh to have refactoring toolss to catch this stuff!

Still - after 2 years, the legacy code has finally started to bite. It took far longer than expected to get here and has been much more sudden in impact. Neat!

The fun thing is I still want to finish this

scrom commented 9 years ago

Note. Sample game to play test this against is s-534174

scrom commented 9 years ago

It needs the player to be in the same location as an infected creature (I think)

scrom commented 9 years ago

confirmed fixed