feat: Add Transport module with repository, controller, and views

- Implemented TransportRepository for managing transport data with methods for listing, finding, saving, and retrieving transport costs. - Created ShopTransportController to handle transport-related actions, including listing, editing, and saving transports. - Added views for transport management: transports list and transport edit forms. - Introduced JavaScript for responsive tabs in transport edit view. - Updated testing suite with comprehensive unit tests for TransportRepository and ShopTransportController. - Increased test coverage with new assertions and scenarios for transport functionalities.
2026-02-14 20:16:18 +01:00
parent 818cd7f2c0
commit 7468dd0d3f
20 changed files with 1215 additions and 377 deletions
--- a/docs/CHANGELOG.md
+++ b/docs/CHANGELOG.md
@@ -4,6 +4,24 @@ Logi zmian z migracji na Domain-Driven Architecture. Najnowsze na gorze.

 ---

+## ver. 0.269 (2026-02-14) - ShopTransport
+
+- **ShopTransport** - migracja `/admin/shop_transport` na Domain + DI + nowe widoki
+  - NOWE: `Domain\Transport\TransportRepository` (`listForAdmin`, `find`, `save`, `allActive`, `allForAdmin`, `findActiveById`, `getTransportCost`, `lowestTransportPrice`, `getApiloCarrierAccountId`)
+  - NOWE: `admin\Controllers\ShopTransportController` (DI) z akcjami `list`, `edit`, `save`
+  - NOWE: widoki `shop-transport/transports-list.php` i `shop-transport/transport-edit.php` + `transport-edit-custom-script.php`
+  - UPDATE: routing i menu admin na kanoniczny URL `/admin/shop_transport/list/`
+  - UPDATE: `admin\factory\ShopTransport`, `front\factory\ShopTransport` przepiete na nowe repozytorium
+  - FIX: `save()` return type `?int` zamiast `int|bool` (spojnosc z PaymentMethod)
+  - FIX: `toSwitchValue()` helper zamiast `=== 'on'` (obsluga '1', 'on', 'true', 'yes')
+  - FIX: `\S::delete_dir()` przeniesione z repozytorium do kontrolera (DDD)
+  - FIX: Medoo `select()` syntax - ORDER w WHERE zamiast 4-arg form
+  - CLEANUP: usuniete legacy `autoload/admin/controls/class.ShopTransport.php`, `autoload/admin/view/class.ShopTransport.php`, `admin/templates/shop-transport/view-list.php`
+  - FIX: `transports-list.php` - zmienna `'viewModel'` zmieniona na `'list'` (zgodnie z `table-list.php` komponentem)
+- Testy: **OK (300 tests, 895 assertions)**
+
+---
+
 ## ver. 0.268 (2026-02-14) - ShopPaymentMethod + Apilo token keepalive

 - **ShopPaymentMethod** - migracja `/admin/shop_payment_method` na Domain + DI + nowe widoki
--- a/docs/DATABASE_STRUCTURE.md
+++ b/docs/DATABASE_STRUCTURE.md
@@ -384,6 +384,37 @@ Metody platnosci sklepu (modul `/admin/shop_payment_method`).

 **Aktualizacja 2026-02-14 (ver. 0.268):** modul `/admin/shop_payment_method` korzysta z `Domain\PaymentMethod\PaymentMethodRepository` przez `admin\Controllers\ShopPaymentMethodController`. Usunieto legacy klasy `admin\controls\ShopPaymentMethod`, `admin\factory\ShopPaymentMethod`, `admin\view\ShopPaymentMethod` oraz widok `admin/templates/shop-payment-method/view-list.php`.

