TauCetiStation/TauCetiClassic

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

Open

#8,143 opened on Oct 31, 2021

View on GitHub
 (2 comments) (0 reactions) (0 assignees)DM (433 forks)batch import
BugGood First Issue

Repository metrics

Stars
 (153 stars)
PR merge metrics
 (Avg merge 27d 14h) (41 merged PRs in 30d)

Description

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

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

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

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

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

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

image

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

Зайти в код

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

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

Contributor guide

Research direction
Find all switch statements that use tgui alert and ensure they handle null return values when the user closes the dialog without selecting an option.
Tech stack
None
Domain
game dev
Issue type
Bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
1
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
Under 1 hour
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Fresh
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
None
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
80