shopPRO/docs/LEGACY_SHOP_REFACTORING_PLAN.md

Plan refaktoryzacji autoload/shop/ — usunięcie legacy klas

Kontekst

Katalog autoload/shop/ zawierał 12 legacy klas (~2 363 linii kodu), które pełniły rolę fasad/wrapperów nad warstwą Domain\. Wszystkie klasy zostały usunięte — logika przeniesiona do warstwy Domain\ / Shared\, wywołania zaktualizowane.

Status: UKOŃCZONE (wszystkie 12 klas usunięte, katalog autoload/shop/ pusty)

Status

Faza	Klasy	Status
1	Transport, ProductSet, Coupon	✅ Ukończona
2	Shop, Search, Basket	✅ Ukończona
3	ProductCustomField, Category, ProductAttribute	✅ Ukończona
4	Promotion	✅ Ukończona
5	Order, Product	✅ Ukończona

---

Faza 1 — Trywialne fasady (0 logiki do migracji)

1.1 class.Transport.php ✅ (~31 linii)

Jedyne zewnętrzne użycie:

• admin\Controllers\ShopOrderController.php → \shop\Transport::transport_list()

Plan:

• Dodać metodę allActiveOrdered() do Domain\Transport\TransportRepository (odpowiednik transport_list())
• Zamienić wywołanie w ShopOrderController
• Usunąć class.Transport.php

1.2 class.ProductSet.php ✅ (~49 linii)

Jedyne zewnętrzne użycie:

• admin\Controllers\ShopProductController.php:224 → \shop\ProductSet::sets_list()

Plan:

• ProductSetRepository::allSets() już istnieje — zamienić wywołanie na $this->productSetRepo->allSets()
• Wstrzyknąć ProductSetRepository do ShopProductController (jeśli jeszcze nie jest)
• Usunąć class.ProductSet.php

1.3 class.Coupon.php ✅ (~84 linii)

Zewnętrzne użycia (3×):

• shop\class.Order.php:171 → new \shop\Coupon($id) (do przeniesienia w fazie Order)
• front\Controllers\ShopOrderController.php:139 → new \shop\Coupon($id)
• admin\Controllers\ShopOrderController.php:170 → new \shop\Coupon($id)

Plan:

• CouponRepository::findByName() / find() już istnieje — dodać findById(int $id): ?array jeśli brak
• Zamienić new \shop\Coupon($id) → $this->couponRepo->findById($id) w obu kontrolerach
• Użycie w class.Order.php — rozwiąże się w fazie Order
• Usunąć class.Coupon.php

---

Faza 2 — Proste wrappery (minimalna logika do przeniesienia)

2.1 class.Shop.php ✅ (~39 linii) — utility cenowe

Zewnętrzne użycia (~24× w szablonach):

• templates/shop-product/product.php, product-mini.php
• templates/shop-search/product-search.php
• templates/shop-basket/alert-product-sets.php
• templates/controls/alert-product-sets.php

Plan:

• Przenieść shortPrice() i isWholeNumber() do Shared\Helpers\Helpers jako metody statyczne
• Zamienić \shop\Shop::shortPrice() → \Shared\Helpers\Helpers::shortPrice() we wszystkich szablonach (replace_all)
• Usunąć class.Shop.php

2.2 class.Search.php ✅ (~70 linii)

Jedyne zewnętrzne użycie:

• front\LayoutEngine.php:337 → \shop\Search::simple_form()

Plan:

• Przenieść simple_form() do front\Views\ShopSearch (nowa klasa View, statyczna)
• Przenieść search_results() i search_products() do front\Controllers\ShopSearchController lub odpowiedniej istniejącej lokalizacji
• Zamienić wywołanie w LayoutEngine
• Usunąć class.Search.php
• Poprawić bug: use shop\Produt; (literówka — brakuje 'c')

2.3 class.Basket.php ✅ (~92 linii)

Zewnętrzne użycia (6× w ShopBasketController):

• validate_basket() — prosta walidacja tablicy sesji
• check_product_quantity_in_stock() — sprawdzenie stanów magazynowych

Plan:

• Przenieść obie metody do Domain\Basket\BasketCalculator (już istnieje)
• Zamienić wywołania w ShopBasketController
• Usunąć class.Basket.php

---

