TauCetiStation / TauCetiClassic

Франкенштейн жив
GNU Affero General Public License v3.0
142 stars 417 forks source link

Спейсьюит получает бурн дамаг вместе с персонажем #6553

Open Sakuya-Izayoi opened 3 years ago

Sakuya-Izayoi commented 3 years ago

Подробное описание проблемы

Если хуман получает бурн дамаг при этом находясь в скафандре (например, сняв шлем), то скафандр будет получать урон вместе с ним

Что должно было произойти

Скафандр не должен дамажиться вместе с хуманом

Что произошло на самом деле

Скафандр получает дамаг вместе с хуманом

Как повторить

  1. надеть скаф
  2. зайти в телекоммы
  3. снять шлем image

Дополнительная информация:

тестилось только с ригом синдиката и только в телекоммах

--- Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/95471082-?utm_campaign=plugin&utm_content=tracker%2F34704297&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F34704297&utm_medium=issues&utm_source=github).
Kortez90 commented 3 years ago

А баг в чем? Костюм получил термический тип повреждений. Сообщение вывелось. https://github.com/TauCetiStation/TauCetiClassic/blob/778f83034e26a2bd4a1bd22c824357ae4184878f/code/modules/clothing/spacesuits/breaches.dm#L121-L124

AndroBetel commented 3 years ago

в том что если атмосферный риг не получает повреждений термических с надетым шлемом то и без шлема он тоже их не должен получать

Kortez90 commented 3 years ago

Почему не должен? Шлема нет, декомпрессия, костюм рвется. В общем, вот цепочка. Когда происходит human/apply_damage, то вызывается handle_suit_punctures(damagetype, damage), который в свою очередь вызывает if(penetrated_dam) SS.create_breaches(damtype, penetrated_dam), который и выводил это сообщение, создавая дыру в костюме.

Sakuya-Izayoi commented 3 years ago

Может костюм должен получать брут, если он рвётся от декомпрессии, а не бёрн от того, что у Хумана мёрзнут уши?

(обратите внимание на ишью, я тестил это на телекоммах, а там давление 1 атмосфера)

LudwigVonChesterfield commented 3 years ago

кажется нужно как-то поделить на урон приходящий "снаружи", и урон связанный с телом самим по себе

Akellazp commented 3 years ago

4731