Assigning certain anti-personnel weapons to Battle Armor fails to register

[VictorMorson]

Several anti-personal weapons - for example the Grenade Launcher (Auto) - can be assigned to BattleArmor. Unfortunately, you cannot drag the weapon onto an arm (showing 0 crits), and when you attempt to auto-assign, the weapon does not mount properly and shows it's listing in the "Body," and isn't usable.

This seems to occur in both Clan and IS BattleArmor.

Steps to Repro:

1. Setup a new battle armor
2. Add a Grenade Launcher (Auto) to the weapons list
3. Attempt to assign the grenade launcher; note it cannot be dragged
4. Autofill the crits and notice it appears in the body, but does not register.

[pheonixstorm]

They should be mounted on an APWM or Armored Glove but after testing this the tooltip only shows them as being able to go on the armored glove. Any other attempt at mounting (such as APWM) fails.

[BLOODWOLF333]

MML 0.1.56 Made a new IS BA Assault type, right clicked the grenade launcher and assigned it to the armored glove and it appears to work correctly. But it is true that you cannot click and drag the grenade launcher over into the armored glove.

[arlith]

Alright, ultimately the issue here is that the Auto Fill was trying to add things that it shouldn't. I updated UnitUtil.isValidLocationto have some BA-specific code. Basically, infantry weapons can't be mounted directly, they must be mounted either in an APM for normal anti-personnel weapons, or in an armored glove for infantry support weapons. The Auto Grenade Launcher is a support weapon and must go in an armored glove. There is an asterisk on the entry in the unallocated equipment table, and the tooltip describes this. This can be slightly confusing but I don't know that there's a better way to communicate that. Now, Auto Fill ignores any infantry weapons, as it should.