Faza 3 — Klasy z logiką cache/atrybutów

3.1 class.ProductCustomField.php ✅ (~74 linii)

Zewnętrzne użycia (2×):

• Domain\Order\OrderRepository.php:643 → \shop\ProductCustomField::getFromCache()
• templates/shop-basket/_partials/product-custom-fields.php:3 → \shop\ProductCustomField::getFromCache()

Plan:

• Dodać findCached(int $id): ?array do Domain\Product\ProductRepository (lub nowy ProductCustomFieldRepository)
• Zamienić wywołania w OrderRepository i szablonie
• Usunąć class.ProductCustomField.php

3.2 class.Category.php ✅ (~84 linii)

Zewnętrzne użycia (2×):

• index.php:225 → \shop\Category::get_category_products_id() (FB Pixel)
• templates/shop-category/category.php:19 → \shop\Category::get_subcategory_by_category()

Plan:

• CategoryRepository::productsId() prawdopodobnie istnieje — sprawdzić i użyć lub dodać
• Dodać subcategoriesCached(int $categoryId): array do CategoryRepository
• Zamienić wywołania
• Usunąć class.Category.php

3.3 class.ProductAttribute.php ✅ (~119 linii)

Zewnętrzne użycia (3 lokalizacje):

• shop\class.Product.php — użycia wewnętrzne (rozwiążą się w fazie Product)
• admin/templates/shop-product/product-combination.php:32 → getAttributeName(), get_value_name()
• templates/shop-product/_partial/product-attribute.php:13 → get_value_name()

Plan:

• AttributeRepository::getAttributeNameById() i getAttributeValueById() już istnieją
• Dodać brakujące metody do AttributeRepository: isValueDefault(), getAttributeOrder(), getAttributeNameByValue() (z Redis cache)
• Zamienić wywołania w szablonach
• Usunąć class.ProductAttribute.php

---

Faza 4 — Klasy z logiką promocji

4.1 class.Promotion.php ✅ (~107 linii)

Zewnętrzne użycia (3 lokalizacje):

• front\Controllers\ShopBasketController.php (6×) → \shop\Promotion::find_promotion()
• admin\Controllers\ShopPromotionController.php → ::$condition_type, ::$discount_type

Plan:

• Przenieść get_active_promotions() do PromotionRepository (z Redis cache)
• Przenieść find_promotion() do PromotionRepository — orkiestracja logiki applyType*
• Przenieść stałe $condition_type / $discount_type jako stałe klasowe do PromotionRepository
• Zamienić wywołania w ShopBasketController i ShopPromotionController
• Usunąć class.Promotion.php

---

Faza 5 — Duże klasy z wieloma zależnościami

5.1 class.Order.php ✅ (~562 linii)

Główna logika do migracji:

• OrderAdminService ma już: setOrderAsPaid(), setOrderAsUnpaid(), changeStatus(), resendConfirmationEmail(), saveOrderByAdmin(), deleteOrder(), saveNotes()
• Brakuje: logika Apilo sync (process_apilo_sync_queue(), sync_apilo_payment(), sync_apilo_status(), queue management)
• Brakuje: send_status_change_email() (ale może być zintegrowane w changeStatus())

Plan:

1. Sprawdzić, czy OrderAdminService obsługuje już Apilo sync (metoda sendOrderToApilo() istnieje)
2. Przenieść process_apilo_sync_queue() + kolejkę Apilo do Domain\Integrations\ApiloService (lub do IntegrationsRepository)
3. Przenieść send_status_change_email() do OrderAdminService (jeśli nie jest częścią changeStatus())
4. Usunąć referencję do new \shop\Coupon() — użyć CouponRepository (z fazy 1.3)
5. Zaktualizować cron.php (jeśli woła process_apilo_sync_queue)
6. Usunąć class.Order.php

5.2 class.Product.php ✅ (~952 linii) — NAJWIĘKSZA KLASA

Stan: Większość logiki zmigowana do ProductRepository, ale klasa pełni rolę entity z ArrayAccess (używana w szablonach jako $product['name']).

Metody do przeniesienia:

