ParadiseSS13/Paradise

/obj/docking_port/mobile keeps a hard reference of the last_caller

Aperta

#19.295 aperta il 5 ott 2022

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)DM (1313 fork)batch import
GC RelatedGood First Issue

Metriche repository

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

Descrizione

Issue Description: The mobile docking ports keep a hard reference to the last caller. /datum/controller/subsystem/shuttle/proc/moveShuttle is the proc which sets it.

[2022-10-05T20:51:06] Found /mob/dead/observer [0x300003b] in /obj/docking_port/mobile's [0x200057f] last_caller var. World -> /obj/docking_port/mobile

Steps to reproduce the problem:

  1. Spawn as a ghost or aghost.
  2. Use any shuttle console and call/send the shuttle
  3. Become a mob or something else to delete the ghost

When did the problem start happening: For a while. I have introduced it 2 years ago in #13208 :(

Extra information: It should use a UID here instead. This should be a relatively easy first GC issue.

Guida contributor

Direzione di ricerca
Il bug si trova in /datum/controller/subsystem/shuttle/proc/moveShuttle che imposta last caller. La correzione consiste nell'utilizzare un UID invece di un riferimento fisso. Controlla la PR #13208 dove è stato introdotto. Implementa un sistema UID per la porta di attracco e modifica la variabile per memorizzare l'UID.
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.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
85