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

Items inside integrated armor are not deleted if mutation is removed #75565

Merged

Conversation

GuardianDll
Copy link
Member

Summary

None

Purpose of change

When integrated mutation is removed, it's content is lost forever. Not anymore

Describe the solution

Add check, that would spill items on the ground if said mutation would be removed

Testing

Before changes
image
After changes
image

Additional context

@Standing-Storm

@github-actions github-actions bot added [C++] Changes (can be) made in C++. Previously named `Code` json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions labels Aug 10, 2024
@GuardianDll GuardianDll changed the title Items are not removed if integrated mutation is deleted Items inside integrated armor are not deleted if mutation is removed Aug 10, 2024
@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Aug 10, 2024
@Standing-Storm
Copy link
Contributor

Telekinetics in shambles.

@Maleclypse Maleclypse merged commit 02b3eee into CleverRaven:master Aug 11, 2024
24 of 29 checks passed
@GuardianDll GuardianDll deleted the integrated_armor_no_item_disappear branch August 11, 2024 08:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions [C++] Changes (can be) made in C++. Previously named `Code` json-styled JSON lint passed, label assigned by github actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants