nextcloud/server

Cleanup Server.php (aliases & services)

Open

#12.225 geöffnet am 2. Nov. 2018

Auf GitHub ansehen
 (3 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)PHP (4.865 Forks)batch import
1. to developgood first issuetechnical debt

Repository-Metriken

Stars
 (34.953 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 20T) (627 gemergte PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Beginnen Sie mit der Untersuchung der Datei lib/private/Server.php, insbesondere der Aliase, die um Zeile 182 registriert sind, und der einfachen Wrapper um die Zeilen 256-258. Überprüfen Sie die vorhandenen Kommentare im Issue auf zusätzlichen Kontext. Überlegen Sie, wie jeder Alias an anderer Stelle in der Codebasis verwendet wird und ob er durch eine direkte Serviceabfrage ersetzt werden kann. Bestimmen Sie für die Wrapper, ob der DI Container die Abhängigkeit direkt auflösen kann. Entfernen Sie nach der Analyse die Aliase und vereinfachen Sie die Wrapper entsprechend.
Tech Stack
php
Domain
backend
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
PHPDependency InjectionGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75