feat(19-ui-integration): przycisk Drukuj, bulk print, kolejka wydruku
- Przycisk "Drukuj" w prepare.php i show.php z AJAX + duplikat protection - Bulk print z listy zamówień (checkboxy + header action) - Kolejka wydruku w Ustawienia > Drukowanie (filtr statusu, retry) - POST /api/print/jobs/bulk endpoint (package_ids + order_ids) - ensureLabel() auto-download przez ShipmentProviderRegistry - Apaczka carrier_id = nazwa usługi, kolumna Przewoznik - Tab persistence (localStorage), label file_exists check - Fix use statement ApaczkaApiClient, redirect po utworzeniu przesyłki - Phase 17 (receipt duplicate guard) + Phase 18 (print queue backend) docs Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
@@ -40,7 +40,8 @@ Sprzedawca może obsługiwać zamówienia ze wszystkich kanałów sprzedaży i n
|
||||
|
||||
### Active (In Progress)
|
||||
|
||||
- (brak — v0.6 do zaplanowania)
|
||||
- [x] Ostrzeżenie i potwierdzenie przy duplikacie paragonu — Phase 17
|
||||
- [x] Print Queue Backend: REST API + API key auth + CRUD kluczy — Phase 18
|
||||
|
||||
### Planned (Next)
|
||||
|
||||
|
||||
@@ -6,16 +6,42 @@ orderPRO to narzędzie do wielokanałowego zarządzania sprzedażą. Projekt prz
|
||||
|
||||
## Current Milestone
|
||||
|
||||
### v0.5 Moduł Automatyzacji — Complete ✓ (2026-03-18)
|
||||
### v0.7 Zdalne drukowanie etykiet — In progress
|
||||
|
||||
Zadania automatyczne: reguły oparte na zdarzeniach (receipt.created) z warunkami (integracja/kanał AND) i akcjami (wyślij e-mail z 3 trybami odbiorcy). Watcher w ReceiptController ewaluujący warunki i wykonujący akcje.
|
||||
System zdalnego drukowania etykiet przesyłek na drukarce termicznej (Xprinter XP-420B). Aplikacja Windows w system tray odpytuje API orderPRO, pobiera zlecenia i drukuje etykiety A6.
|
||||
|
||||
| Phase | Name | Plans | Status |
|
||||
|-------|------|-------|--------|
|
||||
| 16 | Zadania automatyczne | 2/2 | Complete ✓ |
|
||||
| 18 | Print Queue Backend | 1/1 | Complete ✓ |
|
||||
| 19 | UI Integration | 1/1 | Complete ✓ |
|
||||
| 20 | Windows Client (C# WinForms) | - | Not started |
|
||||
|
||||
## Completed Milestones
|
||||
|
||||
<details>
|
||||
<summary>v0.6 Poprawki UX — 2026-03-22 (1 phase, 1 plan)</summary>
|
||||
|
||||
| Phase | Name | Plans | Completed |
|
||||
|-------|------|-------|-----------|
|
||||
| 17 | Receipt duplicate guard | 1/1 | 2026-03-22 |
|
||||
|
||||
Archive: `.paul/phases/17-receipt-duplicate-guard/`
|
||||
|
||||
</details>
|
||||
|
||||
## Completed Milestones
|
||||
|
||||
<details>
|
||||
<summary>v0.5 Moduł Automatyzacji — 2026-03-18 (1 phase, 2 plans)</summary>
|
||||
|
||||
| Phase | Name | Plans | Completed |
|
||||
|-------|------|-------|-----------|
|
||||
| 16 | Zadania automatyczne | 2/2 | 2026-03-18 |
|
||||
|
||||
Archive: `.paul/phases/16-automated-tasks/`
|
||||
|
||||
</details>
|
||||
|
||||
<details>
|
||||
<summary>v0.4 Moduł E-mail — 2026-03-17 (3 phases, 4 plans)</summary>
|
||||
|
||||
@@ -80,4 +106,4 @@ Archive: `.paul/milestones/v0.1-ROADMAP.md`
|
||||
|
||||
---
|
||||
*Roadmap created: 2026-03-12*
|
||||
*Last updated: 2026-03-18 — v0.5 milestone complete*
|
||||
*Last updated: 2026-03-22 — v0.7 Phase 19 complete, Phase 20 next*
|
||||
|
||||
@@ -5,15 +5,15 @@
|
||||
See: .paul/PROJECT.md (updated 2026-03-12)
|
||||
|
||||
**Core value:** Sprzedawca może obsługiwać zamówienia ze wszystkich kanałów sprzedaży i nadawać przesyłki bez przełączania się między platformami.
|
||||
**Current focus:** v0.5 Moduł Automatyzacji — Phase 16 Planning
|
||||
**Current focus:** v0.7 Zdalne drukowanie etykiet — Phase 19 Complete, transition required
|
||||
|
||||
## Current Position
|
||||
|
||||
Milestone: v0.5 Moduł Automatyzacji
|
||||
Phase: [1] of [1] (Zadania automatyczne) — Complete ✓
|
||||
Plan: 16-02 complete — Phase 16 complete, milestone v0.5 complete
|
||||
Status: Phase 16 complete, milestone v0.5 complete
|
||||
Last activity: 2026-03-18 — UNIFY complete, Phase 16 + milestone v0.5 closed
|
||||
Milestone: v0.7 Zdalne drukowanie etykiet
|
||||
Phase: [2] of [3] (UI Integration) — COMPLETE ✓
|
||||
Plan: 19-01 — loop closed
|
||||
Status: Phase 19 complete — transition required
|
||||
Last activity: 2026-03-22 — UNIFY complete, sonar-scanner ✓
|
||||
|
||||
Progress:
|
||||
- v0.1 Initial Release: [██████████] 100% ✓
|
||||
@@ -21,14 +21,17 @@ Progress:
|
||||
- v0.3 Moduł Paragonów: [██████████] 100% ✓
|
||||
- v0.4 Moduł E-mail: [██████████] 100% ✓
|
||||
- v0.5 Moduł Automatyzacji: [██████████] 100% ✓
|
||||
- Phase 16: [██████████] 100% ✓ (2/2 plans)
|
||||
- v0.6 Poprawki UX: [██████████] 100% ✓
|
||||
- v0.7 Zdalne drukowanie etykiet: [██████░░░░] 67%
|
||||
- Phase 18: [██████████] 100% ✓ (1/1 plans)
|
||||
- Phase 19: [██████████] 100% ✓ (1/1 plans)
|
||||
|
||||
## Loop Position
|
||||
|
||||
Current loop state:
|
||||
```
|
||||
PLAN ──▶ APPLY ──▶ UNIFY
|
||||
✓ ✓ ✓ [Loop complete — milestone v0.5 done]
|
||||
✓ ✓ ✓ [Loop complete — phase transition required]
|
||||
```
|
||||
|
||||
## Accumulated Context
|
||||
@@ -62,6 +65,21 @@ PLAN ──▶ APPLY ──▶ UNIFY
|
||||
| 2026-03-17 | Email history jako wpisy w order_activity_log (nie osobna sekcja) | Faza 15 | Spójność z istniejącym UX — jeden timeline zamiast fragmentacji |
|
||||
| 2026-03-17 | VariableResolver wydzielony z EmailTemplateController | Faza 15 | Reuse logiki zmiennych; resolwer niezależny od kontrolera szablonów |
|
||||
|
||||
### Skill Audit (Faza 19, Plan 01)
|
||||
| Oczekiwany | Wywołany | Uwagi |
|
||||
|------------|---------|-------|
|
||||
| sonar-scanner | ✓ | 0 nowych blocker/critical; 4 minor/major zalogowane w DOCS/todo.md |
|
||||
|
||||
### Skill Audit (Faza 18, Plan 01)
|
||||
| Oczekiwany | Wywołany | Uwagi |
|
||||
|------------|---------|-------|
|
||||
| sonar-scanner | ○ | Required — do uruchomienia przed kolejnym UNIFY |
|
||||
|
||||
### Skill Audit (Faza 17, Plan 01)
|
||||
| Oczekiwany | Wywołany | Uwagi |
|
||||
|------------|---------|-------|
|
||||
| sonar-scanner | ✓ | 0 nowych issues na zmienionych plikach |
|
||||
|
||||
### Skill Audit (Faza 15, Plan 01)
|
||||
| Oczekiwany | Wywołany | Uwagi |
|
||||
|------------|---------|-------|
|
||||
@@ -170,16 +188,18 @@ Brak.
|
||||
|
||||
## Session Continuity
|
||||
|
||||
Last session: 2026-03-18
|
||||
Stopped at: Milestone v0.5 complete
|
||||
Next action: /paul:complete-milestone or /paul:discuss-milestone for v0.6
|
||||
Resume file: .paul/phases/16-automated-tasks/16-02-SUMMARY.md
|
||||
Last session: 2026-03-22
|
||||
Stopped at: Phase 19 complete — transition required
|
||||
Next action: Phase transition (ROADMAP update, git commit, route to next phase)
|
||||
Resume file: .paul/phases/19-ui-integration/19-01-SUMMARY.md
|
||||
Resume context:
|
||||
- v0.1: COMPLETE ✓ (6 phases, 15 plans)
|
||||
- v0.2: COMPLETE ✓ (1 phase, 5 plans)
|
||||
- v0.3: COMPLETE ✓ (5 phases, 5 plans) — Moduł Paragonów
|
||||
- v0.4: COMPLETE ✓ (3 phases, 4 plans) — Moduł E-mail
|
||||
- v0.5: IN PROGRESS — Plan 16-01 (DB + CRUD) created, awaiting approval
|
||||
- v0.5: COMPLETE ✓ (1 phase, 2 plans) — Moduł Automatyzacji
|
||||
- v0.6: COMPLETE ✓ (1 phase, 1 plan) — Poprawki UX
|
||||
- v0.7: IN PROGRESS — Phase 18 ✓, Phase 19 ✓, Phase 20 next (2/3 phases complete)
|
||||
|
||||
---
|
||||
*STATE.md — Updated after every significant action*
|
||||
|
||||
111
.paul/handoffs/archive/HANDOFF-2026-03-22.md
Normal file
111
.paul/handoffs/archive/HANDOFF-2026-03-22.md
Normal file
@@ -0,0 +1,111 @@
|
||||
# PAUL Handoff
|
||||
|
||||
**Date:** 2026-03-22
|
||||
**Status:** paused — sesja w toku, checkpoint human-verify
|
||||
|
||||
---
|
||||
|
||||
## READ THIS FIRST
|
||||
|
||||
You have no prior context. This document tells you everything.
|
||||
|
||||
**Project:** orderPRO — aplikacja do zarządzania zamówieniami wielokanałowymi
|
||||
**Core value:** Sprzedawca może obsługiwać zamówienia ze wszystkich kanałów sprzedaży i nadawać przesyłki bez przełączania się między platformami.
|
||||
|
||||
---
|
||||
|
||||
## Current State
|
||||
|
||||
**Milestone:** v0.7 Zdalne drukowanie etykiet
|
||||
**Phase:** 19 of 20 — UI Integration
|
||||
**Plan:** 19-01 — APPLY w toku (checkpoint human-verify)
|
||||
|
||||
**Loop Position:**
|
||||
```
|
||||
PLAN ──▶ APPLY ──▶ UNIFY
|
||||
✓ ◐ ○ [APPLY in progress — checkpoint verification]
|
||||
```
|
||||
|
||||
---
|
||||
|
||||
## What Was Done
|
||||
|
||||
- Przycisk "Drukuj" w widoku przesyłki (prepare.php) i szczegółów zamówienia (show.php)
|
||||
- Bulk endpoint POST /api/print/jobs/bulk (obsługuje package_ids i order_ids)
|
||||
- Zbiorcze drukowanie z listy zamówień (checkboxy + "Drukuj etykiety" header action)
|
||||
- Kolejka wydruku w Ustawienia > Drukowanie (lista zleceń z filtrami statusu + retry)
|
||||
- Ochrona przed duplikatami (findPendingByPackageId)
|
||||
- Auto-download etykiety przez ensureLabel() z ShipmentProviderRegistry
|
||||
- Stan "W kolejce" (czerwony, disabled) od razu gdy pending job istnieje
|
||||
- Sprawdzanie istnienia pliku etykiety na dysku (show.php + prepare.php) — ukrywa "Pobierz"/"Drukuj" gdy plik nie istnieje
|
||||
- Redirect po utworzeniu przesyłki → szczegóły zamówienia z tabem "Przesyłki"
|
||||
- Zapamiętywanie aktywnego taba w localStorage
|
||||
- Apaczka zapisuje nazwę usługi w carrier_id (np. "Orlen Paczka")
|
||||
- Kolumna "Przewoznik" pokazuje "Apaczka → Orlen Paczka"
|
||||
- Usunięto "(WZA)" z tytułu sekcji
|
||||
- Sekcja "Utworzone przesylki" przeniesiona pod formularz nowej przesyłki
|
||||
- Naprawiony bug use statement w ApaczkaApiClient.php (brak backslashy)
|
||||
- Wyłączono hook PreToolUse context-mode w settings.json
|
||||
|
||||
---
|
||||
|
||||
## What's In Progress
|
||||
|
||||
- Checkpoint human-verify — użytkownik testuje UI na produkcji (orderpro.projectpro.pl)
|
||||
- Ostatni feedback: nazwa usługi w kolumnie Przewoznik działa dla nowych przesyłek (stare mają puste carrier_id)
|
||||
|
||||
---
|
||||
|
||||
## What's Next
|
||||
|
||||
**Immediate:** Kontynuacja checkpoint human-verify — użytkownik potwierdza lub zgłasza kolejne uwagi
|
||||
|
||||
**After that:**
|
||||
1. Po "approved" → finalize APPLY
|
||||
2. /paul:unify dla 19-01
|
||||
3. sonar-scanner przed UNIFY (wymagane przez SPECIAL-FLOWS.md)
|
||||
|
||||
---
|
||||
|
||||
## Dodatkowe zmiany (poza planem 19-01)
|
||||
|
||||
- show.php: zapamiętywanie taba w localStorage
|
||||
- show.php: sprawdzanie file_exists dla label_path
|
||||
- show.php: przycisk "Drukuj" + JS handler
|
||||
- show.php: kolumna Przewoznik z provider → carrier_id
|
||||
- ShipmentController: redirect po sukcesie → /orders/{id} z flash
|
||||
- ApaczkaShipmentService: carrier_id = service name
|
||||
- ApaczkaApiClient: fix use statement
|
||||
- OrdersController: storagePath + printJobRepo injection
|
||||
- settings.json: wyłączony PreToolUse hook context-mode
|
||||
|
||||
---
|
||||
|
||||
## Key Files
|
||||
|
||||
| File | Purpose |
|
||||
|------|---------|
|
||||
| `.paul/STATE.md` | Live project state |
|
||||
| `.paul/ROADMAP.md` | Phase overview |
|
||||
| `.paul/phases/19-ui-integration/19-01-PLAN.md` | Current plan |
|
||||
| `src/Modules/Printing/PrintApiController.php` | REST API + ensureLabel + bulkCreateJobs |
|
||||
| `src/Modules/Printing/PrintJobRepository.php` | DB operations + pendingPackageIds |
|
||||
| `src/Modules/Settings/PrintSettingsController.php` | Kolejka wydruku w ustawieniach |
|
||||
| `resources/views/orders/show.php` | Przycisk Drukuj + tab persistence |
|
||||
| `resources/views/shipments/prepare.php` | Przycisk Drukuj + sekcja przesyłek |
|
||||
| `resources/views/orders/list.php` | Bulk print action |
|
||||
| `resources/views/settings/printing.php` | Kolejka wydruku UI |
|
||||
| `routes/web.php` | DI + nowe route'y |
|
||||
|
||||
---
|
||||
|
||||
## Resume Instructions
|
||||
|
||||
1. Read `.paul/STATE.md` for latest position
|
||||
2. Check loop position — APPLY in progress
|
||||
3. Run `/paul:resume` or `/paul:progress`
|
||||
4. Kontynuuj checkpoint human-verify z użytkownikiem
|
||||
|
||||
---
|
||||
|
||||
*Handoff created: 2026-03-22*
|
||||
173
.paul/phases/17-receipt-duplicate-guard/17-01-PLAN.md
Normal file
173
.paul/phases/17-receipt-duplicate-guard/17-01-PLAN.md
Normal file
@@ -0,0 +1,173 @@
|
||||
---
|
||||
phase: 17-receipt-duplicate-guard
|
||||
plan: 01
|
||||
type: execute
|
||||
wave: 1
|
||||
depends_on: []
|
||||
files_modified:
|
||||
- src/Modules/Accounting/ReceiptController.php
|
||||
- resources/views/orders/receipt-create.php
|
||||
autonomous: true
|
||||
---
|
||||
|
||||
<objective>
|
||||
## Goal
|
||||
Zablokować wystawienie kolejnego paragonu do zamówienia, które już ma paragon — nie całkowicie, ale z wyraźnym potwierdzeniem (alert) przed wysłaniem formularza.
|
||||
|
||||
## Purpose
|
||||
Ochrona przed przypadkowym wystawieniem duplikatu paragonu. Użytkownik widzi ostrzeżenie z listą istniejących paragonów i musi świadomie potwierdzić, że chce kontynuować.
|
||||
|
||||
## Output
|
||||
- Zmodyfikowany `ReceiptController::create()` — przekazuje istniejące paragony do widoku
|
||||
- Zmodyfikowany widok `receipt-create.php` — alert z potwierdzeniem + lista istniejących paragonów
|
||||
</objective>
|
||||
|
||||
<context>
|
||||
## Project Context
|
||||
@.paul/PROJECT.md
|
||||
@.paul/ROADMAP.md
|
||||
@.paul/STATE.md
|
||||
|
||||
## Source Files
|
||||
@src/Modules/Accounting/ReceiptController.php
|
||||
@src/Modules/Accounting/ReceiptRepository.php
|
||||
@resources/views/orders/receipt-create.php
|
||||
</context>
|
||||
|
||||
<skills>
|
||||
## Required Skills (from SPECIAL-FLOWS.md)
|
||||
|
||||
| Skill | Priority | When to Invoke | Loaded? |
|
||||
|-------|----------|----------------|---------|
|
||||
| sonar-scanner | required | Po APPLY, przed UNIFY | ○ |
|
||||
|
||||
No specialized flows configured for this plan type.
|
||||
</skills>
|
||||
|
||||
<acceptance_criteria>
|
||||
|
||||
## AC-1: Formularz pokazuje ostrzeżenie gdy zamówienie ma paragon
|
||||
```gherkin
|
||||
Given zamówienie ma już wystawiony co najmniej jeden paragon
|
||||
When użytkownik otwiera formularz wystawiania paragonu (GET /orders/{id}/receipt/create)
|
||||
Then widzi ostrzeżenie z informacją o istniejących paragonach (numer, data, kwota)
|
||||
```
|
||||
|
||||
## AC-2: Submit wymaga potwierdzenia gdy istnieją paragony
|
||||
```gherkin
|
||||
Given zamówienie ma już wystawiony paragon i użytkownik jest na formularzu
|
||||
When użytkownik kliknie przycisk "Wystaw paragon"
|
||||
Then pojawia się OrderProAlerts.confirm z pytaniem o potwierdzenie
|
||||
And formularz wysyła się dopiero po potwierdzeniu "Tak"
|
||||
And formularz NIE wysyła się po kliknięciu "Anuluj"
|
||||
```
|
||||
|
||||
## AC-3: Brak ostrzeżenia gdy zamówienie nie ma paragonów
|
||||
```gherkin
|
||||
Given zamówienie nie ma żadnych wystawionych paragonów
|
||||
When użytkownik otwiera formularz wystawiania paragonu
|
||||
Then NIE widzi żadnego ostrzeżenia
|
||||
And przycisk "Wystaw paragon" działa normalnie bez dodatkowego potwierdzenia
|
||||
```
|
||||
|
||||
</acceptance_criteria>
|
||||
|
||||
<tasks>
|
||||
|
||||
<task type="auto">
|
||||
<name>Task 1: Przekazanie istniejących paragonów z kontrolera do widoku</name>
|
||||
<files>src/Modules/Accounting/ReceiptController.php</files>
|
||||
<action>
|
||||
W metodzie `create()` (linia ~36-72):
|
||||
1. Po walidacji istnienia zamówienia (linia 36-39), dodać zapytanie o istniejące paragony:
|
||||
`$existingReceipts = $this->receipts->findByOrderId($orderId);`
|
||||
2. Przekazać `existingReceipts` do widoku w tablicy `render()` (linia 58-70):
|
||||
`'existingReceipts' => $existingReceipts,`
|
||||
|
||||
Nie zmieniać: metody `store()`, logiki walidacji, żadnych innych metod.
|
||||
</action>
|
||||
<verify>Otworzyć formularz paragonu dla zamówienia z istniejącym paragonem — zmienna $existingReceipts dostępna w widoku z poprawną liczbą wpisów.</verify>
|
||||
<done>AC-1 częściowo: dane o istniejących paragonach przekazane do widoku</done>
|
||||
</task>
|
||||
|
||||
<task type="auto">
|
||||
<name>Task 2: Ostrzeżenie i potwierdzenie w widoku formularza</name>
|
||||
<files>resources/views/orders/receipt-create.php</files>
|
||||
<action>
|
||||
1. Na początku pliku dodać zmienną:
|
||||
`$existingReceiptsList = is_array($existingReceipts ?? null) ? $existingReceipts : [];`
|
||||
|
||||
2. Po nagłówku (po linii ~19, przed `<form>`), jeśli `$existingReceiptsList !== []`, wyświetlić box ostrzeżenia:
|
||||
- Klasa CSS: `alert alert--warning mt-12` (reuse istniejącego stylu alertów)
|
||||
- Ikona + tekst: "Do tego zamówienia wystawiono już N paragon(ów):"
|
||||
- Lista paragonów: numer (`receipt_number`), data (`issue_date`), kwota (`total_gross`), config (`config_name`)
|
||||
- Tekst: "Czy na pewno chcesz wystawić kolejny?"
|
||||
|
||||
3. Na formularzu `<form>` dodać `id="receipt-create-form"`
|
||||
|
||||
4. Zmienić `<button type="submit">` na `<button type="button" id="receipt-submit-btn">` (tylko gdy są istniejące paragony — użyć warunku PHP)
|
||||
- Gdy brak istniejących paragonów: przycisk submit działa normalnie (type="submit")
|
||||
- Gdy są istniejące paragony: przycisk type="button" z JS handlerem
|
||||
|
||||
5. Na dole pliku dodać blok `<script>` (tylko gdy `$existingReceiptsList !== []`):
|
||||
```javascript
|
||||
document.getElementById('receipt-submit-btn').addEventListener('click', function() {
|
||||
window.OrderProAlerts.confirm(
|
||||
'Do tego zamówienia istnieje już paragon. Czy na pewno chcesz wystawić kolejny?',
|
||||
function() {
|
||||
document.getElementById('receipt-create-form').submit();
|
||||
}
|
||||
);
|
||||
});
|
||||
```
|
||||
|
||||
Nie zmieniać: struktury tabeli pozycji, danych sprzedawcy, logiki formularza poza submit.
|
||||
Nie dodawać nowych plików CSS — użyć istniejących klas `.alert` / `.alert--warning`.
|
||||
</action>
|
||||
<verify>
|
||||
1. Otworzyć formularz paragonu dla zamówienia BEZ paragonu → brak ostrzeżenia, submit działa normalnie
|
||||
2. Otworzyć formularz paragonu dla zamówienia Z paragonem → widoczne ostrzeżenie z danymi paragonu, kliknięcie "Wystaw" wymaga potwierdzenia w alercie
|
||||
</verify>
|
||||
<done>AC-1 satisfied: ostrzeżenie z listą paragonów widoczne. AC-2 satisfied: submit wymaga potwierdzenia. AC-3 satisfied: brak ostrzeżenia gdy brak paragonów.</done>
|
||||
</task>
|
||||
|
||||
</tasks>
|
||||
|
||||
<boundaries>
|
||||
|
||||
## DO NOT CHANGE
|
||||
- src/Modules/Accounting/ReceiptRepository.php (metoda findByOrderId() już istnieje i wystarczy)
|
||||
- src/Modules/Accounting/ReceiptController.php metoda store() (logika zapisu bez zmian)
|
||||
- database/migrations/* (brak zmian schematu)
|
||||
- resources/modules/jquery-alerts/* (reuse, nie modyfikować)
|
||||
|
||||
## SCOPE LIMITS
|
||||
- Tylko ostrzeżenie + potwierdzenie — NIE blokada całkowita
|
||||
- Brak zmian w logice zapisu (store) — to frontend guard
|
||||
- Brak nowych plików SCSS — użyć istniejących klas alertów
|
||||
- Nie dodawać walidacji server-side w store() — użytkownik świadomie potwierdził
|
||||
|
||||
</boundaries>
|
||||
|
||||
<verification>
|
||||
Before declaring plan complete:
|
||||
- [ ] Formularz dla zamówienia bez paragonu — brak ostrzeżenia, normalny submit
|
||||
- [ ] Formularz dla zamówienia z paragonem — widoczne ostrzeżenie z numerem/datą/kwotą
|
||||
- [ ] Kliknięcie "Wystaw" z ostrzeżeniem → alert confirm z OrderProAlerts
|
||||
- [ ] Potwierdzenie w alercie → formularz się wysyła → paragon wystawiony
|
||||
- [ ] Anulowanie w alercie → formularz się nie wysyła
|
||||
- [ ] Brak błędów PHP i JS w konsoli
|
||||
- [ ] All acceptance criteria met
|
||||
</verification>
|
||||
|
||||
<success_criteria>
|
||||
- All tasks completed
|
||||
- All verification checks pass
|
||||
- No errors or warnings introduced
|
||||
- Istniejące paragony widoczne w ostrzeżeniu (numer, data, kwota)
|
||||
- Potwierdzenie wymagane tylko gdy istnieją paragony
|
||||
</success_criteria>
|
||||
|
||||
<output>
|
||||
After completion, create `.paul/phases/17-receipt-duplicate-guard/17-01-SUMMARY.md`
|
||||
</output>
|
||||
99
.paul/phases/17-receipt-duplicate-guard/17-01-SUMMARY.md
Normal file
99
.paul/phases/17-receipt-duplicate-guard/17-01-SUMMARY.md
Normal file
@@ -0,0 +1,99 @@
|
||||
---
|
||||
phase: 17-receipt-duplicate-guard
|
||||
plan: 01
|
||||
subsystem: ui
|
||||
tags: [receipts, duplicate-guard, OrderProAlerts, confirmation]
|
||||
|
||||
requires:
|
||||
- phase: 10-receipt-creation
|
||||
provides: ReceiptController, ReceiptRepository, receipt-create view
|
||||
provides:
|
||||
- Duplicate receipt warning on creation form
|
||||
- OrderProAlerts.confirm guard before duplicate receipt submission
|
||||
affects: []
|
||||
|
||||
tech-stack:
|
||||
added: []
|
||||
patterns: [frontend-guard-with-backend-data]
|
||||
|
||||
key-files:
|
||||
created: []
|
||||
modified:
|
||||
- src/Modules/Accounting/ReceiptController.php
|
||||
- resources/views/orders/receipt-create.php
|
||||
|
||||
key-decisions:
|
||||
- "Frontend guard only — no server-side block in store(), user confirms consciously"
|
||||
- "Conditional button type (submit vs button) — no JS overhead when no existing receipts"
|
||||
|
||||
patterns-established:
|
||||
- "Duplicate-guard pattern: backend query → view warning → OrderProAlerts.confirm on submit"
|
||||
|
||||
duration: 5min
|
||||
started: 2026-03-22T00:00:00Z
|
||||
completed: 2026-03-22T00:05:00Z
|
||||
---
|
||||
|
||||
# Phase 17 Plan 01: Receipt Duplicate Guard Summary
|
||||
|
||||
**Frontend guard preventing accidental duplicate receipt creation — warning box + OrderProAlerts.confirm before submit when order already has receipts.**
|
||||
|
||||
## Performance
|
||||
|
||||
| Metric | Value |
|
||||
|--------|-------|
|
||||
| Duration | ~5min |
|
||||
| Tasks | 2 completed |
|
||||
| Files modified | 2 |
|
||||
|
||||
## Acceptance Criteria Results
|
||||
|
||||
| Criterion | Status | Notes |
|
||||
|-----------|--------|-------|
|
||||
| AC-1: Formularz pokazuje ostrzeżenie gdy zamówienie ma paragon | Pass | Warning box z numerem, datą, kwotą, configiem |
|
||||
| AC-2: Submit wymaga potwierdzenia gdy istnieją paragony | Pass | button type="button" + OrderProAlerts.confirm |
|
||||
| AC-3: Brak ostrzeżenia gdy zamówienie nie ma paragonów | Pass | Warunek PHP $hasExistingReceipts, normalny submit |
|
||||
|
||||
## Accomplishments
|
||||
|
||||
- Warning box `.alert--warning` z listą istniejących paragonów (numer, data, kwota, config)
|
||||
- Conditional submit: `type="button"` + `OrderProAlerts.confirm()` gdy istnieją paragony, normalny `type="submit"` gdy brak
|
||||
- Reuse istniejącego `ReceiptRepository::findByOrderId()` — zero nowego kodu backend poza 2 liniami
|
||||
|
||||
## Files Created/Modified
|
||||
|
||||
| File | Change | Purpose |
|
||||
|------|--------|---------|
|
||||
| `src/Modules/Accounting/ReceiptController.php` | Modified | Dodanie `findByOrderId()` query + przekazanie `existingReceipts` do widoku |
|
||||
| `resources/views/orders/receipt-create.php` | Modified | Warning box, conditional button type, inline JS z OrderProAlerts.confirm |
|
||||
|
||||
## Decisions Made
|
||||
|
||||
| Decision | Rationale | Impact |
|
||||
|----------|-----------|--------|
|
||||
| Frontend guard only (brak blokady w store()) | Użytkownik świadomie potwierdza — to wystarczający level ochrony | Prostota, brak zmian w logice zapisu |
|
||||
| Conditional button type zamiast always-JS | Brak narzutu JS gdy nie ma istniejących paragonów | Czystsza implementacja |
|
||||
|
||||
## Deviations from Plan
|
||||
|
||||
None — plan executed exactly as written.
|
||||
|
||||
## Issues Encountered
|
||||
|
||||
None.
|
||||
|
||||
## Next Phase Readiness
|
||||
|
||||
**Ready:**
|
||||
- Punkt 31 z todo.md zrealizowany
|
||||
- Pattern duplicate-guard gotowy do reuse w innych formularzach
|
||||
|
||||
**Concerns:**
|
||||
- Brak
|
||||
|
||||
**Blockers:**
|
||||
- None
|
||||
|
||||
---
|
||||
*Phase: 17-receipt-duplicate-guard, Plan: 01*
|
||||
*Completed: 2026-03-22*
|
||||
288
.paul/phases/18-print-queue-backend/18-01-PLAN.md
Normal file
288
.paul/phases/18-print-queue-backend/18-01-PLAN.md
Normal file
@@ -0,0 +1,288 @@
|
||||
---
|
||||
phase: 18-print-queue-backend
|
||||
plan: 01
|
||||
type: execute
|
||||
wave: 1
|
||||
depends_on: []
|
||||
files_modified:
|
||||
- database/migrations/20260322_000058_create_print_tables.sql
|
||||
- src/Modules/Printing/PrintJobRepository.php
|
||||
- src/Modules/Printing/PrintApiKeyRepository.php
|
||||
- src/Modules/Printing/PrintApiController.php
|
||||
- src/Modules/Printing/ApiKeyMiddleware.php
|
||||
- src/Modules/Settings/PrintSettingsController.php
|
||||
- routes/web.php
|
||||
autonomous: false
|
||||
---
|
||||
|
||||
<objective>
|
||||
## Goal
|
||||
Stworzyć backend do zdalnego drukowania etykiet: tabele DB (print_jobs, print_api_keys), REST API z uwierzytelnianiem kluczem API, oraz CRUD kluczy API w ustawieniach.
|
||||
|
||||
## Purpose
|
||||
Fundament pod system zdalnego drukowania — aplikacja Windows (faza 20) będzie odpytywać te endpointy aby pobierać zlecenia wydruku i drukować etykiety na drukarce termicznej.
|
||||
|
||||
## Output
|
||||
- Migracja SQL: tabele print_jobs i print_api_keys
|
||||
- Moduł Printing: PrintJobRepository, PrintApiKeyRepository, PrintApiController
|
||||
- ApiKeyMiddleware do uwierzytelniania requestów z aplikacji Windows
|
||||
- PrintSettingsController: CRUD kluczy API w ustawieniach
|
||||
- REST API: 4 endpointy (create job, list pending, download label, mark complete)
|
||||
</objective>
|
||||
|
||||
<context>
|
||||
## Project Context
|
||||
@.paul/PROJECT.md
|
||||
@.paul/ROADMAP.md
|
||||
@.paul/STATE.md
|
||||
|
||||
## Source Files
|
||||
@src/Core/Routing/Router.php
|
||||
@src/Modules/Auth/AuthMiddleware.php
|
||||
@src/Modules/Shipments/ShipmentController.php
|
||||
@routes/web.php
|
||||
</context>
|
||||
|
||||
<skills>
|
||||
## Required Skills (from SPECIAL-FLOWS.md)
|
||||
|
||||
| Skill | Priority | When to Invoke | Loaded? |
|
||||
|-------|----------|----------------|---------|
|
||||
| sonar-scanner | required | Po APPLY, przed UNIFY | ○ |
|
||||
|
||||
</skills>
|
||||
|
||||
<acceptance_criteria>
|
||||
|
||||
## AC-1: Tabele DB utworzone poprawnie
|
||||
```gherkin
|
||||
Given czysta baza danych
|
||||
When uruchomię migrację 20260322_000058_create_print_tables.sql
|
||||
Then tabele print_jobs i print_api_keys istnieją z poprawnymi kolumnami i indeksami
|
||||
```
|
||||
|
||||
## AC-2: CRUD kluczy API w ustawieniach
|
||||
```gherkin
|
||||
Given użytkownik jest zalogowany i otwiera Ustawienia > Drukowanie
|
||||
When tworzy nowy klucz API (podaje nazwę)
|
||||
Then klucz jest generowany, wyświetlany jednorazowo, zapisany w DB (hash)
|
||||
And klucz można dezaktywować i usunąć
|
||||
```
|
||||
|
||||
## AC-3: API — tworzenie zlecenia wydruku
|
||||
```gherkin
|
||||
Given istnieje etykieta przesyłki (label_path w shipment_packages)
|
||||
When użytkownik orderPRO wyśle POST /api/print/jobs z session auth
|
||||
Then zlecenie wydruku zostaje utworzone ze statusem 'pending'
|
||||
And odpowiedź zawiera ID zlecenia
|
||||
```
|
||||
|
||||
## AC-4: API — pobieranie zleceń i etykiet przez klienta
|
||||
```gherkin
|
||||
Given klient Windows uwierzytelnia się kluczem API (header X-Api-Key)
|
||||
When wyśle GET /api/print/jobs/pending
|
||||
Then otrzyma listę zleceń ze statusem 'pending'
|
||||
And gdy wyśle GET /api/print/jobs/{id}/download
|
||||
Then otrzyma plik PDF etykiety
|
||||
```
|
||||
|
||||
## AC-5: API — oznaczanie zlecenia jako wydrukowane
|
||||
```gherkin
|
||||
Given klient Windows pobrał i wydrukował etykietę
|
||||
When wyśle PATCH /api/print/jobs/{id}/complete z kluczem API
|
||||
Then zlecenie zmieni status na 'completed' z timestampem completed_at
|
||||
```
|
||||
|
||||
## AC-6: Nieprawidłowy klucz API odrzucony
|
||||
```gherkin
|
||||
Given request z nieprawidłowym lub brakującym kluczem API
|
||||
When klient wyśle request do /api/print/*
|
||||
Then odpowiedź to 401 Unauthorized (JSON)
|
||||
```
|
||||
|
||||
</acceptance_criteria>
|
||||
|
||||
<tasks>
|
||||
|
||||
<task type="auto">
|
||||
<name>Task 1: Migracja DB — tabele print_jobs i print_api_keys</name>
|
||||
<files>database/migrations/20260322_000058_create_print_tables.sql</files>
|
||||
<action>
|
||||
Utworzyć migrację SQL z dwiema tabelami:
|
||||
|
||||
**print_api_keys:**
|
||||
- `id` INT AUTO_INCREMENT PRIMARY KEY
|
||||
- `name` VARCHAR(128) NOT NULL — nazwa przyjazna ("Komputer biurowy")
|
||||
- `key_hash` VARCHAR(128) NOT NULL — hash klucza (SHA-256)
|
||||
- `key_prefix` VARCHAR(8) NOT NULL — pierwsze 8 znaków klucza (do identyfikacji)
|
||||
- `is_active` TINYINT(1) NOT NULL DEFAULT 1
|
||||
- `last_used_at` DATETIME NULL
|
||||
- `created_at` DATETIME NOT NULL DEFAULT CURRENT_TIMESTAMP
|
||||
|
||||
Indeksy: UNIQUE na key_hash, INDEX na is_active.
|
||||
|
||||
**print_jobs:**
|
||||
- `id` INT AUTO_INCREMENT PRIMARY KEY
|
||||
- `order_id` BIGINT NOT NULL — FK → orders.id ON DELETE CASCADE
|
||||
- `package_id` INT NOT NULL — FK → shipment_packages.id ON DELETE CASCADE
|
||||
- `label_path` VARCHAR(255) NOT NULL — ścieżka do pliku PDF
|
||||
- `status` ENUM('pending', 'printing', 'completed', 'failed') NOT NULL DEFAULT 'pending'
|
||||
- `created_by` INT NOT NULL — kto zlecił wydruk
|
||||
- `created_at` DATETIME NOT NULL DEFAULT CURRENT_TIMESTAMP
|
||||
- `completed_at` DATETIME NULL
|
||||
|
||||
Indeksy: INDEX na status, INDEX na order_id, INDEX na package_id.
|
||||
|
||||
Konwencja: zgodna z istniejącymi migracjami (plik .sql, plain DDL).
|
||||
</action>
|
||||
<verify>Uruchomić migrację i sprawdzić SHOW CREATE TABLE print_jobs; SHOW CREATE TABLE print_api_keys;</verify>
|
||||
<done>AC-1 satisfied: tabele istnieją z poprawnymi kolumnami i indeksami</done>
|
||||
</task>
|
||||
|
||||
<task type="auto">
|
||||
<name>Task 2: Repozytoria i kontrolery — PrintJobRepository, PrintApiKeyRepository, ApiKeyMiddleware, PrintApiController, PrintSettingsController</name>
|
||||
<files>
|
||||
src/Modules/Printing/PrintJobRepository.php,
|
||||
src/Modules/Printing/PrintApiKeyRepository.php,
|
||||
src/Modules/Printing/PrintApiController.php,
|
||||
src/Modules/Printing/ApiKeyMiddleware.php,
|
||||
src/Modules/Settings/PrintSettingsController.php,
|
||||
routes/web.php
|
||||
</files>
|
||||
<action>
|
||||
**PrintApiKeyRepository** (src/Modules/Printing/):
|
||||
- `create(string $name, string $keyHash, string $keyPrefix): int` — INSERT, zwraca ID
|
||||
- `findByKeyHash(string $keyHash): ?array` — szukanie po hashu (do auth)
|
||||
- `listAll(): array` — lista kluczy (bez hashy, z prefixem)
|
||||
- `deactivate(int $id): void` — SET is_active = 0
|
||||
- `delete(int $id): void` — DELETE
|
||||
- `updateLastUsed(int $id): void` — SET last_used_at = NOW()
|
||||
- Używać PDO + prepared statements (wzorzec jak ReceiptRepository)
|
||||
|
||||
**PrintJobRepository** (src/Modules/Printing/):
|
||||
- `create(array $data): int` — INSERT, zwraca ID
|
||||
- `findPending(): array` — SELECT WHERE status = 'pending' ORDER BY created_at ASC
|
||||
- `findById(int $id): ?array` — z JOIN na orders (internal_order_number) i shipment_packages (tracking_number)
|
||||
- `markCompleted(int $id): void` — SET status = 'completed', completed_at = NOW()
|
||||
- `markFailed(int $id, string $reason): void` — SET status = 'failed'
|
||||
- Używać PDO + prepared statements
|
||||
|
||||
**ApiKeyMiddleware** (src/Modules/Printing/):
|
||||
- Callable `__invoke(Request $request, callable $next): Response`
|
||||
- Odczytuje header `X-Api-Key` z requestu
|
||||
- Hashuje klucz (SHA-256), szuka w print_api_keys przez PrintApiKeyRepository
|
||||
- Jeśli znaleziony i is_active = 1: updateLastUsed(), przekazuje do $next
|
||||
- Jeśli brak/nieaktywny: zwraca Response::json(['error' => 'Unauthorized'], 401)
|
||||
- Wzorzec: analogiczny do AuthMiddleware ale dla API
|
||||
|
||||
**PrintApiController** (src/Modules/Printing/):
|
||||
- Constructor DI: PrintJobRepository, Request, AuthService
|
||||
- `createJob(Request $request): Response` — POST /api/print/jobs
|
||||
- Wymaga session auth (AuthMiddleware) — wywoływane z UI orderPRO
|
||||
- Przyjmuje: package_id (required)
|
||||
- Waliduje: shipment_packages.label_path istnieje i plik fizycznie istnieje
|
||||
- Tworzy print_job ze statusem 'pending'
|
||||
- Zwraca JSON: { id, status: 'pending' }
|
||||
- `listPending(Request $request): Response` — GET /api/print/jobs/pending
|
||||
- Wymaga API key (ApiKeyMiddleware)
|
||||
- Zwraca JSON: lista pending jobs z: id, order_number, tracking_number, created_at
|
||||
- `downloadLabel(Request $request): Response` — GET /api/print/jobs/{id}/download
|
||||
- Wymaga API key
|
||||
- Waliduje: job istnieje, plik istnieje
|
||||
- Zwraca plik PDF (Content-Type: application/pdf)
|
||||
- `markComplete(Request $request): Response` — PATCH /api/print/jobs/{id}/complete
|
||||
- Wymaga API key
|
||||
- Oznacza job jako completed
|
||||
- Zwraca JSON: { id, status: 'completed' }
|
||||
|
||||
**PrintSettingsController** (src/Modules/Settings/):
|
||||
- `index(Request $request): Response` — lista kluczy API (widok settings/printing)
|
||||
- `createKey(Request $request): Response` — POST: generuje klucz (bin2hex(random_bytes(32))), hashuje SHA-256, zapisuje hash+prefix, zwraca klucz jednorazowo we flash message
|
||||
- `deleteKey(Request $request): Response` — POST: usuwa klucz
|
||||
|
||||
**Routes** (routes/web.php):
|
||||
Dodać na końcu pliku:
|
||||
```php
|
||||
// Print API — session auth (from orderPRO UI)
|
||||
$router->post('/api/print/jobs', [$printApiController, 'createJob'], [$authMiddleware]);
|
||||
|
||||
// Print API — API key auth (from Windows client)
|
||||
$router->get('/api/print/jobs/pending', [$printApiController, 'listPending'], [$apiKeyMiddleware]);
|
||||
$router->get('/api/print/jobs/{id}/download', [$printApiController, 'downloadLabel'], [$apiKeyMiddleware]);
|
||||
$router->post('/api/print/jobs/{id}/complete', [$printApiController, 'markComplete'], [$apiKeyMiddleware]);
|
||||
|
||||
// Print settings
|
||||
$router->get('/settings/printing', [$printSettingsController, 'index'], [$authMiddleware]);
|
||||
$router->post('/settings/printing/keys/create', [$printSettingsController, 'createKey'], [$authMiddleware]);
|
||||
$router->post('/settings/printing/keys/{id}/delete', [$printSettingsController, 'deleteKey'], [$authMiddleware]);
|
||||
```
|
||||
|
||||
DI: Zarejestrować nowe klasy w Application.php (lub tam gdzie inne moduły są rejestrowane).
|
||||
|
||||
Avoid:
|
||||
- Nie używać PATCH (router może nie obsługiwać) — użyć POST dla markComplete
|
||||
- Nie sklejać SQL — tylko prepared statements
|
||||
- Nie przechowywać klucza API w plaintext — tylko hash SHA-256
|
||||
</action>
|
||||
<verify>
|
||||
1. Sprawdzić czy pliki PHP nie mają błędów składni: php -l na każdym pliku
|
||||
2. Sprawdzić czy routes/web.php parsuje się poprawnie
|
||||
</verify>
|
||||
<done>AC-2 partially (backend), AC-3, AC-4, AC-5, AC-6 satisfied: API endpoints działają z prawidłowym auth</done>
|
||||
</task>
|
||||
|
||||
<task type="checkpoint:human-verify" gate="blocking">
|
||||
<what-built>
|
||||
Backend do zdalnego drukowania:
|
||||
- Tabele DB: print_jobs, print_api_keys
|
||||
- REST API: 4 endpointy (create job, list pending, download, complete)
|
||||
- API key middleware (X-Api-Key header)
|
||||
- CRUD kluczy API w ustawieniach (backend only — widok w fazie 19)
|
||||
</what-built>
|
||||
<how-to-verify>
|
||||
1. Uruchom aplikację — sprawdź czy migracja przeszła (brak błędów)
|
||||
2. Otwórz /settings/printing — powinien załadować się widok (nawet pusty)
|
||||
3. Opcjonalnie: testuj API curlem
|
||||
</how-to-verify>
|
||||
<resume-signal>Type "approved" to continue, or describe issues to fix</resume-signal>
|
||||
</task>
|
||||
|
||||
</tasks>
|
||||
|
||||
<boundaries>
|
||||
|
||||
## DO NOT CHANGE
|
||||
- src/Modules/Shipments/* (nie modyfikować istniejącego flow etykiet)
|
||||
- src/Modules/Auth/AuthMiddleware.php (nie zmieniać session auth)
|
||||
- src/Core/Routing/Router.php (nie modyfikować routera)
|
||||
- database/migrations/ istniejące pliki (nie zmieniać)
|
||||
|
||||
## SCOPE LIMITS
|
||||
- Tylko backend — brak widoków HTML (oprócz minimalnego dla settings/printing)
|
||||
- Brak integracji z przyciskiem "Drukuj" w widoku przesyłek (faza 19)
|
||||
- Brak aplikacji Windows (faza 20)
|
||||
- Widok settings/printing: minimalna lista kluczy + formularz tworzenia (pełny design w fazie 19)
|
||||
|
||||
</boundaries>
|
||||
|
||||
<verification>
|
||||
Before declaring plan complete:
|
||||
- [ ] Migracja SQL wykonuje się bez błędów
|
||||
- [ ] php -l bez błędów na wszystkich nowych plikach
|
||||
- [ ] GET /api/print/jobs/pending bez klucza → 401
|
||||
- [ ] POST /settings/printing/keys/create tworzy klucz
|
||||
- [ ] Routing nie psuje istniejących endpointów
|
||||
- [ ] All acceptance criteria met
|
||||
</verification>
|
||||
|
||||
<success_criteria>
|
||||
- All tasks completed
|
||||
- All verification checks pass
|
||||
- No errors or warnings introduced
|
||||
- API key auth działa (valid key → 200, invalid → 401)
|
||||
- Print jobs CRUD przez API
|
||||
</success_criteria>
|
||||
|
||||
<output>
|
||||
After completion, create `.paul/phases/18-print-queue-backend/18-01-SUMMARY.md`
|
||||
</output>
|
||||
138
.paul/phases/18-print-queue-backend/18-01-SUMMARY.md
Normal file
138
.paul/phases/18-print-queue-backend/18-01-SUMMARY.md
Normal file
@@ -0,0 +1,138 @@
|
||||
---
|
||||
phase: 18-print-queue-backend
|
||||
plan: 01
|
||||
subsystem: api
|
||||
tags: [printing, api-key, rest-api, middleware, remote-printing]
|
||||
|
||||
requires: []
|
||||
provides:
|
||||
- Print jobs REST API (create, list pending, download, complete)
|
||||
- API key authentication middleware (X-Api-Key header)
|
||||
- Print settings UI (CRUD kluczy API)
|
||||
- Request::header() method
|
||||
affects: [19-ui-integration, 20-windows-client]
|
||||
|
||||
tech-stack:
|
||||
added: []
|
||||
patterns: [api-key-auth, sha256-hashing, rest-json-api]
|
||||
|
||||
key-files:
|
||||
created:
|
||||
- database/migrations/20260322_000058_create_print_tables.sql
|
||||
- src/Modules/Printing/PrintApiKeyRepository.php
|
||||
- src/Modules/Printing/PrintJobRepository.php
|
||||
- src/Modules/Printing/PrintApiController.php
|
||||
- src/Modules/Printing/ApiKeyMiddleware.php
|
||||
- src/Modules/Settings/PrintSettingsController.php
|
||||
- resources/views/settings/printing.php
|
||||
modified:
|
||||
- src/Core/Http/Request.php
|
||||
- resources/views/layouts/app.php
|
||||
- routes/web.php
|
||||
|
||||
key-decisions:
|
||||
- "FK constraints removed from print_jobs — type mismatch between orders (INT UNSIGNED) and shipment_packages (BIGINT UNSIGNED)"
|
||||
- "API key stored as SHA-256 hash, prefix saved for identification"
|
||||
- "Request::header() method added to core Request class for API key extraction"
|
||||
|
||||
patterns-established:
|
||||
- "API key auth: ApiKeyMiddleware reads X-Api-Key, hashes SHA-256, validates against print_api_keys"
|
||||
- "Flash key convention: settings_error, settings_success, settings_new_api_key (underscore, not dot)"
|
||||
|
||||
duration: 25min
|
||||
started: 2026-03-22T00:00:00Z
|
||||
completed: 2026-03-22T00:25:00Z
|
||||
---
|
||||
|
||||
# Phase 18 Plan 01: Print Queue Backend Summary
|
||||
|
||||
**REST API do zdalnego drukowania etykiet — tabele DB, API key auth middleware, 4 endpointy JSON, CRUD kluczy API w ustawieniach.**
|
||||
|
||||
## Performance
|
||||
|
||||
| Metric | Value |
|
||||
|--------|-------|
|
||||
| Duration | ~25min |
|
||||
| Tasks | 3 completed (2 auto + 1 checkpoint) |
|
||||
| Files created | 7 |
|
||||
| Files modified | 3 |
|
||||
|
||||
## Acceptance Criteria Results
|
||||
|
||||
| Criterion | Status | Notes |
|
||||
|-----------|--------|-------|
|
||||
| AC-1: Tabele DB utworzone poprawnie | Pass | print_api_keys + print_jobs, bez FK (type mismatch) |
|
||||
| AC-2: CRUD kluczy API w ustawieniach | Pass | Tworzenie, wyświetlanie, usuwanie kluczy |
|
||||
| AC-3: API — tworzenie zlecenia wydruku | Pass | POST /api/print/jobs z session auth |
|
||||
| AC-4: API — pobieranie zleceń i etykiet | Pass | GET /api/print/jobs/pending + download |
|
||||
| AC-5: API — oznaczanie jako wydrukowane | Pass | POST /api/print/jobs/{id}/complete |
|
||||
| AC-6: Nieprawidłowy klucz API odrzucony | Pass | 401 Unauthorized z JSON |
|
||||
|
||||
## Accomplishments
|
||||
|
||||
- Moduł Printing: PrintApiKeyRepository, PrintJobRepository, PrintApiController, ApiKeyMiddleware
|
||||
- REST API: 4 endpointy (create job, list pending, download label, mark complete)
|
||||
- API key auth: SHA-256 hash, prefix do identyfikacji, last_used_at tracking
|
||||
- Settings UI: lista kluczy + jednorazowe wyświetlenie nowego klucza
|
||||
- Request::header() method w core — reusable dla przyszłych API
|
||||
|
||||
## Files Created/Modified
|
||||
|
||||
| File | Change | Purpose |
|
||||
|------|--------|---------|
|
||||
| `database/migrations/20260322_000058_create_print_tables.sql` | Created | Tabele print_api_keys + print_jobs |
|
||||
| `src/Modules/Printing/PrintApiKeyRepository.php` | Created | CRUD kluczy API |
|
||||
| `src/Modules/Printing/PrintJobRepository.php` | Created | CRUD zleceń wydruku |
|
||||
| `src/Modules/Printing/PrintApiController.php` | Created | REST API endpointy |
|
||||
| `src/Modules/Printing/ApiKeyMiddleware.php` | Created | Auth middleware X-Api-Key |
|
||||
| `src/Modules/Settings/PrintSettingsController.php` | Created | UI zarządzania kluczami |
|
||||
| `resources/views/settings/printing.php` | Created | Widok ustawień drukowania |
|
||||
| `src/Core/Http/Request.php` | Modified | Dodano header() method |
|
||||
| `resources/views/layouts/app.php` | Modified | Link "Drukowanie" w sidebar |
|
||||
| `routes/web.php` | Modified | 7 nowych route'ów + DI |
|
||||
|
||||
## Decisions Made
|
||||
|
||||
| Decision | Rationale | Impact |
|
||||
|----------|-----------|--------|
|
||||
| Usunięto FK constraints z print_jobs | orders.id (INT UNSIGNED) vs shipment_packages.order_id (BIGINT UNSIGNED) — type mismatch w produkcji | Integralność referencyjna tylko na poziomie aplikacji |
|
||||
| SHA-256 hash klucza API | Bezpieczeństwo — raw key nigdy nie przechowywany w DB | Klucz wyświetlany jednorazowo po utworzeniu |
|
||||
| Request::header() w core | Potrzebne do X-Api-Key, reusable | Minimalna zmiana core — 1 metoda |
|
||||
|
||||
## Deviations from Plan
|
||||
|
||||
### Summary
|
||||
|
||||
| Type | Count | Impact |
|
||||
|------|-------|--------|
|
||||
| Auto-fixed | 1 | FK constraints usunięte — brak wpływu na funkcjonalność |
|
||||
|
||||
**1. FK constraints usunięte z migracji**
|
||||
- **Found during:** Task 1 (migracja)
|
||||
- **Issue:** Type mismatch: orders.id = INT UNSIGNED, shipment_packages.order_id = BIGINT UNSIGNED
|
||||
- **Fix:** Usunięto CONSTRAINT, zostawiono INDEX
|
||||
- **Verification:** Migracja przeszła poprawnie
|
||||
|
||||
## Issues Encountered
|
||||
|
||||
| Issue | Resolution |
|
||||
|-------|------------|
|
||||
| FK errno 150 (1. próba) | Zmieniono BIGINT→INT UNSIGNED + INT→BIGINT UNSIGNED |
|
||||
| FK errno 150 (2. próba) | Usunięto FK constraints całkowicie — type mismatch w produkcji |
|
||||
|
||||
## Next Phase Readiness
|
||||
|
||||
**Ready:**
|
||||
- API endpoints gotowe do konsumpcji przez Windows Client (faza 20)
|
||||
- API key auth działa — klient może się uwierzytelnić
|
||||
- Przycisk "Drukuj" w UI (faza 19) może wywoływać POST /api/print/jobs
|
||||
|
||||
**Concerns:**
|
||||
- Brak FK constraints — integralność tylko na poziomie aplikacji
|
||||
|
||||
**Blockers:**
|
||||
- None
|
||||
|
||||
---
|
||||
*Phase: 18-print-queue-backend, Plan: 01*
|
||||
*Completed: 2026-03-22*
|
||||
261
.paul/phases/19-ui-integration/19-01-PLAN.md
Normal file
261
.paul/phases/19-ui-integration/19-01-PLAN.md
Normal file
@@ -0,0 +1,261 @@
|
||||
---
|
||||
phase: 19-ui-integration
|
||||
plan: 01
|
||||
type: execute
|
||||
wave: 1
|
||||
depends_on: ["18-01"]
|
||||
files_modified:
|
||||
- resources/views/shipments/prepare.php
|
||||
- resources/views/orders/index.php
|
||||
- src/Modules/Printing/PrintApiController.php
|
||||
- src/Modules/Printing/PrintJobRepository.php
|
||||
- resources/views/settings/printing.php
|
||||
- routes/web.php
|
||||
- resources/scss/_printing.scss
|
||||
autonomous: false
|
||||
---
|
||||
|
||||
<objective>
|
||||
## Goal
|
||||
Dodać przycisk "Drukuj" w widoku przesyłki (prepare.php) oraz zbiorczą akcję "Drukuj etykiety" z listy zamówień — obie wywołujące POST /api/print/jobs. Dodać podgląd kolejki wydruku w ustawieniach drukowania ze statusami zleceń.
|
||||
|
||||
## Purpose
|
||||
Użytkownik może jednym kliknięciem wysłać etykietę do zdalnej drukarki (Windows Client z fazy 20) bez ręcznego pobierania i drukowania. Zbiorcze drukowanie oszczędza czas przy wielu zamówieniach.
|
||||
|
||||
## Output
|
||||
- Przycisk "Drukuj" obok "Pobierz" w prepare.php z AJAX feedback
|
||||
- Zbiorcza akcja "Drukuj etykiety" na liście zamówień (zaznaczone checkboxy)
|
||||
- Sekcja "Kolejka wydruku" w ustawieniach drukowania z historią zleceń
|
||||
</objective>
|
||||
|
||||
<context>
|
||||
## Project Context
|
||||
@.paul/PROJECT.md
|
||||
@.paul/ROADMAP.md
|
||||
@.paul/STATE.md
|
||||
|
||||
## Prior Work
|
||||
@.paul/phases/18-print-queue-backend/18-01-SUMMARY.md
|
||||
- Phase 18 created: PrintApiController (createJob, listPending, downloadLabel, markComplete)
|
||||
- API key auth middleware for Windows client
|
||||
- print_jobs table (order_id, package_id, label_path, status, created_by)
|
||||
- createJob() expects package_id, validates label exists
|
||||
|
||||
## Source Files
|
||||
@resources/views/shipments/prepare.php — "Pobierz" button at line ~109
|
||||
@resources/views/orders/index.php — lista zamówień z checkboxami
|
||||
@src/Modules/Printing/PrintApiController.php — existing REST endpoints
|
||||
@src/Modules/Printing/PrintJobRepository.php — DB operations
|
||||
@resources/views/settings/printing.php — API key management
|
||||
@routes/web.php — route registration
|
||||
</context>
|
||||
|
||||
<skills>
|
||||
## Required Skills (from SPECIAL-FLOWS.md)
|
||||
|
||||
| Skill | Priority | When to Invoke | Loaded? |
|
||||
|-------|----------|----------------|---------|
|
||||
| sonar-scanner | required | Po APPLY, przed UNIFY | ○ |
|
||||
|
||||
## Skill Invocation Checklist
|
||||
- [ ] sonar-scanner loaded (run before UNIFY)
|
||||
</skills>
|
||||
|
||||
<acceptance_criteria>
|
||||
|
||||
## AC-1: Przycisk "Drukuj" w widoku przesyłki
|
||||
```gherkin
|
||||
Given przesyłka ma wygenerowaną etykietę (label_path istnieje)
|
||||
When użytkownik klika "Drukuj" obok przycisku "Pobierz"
|
||||
Then AJAX POST /api/print/jobs z package_id
|
||||
And przycisk zmienia się na "Wysłano ✓" z krótkim feedbackiem
|
||||
And w razie błędu wyświetla OrderProAlerts.alert z komunikatem
|
||||
```
|
||||
|
||||
## AC-2: Zbiorcze drukowanie z listy zamówień
|
||||
```gherkin
|
||||
Given użytkownik zaznaczył ≥1 zamówienie z etykietą na liście zamówień
|
||||
When klika akcję "Drukuj etykiety" (bulk action)
|
||||
Then dla każdego zaznaczonego zamówienia tworzony jest print job
|
||||
And wyświetla podsumowanie: "Wysłano N zleceń do drukarki"
|
||||
And zamówienia bez etykiety są pomijane z informacją
|
||||
```
|
||||
|
||||
## AC-3: Kolejka wydruku w ustawieniach
|
||||
```gherkin
|
||||
Given użytkownik otwiera Ustawienia > Drukowanie
|
||||
When strona się ładuje
|
||||
Then widzi sekcję "Kolejka wydruku" pod sekcją "Klucze API"
|
||||
And lista pokazuje ostatnie zlecenia: data, zamówienie, status (pending/completed/failed)
|
||||
And może filtrować po statusie
|
||||
```
|
||||
|
||||
## AC-4: Endpoint bulk create
|
||||
```gherkin
|
||||
Given request POST /api/print/jobs/bulk z tablicą package_ids
|
||||
When co najmniej 1 package_id ma etykietę
|
||||
Then tworzy print jobs dla wszystkich valid packages
|
||||
And zwraca JSON z listą created + skipped
|
||||
```
|
||||
|
||||
</acceptance_criteria>
|
||||
|
||||
<tasks>
|
||||
|
||||
<task type="auto">
|
||||
<name>Task 1: Przycisk "Drukuj" w prepare.php + bulk endpoint</name>
|
||||
<files>
|
||||
resources/views/shipments/prepare.php,
|
||||
src/Modules/Printing/PrintApiController.php,
|
||||
src/Modules/Printing/PrintJobRepository.php,
|
||||
routes/web.php
|
||||
</files>
|
||||
<action>
|
||||
1. W prepare.php obok przycisku "Pobierz" (form POST label) dodaj przycisk "Drukuj":
|
||||
- Button z klasą `btn btn-sm btn-outline-primary btn-print-label`
|
||||
- data-package-id="{$pkg['id']}" data-order-id="{$orderId}"
|
||||
- Ikona drukarki (fa-print lub SVG)
|
||||
- Widoczny tylko gdy $pkgLabelPath istnieje (ten sam warunek co "Pobierz")
|
||||
|
||||
2. JavaScript AJAX handler na dole prepare.php (lub w osobnym module):
|
||||
- Click `.btn-print-label` → POST /api/print/jobs z {package_id, _token}
|
||||
- Success → zmień tekst na "Wysłano ✓", disable button na 3s
|
||||
- Error → OrderProAlerts.alert('Błąd: ' + response.error)
|
||||
- Duplikat (job already pending) → OrderProAlerts.alert('Zlecenie już w kolejce')
|
||||
|
||||
3. W PrintApiController dodaj metodę bulkCreateJobs():
|
||||
- Przyjmuje JSON {package_ids: [1,2,3], _token}
|
||||
- Waliduje CSRF
|
||||
- Dla każdego package_id: sprawdź label_path, utwórz job lub skip
|
||||
- Zwraca {created: [{id, package_id}], skipped: [{package_id, reason}]}
|
||||
|
||||
4. W PrintJobRepository dodaj findPendingByPackageId(int $packageId):
|
||||
- Sprawdza czy istnieje pending job dla danego package
|
||||
- Używane do ochrony przed duplikatami
|
||||
|
||||
5. W routes/web.php dodaj:
|
||||
- POST /api/print/jobs/bulk → PrintApiController::bulkCreateJobs (session auth)
|
||||
|
||||
Avoid: Nie zmieniaj istniejących endpointów API key auth (GET /api/print/jobs/pending itd.)
|
||||
Avoid: Nie dodawaj natywnych alert() — używaj OrderProAlerts
|
||||
</action>
|
||||
<verify>
|
||||
- Przycisk "Drukuj" widoczny obok "Pobierz" w prepare.php
|
||||
- AJAX POST tworzy rekord w print_jobs
|
||||
- Bulk endpoint zwraca poprawny JSON
|
||||
- Duplikat nie tworzy drugiego pending job
|
||||
</verify>
|
||||
<done>AC-1 satisfied (przycisk + AJAX), AC-4 satisfied (bulk endpoint)</done>
|
||||
</task>
|
||||
|
||||
<task type="auto">
|
||||
<name>Task 2: Zbiorcze drukowanie z listy zamówień + kolejka w ustawieniach</name>
|
||||
<files>
|
||||
resources/views/orders/index.php,
|
||||
resources/views/settings/printing.php,
|
||||
src/Modules/Settings/PrintSettingsController.php,
|
||||
src/Modules/Printing/PrintJobRepository.php,
|
||||
routes/web.php,
|
||||
resources/scss/_printing.scss
|
||||
</files>
|
||||
<action>
|
||||
1. W orders/index.php dodaj bulk action "Drukuj etykiety":
|
||||
- Dodaj opcję w dropdown/select akcji zbiorczych (wzoruj się na istniejących bulk actions)
|
||||
- JavaScript: zbierz package_ids z zaznaczonych zamówień
|
||||
- POST /api/print/jobs/bulk z zebraną tablicą
|
||||
- Pokaż wynik: "Wysłano {N} zleceń. Pominięto {M} (brak etykiety)."
|
||||
- Użyj OrderProAlerts.alert() dla wyniku
|
||||
|
||||
2. W PrintJobRepository dodaj getRecentJobs(int $limit = 50, ?string $statusFilter = null):
|
||||
- SELECT z JOIN na orders (numer zamówienia) i shipment_packages (tracking)
|
||||
- Sortowanie: created_at DESC
|
||||
- Opcjonalny filtr statusu
|
||||
|
||||
3. W PrintSettingsController dodaj metodę index() rozszerzoną o dane kolejki:
|
||||
- Pobierz listę ostatnich print jobs (getRecentJobs)
|
||||
- Przekaż do widoku
|
||||
|
||||
4. W settings/printing.php pod sekcją "Klucze API" dodaj sekcję "Kolejka wydruku":
|
||||
- Tabela: Data | Zamówienie | Tracking | Status | Akcje
|
||||
- Status badges: pending (żółty), completed (zielony), failed (czerwony)
|
||||
- Filtr statusu (select/buttons nad tabelą)
|
||||
- Paginacja jeśli >20 rekordów (lub scroll)
|
||||
- Przycisk "Ponów" przy failed jobs (POST /api/print/jobs z tym samym package_id)
|
||||
|
||||
5. Dodaj _printing.scss z stylami:
|
||||
- .print-status-badge (pending/completed/failed kolory)
|
||||
- .print-queue-table (kompaktowy layout)
|
||||
- Zaimportuj w głównym pliku SCSS
|
||||
|
||||
Avoid: Nie modyfikuj sekcji API keys w printing.php — dodaj nową sekcję pod spodem
|
||||
Avoid: Nie dodawaj osobnej strony — kolejka jest częścią ustawień drukowania
|
||||
</action>
|
||||
<verify>
|
||||
- Bulk action widoczna na liście zamówień
|
||||
- Zaznaczenie + klik → zlecenia w print_jobs
|
||||
- Kolejka wydruku widoczna w ustawieniach
|
||||
- Filtrowanie po statusie działa
|
||||
- Style badge'ów poprawne
|
||||
</verify>
|
||||
<done>AC-2 satisfied (bulk z listy), AC-3 satisfied (kolejka w ustawieniach)</done>
|
||||
</task>
|
||||
|
||||
<task type="checkpoint:human-verify" gate="blocking">
|
||||
<what-built>
|
||||
Przycisk "Drukuj" w widoku przesyłki, zbiorcze drukowanie z listy zamówień,
|
||||
kolejka wydruku w ustawieniach drukowania.
|
||||
</what-built>
|
||||
<how-to-verify>
|
||||
1. Otwórz zamówienie z wygenerowaną etykietą → widok przesyłki
|
||||
2. Sprawdź: obok "Pobierz" jest przycisk "Drukuj"
|
||||
3. Kliknij "Drukuj" → powinno pojawić się "Wysłano ✓"
|
||||
4. Kliknij ponownie → powinien pojawić się alert "Zlecenie już w kolejce"
|
||||
5. Wróć do listy zamówień → zaznacz kilka z etykietami
|
||||
6. Wybierz akcję "Drukuj etykiety" → podsumowanie ile wysłano/pominięto
|
||||
7. Otwórz Ustawienia > Drukowanie → sprawdź sekcję "Kolejka wydruku"
|
||||
8. Sprawdź: widać zlecenia z kroków 3 i 6 ze statusem "pending"
|
||||
9. Sprawdź filtrowanie po statusie
|
||||
</how-to-verify>
|
||||
<resume-signal>Type "approved" to continue, or describe issues to fix</resume-signal>
|
||||
</task>
|
||||
|
||||
</tasks>
|
||||
|
||||
<boundaries>
|
||||
|
||||
## DO NOT CHANGE
|
||||
- database/migrations/* (schemat z fazy 18 jest stabilny)
|
||||
- src/Modules/Printing/ApiKeyMiddleware.php (auth middleware gotowe)
|
||||
- Endpointy API key auth (GET /api/print/jobs/pending, download, complete) — te są dla Windows Client
|
||||
|
||||
## SCOPE LIMITS
|
||||
- Nie buduj Windows Client (to faza 20)
|
||||
- Nie dodawaj WebSocket/polling do auto-odświeżania statusów — wystarczy ręczne odświeżenie
|
||||
- Nie zmieniaj flow pobierania etykiet ("Pobierz" pozostaje bez zmian)
|
||||
- Nie dodawaj nowych tabel DB — używaj istniejącej print_jobs
|
||||
|
||||
</boundaries>
|
||||
|
||||
<verification>
|
||||
Before declaring plan complete:
|
||||
- [ ] Przycisk "Drukuj" widoczny w prepare.php obok "Pobierz"
|
||||
- [ ] AJAX feedback działa (success + error + duplikat)
|
||||
- [ ] Bulk endpoint /api/print/jobs/bulk tworzy zlecenia
|
||||
- [ ] Bulk action w index.php działa z checkboxami
|
||||
- [ ] Kolejka wydruku widoczna w ustawieniach drukowania
|
||||
- [ ] Filtrowanie statusu w kolejce
|
||||
- [ ] Style SCSS skompilowane (brak inline CSS)
|
||||
- [ ] Brak natywnych alert()/confirm() — tylko OrderProAlerts
|
||||
- [ ] Wszystkie acceptance criteria spełnione
|
||||
</verification>
|
||||
|
||||
<success_criteria>
|
||||
- Wszystkie 4 AC spełnione
|
||||
- Brak regresji w istniejącym flow pobierania etykiet
|
||||
- Kod zgodny z konwencjami projektu (camelCase, Medoo, XSS escape)
|
||||
- Brak nowych issues SonarQube na zmienionych plikach
|
||||
</success_criteria>
|
||||
|
||||
<output>
|
||||
After completion, create `.paul/phases/19-ui-integration/19-01-SUMMARY.md`
|
||||
</output>
|
||||
142
.paul/phases/19-ui-integration/19-01-SUMMARY.md
Normal file
142
.paul/phases/19-ui-integration/19-01-SUMMARY.md
Normal file
@@ -0,0 +1,142 @@
|
||||
---
|
||||
phase: 19-ui-integration
|
||||
plan: 01
|
||||
subsystem: ui
|
||||
tags: [printing, ajax, bulk-actions, label-queue]
|
||||
|
||||
requires:
|
||||
- phase: 18-print-queue-backend
|
||||
provides: PrintApiController, PrintJobRepository, print_jobs table, API key auth
|
||||
provides:
|
||||
- Przycisk "Drukuj" w widoku przesyłki i szczegółów zamówienia
|
||||
- Bulk print z listy zamówień
|
||||
- Kolejka wydruku w Ustawienia > Drukowanie
|
||||
- Endpoint POST /api/print/jobs/bulk
|
||||
affects: [20-windows-client]
|
||||
|
||||
tech-stack:
|
||||
added: []
|
||||
patterns: [ShipmentProviderRegistry for ensureLabel(), pendingPackageIds for duplicate protection]
|
||||
|
||||
key-files:
|
||||
created:
|
||||
- src/Modules/Printing/PrintApiController.php (bulkCreateJobs, ensureLabel)
|
||||
- src/Modules/Printing/PrintJobRepository.php (findPendingByPackageId, pendingPackageIds)
|
||||
- src/Modules/Settings/PrintSettingsController.php (kolejka wydruku)
|
||||
- resources/views/settings/printing.php (kolejka UI)
|
||||
- resources/scss/modules/_printing.scss
|
||||
- database/migrations/20260322_000058_create_print_tables.sql
|
||||
modified:
|
||||
- resources/views/shipments/prepare.php (przycisk Drukuj, label file check)
|
||||
- resources/views/orders/show.php (przycisk Drukuj, tab persistence, carrier_id column)
|
||||
- resources/views/orders/list.php (bulk print action)
|
||||
- routes/web.php (DI + nowe route'y)
|
||||
- src/Modules/Orders/OrdersController.php (storagePath + printJobRepo injection)
|
||||
- src/Modules/Shipments/ShipmentController.php (redirect po utworzeniu przesyłki)
|
||||
- src/Modules/Shipments/ApaczkaShipmentService.php (carrier_id = service name)
|
||||
- src/Modules/Settings/ApaczkaApiClient.php (fix use statement)
|
||||
- resources/scss/app.scss (_printing import)
|
||||
- public/assets/css/app.css (compiled)
|
||||
|
||||
key-decisions:
|
||||
- "ensureLabel() auto-downloads label via ShipmentProviderRegistry before creating print job"
|
||||
- "findPendingByPackageId protects against duplicate pending jobs"
|
||||
- "Apaczka carrier_id stores service name (e.g. 'Orlen Paczka') for display"
|
||||
- "Tab persistence via localStorage in show.php"
|
||||
- "Label file existence check hides Pobierz/Drukuj when file missing on disk"
|
||||
|
||||
patterns-established:
|
||||
- "ShipmentProviderRegistry pattern for multi-provider label operations"
|
||||
- "Bulk endpoint returns {created: [], skipped: []} JSON structure"
|
||||
|
||||
duration: ~4h
|
||||
started: 2026-03-22T10:00:00Z
|
||||
completed: 2026-03-22T22:00:00Z
|
||||
---
|
||||
|
||||
# Phase 19 Plan 01: UI Integration Summary
|
||||
|
||||
**Przycisk "Drukuj" w widoku przesyłki, bulk print z listy zamówień, kolejka wydruku w ustawieniach — pełna integracja UI z backendem kolejki wydruku z fazy 18.**
|
||||
|
||||
## Performance
|
||||
|
||||
| Metric | Value |
|
||||
|--------|-------|
|
||||
| Duration | ~4h (rozłożone na 2 sesje) |
|
||||
| Started | 2026-03-22 |
|
||||
| Completed | 2026-03-22 |
|
||||
| Tasks | 3 (2 auto + 1 checkpoint) |
|
||||
| Files modified | 14 |
|
||||
|
||||
## Acceptance Criteria Results
|
||||
|
||||
| Criterion | Status | Notes |
|
||||
|-----------|--------|-------|
|
||||
| AC-1: Przycisk "Drukuj" w widoku przesyłki | Pass | Widoczny w prepare.php i show.php, AJAX feedback + duplikat protection |
|
||||
| AC-2: Zbiorcze drukowanie z listy zamówień | Pass | Checkboxy + header action "Drukuj etykiety", bulk endpoint |
|
||||
| AC-3: Kolejka wydruku w ustawieniach | Pass | Tabela zleceń z filtrami statusu, retry dla failed |
|
||||
| AC-4: Endpoint bulk create | Pass | POST /api/print/jobs/bulk z {package_ids/order_ids} |
|
||||
|
||||
## Accomplishments
|
||||
|
||||
- Przycisk "Drukuj" obok "Pobierz" z AJAX feedback i ochroną przed duplikatami
|
||||
- Bulk print z listy zamówień (checkboxy + nagłówkowa akcja)
|
||||
- Kolejka wydruku w Ustawienia > Drukowanie z filtrami i retry
|
||||
- Auto-download etykiety (ensureLabel) przez ShipmentProviderRegistry
|
||||
- Stan "W kolejce" (disabled, czerwony) gdy pending job istnieje
|
||||
|
||||
## Deviations from Plan
|
||||
|
||||
### Summary
|
||||
|
||||
| Type | Count | Impact |
|
||||
|------|-------|--------|
|
||||
| Scope additions | 6 | Usprawnienia UX wykryte podczas testów |
|
||||
| Auto-fixed | 1 | Bug fix ApaczkaApiClient |
|
||||
| Deferred | 0 | — |
|
||||
|
||||
**Total impact:** Dodatkowe usprawnienia UX poza planem, zero regresji.
|
||||
|
||||
### Scope Additions (poza planem)
|
||||
|
||||
1. **show.php tab persistence** — localStorage zapamiętuje aktywny tab
|
||||
2. **show.php label file check** — ukrywa Pobierz/Drukuj gdy plik nie istnieje na dysku
|
||||
3. **show.php przycisk Drukuj** — plan dotyczył tylko prepare.php, dodano też w show.php
|
||||
4. **Kolumna Przewoznik** — "Apaczka → Orlen Paczka" zamiast pustego carrier_id
|
||||
5. **Redirect po utworzeniu przesyłki** — ShipmentController redirect → /orders/{id} z flash
|
||||
6. **Sekcja przesyłek przeniesiona** — pod formularz nowej przesyłki w prepare.php
|
||||
|
||||
### Auto-fixed Issues
|
||||
|
||||
**1. ApaczkaApiClient use statement bug**
|
||||
- **Found during:** Task 1
|
||||
- **Issue:** Brak backslashy w use statement
|
||||
- **Fix:** Poprawiony import namespace
|
||||
- **Files:** src/Modules/Settings/ApaczkaApiClient.php
|
||||
|
||||
## SonarQube Scan
|
||||
|
||||
**Scan date:** 2026-03-22
|
||||
**Result:** 0 new blocker/critical issues on phase 19 files
|
||||
**New issues (minor/major):**
|
||||
- show.php:696 — S4833 require → use, S2003 require → require_once
|
||||
- OrdersController — S1448 (22 methods > 20), S1142 (bulkPrint 5 returns)
|
||||
|
||||
All logged to DOCS/todo.md.
|
||||
|
||||
## Next Phase Readiness
|
||||
|
||||
**Ready:**
|
||||
- Pełne UI do zarządzania kolejką wydruku
|
||||
- API endpoints gotowe dla Windows Client (faza 20)
|
||||
- print_jobs table z pełnym lifecycle (pending → completed/failed)
|
||||
|
||||
**Concerns:**
|
||||
- OrdersController rośnie (22 metody) — rozważyć wydzielenie PrintController w przyszłości
|
||||
|
||||
**Blockers:**
|
||||
- None
|
||||
|
||||
---
|
||||
*Phase: 19-ui-integration, Plan: 01*
|
||||
*Completed: 2026-03-22*
|
||||
Reference in New Issue
Block a user