Changeset 8e1af9c in Klonkt for src/services/AudioEmbedService.js

Timestamp:

08/22/2026 09:11:56 AM (3 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

16fd4fe

Parents:

25dd08f

git-author:

Robin <roboburr@…> (08/22/2026 09:11:36 AM)

git-committer:

Robin <roboburr@…> (08/22/2026 09:11:56 AM)

Message:

Mixtape als posttype, kiesbaar in de composer

Het bandje bestond wel als soort playlist, maar je kon er geen post van maken.
Nu wel: een eigen knop met een eigen teken, en het muziekpaneel deelt hij met
album en playlist -- het verschil zit in de playlist die je insluit, niet in
wat je uploadt.

KEUZE_TYPES en MUZIEK_TYPES in config/post-types.js waren de goede plek; dat
bestand bestaat precies omdat die lijst eerder drie keer los rondslingerde.

Nog twee keer dezelfde binaire vorm opgeruimd, en de tweede was een echte:

• VOLGBAAR kende geen mixtape, dus het type volgde de muziek niet.
• De editor zocht de soort op met de vorm p.kind === 'playlist' ? 'playlist' : 'album'. Het scherm zag dus album terwijl de server mixtape opsloeg: het type verspringt onder je handen bij het bewaren.
• De renderer koos teken en woord met dezelfde tweewegkeuze, waardoor een ingesloten mixtape het jasje van een album droeg.

De SPELER is er nog niet -- een mixtape rendert voorlopig als de gewone lijst.
Wat af is, is dat hij overal zichzelf noemt.

Zes tests, waaronder de opslagweg door de echte route: kent de opslag een type
niet, dan wordt de post zonder melding een gewone post, en dat is precies de
fout waar config/post-types.js voor waarschuwt. Een van de tests riep eerst een
methode aan die niet bestaat en sloeg zichzelf over; die gaat nu door
embedPlaylistShortcodes heen en valt om als je de tweewegkeuze terugzet.

MOD_V naar 51.

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

File:

• src/services/AudioEmbedService.js (modified) (3 diffs)

src/services/AudioEmbedService.js

@@ -10 +10 @@
  * </figure>
  */
+
+// Dezelfde lijst soorten als de server en de editor gebruiken. Zie
+// assets/js/shared/post-music-type.js: die module is puur, dus hij mag hier.
+import { SOORTEN } from '../assets/js/shared/post-music-type.js';
 
 // "Open in" icons (brand-colored via CSS .pat-link--).
@@ -538 +542 @@
 
       const albumDomId = 'album-' + id;
-      const kind = (pl.kind === 'playlist') ? 'playlist' : 'album';
-      const kindLabel = kind === 'playlist' ? '📃 Playlist' : '💿 Album';
+      // De soort zoals hij is opgeslagen. Stond hier als ternair met twee
+      // uitkomsten, en dan draagt een mixtape het jasje en het woord van een
+      // album -- dezelfde vorm die op vier andere plekken al misging.
+      // DE SPELER IS ER NOG NIET. Een mixtape rendert voorlopig als de gewone
+      // lijst; wat hier af is, is dat hij zichzelf goed noemt.
+      const kind = SOORTEN.includes(pl.kind) ? pl.kind : 'album';
+      const KIND_LABEL = { album: '💿 Album', playlist: '📃 Playlist', mixtape: '📼 Mixtape' };
+      const kindLabel = KIND_LABEL[kind] || KIND_LABEL.album;
       const titleH  = this.escape(pl.title || 'Naamloos');
       const artistH = this.escape(pl.artist || '');
@@ -638 +648 @@
             data-pcms-track-url="${firstUrl}"
             data-pcms-album-id="${albumDomId}"
-            aria-label="Speel ${kind === 'playlist' ? 'playlist' : 'album'}">
+            aria-label="Speel ${kind}">
       ${coverH
         ? `<span class="post-album-cover" style="background-image:url('${coverH}')"></span>`