Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Industrial battlemech armor causes NullPointerException #1440

Closed
FixDis opened this issue Feb 25, 2024 · 3 comments · Fixed by #1442
Closed

Industrial battlemech armor causes NullPointerException #1440

FixDis opened this issue Feb 25, 2024 · 3 comments · Fixed by #1442

Comments

@FixDis
Copy link

FixDis commented Feb 25, 2024

Environment

MegaMekLab v0.49.18
Windows 11
Eclipse Adoptium Java Version: 11.0.21

Description

Putting Industrial armor on a tournament-legal battlemech causes a NullPointerException

@HammerGS
Copy link
Member

We need logs.

@HammerGS HammerGS added the Waiting on Author This issue is waiting on a response from its author. label Feb 25, 2024
@FixDis
Copy link
Author

FixDis commented Feb 25, 2024

megameklab.log
?

@HammerGS HammerGS added Bug and removed Waiting on Author This issue is waiting on a response from its author. labels Feb 25, 2024
@HammerGS
Copy link
Member

HammerGS commented Feb 25, 2024

Easy to duplicate, and still exists in the Repo as off todays date.

00:24:55,512 ERROR [megameklab.MegaMekLab] {AWT-EventQueue-0}
megameklab.MegaMekLab.lambda$main$0(MegaMekLab.java:40) - Uncaught Exception Detected
java.lang.NullPointerException
	at megameklab.ui.mek.BMStructureTab.createArmorMountsAndSetArmorType(BMStructureTab.java:507)
	at megameklab.ui.mek.BMStructureTab.armorTypeChanged(BMStructureTab.java:971)
	at megameklab.ui.generalUnit.MVFArmorView.lambda$actionPerformed$2(MVFArmorView.java:351)
	at java.base/java.util.concurrent.CopyOnWriteArrayList.forEach(CopyOnWriteArrayList.java:807)
	at megameklab.ui.generalUnit.MVFArmorView.actionPerformed(MVFArmorView.java:351)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants