feat(per-page-persistence): remember selected rows per page in localStorage

table-list component now saves per_page selection to localStorage and
restores it when navigating back to any list without explicit per_page
in URL. URL parameter takes priority over stored value to ensure
bookmarked links work predictably.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
2026-04-07 08:26:53 +02:00
parent 6d0905d97a
commit 24df01cfde
7 changed files with 251 additions and 9 deletions

View File

@@ -75,6 +75,7 @@ Sprzedawca moĹĽe obsĹugiwać zamĂłwienia ze wszystkich kanaĹĂłw
- [x] Data wystawienia paragonu z dokladnoscia do godziny i minuty (DATE -> DATETIME) — Phase 64 - [x] Data wystawienia paragonu z dokladnoscia do godziny i minuty (DATE -> DATETIME) — Phase 64
- [x] Koszt wysylki jako pozycja paragonu (bugfix buildItemsSnapshot + delivery_price) — Phase 70 - [x] Koszt wysylki jako pozycja paragonu (bugfix buildItemsSnapshot + delivery_price) — Phase 70
- [x] Import atrybutow produktow z shopPRO (attributes + custom_fields w personalizacji) — Phase 71 - [x] Import atrybutow produktow z shopPRO (attributes + custom_fields w personalizacji) — Phase 71
- [x] Zapamiętywanie wybranej liczby wierszy na stronie (per_page) w localStorage — Phase 72
- [ ] Eliminacja zduplikowanego kodu: SslCertificateResolver, ToggleableRepositoryTrait, RedirectPathResolver, ReceiptService — Phase 68 - [ ] Eliminacja zduplikowanego kodu: SslCertificateResolver, ToggleableRepositoryTrait, RedirectPathResolver, ReceiptService — Phase 68
### Active (In Progress) ### Active (In Progress)

View File

@@ -32,6 +32,7 @@ Wersja mobilna aplikacji, modul po module. Cel: pelna uzywalnosc orderPRO na tel
| 69 | Allegro Tracking English Statuses | 1/1 | Complete | | 69 | Allegro Tracking English Statuses | 1/1 | Complete |
| 70 | Receipt Shipping Cost | 1/1 | Complete | | 70 | Receipt Shipping Cost | 1/1 | Complete |
| 71 | Attributes Import | 1/1 | Complete | | 71 | Attributes Import | 1/1 | Complete |
| 72 | Per Page Persistence | 1/1 | Complete |
| TBD | Mobile Orders List | - | Not started | | TBD | Mobile Orders List | - | Not started |
| TBD | Mobile Order Details | - | Not started | | TBD | Mobile Order Details | - | Not started |
| TBD | Mobile Settings | - | Not started | | TBD | Mobile Settings | - | Not started |

View File

