Leantime/leantime

Move all repo calls from controllers to services

Open

#1532 aperta il 31 mar 2023

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)PHP (957 fork)batch import
Help Wantedgood-first-issue

Metriche repository

Star
 (9827 star)
Metriche merge PR
 (Merge medio 12h 22m) (61 PR mergiate in 30 g)

Descrizione

The controllers should not be calling the repos directly. Instead they should call a service method. The service layer should be the layer calling the repo.

As such the task is to go through all controllers and update repository calls to be service calls. If the method does not exist in the service a wrapper service method should be created.

Guida contributor

Direzione di ricerca
Identifica tutti i file controller nel progetto. Per ogni metodo del controller che chiama direttamente un repository (ad esempio `$this >repo >method()`), trova la corrispondente classe di servizio (se esiste). Se il servizio ha già un metodo che incapsula la chiamata al repository, sostituisci la chiamata diretta al repository con il metodo del servizio. Se tale metodo non esiste, crea un nuovo metodo nel servizio appropriato che delega al repository, quindi aggiorna il controller per usare il servizio. Assicurati che il servizio sia iniettato nel controller. Dopo le modifiche, verifica che l'applicazione funzioni ancora.
Tech stack
php
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
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
PHPGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70