Changeset e854ace in Klonkt for src/services/AudioEmbedService.js

Timestamp:

08/17/2026 06:22:36 PM (3 weeks ago)

Author:

Bart <bart@…>

Branches:

main

Children:

acbc9fc

Parents:

995b100

Message:

Een Apple Music-afspeellijst is geen album: het id is geen getal

Barts melding (17-8): het concept "The Mixtape" op boiert.eu toonde in preview
letterlijk [[embed:https://music.apple.com/nl/playlist/romance/pl.u-...]].

Twee fouten achter elkaar. detectProvider herkent Apple Music aan /playlist/ en
kijkt niet verder, maar applemusicIframe eiste [0-9]+ als id. Dat klopt voor een
album of een nummer; een afspeellijst heet pl.u-LdbqzVvI3go5g. Dus null, en
embedMediaShortcodes viel terug op de shortcode zelf.

Die terugval is de tweede fout, en hij maakte de eerste onbegrijpelijk. Een
lezer ziet geen "dit kan ik niet", hij ziet rommel op de pagina. Onherkend gaf
al netjes een melding; herkend-maar-niet-te-bouwen geeft nu dezelfde, want voor
de lezer is het hetzelfde geval.

De vangst blijft krap -- geen slash, vraagteken of hekje -- want hij gaat
rechtstreeks achter embed.music.apple.com/ aan. Een test loopt drie
ontsnappingspogingen langs.

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

File:

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

src/services/AudioEmbedService.js

@@ -239 +239 @@
 
   static applemusicIframe({ url }) {
-    const match = url.match(/music\.apple\.com\/([a-z]{2}\/(?:album|playlist|song)\/[^/?#]+\/[0-9]+)/i);
+    // Een album of nummer heeft een NUMMER als id, een afspeellijst niet: die
+    // heet `pl.u-LdbqzVvI3go5g`. Met alleen [0-9]+ viel elke playlist hier af
+    // en gaf deze functie null -- waarna de shortcode zelf op de pagina kwam.
+    // Barts melding (17-8): het concept "The Mixtape" toonde in preview
+    // letterlijk [[embed:https://music.apple.com/nl/playlist/...]].
+    //
+    // Bewust krap: geen slash, vraagteken of hekje in het id, want wat hier
+    // gevangen wordt gaat rechtstreeks achter https://embed.music.apple.com/ aan.
+    const match = url.match(
+      /music\.apple\.com\/([a-z]{2}\/(?:album|playlist|song)\/[^/?#]+\/(?:[0-9]+|pl\.[A-Za-z0-9_-]+))/i,
+    );
     if (!match) return null;
     const src = `https://embed.music.apple.com/${match[1]}`;
@@ -341 +351 @@
         return `<div class="post-embed-missing"><em>Embed: niet-ondersteunde of ongeldige URL.</em></div>`;
       }
-      return this.generateIframe(detected.provider, detected) || match;
+      // HERKEND maar niet te bouwen is geen reden om de shortcode zelf te
+      // tonen. Dat deed het wel, en dan leest een bezoeker "[[embed:https://...]]"
+      // op de pagina en denkt hij dat er iets stuk is. Onherkend gaf hierboven
+      // al een nette melding; herkend-maar-mislukt hoort dezelfde te geven,
+      // want voor de lezer is het hetzelfde geval.
+      return this.generateIframe(detected.provider, detected)
+        || `<div class="post-embed-missing"><em>Embed: niet-ondersteunde of ongeldige URL.</em></div>`;
     });
   }