+## pp_shop_transports
+Rodzaje transportu sklepu (modul `/admin/shop_transport`).
+
+| Kolumna | Opis |
+|---------|------|
+| id | PK |
+| name | Nazwa (systemowa, readonly) |
+| name_visible | Nazwa widoczna dla klienta |
+| description | Opis metody transportu |
+| status | Status: 1 = aktywny, 0 = nieaktywny |
+| cost | Koszt dostawy (PLN) |
+| max_wp | Maksymalna waga paczki (NULL = bez limitu) |
+| default | Domyslna forma dostawy (0/1) |
+| delivery_free | Czy obsluguje darmowa dostawe (0/1) |
+| apilo_carrier_account_id | ID konta przewoznika w Apilo (NULL gdy brak mapowania) |
+| o | Kolejnosc wyswietlania |
+
+**Uzywane w:** `Domain\Transport\TransportRepository`, `admin\Controllers\ShopTransportController`, `front\factory\ShopTransport`, `admin\factory\ShopTransport`
+
+## pp_shop_transport_payment_methods
+Powiazanie metod transportu z metodami platnosci (tabela lacznikowa).
+
+| Kolumna | Opis |
+|---------|------|
+| id_transport | FK do pp_shop_transports |
+| id_payment_method | FK do pp_shop_payment_methods |
+
+**Uzywane w:** `Domain\Transport\TransportRepository`, `Domain\PaymentMethod\PaymentMethodRepository::forTransport()`
+
+**Aktualizacja 2026-02-14 (ver. 0.269):** modul `/admin/shop_transport` korzysta z `Domain\Transport\TransportRepository` przez `admin\Controllers\ShopTransportController`. Usunieto legacy klasy `admin\controls\ShopTransport`, `admin\view\ShopTransport` oraz widok `admin/templates/shop-transport/view-list.php`.
+
 ## pp_shop_apilo_settings
 Ustawienia integracji Apilo (key-value).

--- a/docs/PROJECT_STRUCTURE.md
+++ b/docs/PROJECT_STRUCTURE.md
@@ -126,6 +126,7 @@ shopPRO/

 ### Tabele checkout
 - `pp_shop_payment_methods` - metody platnosci sklepu (mapowanie `apilo_payment_type_id`)
+- `pp_shop_transports` - rodzaje transportu sklepu (mapowanie `apilo_carrier_account_id`)
 - `pp_shop_transport_payment_methods` - powiazanie metod transportu i platnosci

 Pelna dokumentacja tabel: `DATABASE_STRUCTURE.md`
@@ -215,6 +216,8 @@ autoload/
 │   │   └── CouponRepository.php
 │   ├── ShopStatus/
 │   │   └── ShopStatusRepository.php
+│   ├── Transport/
+│   │   └── TransportRepository.php
 │   └── ...
 ├── admin/
 │   ├── Controllers/                # Nowe kontrolery (namespace \admin\Controllers\)
@@ -232,6 +235,13 @@ autoload/
 - Modul `/admin/shop_payment_method/*` dziala na nowych widokach (`payment-methods-list`, `payment-method-edit`).
 - Usunieto legacy: `autoload/admin/controls/class.ShopPaymentMethod.php`, `autoload/admin/factory/class.ShopPaymentMethod.php`, `autoload/admin/view/class.ShopPaymentMethod.php`, `admin/templates/shop-payment-method/view-list.php`.

+**Aktualizacja 2026-02-14 (ver. 0.269):**
+- Dodano modul domenowy `Domain/Transport/TransportRepository.php`.
+- Dodano kontroler DI `admin/Controllers/ShopTransportController.php`.
+- Modul `/admin/shop_transport/*` dziala na nowych widokach (`transports-list`, `transport-edit`).
+- Usunieto legacy: `autoload/admin/controls/class.ShopTransport.php`, `autoload/admin/view/class.ShopTransport.php`, `admin/templates/shop-transport/view-list.php`.
+- `admin\factory\ShopTransport` i `front\factory\ShopTransport` przepiete na repozytorium.
+
 ### Routing admin (admin\Site::route())
 1. Sprawdź mapę `$newControllers` → utwórz instancję z DI → wywołaj
 2. Jeśli nowy kontroler nie istnieje (`class_exists()` = false) → fallback na `admin\controls\`
--- a/docs/REFACTORING_PLAN.md
+++ b/docs/REFACTORING_PLAN.md
@@ -149,6 +149,7 @@ grep -r "Product::getQuantity" .
 | 18 | ShopCoupon | 0.266 | listForAdmin, find, save, delete, categoriesTree |
 | 19 | ShopStatuses | 0.267 | listForAdmin, find, save, color picker |
 | 20 | ShopPaymentMethod | 0.268 | listForAdmin, find, save, allActive, mapowanie Apilo, DI kontroler |
+| 21 | ShopTransport | 0.269 | listForAdmin, find, save, allActive, allForAdmin, findActiveById, getTransportCost, lowestTransportPrice, getApiloCarrierAccountId, powiazanie z PaymentMethod, DI kontroler |

 ### Product - szczegolowy status
 - ✅ getQuantity (ver. 0.238)
@@ -167,12 +168,12 @@ grep -r "Product::getQuantity" .

 ## Kolejność refaktoryzacji (priorytet)

-1-20: ✅ Cache, Product, Banner, Settings, Dictionaries, ProductArchive, Filemanager, Users, Pages, Integrations, ShopPromotion, ShopCoupon, ShopStatuses, ShopPaymentMethod
+1-21: ✅ Cache, Product, Banner, Settings, Dictionaries, ProductArchive, Filemanager, Users, Pages, Integrations, ShopPromotion, ShopCoupon, ShopStatuses, ShopPaymentMethod, ShopTransport

 Nastepne:
-21. **Order**
-22. **Category**
-23. **ShopAttribute**
+22. **Order**
+23. **Category**
+24. **ShopAttribute**

 ## Form Edit System

@@ -269,7 +270,7 @@ tests/
 │           └── UsersControllerTest.php
 └── Integration/
 ```
