Jacek Pyziak
87203c4321
Phase 02 plans 02-01, 02-02, 02-03: - fix(02-01): dead condition in AllegroShipmentService ZPL page size Both ternary branches returned 'A6'; ZPL now correctly returns 'ZPL' - fix(02-02): add last_status_checked_at cursor to AllegroStatusSyncService New migration adds orders.last_status_checked_at DATETIME NULL with composite index (source, source_updated_at). findOrdersNeedingStatusSync() filters by cursor; markOrderStatusChecked() records timestamp on success. - fix(02-03): replace AllegroOrderSyncStateRepository in ShopproOrdersSyncService New ShopproOrderSyncStateRepository (same table, correct class name). Application.php wires correct repository to correct service. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
37 lines
3.1 KiB
Markdown
37 lines
3.1 KiB
Markdown
1. [x] Na liście zamówień powiększenie zdjęcia produktu na hover nie na onclick, wtedy to nie może być modal zamykany X
|
||
2. [x] Doda<64> rejestracj<63> historii zam<61>wie<69>, i zmiana statusu rejestrowana w Historii zmian zamówienia
|
||
3. [x] Pobranie zamówienia rejestrowane w histori zmian zamówienia
|
||
4. [x] Przy imporcie zamówień musi być pobierania forma wysyłki.
|
||
5. [x] W szczególach zamówienia dorobić opcję zmiany statusu.
|
||
6. [x] W szczeg<65>lach zam<61>wienia 2 razy wy<77>wietla si<73> ID zam<61>wienai z allegro, np: 008d3d60-1743-11f1-b15c-fdb4f87ccfc6
|
||
7. [x] Przy imporcie z allegro liczba przesy<73>ek jest 0.
|
||
8. [x] Kolumna LP w szczeg<65>lach zam<61>wienia jest zbyt szeroka.
|
||
9. [x] Na lis<69>ie zam<61>wie<69> pole po kt<6B>rym jest domy<6D>lnie sortowana czyli data zam<61>wienia jest puste.
|
||
10. [x] Na li<6C>cie zam<61>wie<69> ukry<72> kolumn<6D> ostatnia zmiana.
|
||
11. [x] W ustawieniach doda<64> zak<61>adk<64> Integracja Apaczka. Doda<64> tam pierwsze ustawienie, czyli klucz API.
|
||
12. [] synchronizowa<77> r<>czn<7A> zmian<61> statusu z allegro
|
||
13. [x] W ustawieniach cron https://orderpro.projectpro.pl/settings/cron historia powinna mie<69> stronicowanie
|
||
14. [] border input<75>w, select, textarea, itd zr<7A>b troszk<7A> ciemniejszy
|
||
15. [] W tym miejscu odwróć kolejność: najpierw źródło potem ID, <div class="orders-ref__meta"><span>f6079660-1af8-11f1-a7c9-231cf6ef29d1</span><span>allegro</span></div>
|
||
16. [] Na liście zamówień statusy powinno być pokolorowane zgodnie z ustawieniami.
|
||
17. [] Na liście zamówien jak jest źródło i id zamówienia to zamiast shopPRO musi pisać która integracja konkretnie. Oraz dodajemy napis ID: ...D
|
||
|
||
## SonarQube — post plany 02-02 i 02-03 (skan 2026-03-13)
|
||
30. [] [Sonar 2026-03-13] Brak nowych issues — AllegroStatusSyncService i ShopproOrderSyncStateRepository czyste. Pre-existing issues w ShopproOrdersSyncService (god class) i Application.php niezmienione przez nasze modyfikacje.
|
||
|
||
## SonarQube — post plan 01-01 (skan 2026-03-12)
|
||
28. [] [Sonar 2026-03-12] php:S1142 — AllegroTokenManager::resolveToken() ma 4 returny (powyżej limitu 3) (1x nowe)
|
||
29. [] [Sonar 2026-03-12] php:S112 — AllegroTokenManager rzuca generic RuntimeException zamiast dedykowanej klasy wyjątku (3x nowe)
|
||
|
||
## SonarQube — code quality (327 issues, skan 2026-03-12)
|
||
18. [] php:S112 (95x) — zastąpić generic `new \Exception` konkretnymi klasami wyjątków
|
||
19. [] php:S1142 (57x) — zredukować liczbę `return` w metodach (early return → wydzielić metody)
|
||
20. [] php:S1192 (40x) — wyciągnąć powtarzające się string literals do stałych
|
||
21. [] php:S3776 (31x) — obniżyć złożoność kognitywną metod (wydzielić logikę do pomocniczych metod)
|
||
22. [] Web:S6827 (15x) — dodać brakujące atrybuty `alt` na tagach `<img>`
|
||
23. [] Web:S6819 (12x) — poprawić dostępność HTML (accessibility)
|
||
24. [] php:S1172 (11x) — usunąć nieużywane parametry funkcji
|
||
25. [] php:S3358 (11x) — rozwinąć zagnieżdżone operatory ternarne
|
||
26. [] php:S1448 (6x) — podzielić klasy z za dużą liczbą metod
|
||
27. [] php:S138 (4x) — skrócić zbyt długie metody
|