Changeset 6401681 in Klonkt for src/routes

Timestamp:

08/13/2026 01:08:32 PM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

f59a46c

Parents:

5300652

git-author:

Robin <roboburr@…> (08/13/2026 12:16:47 PM)

git-committer:

Robin <roboburr@…> (08/13/2026 01:08:32 PM)

Message:

Het paneel hoort bij wie je bent, niet bij je bestanden (Robins correctie)

Verhuisd van Beheer -> Audio naar het profielscherm van de site-eigenaar, en
daar staat het NAAST de aliassen -- want dat is dezelfde vraag. Een alias zegt
wie je elders in de fediverse bent, een MBID zegt wie je in het muziekregister
bent. Bij de audiobestanden ging het over beheer van spullen; hier gaat het over
identiteit, en daar hoort het thuis.

DE VERHUIZING VERANDERDE DE VORM, en dat is de moeite waard om te weten. Het
profielscherm IS een formulier, en een formulier in een formulier bestaat niet
in HTML. Dus geen eigen opslaan-knop meer: de keuze zet een verborgen veld en
gaat mee met de Opslaan van de pagina, precies zoals de profielfoto. Dat leest
ook beter -- een halve wijziging die je nog kunt herzien voor je opslaat.

Ontkoppelen is nu het veld leegmaken, geen aparte route. Minder oppervlak.

Het zoek-endpoint ging mee naar /admin/sites/:slug/api/musicbrainz. Een endpoint
onder /admin/audio laten staan voor een scherm in /admin/sites is precies zo'n
naad die later niemand meer kan plaatsen. De teksten heten nu asite.mb_* in
plaats van aaud.mb_*, in alle drie de talen, met een placeholder erbij.

Wat NIET veranderde: de zoekopdracht draait server-side (hun regel geldt per
applicatie), wij kiezen niet voor de artiest, en alleen een echte MBID komt de
kolom in.

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

Location:

src/routes

Files:

• admin-audio.js (modified) (2 diffs)
• admin-sites.js (modified) (5 diffs)

src/routes/admin-audio.js

@@ -23 +23 @@
 import { audioUrl } from '../services/AudioStreamService.js';
 import { mediaDir } from '../config/paths.js';
-import MusicBrainz from '../services/MusicBrainzService.js';
 
 const __dirname = path.dirname(fileURLToPath(import.meta.url));
@@ -384 +383 @@
 
 /** GET /admin/audio/api/albums — distinct list of album names (for datalist) */
