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

[MIRROR] Fixes destructive analyzer being unable to research anything #864

Merged
merged 1 commit into from
Nov 28, 2023

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Skyrat: Skyrat-SS13/Skyrat-tg#25296
Original PR: tgstation/tgstation#79953

About The Pull Request

https://github.com/tgstation/tgstation/blob/62165b992df9e3b969519d6cfd776c6479c40743/code/modules/research/destructive_analyzer.dm#L184 placed after loaded_item is nulled, so it runtimes and fails to unhide any nodes. I've just saved item's type to new variable. Hopefully, it doesn't make any harddels or other scary things.

image

Why It's Good For The Game

less runtimes, more things that works as they should

Changelog

🆑 Iajret
fix: You can now destructively analyze syndicate and abductor items. It works this time, I promise!
/:cl:

… [MDB IGNORE] (#25296)

* Fixes destructive analyzer being unable to research anything (#79953)

## About The Pull Request

https://github.com/tgstation/tgstation/blob/62165b992df9e3b969519d6cfd776c6479c40743/code/modules/research/destructive_analyzer.dm#L184
placed after `loaded_item` is nulled, so it runtimes and fails to unhide
any nodes. I've just saved item's type to new variable. Hopefully, it
doesn't make any harddels or other scary things.

![image](https://github.com/tgstation/tgstation/assets/8430839/7d4994a3-e0eb-47e6-b69c-f18a6b820b4e)
## Why It's Good For The Game
less runtimes, more things that works as they should
## Changelog
:cl:
fix: You can now destructively analyze syndicate and abductor items. It
works this time, I promise!
/:cl:

* Fixes destructive analyzer being unable to research anything

---------

Co-authored-by: Iajret <[email protected]>
@Iajret Iajret merged commit dbd7473 into master Nov 28, 2023
24 checks passed
@Iajret Iajret deleted the upstream-mirror-25296 branch November 28, 2023 08:58
AnywayFarus added a commit that referenced this pull request Nov 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants