If we somehow end up here we should gotobase not gotomission...
make sure some commented out entries were updated to reflect the names of variables that have changed...

all of the fittings code likely needs some re-factoring... it appears to be doing some very odd things
Pass the agentID from the CombatMissionsBehavior over to the Statistics module so that we do not have to look it up again. THis should make the statistics module much happier generally.
This adds a setting (which should at some point just become a debug type setting...) that determines if we set the eve destination to Questors destination in all cases or not. This defaults to off, and has been disabled via a constant that was false for awhile now. This resolves a compile time warning that has been bugging everyone for awhile now.
If we run the refreshMissionItems function and we aren't using the CombatMissionsBehavior set UseFittingManager to false and return from the function early. We do not need to try to read mission XMLs and determine faction fittings, etc if we are not running combat missions.
Change the DebugWindows function to use DE functions directly rather than leaning on the functions in cache.cs that may or may not be initialized yet (like at the login screen).
Make sure that Charactername is not null (like it would be when running "dotnet questor questor" before we are logged in... when at the login screen)
whitespace
Corrects a problem with MoveTo which was padding the distanceToApproach with 5k so we didnt get stuck on things. I changed this to a default of 4k if you do not specify a distance... if you DO specify a distance it'll approach to that exact distance you specify. (in meters).

Fixes CamelCasing for most variables in CombatMissionCtrl
Add some error checking to Cache.Instance.GetAgentMissions

Fix compile error introduced in an earlier commit.
Cache.Instance.Mission = Cache.Instance.GetAgentMission(AgentID);

is causing problems in general... why we thought we could call this willy nilly if we had a mission or not is beyond me but this is likely going to need move loving before it'll be resolved correctly.
update DirectEVE.dll - requires DE support instances!

https//support.thehackerwithin.com

DirectEve
v1.330.4677.43140
Support Instance Needed

<Ganondorf> i just pushed a new directeve
<Pluckyduck> what was the update?
<Ganondorf> crash fix
<Ganondorf> more limitations for free users
<Ganondorf> and a secret bounty :P
<Pluckyduck> =) what are all the limitations?  If i am cool enough to know.
<Ganondorf> right now
<Ganondorf> skills
<Ganondorf> any market method
<Ganondorf> login
<Ganondorf> anomalies
<Ganondorf> without a support instance those will return an error
<Pluckyduck> when you say any market method, as in buying stuff for storylines?
<Ganondorf> yeah
<Pluckyduck> or valuedump
<Ganondorf> that will not work
<Ganondorf> nothing
<Ganondorf> will work
Adds a new setting: "MoveCommonMissionCompletionItemsToItemsHangar" if set to true (the default is true) common mission completion items will be moved to your local hangar

this allows them to automatically be used to complete missions and should be left at true unless you have a VERY good reason to change it.
This setting is useful if you use the corp hangar for your AmmoHangar and LootHangar, if the corp hangar were configured for both of those there was previously no way to get these items back into your local items hangar by default.
fix for compile error: this setting is not yet used, but will eventually be used to optionally sort loot into different containers.
Questor now caches the MissionList unless an update is forced or we haven't retrieved the list yet. The list is supposed to be retrieved again by AgentInteraction.cs after every mission
Adjust the order of these 2 innerspace commends so that WindowText is run 1st and then we run windowtaskbar on, which looks cleaner in the innerspace logs.
Fix CamelCase of local variables in Cleanup.cs, no behavioral changes
make sure that NavigateOnGrid respects the Settings.Instance.AvoidBumpingThings setting

General, cleanup, no behavioral changes
Spelling corrections and whitespace changes - no behavioral changes
Cleanup old no longer in use code that was commented out, rename from local variables to be CamelCased, fix some spelling issues in comments

no behavioral changes