Changeset d8f7d9c in Klonkt for src/services/PlaylistService.js

Timestamp:

08/16/2026 01:44:46 PM (3 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

d68ee3a

Parents:

21257e4

Message:

Playlists: uitgavedatum en MusicBrainz release-id op een album

Stap 1 van shaer-756s. Het album bestaat al als ding -- een playlist met
kind='album', met eigen id, titel, artiest, jaar, hoes en een eigen
AP-collectie. Wat ontbrak om het als Album uit te geven zijn twee velden.

release_date en niet year: die kolom blijft en wordt gewoon getoond,
maar hun AlbumSerializer leest released als een DateField. Een jaartal
als 2024-01-01 versturen is een dag verzinnen, en dat is precies wat we
bij artiesten en albums niet doen. Volledige datum of niets -- ook
2024-02-31 valt af, want Date rolt die stil door naar 2 maart en dan
bewaren we iets anders dan er ingetypt is.

mb_release_id is de tegenhanger van sites.mb_artist_id: dezelfde
verwijzing naar MusicBrainz, een niveau lager.

ALLEEN BIJ kind='album', en dat wordt in PlaylistService afgedwongen en
niet alleen in het scherm. De API ligt open -- de post-editor gebruikt
hem ook -- en een scherm is geen bewaking. Omschakelen naar afspeellijst
maakt de velden ALTIJD leeg, ook als de aanroeper er niets over zei:
anders houdt een album dat je tot mixtape ombouwt zijn uitgavedatum, en
duikt die weer op zodra iemand hem terugzet. Het leespad geeft ze ook
niet door bij een afspeellijst, zodat een oude rij niet alsnog lekt.

In de editor staan de velden onder Type en verdwijnen ze als je naar
playlist schakelt. Daar hoorde een CSS-regel bij: .pl-field is display:
flex en dat wint van het hidden-attribuut -- zonder .pl-field[hidden]
{ display: none } bleven ze gewoon staan.

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

File:

• src/services/PlaylistService.js (modified) (10 diffs)

src/services/PlaylistService.js

@@ -15 +15 @@
 import { v4 as uuid } from 'uuid';
 
+/**
+ * Een volledige datum of niets (shaer-756s).
+ *
+ * STRIKT, en dat is de hele functie. `year` bestaat al en blijft; dit veld
+ * bestaat juist omdat een jaartal geen uitgavedatum is. Zou hij "2024"
+ * doorlaten en er 2024-01-01 van maken, dan stond er straks een dag op de
+ * federatie die niemand ooit heeft ingevoerd -- en dan hadden we het veld net
+ * zo goed niet kunnen toevoegen.
+ *
+ * Ook 2024-02-31 valt af: dat is geen strengheid om de strengheid, Date rolt
+ * hem stilletjes door naar 2 maart en dan slaan we iets anders op dan er
+ * ingetypt is.
+ */
+function normDatum(v) {
+  const s = String(v == null ? '' : v).trim();
+  if (!s) return null;
+  if (!/^\d{4}-\d{2}-\d{2}$/.test(s)) return null;
+  const d = new Date(`${s}T00:00:00Z`);
+  return Number.isNaN(d.getTime()) || d.toISOString().slice(0, 10) !== s ? null : s;
+}
+
+/** Een MusicBrainz-id of niets. Zelfde vorm als sites.mb_artist_id. */
+function normMbid(v) {
+  const s = String(v == null ? '' : v).trim().toLowerCase();
+  return /^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/.test(s) ? s : null;
+}
+
 class PlaylistService {
 
@@ -52 +79 @@
     const rows = db.prepare(`
       SELECT p.id, p.title, p.artist, p.year, p.cover_url, p.kind,
+             p.release_date, p.mb_release_id,
              p.created_at, p.updated_at,
              (SELECT COUNT(*) FROM playlist_tracks WHERE playlist_id = p.id) AS track_count
@@ -65 +93 @@
       cover: r.cover_url || '',
       kind: r.kind || 'album',
+      release_date: r.release_date || '',
+      mb_release_id: r.mb_release_id || '',
       track_count: r.track_count,
       created_at: r.created_at,
@@ -86 +116 @@
     if (!id) return null;
     const p = db.prepare(`
-      SELECT id, title, artist, year, cover_url, kind, created_at, updated_at
+      SELECT id, title, artist, year, cover_url, kind, release_date, mb_release_id, created_at, updated_at
       FROM playlists WHERE site_id = ? AND id = ?
     `).get(siteId, id);
@@ -125 +155 @@
       cover: p.cover_url || fallbackCover,
       kind: (p.kind === 'playlist') ? 'playlist' : 'album',
+      // Leeg als het een afspeellijst is -- de opslag houdt ze daar al leeg,
+      // maar dit is de plek waar de editor leest en die mag niet afhangen van
+      // wat er toevallig in de kolom stond.
+      release_date: p.kind === 'playlist' ? '' : (p.release_date || ''),
+      mb_release_id: p.kind === 'playlist' ? '' : (p.mb_release_id || ''),
       tracks: mappedTracks,
     };
@@ -141 +176 @@
     const kind = data.kind === 'playlist' ? 'playlist' : 'album';
 
+    // Alleen een UITGAVE draagt deze twee. Een afspeellijst heeft geen
+    // uitgavedatum en geen release-id, en dat onderscheid is precies wat de
+    // keuze album/playlist betekent (shaer-cyg). Het afdwingen gebeurt HIER en
+    // niet alleen in het scherm: een scherm kun je omzeilen -- de API ligt open
+    // voor de post-editor -- en dan staat er stille rommel op een mixtape die
+    // later als Album de deur uit gaat.
+    const uitgave = kind === 'album';
+    const releaseDate = uitgave ? normDatum(data.release_date) : null;
+    const mbRelease = uitgave ? normMbid(data.mb_release_id) : null;
+
     const tx = db.transaction(() => {
       db.prepare(`
-        INSERT INTO playlists (id, site_id, title, artist, year, cover_url, kind, created_at, updated_at)
-        VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?)
+        INSERT INTO playlists (id, site_id, title, artist, year, cover_url, kind, release_date, mb_release_id, created_at, updated_at)
+        VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)
       `).run(
         id, siteId, title,
@@ -150 +195 @@
         Number.isFinite(+data.year) && +data.year > 0 ? +data.year : null,
         String(data.cover || '').trim() || null,
-        kind, now, now,
+        kind, releaseDate, mbRelease, now, now,
       );
       this._writeTracks(id, siteId, data.tracks);
@@ -171 +216 @@
     if (!id) return false;
     const existing = db.prepare(
-      'SELECT id FROM playlists WHERE site_id = ? AND id = ?'
+      'SELECT id, kind FROM playlists WHERE site_id = ? AND id = ?'
     ).get(siteId, id);
     if (!existing) return false;
@@ -177 +222 @@
     const fields = [];
     const values = [];
+    // Wat wordt het NA deze wijziging? `kind` hoeft niet in data te staan, dus
+    // val terug op wat er ligt.
+    const nieuwKind = Object.prototype.hasOwnProperty.call(data, 'kind')
+      ? (data.kind === 'playlist' ? 'playlist' : 'album')
+      : (existing.kind || 'album');
     if (Object.prototype.hasOwnProperty.call(data, 'title')) {
       const v = String(data.title || '').trim();
@@ -193 +243 @@
     }
     if (Object.prototype.hasOwnProperty.call(data, 'kind')) {
-      fields.push('kind = ?'); values.push(data.kind === 'playlist' ? 'playlist' : 'album');
+      fields.push('kind = ?'); values.push(nieuwKind);
+    }
+    // De uitgavevelden. Wordt dit een afspeellijst, dan gaan ze ALTIJD leeg --
+    // ook als de aanroeper er niets over zei. Anders houdt een album dat je tot
+    // mixtape ombouwt zijn uitgavedatum en zijn release-id, en die duiken dan
+    // weer op zodra iemand hem terugzet. Een veld dat niet meer mag bestaan
+    // hoort weg te zijn, niet te wachten.
+    if (nieuwKind !== 'album') {
+      fields.push('release_date = ?'); values.push(null);
+      fields.push('mb_release_id = ?'); values.push(null);
+    } else {
+      if (Object.prototype.hasOwnProperty.call(data, 'release_date')) {
+        fields.push('release_date = ?'); values.push(normDatum(data.release_date));
+      }
+      if (Object.prototype.hasOwnProperty.call(data, 'mb_release_id')) {
+        fields.push('mb_release_id = ?'); values.push(normMbid(data.mb_release_id));
+      }
     }
     fields.push('updated_at = ?'); values.push(new Date().toISOString());