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

[0.50.01 Nightly] Loading existing campaign causes clan_tech_knowledge error. #4890

Closed
7 tasks done
Tzahr opened this issue Sep 26, 2024 · 2 comments · Fixed by #4893
Closed
7 tasks done

[0.50.01 Nightly] Loading existing campaign causes clan_tech_knowledge error. #4890

Tzahr opened this issue Sep 26, 2024 · 2 comments · Fixed by #4893

Comments

@Tzahr
Copy link

Tzahr commented Sep 26, 2024

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 MekHQ 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 *

Low (Minor/Nuisance): Minor glitches or cosmetic issues that don’t affect gameplay and occur rarely.

Brief Description *

When loading an older, existing campaign, it will display the following message directly prior to loading:

image

The log shows variations of the following error:

04:11:35,329 ERROR [mekhq.campaign.personnel.Person] {SwingWorker-pool-2-thread-1}
mekhq.campaign.personnel.Person.generateInstanceFromXML(Person.java:2605) - Error restoring advantage: {}

This message will appear multiple times.
Clicking 'OK' through each of them will proceed with the campaign file loading fine. The affected Techs do not seem impacted.

This is highly like related to the removal of the Clan Tech Specialist SPA.

Steps to Reproduce

Load a campaign with Techs that have the Clan Tech SPA.

Operating System *

Windows 11

Java Version *

21.0.3

MekHQ Suite Version *

Free Text (type manually)

Custom MekHQ Version

Nightly 0.50.01 - 6217756

Attach Files

mekhq.log

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
@IllianiCBT
Copy link
Collaborator

Can you upload a campaign file that includes a character with the removed SPA. Will make testing easier.

@IllianiCBT
Copy link
Collaborator

Closing as resolved

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