Changeset bb9e54c in Klonkt

Timestamp:

08/21/2026 08:13:28 AM (3 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

e311202

Parents:

0f09302

Message:

SEO-paneel kon niets opslaan: twee regels stonden in de SQL-string

In f50a84b belandden de twee const-regels van de MusicBrainz-koppeling BINNEN
de template-literal van de UPDATE. Daarmee waren ze geen code maar tekst in de
SQL, en faalde elke opslag op /admin/seo met near "/": syntax error.

Niet alleen de koppeling: het hele paneel. Titelsjabloon, canonical,
omschrijving, og:image, alle verificatiecodes, publisher -- alles ging door
dezelfde UPDATE.

Waarom niemand het zag. Het is geldig JavaScript, dus node --check, de lader en
de suite bleven groen; de fout leeft pas bij prepare(). En het zoekscherm loopt
langs een aparte GET, dus 'Ben jij dit?' toonde de artiest netjes -- alleen
bewaren deed niets. Het actor-document liet daarom terecht geen schema:sameAs
zien: er stond niets in de kolom. Gemeten op soundfabrics: mb_artist_id is
null, op alle instanties.

Vier routetests erbij, door de echte route heen, plus de tweede helft: dat de
koppeling ook echt als schema:sameAs op de actor komt. Controleproef gedraaid --
met de oude opstelling vallen alle vier om terwijl node --check groen blijft.

Co-Authored-By: Claude Opus 5 <noreply@…>

Files:

• src/routes/admin-seo.js (modified) (2 diffs)
• test/admin-seo-save.test.js (added)

src/routes/admin-seo.js

@@ -95 +95 @@
   const schemaType = f.schema_type === 'Organization' ? 'Organization' : 'Person';
 
+  // De MusicBrainz-koppeling (shaer-mbz). Alleen een echte MBID komt de kolom
+  // in: zonder deze zeef sluipt er een URL of een handle in het veld dat naar
+  // buiten gaat, en het gaat naar TWEE uitgangen -- de JSON-LD en de actor.
+  // Leeg is een geldige keuze; dat is ontkoppelen.
+  //
+  // Deze twee regels stonden BINNEN de template-literal hieronder, dus ze waren
+  // geen code maar tekst in de SQL. Daardoor faalde elke opslag op dit paneel
+  // met `near "/": syntax error` -- niet alleen de koppeling, ook het
+  // titelsjabloon, de omschrijving en alle verificatiecodes.
+  const mbRuw = String(f.mb_artist_id || '').trim().toLowerCase();
+  const mbArtistId = isMbid(mbRuw) ? mbRuw : null;
+
   db.prepare(`
     UPDATE sites SET
@@ -118 +130 @@
       updated_at = CURRENT_TIMESTAMP
     WHERE id = ?
-  // De MusicBrainz-koppeling (shaer-mbz). Alleen een echte MBID komt de kolom
-  // in: zonder deze zeef sluipt er een URL of een handle in het veld dat naar
-  // buiten gaat, en het gaat naar TWEE uitgangen -- de JSON-LD en de actor.
-  // Leeg is een geldige keuze; dat is ontkoppelen.
-  const mbRuw = String(f.mb_artist_id || '').trim().toLowerCase();
-  const mbArtistId = isMbid(mbRuw) ? mbRuw : null;
-
   `).run(
     f.robots_index ? 1 : 0,