nextcloud/server

Cleanup Server.php (aliases & services)

Ouverte

#12 225 ouverte le 2 nov. 2018

Voir sur GitHubGuide contributeur
 (3 commentaires) (0 réaction) (0 personne assignée)PHP (4 865 forks)batch import
1. to developgood first issuetechnical debt

Métriques du dépôt

Stars
 (34 953 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

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.

Guide contributeur

Direction de recherche
Commencez par examiner le fichier lib/private/Server.php, en particulier les alias enregistrés autour de la ligne 182 et les wrappers simples autour des lignes 256-258. Consultez les commentaires existants dans l'issue pour tout contexte supplémentaire. Examinez comment chaque alias est utilisé ailleurs dans la base de code et s'il peut être remplacé par une requête directe au service. Pour les wrappers, déterminez si le conteneur DI peut résoudre directement la dépendance. Après analyse, supprimez les alias et simplifiez les wrappers en conséquence.
Stack technique
php
Domaine
backend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
PHPDependency InjectionGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75