Changeset d8f7d9c in Klonkt for src/assets/js/mod/playlist-editor.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/assets/js/mod/playlist-editor.js (modified) (4 diffs)

src/assets/js/mod/playlist-editor.js

@@ -63 +63 @@
 
     // Existing playlist data when editing
-    let initial = { title: '', artist: '', year: '', cover: '', kind: 'album', track_ids: [] };
+    let initial = { title: '', artist: '', year: '', cover: '', kind: 'album', release_date: '', mb_release_id: '', track_ids: [] };
     if (isEdit && opts.id) {
       try {
@@ -102 +102 @@
                     <option value="playlist" ${initial.kind === 'playlist' ? 'selected' : ''}>📃 Playlist (track-covers)</option>
                   </select>
+                </label>
+                <label class="pl-field pl-uitgave">
+                  <span>Uitgavedatum</span>
+                  <input type="date" id="pli-release-date" value="${esc(initial.release_date || '')}">
+                </label>
+                <label class="pl-field pl-uitgave">
+                  <span>MusicBrainz release-id</span>
+                  <input type="text" id="pli-mb-release" maxlength="36" spellcheck="false"
+                         placeholder="00000000-0000-0000-0000-000000000000"
+                         value="${esc(initial.mb_release_id || '')}">
                 </label>
                 <div class="pl-field pl-field-full">
@@ -157 +167 @@
     const yearEl = $('#pli-year');
     const kindEl = $('#pli-kind');
+    const releaseEl = $('#pli-release-date');
+    const mbReleaseEl = $('#pli-mb-release');
     const coverEl = $('#pli-cover');
+
+    // Uitgavedatum en release-id horen bij een ALBUM, niet bij een
+    // afspeellijst -- dat is wat de keuze album/playlist betekent. Ze
+    // verdwijnen dus als je omschakelt, en de server maakt ze dan ook leeg;
+    // dit scherm is de uitleg, niet de bewaking.
+    const toonUitgave = () => {
+      const album = kindEl.value !== 'playlist';
+      for (const el of backdrop.querySelectorAll('.pl-uitgave')) el.hidden = !album;
+    };
+    kindEl.addEventListener('change', toonUitgave);
+    toonUitgave();
     const coverThumb = $('#pli-cover-thumb');
     const saveBtn = $('#pli-save');
@@ -376 +399 @@
         cover:  coverEl.value.trim(),
         kind:   kindEl.value === 'playlist' ? 'playlist' : 'album',
+        release_date:  releaseEl.value.trim(),
+        mb_release_id: mbReleaseEl.value.trim(),
         tracks: selected.slice(),
       };