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

Fix 3917 jump vehicles with motive damage #4940

Merged

Conversation

Sleet01
Copy link
Collaborator

@Sleet01 Sleet01 commented Dec 3, 2023

This adds the ability for CVs that are "immobilized" by Motive System hits to jump.

Per linked issue, CVs with jump jets should be able to jump even when motive system damage has reduced MP to 0.
When an "immobilized" CV with jump jets jumps, it does not get the -4 TMM for being immobile.
Additionally, when the crew is Stunned, they can still move up to Cruise MP, which means they can still jump.

States that now prevent JJ-equipped CVs from jumping:

  • Vehicle destroyed (ob.)
  • Engine destroyed
  • Crew killed
  • Crew ejected
  • Crew unconscious (not sure if this is possible for CVs but included in existing checks)
  • Engine shut down (again, not sure if possible but included per some other checks)

Added a unit test to EntityTest.java to verify correct handling of these states.

Close #3917

@NickAragua NickAragua merged commit bcb248d into MegaMek:master Dec 5, 2023
4 checks passed
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 this pull request may close these issues.

Errata: Vehicle jump jets should be usable with "immobile" status caused by motive hits if engine is undamaged
2 participants