phase: 06-sonarqube-quality
plan: 02
subsystem: api
tags: [sonarqube, refactoring, validation, php]
requires:
- phase: 06-sonarqube-quality
provides: Faza aktywna — brak twardych zależności między planami 06-xx
provides:
- AllegroIntegrationController z metodami save/saveImportSettings/oauthCallback o ≤3 return each
- ShopproIntegrationsController z metodami save/saveStatusMappings/syncStatuses o ≤3 return each
- Wzorzec validateXxxInput(): ?string oraz validateXxxAccess(): ?Response dla walidacji w kontrolerach
affects: [06-05-god-classes]
tech-stack:
added: []
patterns: [validation-helper-method, access-guard-method]
key-files:
created: []
modified:
- src/Modules/Settings/AllegroIntegrationController.php
- src/Modules/Settings/ShopproIntegrationsController.php
key-decisions:
- "Wzorzec ?string dla walidacji danych wejściowych (validateSaveInput, validateImportSettingsInput, validateOAuthCallbackParams)"
- "Wzorzec ?Response dla strażników dostępu CSRF+notFound (validateSaveAccess, validateCsrfAndIntegrationAccess)"
- "Restrukturyzacja try/catch w ShopproIntegrationsController::save() — brak return wewnątrz bloku, $redirectTo mutuje na sukces"
patterns-established:
- "validateXxxInput(): ?string — zwraca komunikat błędu lub null; wywołujący robi Flash::set + redirect w jednym if"
- "validateXxxAccess(): ?Response — łączy CSRF + not-found w jeden guard zwracający gotowy Response lub null"
duration: 20min
started: 2026-03-13T00:00:00Z
completed: 2026-03-13T00:00:00Z
Phase 6 Plan 02: S1142 Return Statements Reduction — Summary
Wydzielono walidacje do private helperów w AllegroIntegrationController i ShopproIntegrationsController, redukując return statements do ≤3 we wszystkich 6 metodach (łącznie z 9+6+5+4+4+4 → 3 każda).
Performance
| Metric |
Value |
| Duration |
~20 min |
| Started |
2026-03-13 |
| Completed |
2026-03-13 |
| Tasks |
2/2 completed |
| Files modified |
2 |
Acceptance Criteria Results
| Criterion |
Status |
Notes |
| AC-1: AllegroIntegrationController::save() ≤3 return |
Pass |
6→3 (wydzielono validateSaveInput) |
| AC-2: AllegroIntegrationController::saveImportSettings() ≤3 return |
Pass |
5→3 (wydzielono validateImportSettingsInput) |
| AC-3: AllegroIntegrationController::oauthCallback() ≤3 return |
Pass |
4→3 (wydzielono validateOAuthCallbackParams) |
| AC-4: ShopproIntegrationsController::save() ≤3 return |
Pass |
9→3 (validateSaveAccess + validateSaveInput) |
| AC-5: Brak regresji funkcjonalnej |
Pass |
Czysto strukturalna refaktoryzacja — logika, komunikaty, redirect paths bez zmian |
Accomplishments
- 6 metod zredukowanych do ≤3 return statements (AllegroIntegrationController: 3 metody; ShopproIntegrationsController: 3 metody)
- Ustalono dwa wzorce helperów wielokrotnego użytku:
?string dla walidacji danych, ?Response dla strażników dostępu
validateCsrfAndIntegrationAccess() w ShopproIntegrationsController współdzielony przez saveStatusMappings() i syncStatuses()- Try/catch w ShopproIntegrationsController::save() restrukturyzowany do jednego return na końcu metody
Task Commits
| Task |
Commit |
Type |
Description |
| Task 1+2: AllegroIntegrationController + ShopproIntegrationsController |
028c46c |
feat |
Redukcja return statements we wszystkich 6 metodach |
Files Created/Modified
| File |
Change |
Purpose |
src/Modules/Settings/AllegroIntegrationController.php |
Modified |
+3 private metody walidacji; save/saveImportSettings/oauthCallback zrefaktoryzowane |
src/Modules/Settings/ShopproIntegrationsController.php |
Modified |
+3 private metody walidacji/dostępu; save/saveStatusMappings/syncStatuses zrefaktoryzowane |
Decisions Made
| Decision |
Rationale |
Impact |
Wzorzec ?Response dla CSRF+not-found guard zamiast ?string |
CSRF i not-found mają różne redirect URL — zwrot gotowego Response czystszy niż tuple |
validateSaveAccess() i validateCsrfAndIntegrationAccess() spójne z istniejącym validateCsrf() w Allegro |
| Brak internal return w try/catch ShopproIntegrationsController::save() |
Pozwala na pojedynczy return na końcu metody |
$redirectTo mutuje na sukces, pozostaje $redirectTo przy błędzie |
| Jeden commit dla obu plików |
Obie zmiany to jeden spójny wzorzec refaktoryzacji |
Historia spójna |
Deviations from Plan
Summary
| Type |
Count |
Impact |
| Scope additions |
0 |
— |
| Auto-fixed |
0 |
— |
| Deferred |
1 |
Skill gap: sonar-scanner |
Total impact: Plan wykonany zgodnie ze specyfikacją.
Deferred Items
- sonar-scanner pominięty — narzędzie nie jest zainstalowane w PATH (konsekwentnie pomijane od Fazy 03). Violations S1142 powinny spaść o ~12-15 pozycji po uruchomieniu skanera.
Skill Audit
| Oczekiwany |
Wywołany |
Uwagi |
| sonar-scanner |
○ |
Pominięto — brak instalacji w PATH |
Issues Encountered
| Issue |
Resolution |
| ShopproIntegrationsController::save() miała 9 returns (plan mówił o 5) |
Wydzielono dwa helpery zamiast jednego; wzorzec ?Response dla CSRF+not-found guard |
Next Phase Readiness
Ready:
- Wzorzec validateXxxInput()/validateXxxAccess() ustalony — do użycia w kolejnych planach
- Codebase w stabilnym stanie; obie klasy syntaktycznie poprawne
Concerns:
- sonar-scanner nadal nieinstalowany — liczba aktualnych S1142 violations nieznana
- Plan 06-05 (god classes) zależy od 06-04 — kolejność: 06-06 → 06-04 → 06-05
Blockers: None
Phase: 06-sonarqube-quality, Plan: 02
Completed: 2026-03-13
Jacek Pyziak
ae976b7964