diff --git a/TEKNISK_BESKRIVNING.md b/TEKNISK_BESKRIVNING.md index e59d10c0..e44aba3f 100644 --- a/TEKNISK_BESKRIVNING.md +++ b/TEKNISK_BESKRIVNING.md @@ -1030,6 +1030,60 @@ Kvitto-alias lagrar mappningar frÃ¥n kvittots rÃ¥text till produkt-ID. Nä --- +## Arkitektur: Privata produkter och kvittoimport produktval (2026-05-01) + +### Bakgrund + +Kvittoimportflödet (Fas 6b) behövde ett sätt för användare att koppla kvittorader till produkter i systemet. Tre problem identifierades: + +1. **Produktsökning gav nollträffar** när AI-kategorins namn (t.ex. "Frukt & Grönt") matades in som söksträng i produktfältet — produkter heter inte samma sak som kategorier. +2. **AI-föreslagna kategorier var ofta L1/L2** (t.ex. "Mat") men produkter är kopplade till L3-noder. Att filtrera på bara det exakta kategori-ID:t gav 0 produkter. +3. **Saknad produkt** — om varan på kvittot inte finns i systemet alls (t.ex. "Röda äpplen") hade användaren ingen väg framåt. + +### Lösning 1: Tvåstegs-picker (Kategori → Produkt) + +En ny widget `CategoryThenProductPicker` (i `lib/core/ui/`) orkestrerar ett tvåstegsflöde: + +**Steg 1 — Kategoriträdet:** Användaren bläddrar i L1 → L2 → L3 eller söker fritext. Sökresultaten visar hela sökvägen som brödsmula. Alla icke-lövnoder (L1, L2) har en "Välj"-knapp som fallback om man vill stoppa på mellannivå. + +**Steg 2 — Produktpicker:** `ProductPickerField.showSheet()` öppnas filtrerad på den valda kategorins ID **plus alla ättlingars ID:n** (insamlade rekursivt via `_collectIds()`). Det innebär att valet av "Mat > Frukt & Grönt" ger alla produkter i samtliga L3-underkategorier. + +``` +L1: Mat + L2: Frukt & Grönt ← välj här → ger produkter från alla L3 nedan + L3: Äpplen ← välj här → ger bara äpple-produkter + L3: Bananer + L3: Citrusfrukter +``` + +### Lösning 2: AI-direkthopp + +Om `preselectedCategoryId` finns (AI-förslag) hoppar `CategoryThenProductPicker.show()` **direkt till steg 2** utan att visa trädet. Användaren klickar på AI-chip → produktpickern öppnas direkt med rätt kategorifilter. Om kategorin inte hittas i trädet visas trädet som fallback. + +### Lösning 3: Privata produkter + +**Problemet:** `POST /products` (admin) skapar globala produkter. Vanliga användare ska kunna skapa egna produkter utan att smutsa ned den globala produktkatalogen. + +**Modell:** `Product` fick ett nytt fält `isPrivate: Boolean @default(false)`. Privata produkter är kopplade till en specifik användare via `ownerId` och syns bara för den användaren. + +**`normalizedName`-prefix:** För att undvika kollision i det unika indexet prefixas privata produkters normaliserade namn med `private:{userId}:`. Det innebär att "röda äpplen" för användare 5 lagras som `private:5:roda applen` och kolliderar varken med en global produkt med samma namn eller en annan användares privata produkt. + +**Nya endpoints:** + +| Endpoint | Åtkomst | Beskrivning | +|---|---|---| +| `GET /products` | Publik | Returnerar bara globala produkter (`isPrivate: false`) | +| `GET /products/mine` | Inloggad | Returnerar inloggad användares privata produkter | +| `POST /products/private` | Inloggad | Skapar en privat produkt kopplad till inloggad användare | + +**Migration:** `backend/prisma/migrations/20260501000000_add_product_is_private/migration.sql` + +### Flutter-integration + +- `_loadProducts()` i `receipt_import_tab.dart` laddar `GET /products` och `GET /products/mine` parallellt via `Future.wait` och slår ihop till en lista. +- `_EditDialogState` håller en lokal `_localProducts`-kopia som utökas om en ny produkt skapas under dialogen — utan ytterligare nätverksanrop. +- "Skapa ny"-dialogen i produktpickern är förifylld med den aktuella söksträngen för snabbt namngivande. + ## Arkitektur: User-scope för baslager och matplan (2026-04-22) > **Designregel: User-scope vid ny funktionalitet** diff --git a/flutter/lib/core/ui/category_then_product_picker.dart b/flutter/lib/core/ui/category_then_product_picker.dart index ee80d443..5e5a9700 100644 --- a/flutter/lib/core/ui/category_then_product_picker.dart +++ b/flutter/lib/core/ui/category_then_product_picker.dart @@ -73,7 +73,7 @@ class CategoryThenProductPicker { } // Samla alla kategori-IDs i den valda grenen (inkl. ättlingar) - final categoryIds = _collectIds(selectedCategory); + final categoryIds = _collectIds(selectedCategory!); // Filtrera produkter på dessa kategorier final filtered = products @@ -84,7 +84,7 @@ class CategoryThenProductPicker { // Bygg eventuell onCreate-callback med categoryId inbunden final onCreateBound = onCreate == null ? null - : (String name) => onCreate(name, selectedCategory.id); + : (String name) => onCreate(name, selectedCategory!.id); // Steg 2 — välj produkt if (!context.mounted) return null; @@ -92,7 +92,7 @@ class CategoryThenProductPicker { context, products: useList, value: currentProductId, - label: 'Produkt i "${selectedCategory.name}"', + label: 'Produkt i "${selectedCategory!.name}"', categoryFilter: null, // redan förfiltrerat onCreate: onCreateBound, ); diff --git a/flutter/teknisk_beskrivning_flutter.md b/flutter/teknisk_beskrivning_flutter.md index 29043d43..16a7050d 100644 --- a/flutter/teknisk_beskrivning_flutter.md +++ b/flutter/teknisk_beskrivning_flutter.md @@ -2,7 +2,65 @@ Viktigt att komma ihåg vid implementering av nya funktioner och kodning är att inte använda Windows-sökvägar. Använd inte `c:/dev/recipe-app/...` eftersom bygg- och testmiljön är på en remote Ubuntu-server. Utveckling sker lokalt och test samt drift sker på remote server. Säkerställ att inga absoluta Windows-sökvägar används i koden, för att stödja bygg och drift på Linux/Ubuntu. -## Senaste ändringar (2026-05-01) +## Senaste ändringar (2026-05-01, session 2) + +### Tvåstegs-picker: Kategori → Produkt + +Problembeskrivning: AI:n kan föreslå en kategori men produktpickern sökte bara på produktnamn, vilket gav nollresultat när kategorinamnet matades in som söktext. Lösningen är ett nytt separat arbetsflöde för produktval via kategoriträdet. + +**Ny widget: `lib/core/ui/category_then_product_picker.dart`** + +`CategoryThenProductPicker.show()` är en statisk metod som orkestrerar hela flödet: + +1. **Steg 1 — Välj kategori:** Öppnar ett bottenark (`_CategoryPickerSheet`) med hela kategoriträdet (L1 → L2 → L3). Trädet är sökbart — sökfältet filtrerar till matchande lövnoder och visar hela sökvägen som brödsmula (t.ex. *Mat > Frukt & Grönt > Äpplen*). +2. **Steg 2 — Välj produkt:** Öppnar `ProductPickerField.showSheet()` filtrerad på alla produkter som tillhör den valda kategorin **eller någon av dess ättlingar** (L1 samlar alltså in L2- och L3-produkter rekursivt via `_collectIds()`). + +**Inbyggt AI-stöd:** Om `preselectedCategoryId` skickas in (från AI-förslaget) hoppar `show()` direkt till steg 2 — kategoriträdet visas aldrig. Om kategorin inte hittas i trädet faller den tillbaka till att visa trädet. + +**Fallback för L1/L2-noder:** Mellanliggande noder (L1, L2) som inte är löv har en liten "Välj"-knapp till höger i raden. Klick på kategorinamnet/pilen expanderar/kollapsar som vanligt; klick på "Välj" väljer kategorin och öppnar produktpickern direkt. + +**Trädknapp i redigeringsdialogen:** Bredvid det vanliga produktsökfältet finns nu en `OutlinedButton` med trädikon (🌳). Klick öppnar tvåstegs-pickern utan AI-förval. + +### Skapa ny privat produkt i importflödet + +Om inget i produktlistan matchar kvittoradens vara kan användaren skapa en egen produkt direkt från produktpickern. + +**Flöde:** +1. Välj kategori (via trädet eller AI-direkthopp). +2. Produktpickern visas med en **"Skapa ny"**-knapp i rubrikraden. +3. En enkel dialog öppnas med ett namnfält (förfyllt med söksträngen om sådan finns). +4. Vid bekräftelse anropas `POST /products/private` — produkten skapas som privat och user-scopad. +5. Den nya produkten läggs till i den lokala produktlistan och väljs direkt. + +**Privata produkter — arkitektur:** + +| | Globala produkter | Privata produkter | +|---|---|---| +| Endpoint (hämta) | `GET /products` | `GET /products/mine` | +| Endpoint (skapa) | `POST /products` (admin) | `POST /products/private` (alla inloggade) | +| Synlighet | Alla användare | Bara ägaren | +| `isPrivate` | `false` | `true` | +| `normalizedName` | `normalize(name)` | `private:{userId}:{normalize(name)}` | +| Kategori | Global (admin-only) | Väljs vid skapandet, global kategori | + +Importfliken laddar globala och privata produkter parallellt via `Future.wait` och slår ihop dem till en gemensam `_products`-lista. Den lokala `_localProducts`-listan i `_EditDialogState` utökas om en ny produkt skapas under dialogen, utan att en ny nätverksanrop krävs. + +### Redigeringsdialog — sammanfattning av alla fält + +`_EditDialog` i `receipt_import_tab.dart` innehåller nu: +- **AI-chip (grön):** Klickbar. Om AI föreslog en matchad produkt direkt → väljer den. Om AI föreslog en kategori → öppnar produktpickern filtrerad på den kategorin (utan att visa trädet). +- **Destinationsväljare:** `SegmentedButton` — Inventarie eller Baslager. +- **Produktfält + trädknapp:** `ProductPickerField` (fritext-sökning) + knapp för att öppna tvåstegs-picker. +- **Antal/Enhet:** Visas bara vid Inventarie-destination. + +### Designregel bekräftad: User-scope + +User-scope-principen dokumenterades formellt i båda tekniska beskrivningarna (2026-05-01). Privata produkter är det första exemplet på mönstret för resurser som är varken globala (alla ser dem) eller fullt user-owned (bara ägaren ser dem): +- `Product.isPrivate = true` + `Product.ownerId = userId` +- `normalizedName`-prefix undviker databaskollision med globala produkter +- Migration: `20260501000000_add_product_is_private` + +## Senaste ändringar (2026-05-01, session 1) **Kvittoimport Fas 6b — komplett:** - Granskningssteg i `receipt_import_tab.dart` med per-rad checkbox, redigeringsdialog och destinationsväljare.