Parcourir la source

171 af + presave-correctie van Bob vastgelegd

Nagemeten: 0 van 1591 records zonder plaats, de "null" in de Home-slider is weg.

Correctie die in CLAUDE.md hoort: het EVENEMENT moet opnieuw opgeslagen worden,
niet de horecagelegenheid. De presave-hook hangt aan het event.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
bob il y a 8 heures
Parent
commit
b6b8538209
2 fichiers modifiés avec 12 ajouts et 6 suppressions
  1. 5 1
      CLAUDE.md
  2. 7 5
      TASKS.md

+ 5 - 1
CLAUDE.md

@@ -4499,7 +4499,11 @@ Client-side zoeken op Home kan niet: de tabs zijn gepagineerd.
 kapotte presave.** `custom_node_presave()` kopieert plaats/geo/adres van de
 horecagelegenheid **op het moment dat het event wordt opgeslagen**. Had de zaak
 toen nog geen `field_hor_municipality_town`, dan blijft het event leeg totdat
-het opnieuw wordt opgeslagen (`node_save()` via drush volstaat). Nagemeten
+het opnieuw wordt opgeslagen (`node_save()` via drush volstaat).
+⚠️ **Het EVENEMENT moet opnieuw opgeslagen worden, niet de horecagelegenheid**
+(Bob, 2026-09-19). De hook hangt aan het event; de zaak opslaan doet niets voor
+al bestaande events. Eerst de zaak een plaats geven, dán elk betrokken event
+opnieuw opslaan. Nagemeten
 2026-09-15: 14 willekeurige recente events en 50/50 op `services_1` hadden
 allemaal een plaats — ga dus niet in de module zoeken. De view leest `plaats`
 via `taxonomy_index`, en die wordt alleen voor **gepubliceerde** nodes gevuld.

+ 7 - 5
TASKS.md

@@ -482,11 +482,13 @@ en daar zit **geen `valueOrDefault`** omheen (staat zo in `CLAUDE.md`). Een lege
 *Paul van Loo "Solo"*, allebei op adres `Kerkberg 4` (Theater Landgraaf).
 `adres` is nergens leeg, alleen `plaats`.
 
-**Snelste fix is content, niet code.** Dit is het bekende presave-patroon:
-`custom_node_presave()` kopieert plaats/geo/adres van de horecagelegenheid **op
-het moment dat het event wordt opgeslagen**. Had Theater Landgraaf toen nog geen
-`field_hor_municipality_town`, dan blijft het event leeg. Geef de zaak een plaats
-en sla de twee nodes opnieuw op (`node_save()` via drush volstaat).
+**✅ AF (Bob, 2026-09-19).** Nagemeten over dezelfde 1591 records met
+cache-buster: **0 rijen zonder plaats**, de "null" is weg.
+
+⚠️ **Les die Bob erbij gaf en die je moet onthouden: de horecagelegenheid opslaan
+is NIET genoeg — je moet het EVENEMENT opslaan.** `custom_node_presave()` hangt
+aan het event en kopieert dán van de zaak; de zaak opslaan raakt bestaande events
+niet. Staat nu ook in `CLAUDE.md`.
 
 **App-kant: bewust niet gebouwd.** Een guard om die twee nodes heen bouwen kost
 een conditie op een `Combine Text` voor 0,1 % van de records. Wel het noemen