Index: src/assets/js/mod/admin-audio.js
===================================================================
--- src/assets/js/mod/admin-audio.js (revision 5300652c5fdc930a614f60c65e8abb4e56060f4f)
+++ src/assets/js/mod/admin-audio.js (revision 64016815dd12ec86d05866d6d71bca783024094d)
@@ -452,92 +452,3 @@
});
})();
-
- // MusicBrainz-paneel (shaer-mbz). Binnen run(), niet op moduleniveau:
- // deze modules krijgen bij elke paginawissel opnieuw init(), en een blok dat
- // maar een keer per sessie draait is precies wat shaer-5s1 opleverde.
- wireMusicBrainz();
}
-
-// ── MusicBrainz: ben jij dit? (shaer-mbz, stap 2) ─────────────────
-//
-// De kandidaten komen van de server, want MusicBrainz eist een verzoek per
-// seconde per APPLICATIE en een User-Agent met contact -- allebei niet vanuit
-// een browser af te dwingen.
-//
-// WIJ KIEZEN NIET. Ook niet als er precies een treffer is: een verkeerd
-// geraden MBID zet jouw naam onder andermans werk. De knop staat er, de klik
-// is van de artiest.
-function wireMusicBrainz() {
- const knop = document.getElementById('mb-zoek-btn');
- const veld = document.getElementById('mb-q');
- const uit = document.getElementById('mb-uit');
- if (!knop || !veld || !uit || knop.__wired) return;
- knop.__wired = true;
-
- const el = (tag, cls, tekst) => {
- const e = document.createElement(tag);
- if (cls) e.className = cls;
- if (tekst != null) e.textContent = tekst; // textContent: nooit HTML uit een vreemd register
- return e;
- };
-
- async function zoek() {
- const q = (veld.value || '').trim();
- if (!q) return;
- uit.hidden = false;
- uit.replaceChildren(el('p', 'ax-hint', T.aaud_mb_busy || 'Zoeken…'));
- knop.disabled = true;
- try {
- const r = await fetch(`api/musicbrainz?q=${encodeURIComponent(q)}`, { credentials: 'same-origin' });
- const j = await r.json();
- toon((j && j.kandidaten) || []);
- } catch {
- uit.replaceChildren(el('p', 'ax-hint', T.aaud_mb_fail || 'MusicBrainz is even niet bereikbaar.'));
- } finally {
- knop.disabled = false;
- }
- }
-
- function toon(kandidaten) {
- if (!kandidaten.length) {
- uit.replaceChildren(el('p', 'ax-hint', T.aaud_mb_none || 'Niets gevonden.'));
- return;
- }
- const lijst = el('ul', 'mb-lijst');
- for (const k of kandidaten) {
- const li = el('li', 'mb-kandidaat');
- li.appendChild(el('strong', null, k.naam));
- // De toelichting is het hele punt: er zijn drie bands die Nirvana heten,
- // en zonder dit veld kiest iemand de verkeerde.
- const bij = [k.toelichting, k.soort, k.land, k.jaren].filter(Boolean).join(' · ');
- if (bij) li.appendChild(el('small', 'mb-bij', bij));
- const open = el('a', 'mb-open', T.aaud_mb_open || 'Bekijk op MusicBrainz');
- open.href = k.url; open.target = '_blank'; open.rel = 'noopener';
- li.appendChild(open);
- li.appendChild(kiesForm(k));
- lijst.appendChild(li);
- }
- uit.replaceChildren(lijst);
- }
-
- function kiesForm(k) {
- const f = document.createElement('form');
- f.method = 'POST';
- f.action = 'musicbrainz/link';
- for (const [naam, waarde] of [['_csrf', csrf()], ['mbid', k.mbid], ['naam', k.naam]]) {
- const i = document.createElement('input');
- i.type = 'hidden'; i.name = naam; i.value = waarde || '';
- f.appendChild(i);
- }
- const b = el('button', 'ax-btn', T.aaud_mb_pick || 'Dit ben ik');
- b.type = 'submit';
- f.appendChild(b);
- return f;
- }
-
- // Het token staat al in elk formulier op deze pagina; er is er geen apart voor.
- const csrf = () => (document.querySelector('input[name="_csrf"]') || {}).value || '';
-
- knop.addEventListener('click', zoek);
- veld.addEventListener('keydown', (e) => { if (e.key === 'Enter') { e.preventDefault(); zoek(); } });
-}
Index: src/assets/js/mod/admin-site-edit.js
===================================================================
--- src/assets/js/mod/admin-site-edit.js (revision 5300652c5fdc930a614f60c65e8abb4e56060f4f)
+++ src/assets/js/mod/admin-site-edit.js (revision 64016815dd12ec86d05866d6d71bca783024094d)
@@ -134,3 +134,99 @@
}
})();
+
+ // MusicBrainz-paneel (shaer-mbz). Binnen run(), niet op moduleniveau: deze
+ // modules krijgen bij elke paginawissel opnieuw init(), en een blok dat maar
+ // een keer per sessie draait is precies wat shaer-5s1 opleverde.
+ wireMusicBrainz();
}
+
+// ── MusicBrainz: ben jij dit? (shaer-mbz) ─────────────────────────
+//
+// De kandidaten komen van de server, want MusicBrainz eist een verzoek per
+// seconde per APPLICATIE en een User-Agent met contact -- allebei niet vanuit
+// een browser af te dwingen.
+//
+// WIJ KIEZEN NIET. Ook niet als er precies een treffer is: een verkeerd geraden
+// MBID zet jouw naam onder andermans werk. De knop staat er, de klik is van de
+// artiest. En de keuze gaat mee met de Opslaan-knop van de pagina -- een eigen
+// formulier kan hier niet, want deze pagina IS er een.
+function wireMusicBrainz() {
+ const knop = document.getElementById('mb-zoek-btn');
+ const veld = document.getElementById('mb-q');
+ const uit = document.getElementById('mb-uit');
+ const idVeld = document.getElementById('mb-id');
+ const naamVeld = document.getElementById('mb-naam');
+ const huidig = document.getElementById('mb-huidig');
+ if (!knop || !veld || !uit || !idVeld || knop.__wired) return;
+ knop.__wired = true;
+
+ const el = (tag, cls, tekst) => {
+ const e = document.createElement(tag);
+ if (cls) e.className = cls;
+ // textContent en nooit innerHTML: dit is tekst uit een vreemd register.
+ if (tekst != null) e.textContent = tekst;
+ return e;
+ };
+
+ function zet(mbid, naam) {
+ idVeld.value = mbid || '';
+ naamVeld.value = naam || '';
+ if (huidig) {
+ huidig.hidden = !mbid;
+ const n = document.getElementById('mb-huidig-naam');
+ const a = document.getElementById('mb-huidig-link');
+ if (n) n.textContent = naam || mbid || '';
+ if (a) a.href = mbid ? `https://musicbrainz.org/artist/${mbid}` : '#';
+ }
+ uit.hidden = true;
+ uit.replaceChildren();
+ }
+
+ const wis = document.getElementById('mb-wis');
+ if (wis) wis.addEventListener('click', () => zet('', ''));
+
+ async function zoek() {
+ const q = (veld.value || '').trim();
+ if (!q) return;
+ uit.hidden = false;
+ uit.replaceChildren(el('p', 'form-hint', T.asite_mb_busy || 'Zoeken…'));
+ knop.disabled = true;
+ try {
+ const r = await fetch(`api/musicbrainz?q=${encodeURIComponent(q)}`, { credentials: 'same-origin' });
+ const j = await r.json();
+ toon((j && j.kandidaten) || []);
+ } catch {
+ uit.replaceChildren(el('p', 'form-hint', T.asite_mb_fail || 'MusicBrainz is even niet bereikbaar.'));
+ } finally {
+ knop.disabled = false;
+ }
+ }
+
+ function toon(kandidaten) {
+ if (!kandidaten.length) {
+ uit.replaceChildren(el('p', 'form-hint', T.asite_mb_none || 'Niets gevonden.'));
+ return;
+ }
+ const lijst = el('ul', 'mb-lijst');
+ for (const k of kandidaten) {
+ const li = el('li', 'mb-kandidaat');
+ li.appendChild(el('strong', null, k.naam));
+ // De toelichting is het hele punt: er zijn drie bands die Nirvana heten,
+ // en zonder dit veld kiest iemand de verkeerde.
+ const bij = [k.toelichting, k.soort, k.land, k.jaren].filter(Boolean).join(' · ');
+ if (bij) li.appendChild(el('small', 'mb-bij', bij));
+ const open = el('a', 'mb-open', T.asite_mb_open || 'Bekijk op MusicBrainz');
+ open.href = k.url; open.target = '_blank'; open.rel = 'noopener';
+ li.appendChild(open);
+ const b = el('button', 'btn', T.asite_mb_pick || 'Dit ben ik');
+ b.type = 'button';
+ b.addEventListener('click', () => zet(k.mbid, k.naam));
+ li.appendChild(b);
+ lijst.appendChild(li);
+ }
+ uit.replaceChildren(lijst);
+ }
+
+ knop.addEventListener('click', zoek);
+ veld.addEventListener('keydown', (e) => { if (e.key === 'Enter') { e.preventDefault(); zoek(); } });
+}
Index: src/routes/admin-audio.js
===================================================================
--- src/routes/admin-audio.js (revision 5300652c5fdc930a614f60c65e8abb4e56060f4f)
+++ src/routes/admin-audio.js (revision 64016815dd12ec86d05866d6d71bca783024094d)
@@ -23,5 +23,4 @@
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,57 +383,4 @@
/** 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;
Index: src/routes/admin-sites.js
===================================================================
--- src/routes/admin-sites.js (revision 5300652c5fdc930a614f60c65e8abb4e56060f4f)
+++ src/routes/admin-sites.js (revision 64016815dd12ec86d05866d6d71bca783024094d)
@@ -27,4 +27,5 @@
import { mediaDir } from '../config/paths.js';
import AP from '../services/ActivityPubService.js';
+import MusicBrainz from '../services/MusicBrainzService.js';
@@ -322,4 +323,23 @@
// ==================== 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,4 +366,10 @@
// 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,4 +381,5 @@
profile_links = ?,
ap_aliases = ?,
+ mb_artist_id = ?, mb_artist_name = ?,
is_public = ?, robots_index = ?, require_login_to_comment = ?,
enable_audio_player = ?,
@@ -374,4 +401,6 @@
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,
Index: src/services/i18n.js
===================================================================
--- src/services/i18n.js (revision 5300652c5fdc930a614f60c65e8abb4e56060f4f)
+++ src/services/i18n.js (revision 64016815dd12ec86d05866d6d71bca783024094d)
@@ -360,14 +360,15 @@
'aaud.fedi_off': 'Niet op de fediverse gedeeld (alleen webspeler, bestand verborgen) — klik om te delen',
'aaud.embed_player': 'Embedbare speler',
- 'aaud.mb_title': 'Ben jij dit in MusicBrainz?',
- 'aaud.mb_hint': 'MusicBrainz is het open register waar muziekdiensten je aan herkennen. Zoek jezelf op en kies je eigen pagina — dan weten anderen dat jouw muziek van jou is.',
- 'aaud.mb_search': 'Zoeken',
- 'aaud.mb_none': 'Niets gevonden. Sta je er nog niet in? Dan kun je jezelf aanmelden op musicbrainz.org — dat kan alleen daar, niet vanuit Klonkt.',
- 'aaud.mb_pick': 'Dit ben ik',
- 'aaud.mb_linked': 'Gekoppeld aan',
- 'aaud.mb_unlink': 'Ontkoppelen',
- 'aaud.mb_open': 'Bekijk op MusicBrainz',
- 'aaud.mb_busy': 'Zoeken…',
- 'aaud.mb_fail': 'MusicBrainz is even niet bereikbaar.',
+ 'asite.mb_title': 'Ben jij dit in MusicBrainz?',
+ 'asite.mb_hint': 'MusicBrainz is het open register waar muziekdiensten je aan herkennen. Zoek jezelf op en kies je eigen pagina — dan weten anderen dat jouw muziek van jou is.',
+ 'asite.mb_search': 'Zoeken',
+ 'asite.mb_placeholder': 'Je artiestennaam',
+ 'asite.mb_none': 'Niets gevonden. Sta je er nog niet in? Dan kun je jezelf aanmelden op musicbrainz.org — dat kan alleen daar, niet vanuit Klonkt.',
+ 'asite.mb_pick': 'Dit ben ik',
+ 'asite.mb_linked': 'Gekoppeld aan',
+ 'asite.mb_unlink': 'Ontkoppelen',
+ 'asite.mb_open': 'Bekijk op MusicBrainz',
+ 'asite.mb_busy': 'Zoeken…',
+ 'asite.mb_fail': 'MusicBrainz is even niet bereikbaar.',
'aaud.embed_hint': 'Plak deze code op je eigen website/blog om je muziek met deze speler in te sluiten:',
'aaud.preview_player': 'Speler-voorbeeld openen',
@@ -1317,14 +1318,15 @@
'aaud.fedi_off': 'Not shared on the fediverse (web player only, file hidden) — click to share',
'aaud.embed_player': 'Embeddable player',
- 'aaud.mb_title': 'Is this you on MusicBrainz?',
- 'aaud.mb_hint': 'MusicBrainz is the open register music services recognise you by. Look yourself up and pick your own page — then others know your music is yours.',
- 'aaud.mb_search': 'Search',
- 'aaud.mb_none': 'Nothing found. Not in there yet? You can add yourself on musicbrainz.org — that can only be done there, not from Klonkt.',
- 'aaud.mb_pick': 'This is me',
- 'aaud.mb_linked': 'Linked to',
- 'aaud.mb_unlink': 'Unlink',
- 'aaud.mb_open': 'View on MusicBrainz',
- 'aaud.mb_busy': 'Searching…',
- 'aaud.mb_fail': 'MusicBrainz is unreachable right now.',
+ 'asite.mb_title': 'Is this you on MusicBrainz?',
+ 'asite.mb_hint': 'MusicBrainz is the open register music services recognise you by. Look yourself up and pick your own page — then others know your music is yours.',
+ 'asite.mb_search': 'Search',
+ 'asite.mb_placeholder': 'Your artist name',
+ 'asite.mb_none': 'Nothing found. Not in there yet? You can add yourself on musicbrainz.org — that can only be done there, not from Klonkt.',
+ 'asite.mb_pick': 'This is me',
+ 'asite.mb_linked': 'Linked to',
+ 'asite.mb_unlink': 'Unlink',
+ 'asite.mb_open': 'View on MusicBrainz',
+ 'asite.mb_busy': 'Searching…',
+ 'asite.mb_fail': 'MusicBrainz is unreachable right now.',
'aaud.embed_hint': 'Paste this code on your own website/blog to embed your music with this player:',
'aaud.preview_player': 'Open player preview',
@@ -2273,14 +2275,15 @@
'aaud.fedi_off': 'Nicht im Fediverse geteilt (nur Web-Player, Datei verborgen) — zum Teilen klicken',
'aaud.embed_player': 'Einbettbarer Player',
- 'aaud.mb_title': 'Bist das du bei MusicBrainz?',
- 'aaud.mb_hint': 'MusicBrainz ist das offene Register, an dem Musikdienste dich erkennen. Suche dich und wähle deine eigene Seite — dann wissen andere, dass deine Musik von dir ist.',
- 'aaud.mb_search': 'Suchen',
- 'aaud.mb_none': 'Nichts gefunden. Noch nicht dabei? Du kannst dich auf musicbrainz.org eintragen — das geht nur dort, nicht aus Klonkt heraus.',
- 'aaud.mb_pick': 'Das bin ich',
- 'aaud.mb_linked': 'Verknüpft mit',
- 'aaud.mb_unlink': 'Trennen',
- 'aaud.mb_open': 'Bei MusicBrainz ansehen',
- 'aaud.mb_busy': 'Suche läuft…',
- 'aaud.mb_fail': 'MusicBrainz ist gerade nicht erreichbar.',
+ 'asite.mb_title': 'Bist das du bei MusicBrainz?',
+ 'asite.mb_hint': 'MusicBrainz ist das offene Register, an dem Musikdienste dich erkennen. Suche dich und wähle deine eigene Seite — dann wissen andere, dass deine Musik von dir ist.',
+ 'asite.mb_search': 'Suchen',
+ 'asite.mb_placeholder': 'Dein Künstlername',
+ 'asite.mb_none': 'Nichts gefunden. Noch nicht dabei? Du kannst dich auf musicbrainz.org eintragen — das geht nur dort, nicht aus Klonkt heraus.',
+ 'asite.mb_pick': 'Das bin ich',
+ 'asite.mb_linked': 'Verknüpft mit',
+ 'asite.mb_unlink': 'Trennen',
+ 'asite.mb_open': 'Bei MusicBrainz ansehen',
+ 'asite.mb_busy': 'Suche läuft…',
+ 'asite.mb_fail': 'MusicBrainz ist gerade nicht erreichbar.',
'aaud.embed_hint': 'Füge diesen Code auf deiner eigenen Website/deinem Blog ein, um deine Musik mit diesem Player einzubetten:',
'aaud.preview_player': 'Player-Vorschau öffnen',
Index: src/views/pages/admin-audio.ejs
===================================================================
--- src/views/pages/admin-audio.ejs (revision 5300652c5fdc930a614f60c65e8abb4e56060f4f)
+++ src/views/pages/admin-audio.ejs (revision 64016815dd12ec86d05866d6d71bca783024094d)
@@ -12,36 +12,4 @@
<% if (success) { %>
<%= success %>
<% } %>
<% if (error) { %>
<%= error %>
<% } %>
-
- <%# ── MUSICBRAINZ ─────────────────────────────────────────────
- Wie ben jij, buiten Klonkt? Een MBID is geen vocabulaire maar een
- register: eraan koppelen is als je ISBN noemen. Andere muziekdiensten
- -- Funkwhale voorop -- herkennen je eraan zonder dat wij hun model
- hoeven over te nemen (shaer-mbz). %>
-
-
- <%# Ontkoppelen moet kunnen: een verkeerde koppeling zet jouw naam onder
- andermans werk, en dat hoort met een klik terug te draaien zijn. %>
-
- <% } %>
-
-
-
-
-
-
-
<%# ── UPLOAD ──────────────────────────────────────────────── %>
Index: src/views/pages/admin-site-edit.ejs
===================================================================
--- src/views/pages/admin-site-edit.ejs (revision 5300652c5fdc930a614f60c65e8abb4e56060f4f)
+++ src/views/pages/admin-site-edit.ejs (revision 64016815dd12ec86d05866d6d71bca783024094d)
@@ -190,4 +190,36 @@
+ <%# ── MUSICBRAINZ ─────────────────────────────────────────────
+ Naast de aliassen, want het is dezelfde vraag: wie ben jij elders?
+ Een MBID is geen vocabulaire maar een register -- eraan koppelen is als
+ je ISBN noemen, en muziekdiensten herkennen je eraan zonder dat wij hun
+ model hoeven over te nemen (shaer-mbz).
+
+ GEEN EIGEN FORMULIER: deze pagina IS er een, en een formulier in een
+ formulier bestaat niet in HTML. De keuze zet een verborgen veld en gaat
+ mee met de Opslaan-knop van de pagina, net als de profielfoto. %>
+
+