Parcourir la source

P1-6: verse audit toont ~45 treffers i.p.v. 13 (evenement_horecagelegenheid grotendeels gemist eerder); P1-8: EventCurrent-knoop niet meer geblokkeerd, Bob's keuze vastgelegd

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
bob il y a 1 mois
Parent
commit
202790a329
1 fichiers modifiés avec 44 ajouts et 22 suppressions
  1. 44 22
      TASKS.md

+ 44 - 22
TASKS.md

@@ -224,32 +224,52 @@ leverbaar-veld per horecagelegenheid. Wacht op Bob: eerst het
 API-endpoint aan Drupal-kant configureren. Daarna tonen/verbergen op
 API-endpoint aan Drupal-kant configureren. Daarna tonen/verbergen op
 basis van het echte veld i.p.v. de huidige dode tap.
 basis van het echte veld i.p.v. de huidige dode tap.
 
 
-**P1-6 · Eigenaar: Bob.** Letterlijke veldnaam i.p.v. nette placeholder
-bij ontbrekende data — audit afgerond 2026-08-04, 13 treffers. Patroon:
-`valueOrDefault<String>(<bron>, '<default>')` waarbij de default gelijk
-is aan de veld-/functienaam. Fix is mechanisch (Default Value-tekst
-aanpassen in de builder op het Text-widget), maar raakt waarschijnlijk
-dezelfde "Set from Variable"-property als de Empty-URL-fix — zelfde
-risico-inschatting als het CachedNetworkImage-patroon.
-- `horecagelegenheidoverzicht_kaart_widget.dart:357/381/408` — titel/adres/plaats
-- `event_current_widget.dart:416` — nid
-- `evenement_info_widget.dart:134/156/211/236/291` — adres/plaats/entreeprijs/entreetoelichting/contact
-- `evenement_component_widget.dart:216/238` — title/eventDate
-- Grensgevallen (geen letterlijke veldnaam maar wel technische/zinloze
+**P1-6 · Eigenaar: Bob (mechanisch, maar groot — spreid over
+sessies).** Letterlijke veldnaam i.p.v. nette placeholder bij
+ontbrekende data. **2026-08-05: audit herhaald (Claude, verse grep) —
+scope flink groter dan de eerdere 13 treffers van 2026-08-04, met name
+`evenement_horecagelegenheid_widget.dart` bleek nog niet meegenomen.**
+Patroon overal hetzelfde: `valueOrDefault<String>(<bron>, '<default>')`
+waarbij de default gelijk is aan de veld-/functienaam. Fix is
+mechanisch (Default Value-tekst aanpassen in de builder op het
+Text-widget), maar raakt dezelfde "Set from Variable"-property als de
+Empty-URL-fix — zelfde risico-inschatting als het
+CachedNetworkImage-patroon. **Niet in één sessie proberen af te maken**
+gezien de omvang hieronder (~45 treffers); pak het file-voor-file op.
+- `horecagelegenheidoverzicht_kaart_widget.dart:359/383/410` — titel/adres/plaats
+- `evenement_info_widget.dart:136/158/213/238/293` — adres/plaats/entreeprijs/entreetoelichting/contact
+- `evenement_component_widget.dart:221/242/317/339` — title/eventDate/adres/plaats
+  (317/339 zijn **nieuw t.o.v. de 2026-08-04-audit**, waren toen nog
+  niet aanwezig of gemist)
+- `horecagelegenheid_current_widget.dart:377/500/747` — title/logo/content
+  (**volledig nieuw gevonden, stond niet in de vorige audit**)
+- `event_current_widget.dart:327/418/442` — title/nid/date (327 en 442
+  zijn **nieuw t.o.v. de 2026-08-04-audit**, alleen nid stond er al in)
+- `evenement_horecagelegenheid_widget.dart` — **grootste blok, ~30
+  treffers, volledig nieuw t.o.v. de vorige audit** (waarschijnlijk
+  pas zichtbaar geworden na de P1-image-crash-fix-uitbreiding van dit
+  component): regels 178(HorecaNaam), 418(establishmentnid),
+  445(email), 476(Logo), 499(Plaats), 533(Adres), 577/586/597
+  (Telefoon, 3x — icoon/tekst/tooltip-varianten), 642(menukaart),
+  710(entreeprijs), 743(EntreeToelichting), 818/827/843(facebook, 3x),
+  885/894/910(instagram, 3x), 952/961(twitter, 2x),
+  1016/1025/1041(Website, 3x), 1083/1092/1108(urluk, 3x),
+  1194(Openingstijden), 1222(OpeningstijdenUitzondering),
+  1310(Bezorgtijden), 1337(bezorgkosten), 1364(bestellink).
+- Grensgeval (geen letterlijke veldnaam maar wel technische/zinloze
   placeholder, zelfde fix-aanpak): `p_uitgaan_slider_kaart_component_widget.dart:284`
   placeholder, zelfde fix-aanpak): `p_uitgaan_slider_kaart_component_widget.dart:284`
