TauCetiStation/TauCetiClassic

Не учитывание нулла в конструкции tgui_alert'a в swtich'e

Open

#8,143 创建于 2021年10月31日

在 GitHub 查看
 (2 评论) (0 反应) (0 负责人)DM (433 fork)batch import
BugGood First Issue

仓库指标

Star
 (153 star)
PR 合并指标
 (平均合并 27天 14小时) (30 天内合并 41 个 PR)

描述

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

Изначально использовался бьондовский алерт, который мог вернуть только значение своих кнопок, но с вводом tgui_alert появилась возможность вернуть и null при клике на крестик. В итоге когда массово заменялись алерты на тгуи_алерты, появились ситуации с конструкцией switch(tgui_alert()), где учтено только возвращение значений кнопок.

Например, вот ПР https://github.com/TauCetiStation/TauCetiClassic/pull/8136 , где видна проблема и способ решения.

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

Свитчи с тгуи_алертом учитывают кнопки и нулл. Либо такой свитч заменяется на ифку, которая подразумевает только 1 ответ и действие во всех других случаях.

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

image

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

Зайти в код

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

Довольно легко пофиксить

贡献者指南