Changeset 0beafed in Klonkt

Timestamp:

06/19/2026 07:09:11 AM (3 months ago)

Author:

roboburr <roboburr@…>

Branches:

main

Children:

6b8afb1

Parents:

84325b8

Message:

fix: site-wide audio player never received tracks (loadAudioTracks selected m.url, which doesn't exist)

The media table has no url column (verified on demo + sound-fabrics with 91 tracks),
so the query "SELECT m.url AS media_url" always failed silently -> res.locals.audioTracks=[]
-> PCMS_SITE_TRACKS empty -> the persistent site player was blank everywhere. Now
selecting m.filename and building the stream URL via audioUrl(filename), as admin-audio
does. Also fixes the source for the embeddable player (#7).

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

File:

• src/middleware/site.js (modified) (3 diffs)

src/middleware/site.js

@@ -11 +11 @@
 import db from '../config/database.js';
 import { getTenancy } from '../services/SettingsService.js';
+import { audioUrl } from '../services/AudioStreamService.js';
 
 /**
@@ -76 +77 @@
 
   try {
-    res.locals.audioTracks = db.prepare(`
-      SELECT t.id, t.title, t.artist, t.duration, t.position,
-             m.url AS media_url
+    // m.filename = de kale bestandsnaam; de speelbare URL is de gated stream-route
+    // (audioUrl). De media-tabel heeft GEEN url-kolom — de oude query selecteerde
+    // m.url en faalde dus altijd stil (lege speler). Nu bouwen we de URL uit filename.
+    const rows = db.prepare(`
+      SELECT t.id, t.title, t.artist, t.duration, t.position, m.filename
       FROM audio_tracks t
       LEFT JOIN media m ON m.id = t.media_id
@@ -84 +87 @@
       ORDER BY t.position ASC, t.created_at ASC
     `).all(site.id);
+    res.locals.audioTracks = rows.map((r) => ({
+      id: r.id, title: r.title, artist: r.artist, duration: r.duration, position: r.position,
+      media_url: r.filename ? audioUrl(r.filename) : null,
+    }));
   } catch (e) {
     // media table might not be queryable in some test setups — fall back gracefully