-  toont `'def'` als datum ontbreekt; `evenement_horecagelegenheid_widget.dart:406`
-  toont `'establishmentnid'`.
+  toont `'def'` als datum ontbreekt.
 - Niet-verdachte gevallen bewust genegeerd (echte inhoudelijke
 - Niet-verdachte gevallen bewust genegeerd (echte inhoudelijke
   fallback, geen technische naam): `'Evenement'`
   fallback, geen technische naam): `'Evenement'`
-  (`kaart_tabel_uitgaan_comp_widget.dart:214`,
-  `kaart_tabel_uitgaan_s_comp_widget.dart:121`) en `'Uitgaan'`
-  (`tag_categorie_component_widget.dart:111`).
+  (`kaart_tabel_uitgaan_comp_widget.dart:216`,
+  `kaart_tabel_uitgaan_s_comp_widget.dart:123`) en `'Uitgaan'`
+  (`tag_categorie_component_widget.dart:113`).
 
 
 **P1-7 · Eigenaar: Bob.** Account/profielscherm (wachtwoord wijzigen,
 **P1-7 · Eigenaar: Bob.** Account/profielscherm (wachtwoord wijzigen,
 uitloggen, account verwijderen). Bovenop de login/favorieten-P0-basis.
 uitloggen, account verwijderen). Bovenop de login/favorieten-P0-basis.
 Account verwijderen heeft mogelijk AVG-implicaties aan Drupal-kant.
 Account verwijderen heeft mogelijk AVG-implicaties aan Drupal-kant.
 
 
-**P1-8 · Eigenaar: Claude — bezig (sessie Home-cluster).**
+**P1-8 · Eigenaar: Onbepaald (2 losse restpunten, zie onder).**
 Toegankelijkheid & navigatie, restpunten:
 Toegankelijkheid & navigatie, restpunten:
 - Login-pagina: 2 icon-only knoppen zonder tooltip (veld wissen +
 - Login-pagina: 2 icon-only knoppen zonder tooltip (veld wissen +
   wachtwoord tonen/verbergen) — `login_widget.dart` (suffixIcon van
   wachtwoord tonen/verbergen) — `login_widget.dart` (suffixIcon van
@@ -290,10 +310,12 @@ Toegankelijkheid & navigatie, restpunten:
   gewone body-`Column` werkte wel meteen. Builder-observatie, geen
   gewone body-`Column` werkte wel meteen. Builder-observatie, geen
   bevestigde bug — als Bob de knop liever in de AppBar wil, eerst dat
   bevestigde bug — als Bob de knop liever in de AppBar wil, eerst dat
   invoegen zelf proberen.
   invoegen zelf proberen.
-  **`EventCurrent` nog open** — geen eenduidige "hoofdlijst"-pagina
-  voor events (`EventWidget`-route is een orphan, zie P2-7); Bob moet
-  kiezen welke pagina hier geldt (vermoedelijk `PUitgaanPage` met de
-  huidige `plaats`) voordat dit gebouwd kan worden.
+  **`EventCurrent` nog open, maar niet meer geblokkeerd op een
+  designvraag — Bob's keuze (2026-08-05): `PUitgaanPage` met de
+  huidige `plaats`.** Zelfde patroon als bij `HorecagelegenheidCurrent`
+  hierboven: nieuwe Button onderaan de body-Column → **Navigate To →
+  PUitgaanPage**, parameter `plaats` = App State `gemeenteSelectId`.
+  Nog te bouwen.
 - Deep-link/query-param robuustheid checken (ongeldig/verlopen `nid`)
 - Deep-link/query-param robuustheid checken (ongeldig/verlopen `nid`)
   — **grotendeels gecheckt 2026-08-04, code-niveau, geruststellend.**
   — **grotendeels gecheckt 2026-08-04, code-niveau, geruststellend.**
   `HorecagelegenheidCurrentWidget` en `EventCurrentWidget` gebruiken
   `HorecagelegenheidCurrentWidget` en `EventCurrentWidget` gebruiken