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

[Aftershock] Fix Police bots (Again) #79040

Merged
merged 3 commits into from
Jan 10, 2025

Conversation

QuillInkwell
Copy link
Contributor

Summary

Mods "[Aftershock] Fix the three police robots not dropping the correct corpse"

Purpose of change

A couple months back I noticed when implementing the Enforcer station that the police robots I had placed kept becoming lab bots after ever reload. I learned this was due to a migration on the base game. And was recommended to change the ids on the Aftershock robots to dodge the migration. Which I did. Except at the time I did not understand how "broken" robots worked and misplaced the broken prefix. Now whenever a copbot, eyebot, or riotbot dies it attempts to place a corpse item that does not exist.

Describe the solution

Once again change all the item ids on the robot corpses to use the correct placement of the broken prefix. Implement yet another migration to migrate any from the old and incorrect ids to the new and correct ones.

Describe alternatives you've considered

Leave the bug in. Not introduce a new migration since technically this bug has not made it into any stable versions.

Testing

Loaded my world where I was plagued by the bug. Game stopped throwing errors about the robot corpse id not existing. Robot corpses became the correct item.

Loaded a test world. Place the items, and the monster. Reloaded. Verified no items were replaced by vanilla migration or otherwise.

Additional context

@github-actions github-actions bot added [JSON] Changes (can be) made in JSON Mods Issues related to mods or modding Spawn Creatures, items, vehicles, locations appearing on map Mods: Aftershock Anything to do with the Aftershock mod labels Jan 9, 2025
@github-actions github-actions bot added the astyled astyled PR, label is assigned by github actions label Jan 9, 2025
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@github-actions github-actions bot added the json-styled JSON lint passed, label assigned by github actions label Jan 9, 2025
@John-Candlebury
Copy link
Member

Thanks! I noticed this while making the Zenit but got sidelined by the mutation stuff.

@John-Candlebury John-Candlebury merged commit 43ccb71 into CleverRaven:master Jan 10, 2025
27 checks passed
@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Jan 10, 2025
@QuillInkwell QuillInkwell deleted the Aftershock_robot_fix branch January 10, 2025 05:29
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 [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions Mods: Aftershock Anything to do with the Aftershock mod Mods Issues related to mods or modding Spawn Creatures, items, vehicles, locations appearing on map
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants