Reload - Code cleanup and various improvements

[johnb432]

When merged this pull request will:

• Title.
• Improvements/Changes:
  • Previously, when linking a belt, a magazine was removed upon the action start from the unit's (who was trying to link) inventory and readded in case of failure. This PR changes it: It now only removes the magazine at the end of the progressbar upon successful completion. It will check if it was able to remove the magazine; if not, the interaction is cancelled.
  • Upon completion or failure of a belt linking, a notification shows up. Reload/ReloadLauncher - Add notifications when buddy loading or linking #9202 (or a follow-up PR) could be made to disable these notifications.
  • Previously, there was some very specific behaviour concerning UGLs and checking of their ammo. This has been expanded to where if any secondary muzzle has no ammo left, it will show the number of compatible magazines on the unit.
  • CBA events are now added to connected machines that don't have interfaces. I imagine this could cause issues if the unit you are trying to link a belt to is local to a dedicated server or headless client. I don't remember coming across problems stemming from this, but it's a safety measure that doesn't impact performance greatly.
  • If displayNameShort for static weapon magazines can't be found, displayName is used instead. I don't recall encountering an issue caused from the lack of a displayNameShort entry, but it's a small safeguard that doesn't cost much.

IMPORTANT

• If the contribution affects the documentation, please include your changes in this pull request so the documentation will appear on the website.
• Development Guidelines are read, understood and applied.
• Title of this PR uses our standard template Component - Add|Fix|Improve|Change|Make|Remove {changes}.