reorganize Arm.cs to use similar code to Unloadloot, we now wait for all itemlocks to clear after moving any item from hangar to hangar

we also use the same routines to arm BringItem, BringOptionalItem, CapBoosters, Drones, etc
This also moves CombatShipName to Combat.cs
If you notice any issues in UnloadLoot or Arm please open an issue on github and include a snippit from the console log: https://github.com/ISeeDEDPpl/Questor/issues