ParadiseSS13/Paradise

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

Aberta

#19.295 aberto em 5 de out. de 2022

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

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

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.

Guia do colaborador

Direção de pesquisa
O bug está em /datum/controller/subsystem/shuttle/proc/moveShuttle que define last caller. A correção é usar um UID em vez de uma referência fixa. Verifique a PR #13208 onde foi introduzido. Implemente um sistema de UID para a porta de acoplamento e modifique a variável para armazenar o UID.
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.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Git
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
85