Removed the use of a static instance and instead have the Variables/CombatScript instance in the constructor/method of classes that require them.  The LoopTask created in the AutomaticPathMaker now gets interrupted when required to stop (I'm not sure this is the right way to stop it).