Skip to content

Fix #7697: Fixed New Day Advance when Character is Illiterate But Has No Languages/Any Skill#7701

Merged
HammerGS merged 1 commit into
MegaMek:mainfrom
IllianiBird:literacyFix
Oct 15, 2025
Merged

Fix #7697: Fixed New Day Advance when Character is Illiterate But Has No Languages/Any Skill#7701
HammerGS merged 1 commit into
MegaMek:mainfrom
IllianiBird:literacyFix

Conversation

@IllianiBird

Copy link
Copy Markdown
Collaborator

Fix #7697

This PR fixes an Advance Day error caused when a character is both Illiterate but also has no Languages/Any skill.

Giving the character Languages/Any (even at level 0) will work around this issue.

@IllianiBird IllianiBird self-assigned this Oct 13, 2025
@IllianiBird IllianiBird requested a review from a team as a code owner October 13, 2025 01:39
@IllianiBird IllianiBird added Bug Personnel Personnel-related Issues SPA Any issue relating to SPA's. Includes Bugs and RFE Severity: High Issues described as high severity as per the new issue form labels Oct 13, 2025
@codecov

codecov Bot commented Oct 13, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 12.14%. Comparing base (2f6653c) to head (e3dc9a9).
⚠️ Report is 48 commits behind head on main.

Files with missing lines Patch % Lines
MekHQ/src/mekhq/campaign/personnel/Person.java 0.00% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main    #7701      +/-   ##
============================================
- Coverage     12.14%   12.14%   -0.01%     
+ Complexity     7321     7316       -5     
============================================
  Files          1210     1210              
  Lines        152464   152475      +11     
  Branches      23103    23107       +4     
============================================
- Hits          18523    18516       -7     
- Misses       132028   132047      +19     
+ Partials       1913     1912       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@HammerGS HammerGS merged commit 1bcfb57 into MegaMek:main Oct 15, 2025
7 checks passed
@IllianiBird IllianiBird deleted the literacyFix branch November 13, 2025 07:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Personnel Personnel-related Issues Severity: High Issues described as high severity as per the new issue form SPA Any issue relating to SPA's. Includes Bugs and RFE

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Issue] Long advance Not functioning correctly

2 participants