Update directeve
Fixes a potential null reference...

Adds more debugging to help toward fixing (still broken!): https://github.com/ISeeDEDPpl/Questor/issues/434
removes some noise from the debugCombatTarget logs
Adjust the Debug logging for potentialCombatTargets to use the __entities list we are building... this may take some more logging changes before its perfect

Adjust the EntityStatistics log that is spit out when using the LogWhatIsOnGrid action (in mission XML) or the manual button states tab "log entities on grid" to include IsNPCByGroupID info
Add EntityCache IsIgnored

Before we target anything we make sure that it is not ignored...
This also removes the IsSentry checks from some Entitycache methods so that we can and do kill sentries when using specific "kill" actions
fix for null ref... this was caused in the last commit. fixed now.
Change Cache.Instance.IgnoreTargets.Contains stuff to use EntityCache IsIgnored
Fix for E-War priorityTarget issue, why the hell this was in there this way Ill never know.
ok, the comments int he code for the E-war target handling were incorrect, reverting the last change the fixing the comments to not totally suck.
target priority targets (usually e-war) and then primarytarget (basically kill command targets)...
Push the check for sentries down a layer... we dont check generally when pulling the list of valid targets, we do however filter them out (or not) when targeting (non-priority targets) in the TargetCombatants() routine. This means that you can use a Kill action and it will DEFINATELY work...
Fix clearpocket action to obey the KillSentries setting
Corrects these actions to filter out sentries if KillSentries == false
lower the NextOrbit timer when orbiting something to 10 seconds plus a random time between 1 and 15 seconds
Fix panic resume issues. This makes sure that questor is waiting at least 30 seconds after the last session change before trying to undock again (and adds some randomness to the timer.)
sort bookmarks by local system first...
unused local variable
Spacing/formatting fixes
explicitly define what type of var to use, in my opinion it makes it more clear what is going on...
Add anything that "IsSentry" to the NPCByGroupID list...
Revert "explicitly define what type of var to use, in my opinion it makes it more clear what is going on..."

This reverts commit 20ef28d2a17feba3c5559bbaad2b3cf5c4481842.