TauCetiStation / TauCetiClassic

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

Спам ядерной бомбы #6501

Open Akellazp opened 3 years ago

Akellazp commented 3 years ago

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

При пленте бомбы после опускания якоря при любом действии в интерфейсе бомбы пишется то что опускаются болты

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

Спама нет

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

image

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

Заплентить нюку на 600 секунд

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

Issue reported from Round ID: 37623 (Tau Ceti Classic II (RU)) Testmerges: #6400 Reporting client version: 513.1536

--- Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/95471117-?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/ce050176bc65611790c29a8a30cd9d8ef9501259/code/game/gamemodes/nuclear/nuclearbomb.dm#L185-L190

И почему-то, точно такое же сообщение отображается при нажатии на кнопку Anchor. https://github.com/TauCetiStation/TauCetiClassic/blob/ce050176bc65611790c29a8a30cd9d8ef9501259/code/game/gamemodes/nuclear/nuclearbomb.dm#L324-L331

Впрочем, проблема именно в сообщении, которое вызывается ударом руки. Каждое взаимодействие с бомбой, у которой сработал верб make_deployable() вызывает это сообщение, так как единственное условие вызова этого сообщения - if(removal_stage < 5). Существование верба под сомнением, так как он не делает ничего, кроме проверок состояния пользователя и переключения бесполезной переменной deployable.

Akellazp commented 3 years ago

image

Kortez90 commented 3 years ago

Здорово. Но наверное все же надо заполнять раздел "Как повторить".

Akellazp commented 3 years ago

там написано, здарова