nextcloud/server

Cleanup Server.php (aliases & services)

Open

#12.225 aberto em 2 de nov. de 2018

Ver no GitHub
 (3 comments) (0 reactions) (0 assignees)PHP (4.865 forks)batch import
1. to developgood first issuetechnical debt

Métricas do repositório

Stars
 (34.953 stars)
Métricas de merge de PR
 (Mesclagem média 20d) (627 fundiu PRs em 30d)

Description

As discussed with @MorrisJobke

We regsiter a lot of plain text aliases like https://github.com/nextcloud/server/blob/9319d557a4b1156d82d824cdb1b7d6f07ad3fbde/lib/private/Server.php#L182

They should all die. Services should be queries properly using their servicename.

Ideally we also remove as much simple wrappers as possible like: https://github.com/nextcloud/server/blob/9319d557a4b1156d82d824cdb1b7d6f07ad3fbde/lib/private/Server.php#L256-L258 The DI can handle that all perfectly by itself when it is actually needed.

Guia do colaborador

Direção de pesquisa
Comece examinando o arquivo lib/private/Server.php, especificamente os aliases registrados por volta da linha 182 e os wrappers simples nas linhas 256-258. Revise os comentários existentes na issue para qualquer contexto adicional. Considere como cada alias é usado em outras partes da codebase e se pode ser substituído por uma consulta direta ao serviço. Para os wrappers, determine se o contêiner de DI pode resolver a dependência diretamente. Após a análise, remova os aliases e simplifique os wrappers de acordo.
Pilha de tecnologia
php
Domain
backend
Tipo Issue
Refatoração
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.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
PHPDependency InjectionGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75