@@ -5,19 +5,19 @@
See: .paul/PROJECT.md (updated 2026-04-07) See: .paul/PROJECT.md (updated 2026-04-07)
**Core value:** Sprzedawca moze obslugiwac zamowienia ze wszystkich kanalow sprzedazy i nadawac przesylki bez przelaczania sie miedzy platformami. **Core value:** Sprzedawca moze obslugiwac zamowienia ze wszystkich kanalow sprzedazy i nadawac przesylki bez przelaczania sie miedzy platformami.
**Current focus:** Milestone v3.0 - Phase 71 complete, ready for next PLAN **Current focus:** Milestone v3.0 - Phase 72 complete, ready for next PLAN
## Current Position ## Current Position
Milestone: v3.0 Mobile Responsive - In progress Milestone: v3.0 Mobile Responsive - In progress
Phase: 71 (Attributes Import) — Complete Phase: 72 (Per Page Persistence) — Complete
Plan: 71-01 unified Plan: 72-01 unified
Status: Loop complete, ready for next PLAN Status: Loop complete, ready for next PLAN
Last activity: 2026-04-07 — Unified .paul/phases/71-attributes-import/71-01-PLAN.md Last activity: 2026-04-07 — Unified .paul/phases/72-per-page-persistence/72-01-PLAN.md
Progress: Progress:
- Milestone: [#######...] ~74% - Milestone: [########..] ~75%
- Phase 71: [##########] 100% - Phase 72: [##########] 100%
## Loop Position ## Loop Position
@@ -30,11 +30,11 @@ PLAN --> APPLY --> UNIFY
## Session Continuity ## Session Continuity
Last session: 2026-04-07 Last session: 2026-04-07
Stopped at: Plan 71-01 unified Stopped at: Plan 72-01 unified
Next action: Run /paul:plan for the next prioritized phase Next action: Run /paul:plan for the next prioritized phase
Resume file: .paul/phases/71-attributes-import/71-01-SUMMARY.md Resume file: .paul/phases/72-per-page-persistence/72-01-SUMMARY.md
## Git State ## Git State
Last commit: 278f44b Last commit: 6d0905d
Branch: main Branch: main

View File

@@ -0,0 +1,124 @@
---
phase: 72-per-page-persistence
plan: 01
type: execute
wave: 1
depends_on: []
files_modified: [resources/views/components/table-list.php]
autonomous: true
---
<objective>
## Goal
Zapamiętywanie wybranej liczby wierszy na stronie (per_page) w localStorage przeglądarki na liście zamówień i innych listach korzystających z komponentu table-list.
## Purpose
Użytkownik zmienia per_page na 50 lub 100, ale po powrocie na listę wartość resetuje się do domyślnej 20. Zmiana powinna być trwała w danej przeglądarce.
## Output
Zmodyfikowany `resources/views/components/table-list.php` — zapis per_page do localStorage przy zmianie i odczyt przy ładowaniu strony.
</objective>
<context>
## Source Files
@resources/views/components/table-list.php (linie 311-325 per-page selector, 329-432 localStorage, 428-432 change handler)
</context>
<acceptance_criteria>
## AC-1: Zapis per_page do localStorage
```gherkin
Given użytkownik jest na liście zamówień z per_page=20
When zmieni per_page na 50
Then wartość 50 zostaje zapisana w localStorage pod kluczem tableList_[basePath]_[listKey]_per_page
```
## AC-2: Odczyt per_page z localStorage
```gherkin
Given użytkownik wcześniej wybrał per_page=50 (zapisane w localStorage)
When wchodzi na listę zamówień bez parametru per_page w URL
Then formularz per_page automatycznie ustawia 50 i strona przeładowuje się z per_page=50
```
## AC-3: Parametr URL ma priorytet nad localStorage
```gherkin
Given użytkownik ma per_page=50 w localStorage
When wchodzi na listę z jawnym per_page=20 w URL
Then wyświetla 20 rekordów (URL ma priorytet) i localStorage zostaje zaktualizowane na 20
```
## AC-4: Brak pętli przeładowań
```gherkin
Given użytkownik ma per_page=50 w localStorage i wchodzi na stronę
When strona przeładowuje się z per_page=50
Then nie następuje kolejne przeładowanie (brak nieskończonej pętli)
```
</acceptance_criteria>
<tasks>
<task type="auto">
<name>Task 1: Dodanie persistence per_page w table-list.php</name>
<files>resources/views/components/table-list.php</files>
<action>
1. Dodać klucz localStorage: `var perPageKey = storagePrefix + '_per_page';` (obok istniejących filterKey, colsKey, queryKey ~linia 336)
2. W handlerze change selecta per_page (~linia 428-432): przed `form.submit()` zapisać wybraną wartość do localStorage:
`writeJson(perPageKey, parseInt(select.value, 10));`
3. Na końcu sekcji DOMContentLoaded (po istniejących restore bloków), dodać logikę restore per_page:
- Odczytać `readJson(perPageKey, null)`
- Jeśli wartość istnieje i jest różna od aktualnie wybranej w selekcie per_page:
- Sprawdzić czy per_page jest obecne w URL (URLSearchParams)
- Jeśli per_page NIE jest w URL (użytkownik wszedł bez jawnego parametru):
- Ustawić wartość w selekcie i submitować formularz (to spowoduje przeładowanie z prawidłowym per_page w URL)
- Jeśli per_page JEST w URL: zaktualizować localStorage z wartości z URL (sync)
- Jeśli wartość jest taka sama jak w selekcie: nic nie rób (brak pętli)
Wzorzec: użyć istniejących helperów readJson/writeJson.
Unikać: nieskończonej pętli przeładowań — kluczowy warunek: nie submitować jeśli wartość w selekcie już odpowiada localStorage.
</action>
<verify>
1. Otworzyć /orders/list — domyślnie 20
2. Zmienić na 50 — strona przeładowuje z per_page=50
3. Wejść na /orders/list (bez per_page w URL) — automatycznie przeładuje z per_page=50
4. Wejść na /orders/list?per_page=20 — wyświetli 20 i zaktualizuje localStorage
5. Sprawdzić brak pętli przeładowań
</verify>
<done>AC-1, AC-2, AC-3, AC-4 satisfied</done>
</task>
</tasks>
<boundaries>
## DO NOT CHANGE
- Backend PHP (OrdersController, OrdersRepository)
- Struktura HTML per-page selectora
- Inne localStorage keys (filters_open, hidden_cols, query)
## SCOPE LIMITS
- Tylko frontend JS w table-list.php
- Bez zmian backendu — per_page nadal przychodzi jako query param
</boundaries>
<verification>
Before declaring plan complete:
- [ ] Brak błędów JS w konsoli przeglądarki
- [ ] Per_page zapisuje się w localStorage po zmianie
- [ ] Per_page odtwarza się po wejściu bez parametru w URL
- [ ] Brak nieskończonej pętli przeładowań
- [ ] Inne listy (np. paragony) też korzystają z persistence
</verification>
<success_criteria>
- Wybrana liczba wierszy jest zapamiętywana w localStorage
- Wejście na listę bez per_page w URL automatycznie stosuje zapamiętaną wartość
- Brak regresji w istniejącej funkcjonalności table-list
</success_criteria>
<output>
After completion, create `.paul/phases/72-per-page-persistence/72-01-SUMMARY.md`
</output>

View File

@@ -0,0 +1,94 @@
---
phase: 72-per-page-persistence
plan: 01
subsystem: ui
tags: [localStorage, pagination, table-list, UX]
requires: []
provides:
- Per page persistence via localStorage for all table-list instances
affects: []
tech-stack:
added: []
patterns: [per-page localStorage key in table-list component]
key-files:
created: []
modified: [resources/views/components/table-list.php]
key-decisions:
- "Dedykowany klucz per_page zamiast polegania na query restore"
- "URL per_page ma priorytet nad localStorage — sync a nie override"
patterns-established:
- "perPageKey = storagePrefix + '_per_page' — nowy klucz w rodzinie tableList_ storage"
duration: 5min
started: 2026-04-07T12:20:00Z
completed: 2026-04-07T12:25:00Z
---
# Phase 72 Plan 01: Per Page Persistence Summary
**Zapamiętywanie wybranej liczby wierszy w localStorage — per_page persist dla wszystkich list table-list.**
## Performance
| Metric | Value |
|--------|-------|
| Duration | ~5min |
| Tasks | 1 completed |
| Files modified | 1 |
## Acceptance Criteria Results
| Criterion | Status | Notes |
|-----------|--------|-------|
| AC-1: Zapis per_page do localStorage | Pass | writeJson przy change + sync z URL |
| AC-2: Odczyt per_page z localStorage | Pass | Auto-submit gdy brak per_page w URL |
| AC-3: URL ma priorytet nad localStorage | Pass | urlParams.has('per_page') → sync only |
| AC-4: Brak pętli przeładowań | Pass | Po submit URL zawiera per_page → brak ponownego submit |
## Accomplishments
- Dodano `perPageKey` do rodziny kluczy localStorage w table-list.php
- Zapis przy zmianie selecta per_page (przed submit)
- Odczyt i auto-redirect przy wejściu bez per_page w URL
- Sync localStorage z URL gdy per_page jawnie podane
## Files Created/Modified
| File | Change | Purpose |
|------|--------|---------|
| `resources/views/components/table-list.php` | Modified | perPageKey + save/restore logic |
## Decisions Made
| Decision | Rationale | Impact |
|----------|-----------|--------|
| Dedykowany klucz per_page | Query restore przywraca caly URL (status, page, search) — per_page powinien byc niezalezny | Mozna zmienic per_page bez resetowania filtrow |
| URL ma priorytet | Jawny link z per_page=20 powinien dzialac przewidywalnie | Brak niespodzianek przy linkach bookmarkowanych |
## Deviations from Plan
None — plan executed exactly as written
## Issues Encountered
None
## Next Phase Readiness
**Ready:**
- Wszystkie listy table-list automatycznie korzystaja z persistence per_page
**Concerns:**
- None
**Blockers:**
- None
---
*Phase: 72-per-page-persistence, Plan: 01*
*Completed: 2026-04-07*

View File

@@ -1,5 +1,13 @@
# Tech Changelog # Tech Changelog
## 2026-04-07 (Phase 72 - Per Page Persistence, Plan 01)
- `resources/views/components/table-list.php`:
- Dodano klucz localStorage `perPageKey` (`tableList_[path]_[key]_per_page`).
- Przy zmianie selecta per_page: zapis do localStorage przed submit formularza.
- Przy ladowaniu strony: odczyt z localStorage i auto-submit jesli per_page nie jest w URL.
- URL per_page ma priorytet nad localStorage (sync, nie override).
- Brak zmian backendu — per_page nadal przychodzi jako query param.
## 2026-04-07 (Phase 71 - Attributes Import, Plan 01) ## 2026-04-07 (Phase 71 - Attributes Import, Plan 01)
- `ShopproOrderMapper::extractPersonalization()`: rozszerzono o odczyt pola `attributes` z API shopPRO. - `ShopproOrderMapper::extractPersonalization()`: rozszerzono o odczyt pola `attributes` z API shopPRO.
- Metoda iteruje po `['attributes', 'custom_fields']` i laczy niepuste wyniki separatorem `\n`. - Metoda iteruje po `['attributes', 'custom_fields']` i laczy niepuste wyniki separatorem `\n`.

View File

@@ -337,6 +337,7 @@ $buildUrl = static function (array $params = []) use ($basePath, $query): string
var colsKey = storagePrefix + '_hidden_cols'; var colsKey = storagePrefix + '_hidden_cols';
var queryKey = storagePrefix + '_query'; var queryKey = storagePrefix + '_query';
var clearKey = storagePrefix + '_cleared'; var clearKey = storagePrefix + '_cleared';
var perPageKey = storagePrefix + '_per_page';
function readJson(key, fallback) { function readJson(key, fallback) {
try { try {
@@ -427,9 +428,22 @@ $buildUrl = static function (array $params = []) use ($basePath, $query): string
root.querySelectorAll('.js-per-page-select').forEach(function(select) { root.querySelectorAll('.js-per-page-select').forEach(function(select) {
select.addEventListener('change', function() { select.addEventListener('change', function() {
writeJson(perPageKey, parseInt(select.value, 10));
var form = select.closest('form'); var form = select.closest('form');
if (form) form.submit(); if (form) form.submit();
}); });
var urlParams = new URLSearchParams(window.location.search);
if (urlParams.has('per_page')) {
writeJson(perPageKey, parseInt(select.value, 10));
} else {
var savedPerPage = readJson(perPageKey, null);
if (savedPerPage !== null && parseInt(select.value, 10) !== savedPerPage) {
select.value = savedPerPage;
var form = select.closest('form');
if (form) form.submit();
}
}
}); });
var selectAll = root.querySelector('.js-table-select-all'); var selectAll = root.querySelector('.js-table-select-all');