Changes in src/services/PlaylistService.js [7090fd2:834bcc3] in Klonkt

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 {
 
@@ -79 +52 @@
     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
@@ -93 +65 @@
       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,
@@ -116 +86 @@
     if (!id) return null;
     const p = db.prepare(`
-      SELECT id, title, artist, year, cover_url, kind, release_date, mb_release_id, created_at, updated_at
+      SELECT id, title, artist, year, cover_url, kind, created_at, updated_at
       FROM playlists WHERE site_id = ? AND id = ?
     `).get(siteId, id);
@@ -155 +125 @@
       cover: p.cover_url || fallbackCover,
       kind: (p.kind === 'playlist') ? 'playlist' : 'album',
-      // created_at hoort erbij omdat de AP-kant er `published` van maakt. Zonder
-      // dit veld viel buildAlbumObject terug op 1970, en dat stond op 16-8
-      // gewoon op de federatie.
-      created_at: p.created_at,
-      // 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,
     };
@@ -180 +141 @@
     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, release_date, mb_release_id, created_at, updated_at)
-        VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)
+        INSERT INTO playlists (id, site_id, title, artist, year, cover_url, kind, created_at, updated_at)
+        VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?)
       `).run(
         id, siteId, title,
@@ -199 +150 @@
         Number.isFinite(+data.year) && +data.year > 0 ? +data.year : null,
         String(data.cover || '').trim() || null,
-        kind, releaseDate, mbRelease, now, now,
+        kind, now, now,
       );
       this._writeTracks(id, siteId, data.tracks);
@@ -220 +171 @@
     if (!id) return false;
     const existing = db.prepare(
-      'SELECT id, kind FROM playlists WHERE site_id = ? AND id = ?'
+      'SELECT id FROM playlists WHERE site_id = ? AND id = ?'
     ).get(siteId, id);
     if (!existing) return false;
@@ -226 +177 @@
     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();
@@ -247 +193 @@
     }
     if (Object.prototype.hasOwnProperty.call(data, 'kind')) {
-      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('kind = ?'); values.push(data.kind === 'playlist' ? 'playlist' : 'album');
     }
     fields.push('updated_at = ?'); values.push(new Date().toISOString());