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

no pain on switching species to NO_PAIN one #12507

Merged
merged 2 commits into from
Dec 8, 2023
Merged

no pain on switching species to NO_PAIN one #12507

merged 2 commits into from
Dec 8, 2023

Conversation

muWander
Copy link
Contributor

@muWander muWander commented Nov 23, 2023

Описание изменений

Прок set_species ставит болевой шок на 0 если у целевого гуманоида флаг игнорирования боли, сейчас если раса гуманоида сменяется в течение раунда шок остается навсегда без возможности снять кроме как педально
resolves #9654

Почему и что этот ПР улучшит

Авторство

Чеинжлог

🆑

  • bugfix: Превращение в слаймо-человека токсином ксенобиолога больше не чревато вечным замедлением.
  • bugfix: Скелеты больше не замедляются навечно, если были ранены при жизни.

@TauKitty
Copy link
Contributor

Changelog status: ✔️

@TauKitty TauKitty added the Fix label Nov 23, 2023
@@ -1398,6 +1398,10 @@

maxHealth = species.total_health

if(species.flags[NO_PAIN])
shock_stage = 0
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

я бы предпочёл заменить это на геттеры-сеттеры и тогда проблема была бы на 100% решена потому-что при получении флага геттер бы возвращал 0

@NinjaPikachuska NinjaPikachuska dismissed their stale review November 26, 2023 11:24

подсказал в личке что делать с геттером и сеттером, ждем пока их

Copy link
Member

@volas volas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

тут вроде реквест, я просто статус изменяю

@volas
Copy link
Member

volas commented Dec 8, 2023

в целом и так ок, гетеры/сетеры было бы хорошим бонусом но можно потом получить

@volas volas merged commit 235472b into TauCetiStation:master Dec 8, 2023
TauKitty added a commit to TauCetiStation/ClassicChangelog that referenced this pull request Dec 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Поломанные слизнехуманы должны быть пофикшены
5 participants