TauCetiStation/TauCetiClassic

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

Open

#8 143 ouverte le 31 oct. 2021

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)DM (433 forks)batch import
BugGood First Issue

Métriques du dépôt

Stars
 (153 stars)
Métriques de merge PR
 (Merge moyen 27j 14h) (41 PRs mergées en 30 j)

Description

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

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

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

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

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

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

image

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

Зайти в код

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

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

Guide contributeur