Changeset 9026de5 in Klonkt for src/services/music/index.js

Timestamp:

08/16/2026 02:25:04 PM (3 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

749605a

Parents:

7090fd2

Message:

Album: published en released lenen de datum van de uitgavepost

Robins voorstel, en mijn eerste antwoord was te voorzichtig. Ik wilde
released leeg laten omdat een plaat uit 2018 die je vandaag post dan
zou beweren vandaag uit te komen. Dat gebeurt ook -- maar bij de meeste
Klonkt-sites IS de post het uitbrengen, en GEEN datum is slechter dan
een datum die op het gewone geval klopt. Het handmatige veld is precies
het gereedschap voor de uitzondering, en dat wint altijd.

year vult nog steeds niets aan, en dat is geen inconsequentie: een
jaartal is geen dag, terwijl de postdatum een gebeurtenis is die
werkelijk heeft plaatsgevonden. Verzinnen versus afleiden.

GEPLANDE POSTS, Robins tweede vraag. Het pad klopt op alle drie de
schakels: zolang de post 'scheduled' is vindt uitgavePost hem niet (die
filtert op status='published') en heeft de plaat dus nog geen
uitgavedatum -- hij is immers nog niet uit. De Scheduler zet daarna
published_at op COALESCE(published_at, publish_at, CURRENT_TIMESTAMP),
oftewel op de GEPLANDE tijd, en wij lezen COALESCE(published_at,
created_at). Een server die een uur plat lag levert dus geen uur te late
uitgavedatum op, en created_at wint nooit bij een gepubliceerde post.
Er staat nu een test die precies dat afloopt, met een worker die expres
te laat draait.

uitgavePost wordt in buildPlaylistCollection nog EEN keer aangeroepen en
twee keer gebruikt: het Album leent er zijn datums en titel van,
leenVanPost zijn tekst en tags. Twee losse aanroepen zouden uiteen kunnen
lopen, en dan staat er weer iets anders op het ingesloten object dan op
zijn eigen URI -- de fout van een uur geleden.

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

File:

• src/services/music/index.js (modified) (5 diffs)

src/services/music/index.js

@@ -485 +485 @@
   if (!pl) return null;
   const abs = (u) => !u ? null : (/^https?:/i.test(u) ? u : `${base}${u.startsWith('/') ? '' : '/'}${u}`);
-  // GEEN epoch als terugval. `published` is bij hen verplicht, maar 1970 is een
-  // ANTWOORD en geen ontbrekend veld -- en dat is erger: een lezer kan een gat
-  // opmerken, een leugen niet. playlists.created_at heeft een default, dus als
-  // hij hier ontbreekt is er een leespad dat de kolom laat vallen. Dat willen we
-  // zien, niet maskeren. (Zo kwam op 16-8 de route boven water die id, title,
-  // artist, year, cover_url en kind selecteerde en de rest niet.)
-  const wanneer = pl.created_at ? new Date(pl.created_at).toISOString() : null;
+  // WANNEER IS DEZE PLAAT GEPUBLICEERD. De post die hem uitbrengt gaat voor, en
+  // niet als noodgreep maar omdat hij het beter weet: playlists.created_at is
+  // het moment waarop de RIJ is aangemaakt, en dat kan weken eerder zijn terwijl
+  // je nog aan het samenstellen was. AS2 `published` vraagt wanneer het object
+  // openbaar werd, en dat is de post.
+  //
+  // GEEN epoch als laatste terugval. `published` is bij hen verplicht, maar 1970
+  // is een ANTWOORD en geen ontbrekend veld -- en dat is erger: een lezer kan een
+  // gat opmerken, een leugen niet. Zo kwam op 16-8 de route boven water die id,
+  // title, artist, year, cover_url en kind selecteerde en de rest niet.
+  //
+  // OOK VOOR `released`, en daar had ik het eerst mis (Robin, 16-8). Mijn
+  // bezwaar was: post je vandaag een plaat uit 2018, dan beweert dit dat hij
+  // vandaag uitkwam. Dat gebeurt ook -- maar bij de meeste Klonkt-sites IS de
+  // post het uitbrengen, en GEEN datum is slechter dan een datum die op het
+  // gewone geval klopt. Het handmatige veld is precies het gereedschap voor de
+  // uitzondering: bij een heruitgave vul je hem in en die wint.
+  const postDatum = (pl._post && pl._post.uit_wanneer) ? new Date(pl._post.uit_wanneer) : null;
+  const wanneer = postDatum ? postDatum.toISOString()
+    : (pl.created_at ? new Date(pl.created_at).toISOString() : null);
   // Dezelfde lening als in buildPlaylistCollection: de post die de plaat
   // uitbrengt geeft zijn titel, en de eigen titel blijft als alsoKnownAs staan.
@@ -504 +517 @@
   };
   if (titel !== pl.title) album.alsoKnownAs = pl.title;
-  // `released` alleen als er een ECHTE datum is. `year` vult hem niet aan: een
-  // jaartal is geen dag, en dat is de reden dat release_date bestaat.
+  // Het ingevulde veld wint altijd; anders de DAG waarop de post verscheen.
+  // `year` vult hem nog steeds niet aan, en dat is geen inconsequentie: een
+  // jaartal is geen dag, terwijl de postdatum een gebeurtenis is die werkelijk
+  // heeft plaatsgevonden. Het verschil is verzinnen versus afleiden.
   if (pl.release_date) album.released = pl.release_date;
+  else if (postDatum) album.released = postDatum.toISOString().slice(0, 10);
   if (pl.mb_release_id) album.musicbrainzId = pl.mb_release_id;
   const hoes = abs(pl.cover_url || null);
@@ -522 +538 @@
   const hostPosts = site.id ? trackHostPosts(site.id) : null;
   const albums = site.id ? trackAlbums(site.id) : null;
+  // EEN keer opzoeken en twee keer gebruiken: het Album leent er zijn datums en
+  // titel van, leenVanPost onderaan zijn tekst en tags. Twee losse aanroepen
+  // zouden niet alleen dubbel werk zijn maar ook uiteen kunnen lopen -- en dan
+  // staat er weer iets anders op het ingesloten object dan op zijn eigen URI.
+  const post = site.id ? uitgavePost(site.id, playlist.id) : null;
   const items = (rows || []).map((r) => buildTrackAudio(base, site, r, { coverFallback: playlist.cover_url || null, hostPosts, albums }));
   const out = pagedCollection(`${actorId(base, site.slug)}/playlists/${playlist.id}`, items, {
@@ -547 +568 @@
   // staat toch al ingesloten op de track.
   if ((playlist.kind || 'album') === 'album') {
-    const album = buildAlbumObject(base, site, playlist);
+    const album = buildAlbumObject(base, site, { ...playlist, _post: post });
     for (const veld of ['published', 'released', 'musicbrainzId', 'artist_credit', 'image']) {
       if (album[veld] !== undefined) out[veld] = album[veld];
     }
   }
-  return leenVanPost(base, site, out, uitgavePost(site.id, playlist.id));
+  return leenVanPost(base, site, out, post);
 }
 
@@ -577 +598 @@
   try {
     const rijen = db.prepare(`
-      SELECT id, slug, title, excerpt, content, cover_image_url, tags
+      SELECT id, slug, title, excerpt, content, cover_image_url, tags,
+             COALESCE(published_at, created_at) AS uit_wanneer
       FROM posts
       WHERE site_id = ? AND status = 'published'