fix for QuestorManager action grab: thank you, Daxten
Fix for possible exceptions when logging in Arm.MoveDrones
add more logging to the "outofammo" condition caused by a bringitem action in a mission XML and a missing item (bringitem - the item is required, bringoptionalitem the item is optional)

this also changes the bring and bringoptional portions of arm.moveitems to only check the ammo and loot hangars if they are defined (if they are not defined then we only need to check the itemhangar)
Stacksize is correct in this case, quantity shows -1 for constructed items (like drones that were in your drone bay)

Enforce CombatShipName and SalvageShipName being filled in, you do not have to have a salvageship, but you do have to have a unique shipName configured for each.
Merge branch 'BleedingEdge' of git://github.com/ISeeDEDPpl/Questor into BleedingEdge

Conflicts:
	Questor.Modules/Actions/Grab.cs