ParadiseSS13/Paradise

Can not turn off thresholds in Air Alarm UI

Aperta

#24.643 aperta il 16 mar 2024

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)DM (1313 fork)batch import
Good First IssueOversight

Metriche repository

Star
 (445 stelle)
Metriche merge PR
 (Merge medio 13g 5h) (90 PR mergiate in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Esamina la logica di clamping della soglia nel codice dell'allarme aria, in particolare la riga menzionata nell'issue. Trova dove 1 viene fissato a 0 e modifica la condizione per consentire 1 come valore di spegnimento valido.
Tech stack
Nessuno
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
DMGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75