Jacek Pyziak
3a9cfcd4a2
Plany dla php:S112, S1142, S1192, S3776, S1448, S138 — gotowe do APPLY. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
12 KiB
12 KiB
phase, plan, type, wave, depends_on, files_modified, autonomous
| phase | plan | type | wave | depends_on | files_modified | autonomous | |||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| 06-sonarqube-quality | 05 | execute | 3 |
|
|
false |
Purpose
God classes naruszają SRP i S1448. ShopproOrdersSyncService z 39 metodami i 1192 liniami jest najtrudniejszą do testowania klasą w projekcie. Podział ułatwia izolowane modyfikacje integracji Shoppro bez ryzyka regresi w całym module.
Output
5 nowych klas/kontrolerów, ShopproOrdersSyncService spada do ~15 metod, AllegroIntegrationController do ~12. S1448 spada z 6x do 1x (ShopproIntegrationsController pozostaje na następny plan).
Source Files
@src/Modules/Settings/ShopproOrdersSyncService.php @src/Modules/Settings/AllegroIntegrationController.php @routes/web.php
## Required Skills (from SPECIAL-FLOWS.md)| Skill | Priority | When to Invoke | Loaded? |
|---|---|---|---|
| sonar-scanner | required | Po APPLY, przed UNIFY | ○ |
| /code-review | optional | Po implementacji, przed UNIFY | ○ |
Skill Invocation Checklist
- sonar-scanner uruchomiony po zakończeniu APPLY
- /code-review rozważyć (duże zmiany strukturalne)
<acceptance_criteria>
AC-1: ShopproOrderMapper wydzielony
Given ShopproOrdersSyncService zawiera 15+ metod mapujących (mapOrderAggregate, mapAddresses, mapItems, etc.)
When wydzielasz je do src/Modules/Settings/ShopproOrderMapper.php
Then ShopproOrderMapper zawiera wszystkie metody mapowania; ShopproOrdersSyncService wstrzykuje go przez konstruktor
AC-2: ShopproProductImageResolver wydzielony
Given ShopproOrdersSyncService zawiera resolveProductImagesForOrder() i fetchPrimaryProductImageUrl()
When wydzielasz je do src/Modules/Settings/ShopproProductImageResolver.php
Then ShopproProductImageResolver obsługuje image fetching; ShopproOrdersSyncService używa go przez konstruktor
AC-3: ShopproOrdersSyncService ma ≤20 metod
Given ShopproOrdersSyncService ma 39 metod
When wydzielono ShopproOrderMapper i ShopproProductImageResolver
Then ShopproOrdersSyncService ma ≤20 metod (sync + orchestration + state management)
AC-4: AllegroIntegrationController podzielony
Given AllegroIntegrationController ma 30 metod (status mapping, delivery mapping, main settings)
When wydzielasz AllegroStatusMappingController i AllegroDeliveryMappingController
Then każdy kontroler ma ≤15 metod; routes/web.php zaktualizowane
AC-5: Brak regresji
Given Shoppro sync i Allegro integration działają przed podziałem
When podział jest czysto strukturalny (move method, nie zmiana logiki)
Then sync Shoppro importuje zamówienia; mapowania Allegro zapisują się; OAuth działa
</acceptance_criteria>
Task 1: Wydziel ShopproOrderMapper z ShopproOrdersSyncService src/Modules/Settings/ShopproOrderMapper.php, src/Modules/Settings/ShopproOrdersSyncService.php Przeczytaj ShopproOrdersSyncService dokładnie. Zidentyfikuj metody mapujące dane zamówień.**Metody do przeniesienia do ShopproOrderMapper:**
Przenieś wszystkie private metody których jedynym zadaniem jest konwersja danych API → struktury wewnętrzne:
- mapOrderAggregate() — główna metoda mapowania
- mapAddresses() — mapowanie adresów
- mapItems() — mapowanie pozycji
- mapPayments() — mapowanie płatności
- mapShipments() — mapowanie wysyłek
- mapNotes() — mapowanie notatek
- buildInvoiceAddress() — budowanie adresu faktury
- normalizeOrderId(), normalizePaidFlag(), mapPaymentStatus(), sanitizePlainText()
- buildDeliveryMethodLabel(), formatMoneyCompact(), normalizeMediaUrl()
- toFloatOrNull(), toFloatOrDefault(), readPath(), readSinglePath(), composeName()
- hasAddressData(), resolveInvoiceRequested(), parsePickupPoint()
**ShopproOrderMapper (nowy plik):**
- namespace App\Modules\Settings
- class ShopproOrderMapper
- Konstruktor przyjmuje zależności potrzebne metodom (np. ShopproOrderSyncStateRepository jeśli potrzebne)
- Przenieś metody bez zmiany logiki (tylko zmień z private na public te, które sync() wywołuje bezpośrednio)
**ShopproOrdersSyncService (aktualizacja):**
- Dodaj private ShopproOrderMapper $mapper; w konstruktorze
- CronHandlerFactory tworzy ShopproOrdersSyncService — sprawdź src/Modules/Cron/CronHandlerFactory.php i zaktualizuj kompozycję
- Zastąp wywołania $this->mapOrderAggregate() → $this->mapper->mapOrderAggregate()
- Usuń przeniesione metody z ShopproOrdersSyncService
UWAGA: Sprawdź CronHandlerFactory.php — musi przekazać nowy ShopproOrderMapper do konstruktora.
**ShopproOrdersSyncService (finalizacja):**
- Dodaj private ShopproProductImageResolver $imageResolver;
- Zastąp $this->resolveProductImagesForOrder() → $this->imageResolver->resolveProductImagesForOrder()
- Po Task 1 + Task 2 ShopproOrdersSyncService powinien mieć ≤20 metod
**CronHandlerFactory.php (aktualizacja):**
- Sprawdź gdzie tworzona jest instancja ShopproOrdersSyncService
- Dodaj nowe zależności: new ShopproOrderMapper(...), new ShopproProductImageResolver($shopproApiClient)
- Przekaż do konstruktora ShopproOrdersSyncService
Sprawdź czy ShopproOrderMapper też potrzebuje ShopproApiClient lub innych zależności — jeśli tak, przekaż z CronHandlerFactory.
**Podział metod:**
AllegroStatusMappingController (nowy):
- saveStatusMapping()
- saveStatusMappingsBulk()
- deleteStatusMapping()
- syncStatusesFromAllegro()
- buildImportImageWarningMessage() (private helper)
- reasonLabel() (private helper)
AllegroDeliveryMappingController (nowy):
- saveDeliveryMappings()
- loadDeliveryServices()
- (prywatne helpery potrzebne tym metodom)
AllegroIntegrationController (zostaje z):
- index()
- save()
- saveImportSettings()
- startOAuth()
- oauthCallback()
- refreshOAuthToken()
- importSingleOrder()
- (prywatne walidatory i helpery potrzebne tym metodom)
**Nowe pliki kontrolerów:**
- Skopiuj konstruktor i zależności z AllegroIntegrationController
- namespace App\Modules\Settings
- Przenieś metody bez zmiany logiki
**routes/web.php:**
- Utwórz instancje nowych kontrolerów (ten sam wzorzec co istniejące)
- Przepnij route definicje dla:
- status mapping routes → AllegroStatusMappingController
- delivery mapping routes → AllegroDeliveryMappingController
- AllegroIntegrationController obsługuje resztę
Sprawdź które zależności ($this->repository, $this->translator etc.) są potrzebne każdemu kontrolerowi.
DO NOT CHANGE
- Logika biznesowa w żadnej z przenoszonych metod
- Publiczne sygnatury metod (nazwy, parametry)
- Pliki widoków i SCSS
- ShopproIntegrationsController — zakres następnego planu (06-06 jeśli powstanie)
- Inne moduły (Orders, Shipments, Auth, etc.)
SCOPE LIMITS
- Podział 2 klas (ShopproOrdersSyncService + AllegroIntegrationController)
- ShopproIntegrationsController pomijamy w tym planie (30 metod — podobny refactoring)
- Nie zmieniaj publicznych URL routes — tylko który controller je obsługuje
- CronHandlerFactory — tylko minimalne zmiany potrzebne dla nowych dependencies
<success_criteria>
- 4 nowe pliki PHP
- ShopproOrdersSyncService ≤20 metod
- AllegroIntegrationController ≤15 metod
- Zero błędów składniowych
- Shoppro import działa (human-verify passed)
- SonarQube S1448 spada z 6 do ≤2 </success_criteria>