• getFromCache() — cache wrappera entity → przenieść do ProductRepository::findCached()
• getDefaultCombinationPrices() → ProductRepository
• generateSubtitleFromAttributes() → ProductRepository
• getProductDataBySelectedAttributes() → ProductRepository
• is_product_on_promotion() → ProductRepository
• product_sets_when_add_to_basket() → ProductSetRepository
• add_visit() → ProductRepository
• getProductImg() / getProductUrl() → ProductRepository
• searchProductsByName() / searchProductByNameAjax() / searchProductsByNameCount() → ProductRepository
• is_stock_0_buy() → ProductRepository
• get_product_permutation_quantity_options() → ProductRepository
• calculate_basket_product_price() → Domain\Basket\BasketCalculator
• get_product_id_by_attributes() / get_product_permutation_hash() / get_product_attributes() → ProductRepository lub AttributeRepository
• array_cartesian() / permutations() → ProductRepository (helper prywatny)
• generate_sku_code() → ProductRepository
• product_meta() → ProductRepository

Kluczowy problem: szablony używają $product['field'] przez ArrayAccess. Po migracji szablony będą dostawać zwykłe array z ProductRepository — to jest kompatybilne, bo $product['field'] działa tak samo na tablicy.

Plan:

1. Etapami przenosić metody statyczne do ProductRepository / AttributeRepository
2. Przenieść calculate_basket_product_price() do BasketCalculator
3. Zamienić Product::getFromCache() → ProductRepository::findCached() (zwraca tablicę)
4. Zaktualizować szablony i kontrolery
5. Usunąć class.Product.php

---

Pliki do modyfikacji (podsumowanie)

Plik	Zmiany
autoload/Domain/Transport/TransportRepository.php	+ allActiveOrdered()
autoload/Domain/Basket/BasketCalculator.php	+ validateBasket(), checkStock(), calculateProductPrice()
autoload/Domain/Promotion/PromotionRepository.php	+ getActivePromotions(), findPromotion(), stałe
autoload/Domain/Product/ProductRepository.php	+ ~15 metod z class.Product.php
autoload/Domain/Attribute/AttributeRepository.php	+ isValueDefault(), getAttributeOrder(), getAttributeNameByValue()
autoload/Domain/Category/CategoryRepository.php	+ subcategoriesCached(), categoryProductIds()
autoload/Domain/Order/OrderAdminService.php	+ email statusu, sprawdzenie Apilo
autoload/Domain/Integrations/	+ ApiloSyncService (queue Apilo)
autoload/Domain/Coupon/CouponRepository.php	+ findById()
autoload/Shared/Helpers/Helpers.php	+ shortPrice(), isWholeNumber()
autoload/front/Views/ShopSearch.php	NOWY — simple_form()
autoload/front/LayoutEngine.php	zamiana \shop\Search → \front\Views\ShopSearch
autoload/front/Controllers/ShopBasketController.php	zamiana \shop\Basket, \shop\Promotion
autoload/admin/Controllers/ShopOrderController.php	zamiana \shop\Coupon, \shop\Transport
autoload/admin/Controllers/ShopProductController.php	zamiana \shop\ProductSet
autoload/admin/Controllers/ShopPromotionController.php	zamiana ::$condition_type, ::$discount_type
autoload/front/Controllers/ShopOrderController.php	zamiana \shop\Coupon
Szablony (templates/)	zamiana \shop\Shop::shortPrice(), \shop\ProductAttribute::*, \shop\Category::*
index.php	zamiana \shop\Category::get_category_products_id()
autoload/Domain/Order/OrderRepository.php	zamiana \shop\ProductCustomField

Weryfikacja

Po każdej fazie:

1. Uruchomić ./test.ps1 — wszystkie 610+ testów muszą przechodzić
2. Grep po \shop\ w całym codebase — upewnić się, że usunięta klasa nie jest nigdzie używana
3. Po zakończeniu wszystkich faz: grep -r "\\\\shop\\\\" autoload/ templates/ admin/ index.php ajax.php cron.php api.php powinien zwracać 0 wyników

Szacunek złożoności

Faza	Klasy	Linii do usunięcia	Złożoność
1	Transport, ProductSet, Coupon	~164	Niska
2	Shop, Search, Basket	~201	Niska-średnia
3	ProductCustomField, Category, ProductAttribute	~277	Średnia
4	Promotion	~107	Średnia
5	Order, Product	~1 514	Wysoka
Razem	12 klas	~2 363