nextcloud/server

Cleanup Server.php (aliases & services)

Open

#12.225 aperta il 2 nov 2018

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)PHP (4865 fork)batch import
1. to developgood first issuetechnical debt

Metriche repository

Star
 (34.953 star)
Metriche merge PR
 (Merge medio 20g) (627 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Inizia esaminando il file lib/private/Server.php, in particolare gli alias registrati intorno alla riga 182 e i wrapper semplici intorno alle righe 256-258. Rivedi i commenti esistenti nell'issue per eventuali contesti aggiuntivi. Considera come ciascun alias viene utilizzato altrove nella codebase e se può essere sostituito da una query diretta al servizio. Per i wrapper, determina se il contenitore DI può risolvere direttamente la dipendenza. Dopo l'analisi, rimuovi gli alias e semplifica di conseguenza i wrapper.
Tech stack
php
Dominio
backend
Tipo issue
Refactoring
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.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
PHPDependency InjectionGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75