Infantry with Locomotor=Tunnel can work, or move like vehicles with Locomotor=Tunnel

[yawnallday]

Description

Infantry and aircraft (is it really necessary?) will directly use the tunneling method to move, regardless of distance, and ignore "AllowBurrowing=no". Phobos (DevBuild#44) disrupts the way infantry moves underground, and if enabled, it will cause EIP

Phobos Version

b48

Conditions to reproduce

No response

INI code

[InfantryType]
Locomotor={4A582743-9839-11d1-B709-00A024DDAFD1} ;OR Locomotor=Tunnel
MovementZone=Subterannean

Steps to reproduce

1.defines infantry：Locomotor=Tunnel
2.defines infantry：MovementZone=Subterannean
3.
...

Expected behaviour

Infantry with Locomotor=Tunnel can act normally, or move like vehicles with Locomotor=Tunnel

Actual behaviour

Infantry with Locomotor=Tunnel It will cause the game to crash

Additional context

No response

Checklist

• The issue happens on the latest official version of Phobos and wasn't fixed yet.
• I agree to elaborate the details if requested and provide thorough testing if the bugfix is implemented.
• I added a very descriptive title to this issue.
• I used the GitHub search and read the issue list to find a similar issue and didn't find it.
• I have attached as much information as possible (screenshots, gifs, videos, debug and exception logs, etc).

[TaranDahl]

I think this should be regarded as an incorrect usage. Because this Locomotor is obviously not designed to be applied to infantry. The direct criterion for distinction is whether this Locomotor uses FindInfantrySubposition to determine the destination. Jumpjet, Teleport, and Walk all do this, while Tunnel doesn't.
So I think no action needs to be taken until the day we officially apply the Tunnel loco to other types.
Needs others' opinion.

[TaranDahl]

@Phobos-developers/t1-maintainers

[Starkku]

The piggybacking part where game decides whether or not to use Drive or Tunnel based on distance to new destination is implemented only for UnitClass, no equivalent exists for other types. It is clearly not fully functional for infantry.

That said it probably still shouldn't crash especially if it doesn't in vanilla. Could either add whatever sanity checks are required to prevent Phobos from crashing or prevent this situation from arising at parsing time.