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

[49.7] Magma crust breakthrough and liquid magma damage not working as expected #3727

Closed
Gribbly1 opened this issue Jun 22, 2022 · 3 comments

Comments

@Gribbly1
Copy link

Bundled Megamek with MekHQ 0.49.7
Windows 10, Java Version: 11.0.13

save1.sav.gz
image

image

image

A) MM seems to check for breaking through a magma crust when turning in that hex, when it should only check upon entering the hex.

save2.sav.gz

image

B) No damage is being applied for breaking through the crust until leaving the hex the following turn. If the crust breaks and converts to liquid magma the exposed locations should take damage immediately in the movement phase where that occurs.

The wording of the rule (TacOps advanced ground movement) is that damage should be caused when starting a movement phase in liquid magma or if entering the hex. I guess this might be an issue with the order of movement and determining when the hex becomes magma - strictly the unit enters a crust hex first, which then changes.

C) Also the additional second damage amount is not being applied for starting and ending the movement phase in liquid magma.

@pheonixstorm
Copy link
Collaborator

pheonixstorm commented Jul 9, 2022

Disregard, I read the section wrong. Retesting...

@Windchild292
Copy link
Contributor

Fixed by #3991

Windchild292 added a commit that referenced this issue Nov 28, 2022
Fixes #3727 Magma crust breakthrough and liquid magma damage not working as expected
@Gribbly1
Copy link
Author

Gribbly1 commented Dec 1, 2022

Excellent, thanks very much.

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

No branches or pull requests

3 participants