Incorrect minimum doors required for cargo bays

[Setsul]

Prerequisites and Pre-Issue Checklist

• I'm reporting the issue to the correct repository:

• MegaMek

• MegaMekLab

• MekHQ

• I've tested the issue against at least the latest MILESTONE version

• I've asked on the MegaMek Discord about the error

• I've reviewed the BattleTech rules and MegaMekLab documentation, and I've confirmed that something isn't working as intended.

• I've searched the Github tracker and haven't found the issue listed

Severity *

Medium (Gameplay Limitation): Non-core functionality is impaired, providing a suboptimal but playable experience.

Brief Description *

A: The number of Doors on a Cargo Bay on Large Craft can not be reduced below 1.
B: This can also cause an IllegalArgumentException on saved (canonical) units that possess Cargo Bays with 0 Doors.
C: This also makes it impossible to add further Cargo Bays to Large Craft that are using the maximum number of Doors.

Steps to Reproduce

A1. Add a Cargo Bay to a Large Craft.
A2. Try to reduce the number of Doors below 1.
A3. It won't work.

B1. Load any unit with a Cargo Bay with 0 Doors. E.g. Overlord (2762)
B2. Click on the Cargo Bay's number of Doors.
B3. IllegalArgumentException.

C1. Load any unit that uses its maximum number of Doors. E.g. Overlord (2762)
C2. Select Cargo as Bay type to add.
C3. "Add Bay" is unselectable/greyed out.

Cargo Bays are expected to behave like Infantry Bays, with no Doors being required.

Operating System *

Linux

Java Version *

Temurin 17.0.13+11

MegaMek Suite Version *

Free Text (type manually)

Custom MegaMekLab Version

v0.50.01

Attach Files

No response

Final Checklist

• I've checked to make sure that this issue has not already been filed
• I'm reporting only one issue in this ticket for clarity and focus

[Sleet01]

This will likely need updates in two places:

1. the MML UI, specifically around bays;
2. the megamek/common/Bay.java class (and subclasses) where minimums and maximums are defined.