Просмотр исходного кода

TASKS.md/CLAUDE.md: P1-6 titel-correctie gedocumenteerd (HorecagelegenheidoverzichtKaart), nieuwe CLAUDE.md-notitie over drupalRequest vs. declared API Calls als databron

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
bob 3 недель назад
Родитель
Сommit
c906d543ba
2 измененных файлов с 24 добавлено и 0 удалено
  1. 15 0
      CLAUDE.md
  2. 9 0
      TASKS.md

+ 15 - 0
CLAUDE.md

@@ -1187,6 +1187,21 @@ exacte veldnaam+type-specificatie (kost hem seconden per veld).
   die wél (bv. om een eigen foutmelding te tonen), dan moet de
   die wél (bv. om een eigen foutmelding te tonen), dan moet de
   functie-contract opnieuw aangepast worden en alle bestaande
   functie-contract opnieuw aangepast worden en alle bestaande
   aanroepers gecontroleerd.
   aanroepers gecontroleerd.
+- **`drupalRequest` is géén centraal punt om lege/`null`-veldwaarden
+  op te vangen voor de meeste widgets — bevestigd 2026-08-19 (Claude,
+  `grep -rn "drupalRequest(" lib/`).** Deze custom action wordt
+  alleen aangeroepen vanuit `favorieten_widget.dart` (2x) en het dode
+  `lib/kanweg/`. Vrijwel alle andere data (o.a. alles achter P1-6's
+  `valueOrDefault<String>(<bron>, '<placeholder>')`-patroon, zoals
+  `HorecagelegenheidoverzichtKaart`/`evenement_info_widget.dart`) komt
+  binnen via FlutterFlow's **ingebouwde declared API Calls**
+  (`api_calls.dart`, bv. `HorecagelegenheidoverzichtCall`) — die lopen
+  nooit langs `drupalRequest`, en die response heeft **geen centraal
+  transform-punt**: elk veld wordt los, inline via `getJsonField(...)`
+  op zijn eigen plek in de widget-code uitgelezen. Een "lege string →
+  spatie"-fix in `drupal_request.dart` zou dus alleen de 3
+  Favorieten/kanweg-aanroepen raken — voor de rest blijft de
+  per-widget Default Value-route (zie P1-6) de enige praktische fix.
 
 
 - **API-call `cache: true` werkt functioneel correct** — `ApiCallOptions`
 - **API-call `cache: true` werkt functioneel correct** — `ApiCallOptions`
   (`lib/backend/api_requests/api_manager.dart`) `extends Equatable` met
   (`lib/backend/api_requests/api_manager.dart`) `extends Equatable` met

+ 9 - 0
TASKS.md

@@ -747,6 +747,15 @@ verwijderd), `horecagelegenheid_current_widget.dart` (3 velden, commit
 `14f2f14`), `event_current_widget.dart` (title/date, commit
 `14f2f14`), `event_current_widget.dart` (title/date, commit
 `82b0fb6`). Alles geverifieerd via verse export + `flutter analyze`
 `82b0fb6`). Alles geverifieerd via verse export + `flutter analyze`
 (0 errors). Uit deze lijst verwijderd.)*
 (0 errors). Uit deze lijst verwijderd.)*
+*(Correctie 2026-08-19, live pair-fix sessie: het genoemde 8e veld —
+`horecagelegenheidoverzicht_kaart_widget.dart` titel/adres/plaats —
+stond hierboven al als "afgerond" vermeld, maar `titel` bleek bij
+verse export nog steeds de kale `' titel'`-placeholder te tonen
+(zelfde "leek opgeslagen, was het niet"-patroon als elders in
+`CLAUDE.md`). Bob heeft `titel` alsnog op Default Value
+`'Titel onbekend'` gezet (consistent met de al langer werkende
+`adres`/`plaats` → `'Adres onbekend'`/`'Plaats onbekend'`). Nu pas
+écht bevestigd via verse export voor alle 3 velden.)*
 
 
 **Resterend, bewust laag-prioriteit (geen "— bezig" claim):**
 **Resterend, bewust laag-prioriteit (geen "— bezig" claim):**
 - `evenement_component_widget.dart` (title/eventDate/adres/plaats,
 - `evenement_component_widget.dart` (title/eventDate/adres/plaats,