-/**
- * "Ben jij dit?" -- kandidaten uit MusicBrainz (shaer-mbz, stap 1).
- *
- * De zoekopdracht draait HIER en niet in de browser: MusicBrainz staat een
- * verzoek per seconde toe per APPLICATIE, en dat is alleen af te dwingen als
- * alles langs een plek gaat. Bovendien eisen ze een User-Agent met contact, en
- * die kan een browser niet zetten.
- *
- * De keuze blijft van de artiest. Wij tonen kandidaten met hun toelichting; we
- * kiezen er niet zelf een, ook niet als er maar een treffer is -- een verkeerd
- * geraden MBID koppelt iemand aan het werk van een ander.
- */
-router.get('/api/musicbrainz', requireGod, async (req, res) => {
-  const site = res.locals.site;
-  if (!site) return res.status(404).json({ error: 'no_site' });
-  // Standaard de artiestennaam die al in de site staat: negen van de tien keer
-  // is dat precies waar iemand op zou zoeken.
-  const q = String(req.query.q || site.author || site.title || '').trim();
-  if (!q) return res.json({ ok: true, q: '', kandidaten: [] });
-  const kandidaten = await MusicBrainz.zoekArtiesten(q);
-  res.json({
-    ok: true,
-    q,
-    gekoppeld: site.mb_artist_id
-      ? { mbid: site.mb_artist_id, naam: site.mb_artist_name || '', url: MusicBrainz.artiestUrl(site.mb_artist_id) }
-      : null,
-    kandidaten,
-  });
-});
-
-/**
- * De keuze vastleggen. Alleen een echte MBID komt de kolom in: de naam die we
- * ernaast bewaren is voor het scherm, de MBID is het enige dat naar buiten gaat.
- */
-router.post('/musicbrainz/link', requireGod, (req, res) => {
-  const site = res.locals.site;
-  if (!site) return res.status(404).end();
-  const mbid = String(req.body.mbid || '').trim().toLowerCase();
-  const naam = String(req.body.naam || '').trim().slice(0, 200);
-  const terug = (res.locals.siteUrlBase || '') + '/admin/audio';
-  if (!MusicBrainz.isMbid(mbid)) return res.redirect(`${terug}?error=` + encodeURIComponent('Geen geldige MusicBrainz-id.'));
-  db.prepare('UPDATE sites SET mb_artist_id = ?, mb_artist_name = ? WHERE id = ?').run(mbid, naam || null, site.id);
-  res.redirect(`${terug}?success=` + encodeURIComponent('Gekoppeld aan MusicBrainz.'));
-});
-
-/** Terugdraaien. Een verkeerde koppeling zet jouw naam onder andermans werk. */
-router.post('/musicbrainz/unlink', requireGod, (req, res) => {
-  const site = res.locals.site;
-  if (!site) return res.status(404).end();
-  db.prepare('UPDATE sites SET mb_artist_id = NULL, mb_artist_name = NULL WHERE id = ?').run(site.id);
-  res.redirect((res.locals.siteUrlBase || '') + '/admin/audio?success=' + encodeURIComponent('Ontkoppeld.'));
-});
-
 router.get('/api/albums', requireGod, (req, res) => {
   const site = res.locals.site;

src/routes/admin-sites.js

@@ -27 +27 @@
 import { mediaDir } from '../config/paths.js';
 import AP from '../services/ActivityPubService.js';
+import MusicBrainz from '../services/MusicBrainzService.js';
 
 
@@ -322 +323 @@
 
 // ==================== SAVE ====================
+/**
+ * "Ben jij dit?" -- kandidaten uit MusicBrainz (shaer-mbz).
+ *
+ * De zoekopdracht draait HIER en niet in de browser: MusicBrainz staat een
+ * verzoek per seconde toe per APPLICATIE, en dat is alleen af te dwingen als
+ * alles langs een plek gaat. Bovendien eisen ze een User-Agent met contact, en
+ * die kan een browser niet zetten.
+ *
+ * Wij kiezen NIET. Ook niet als er precies een treffer is: een verkeerd geraden
+ * MBID zet jouw naam onder andermans werk.
+ */
+router.get('/:slug/api/musicbrainz', requireSiteManagerBySlug, async (req, res) => {
+  const site = db.prepare('SELECT title, mb_artist_id, mb_artist_name FROM sites WHERE slug = ?').get(req.params.slug);
+  if (!site) return res.status(404).json({ error: 'no_site' });
+  const q = String(req.query.q || site.title || '').trim();
+  if (!q) return res.json({ ok: true, q: '', kandidaten: [] });
+  res.json({ ok: true, q, kandidaten: await MusicBrainz.zoekArtiesten(q) });
+});
+
 router.post('/:slug/save', requireSiteManagerBySlug, async (req, res) => {
   const site = db.prepare('SELECT id, ap_aliases FROM sites WHERE slug = ?').get(req.params.slug);
@@ -346 +366 @@
   // accent: only accept colors from the curated ACCENTS list. Falls back to
   // the orange default if the submitted value isn't recognised.
+  // De MusicBrainz-koppeling (shaer-mbz). Alleen een echte MBID komt de kolom
+  // in: zonder deze zeef sluipt er een hele URL of een handle in het veld dat
+  // straks naar buiten gaat. Leeg is een geldige keuze -- dat is ontkoppelen.
+  const mbRuw = String(f.mb_artist_id || '').trim().toLowerCase();
+  const mbArtistId = MusicBrainz.isMbid(mbRuw) ? mbRuw : null;
+
   const accent = ThemeService.validateAccent(f.accent) || '#e8b04b';
 
@@ -355 +381 @@
       profile_links = ?,
       ap_aliases = ?,
+      mb_artist_id = ?, mb_artist_name = ?,
       is_public = ?, robots_index = ?, require_login_to_comment = ?,
       enable_audio_player = ?,
@@ -374 +401 @@
     profileLinksJson,
     apAliasesJson,
+    mbArtistId,
+    mbArtistId ? (String(f.mb_artist_name || '').trim().slice(0, 200) || null) : null,
     f.is_public ? 1 : 0,
     f.robots_index ? 1 : 0,