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] Fix void adaptation not reapplying when born from a headslug #970

Merged
merged 1 commit into from
Dec 5, 2023

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Skyrat: Skyrat-SS13/Skyrat-tg#25422
Original PR: tgstation/tgstation#80108

About The Pull Request

This bug happend because of 2 things,

  1. the traits where applied to the current mob on purchase
  2. When head slug calls regain_powers, it uses the grant proc

There are 2 solutions for this. Either make void adapatation apply its effects on grant or change regain powers to be on purchase.

This would break admin bussing non ling events into a lings innate_powers. but saves having to rewrite all ling thingies from on_purchase to on_grant

Why It's Good For The Game

Permantly losing void adaptation because of last resort is meh

Changelog

🆑 Autisem
fix: Rebirthing from headslug properly reapplys void adaptation
/:cl:

… [MDB IGNORE] (#25422)

* Fix void adaptation not reapplying when born from a headslug (#80108)

## About The Pull Request

This bug happend because of 2 things,
1. the traits where applied to the current mob on *purchase*
2. When head slug calls regain_powers, it uses the grant proc

There are 2 solutions for this. Either make void adapatation apply its
effects on grant or change regain powers to be on purchase.

This would break admin bussing non ling events into a lings
innate_powers. but saves having to rewrite all ling thingies from
on_purchase to on_grant

## Why It's Good For The Game

Permantly losing void adaptation  because of last resort is meh

## Changelog
:cl:
fix: Rebirthing from headslug properly reapplys void adaptation
/:cl:

* Fix void adaptation not reapplying when born from a headslug

---------

Co-authored-by: Autisem <[email protected]>
@ReezeBL ReezeBL merged commit 1c7a6c4 into master Dec 5, 2023
22 of 23 checks passed
@ReezeBL ReezeBL deleted the upstream-mirror-25422 branch December 5, 2023 07:09
AnywayFarus added a commit that referenced this pull request Dec 5, 2023
Iajret pushed a commit that referenced this pull request Feb 17, 2024
* Fixes a runtime in engaged_role_play_check

* Inverts the return value

* Update ev_roleplay_check.dm

* Update ev_roleplay_check.dm
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