-**Łącznie: 280 testów, 828 asercji**
+**Łącznie: 300 testów, 895 asercji**

 Pelna dokumentacja testow: `TESTING.md`

--- a/docs/TESTING.md
+++ b/docs/TESTING.md
@@ -36,7 +36,7 @@ Alternatywnie (Git Bash):
 Ostatnio zweryfikowano: 2026-02-14

 ```text
-OK (280 tests, 828 assertions)
+OK (300 tests, 895 assertions)
 ```

 ## Struktura testow
@@ -57,6 +57,7 @@ tests/
 |   |   |-- Promotion/PromotionRepositoryTest.php
 |   |   |-- Settings/SettingsRepositoryTest.php
 |   |   |-- ShopStatus/ShopStatusRepositoryTest.php
+|   |   |-- Transport/TransportRepositoryTest.php
 |   |   `-- User/UserRepositoryTest.php
 |   `-- admin/
 |       `-- Controllers/
@@ -69,6 +70,7 @@ tests/
 |           |-- ShopPaymentMethodControllerTest.php
 |           |-- ShopPromotionControllerTest.php
 |           |-- ShopStatusesControllerTest.php
+|           |-- ShopTransportControllerTest.php
 |           `-- UsersControllerTest.php
 `-- Integration/
 ```
@@ -351,3 +353,25 @@ OK (254 tests, 736 assertions)
 Nowe testy dodane 2026-02-14:
 - `tests/Unit/Domain/ShopStatus/ShopStatusRepositoryTest.php` (9 testow: find z ID=0, find null apilo, save update, save z ID=0, empty apilo sets null, reject negative ID, getApiloStatusId, getByIntegrationStatusId, allStatuses, whitelist sortowania)
 - `tests/Unit/admin/Controllers/ShopStatusesControllerTest.php` (5 testow: kontrakty metod, brak aliasow legacy, return types, DI konstruktora)
+
+## Aktualizacja suite (ShopPaymentMethod refactor, ver. 0.268)
+Ostatnio zweryfikowano: 2026-02-14
+
+```text
+OK (280 tests, 828 assertions)
+```
+
+Nowe testy dodane 2026-02-14:
+- `tests/Unit/Domain/PaymentMethod/PaymentMethodRepositoryTest.php` (14 testow: find invalid/null/normalize, save update/null/non-numeric apilo, listForAdmin whitelist, allActive, allForAdmin, findActiveById, isActive, getApiloPaymentTypeId, forTransport)
+- `tests/Unit/admin/Controllers/ShopPaymentMethodControllerTest.php` (5 testow: kontrakty metod, brak aliasow legacy, return types, DI konstruktora)
+
+## Aktualizacja suite (ShopTransport refactor, ver. 0.269)
+Ostatnio zweryfikowano: 2026-02-14
+
+```text
+OK (300 tests, 895 assertions)
+```
+
+Nowe testy dodane 2026-02-14:
+- `tests/Unit/Domain/Transport/TransportRepositoryTest.php` (14 testow: find invalid/null/normalize/nullables, save insert/update/failure/default reset/switch normalization, listForAdmin whitelist, allActive, getApiloCarrierAccountId, getTransportCost, allForAdmin)
+- `tests/Unit/admin/Controllers/ShopTransportControllerTest.php` (5 testow: kontrakty metod, brak aliasow legacy, return types, DI konstruktora z 2 repo)