ParadiseSS13/Paradise

Can not turn off thresholds in Air Alarm UI

Aberta

#24.643 aberto em 16 de mar. de 2024

Ver no GitHubGuia do colaborador
 (0 comentário) (0 reação) (0 responsável)DM (1.313 forks)batch import
Good First IssueOversight

Métricas do repositório

Stars
 (445 estrelas)
Métricas de merge de PR
 (Mesclagem média 13d 5h) (90 fundiu PRs em 30d)

Description

Exploit Reports

  • I confirm this issue is not an exploit. (Required)

BYOND Version

515.1633

Issue Description

Air Alarms' thresholds can not be turned off by setting them to -1.

What did you expect to happen?

Setting thresholds to -1 should turn them off.

What happened instead?

Setting thresholds to -1 clamps it to 0.

Why is this bad/What are the consequences?

Slight inconvenience over setting the threshold to min/max.

Steps to reproduce the issue.

  1. Open an Air Alarm.
  2. Go to Thresholds.
  3. Set one of them to -1.

When did the problem start happening?

Since TGUI Inputs were introduced.

Extra information

https://github.com/ParadiseSS13/Paradise/blob/521abceae83be85905923cb7253c8eb85a3d6bb7/code/modules/atmospherics/machinery/airalarm.dm#L852

Relevant log output/runtime error

No response

Guia do colaborador

Direção de pesquisa
Examine a lógica de clamping do limite no código do alarme de ar, especificamente a linha referenciada no problema. Encontre onde 1 é fixado em 0 e modifique a condição para permitir 1 como um valor de desligamento válido.
Pilha de tecnologia
None
Domain
backend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
DMGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75