MegaMek / mekhq

MekHQ is a java helper program for the MegaMek game that allows users to load a list of entities from an XML file, perform repairs and customizations, and then save the new entities to another XML file that can be loaded into MegaMek.
http://megamek.org
131 stars 168 forks source link

Combat Action being awarded multiple times #4919

Closed repligator closed 2 days ago

repligator commented 4 days ago

Prerequisites and Pre-Issue Checklist

Severity *

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

Brief Description *

The combat action award is being issued after every scenario, and possibly every month as well, even if the pilot already has the award.

Steps to Reproduce

  1. Load the 1st MHQ save.
  2. Note that all 5 pilots in the pending scenario have combat experience.
  3. Start the pending scenario. /victory on the first turn.
  4. Observe as all the pilots are issued with combat action awards.

Operating System *

Linux

Java Version *

17

MekHQ Suite Version *

Free Text (type manually)

Custom MekHQ Version

https://github.com/MegaMek/mekhq/commit/7779b817d1b00ec50c9dd1912913caed20774cd2

Attach Files

CBS302530250409.cpnx.gz mekhq.log

Final Checklist

IllianiCBT commented 4 days ago

To confirm: you are not using any custom award sets?

repligator commented 4 days ago

Everything is vanilla, nothing custom.

IllianiCBT commented 2 days ago

So, what's happened is these awards were issued prior to the fix I put in the other day, so they weren't being stored correctly. This means that while the characters got the awards from their Awards, when you next loaded the save all those Awards were lost, so when autoAwards checked to see if the characters were eligible it found that they were, because insofar as mhq is concerned, they haven't received any awards.

The workaround is to turn off xp and edge awards in campaign options, manually run the award ceremony, issue all the awards it recommends. Turn back on xp and edge awards (if you use those settings), and you'll be good from now on.

Closing as resolved.