Changeset ea3e722 in Klonkt for src/services/music/index.js

Timestamp:

08/09/2026 08:34:57 AM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

5ae4d61

Parents:

d60c498

Message:

Een hashtag hoort te staan zoals hij geschreven is (Robins punt, 9-8)

Stond een tag zowel in het tagveld als in het lijf van de post, dan won de vorm
uit het veld. Die gaat door tagParts, want een hashtag mag geen spaties
bevatten en "Doen we Niet" wordt daar #DoenWeNiet van. Nodig voor een tag die
alleen in het veld staat -- maar als iemand in zijn tekst #DoenweNiet heeft
getypt is dat geen benadering meer maar de tag zelf, en die verdween als
duplicaat van de genormaliseerde vorm.

Nu gaat het lijf voor: de geschreven vorm wint, en het tagveld vult aan wat er
niet staat. De slug in de href blijft kleingeschreven -- dat is een adres, geen
naam.

En de uitgave-collectie gebruikt nu diezelfde lijst in plaats van een eigen: ze
miste de tags uit de tekst helemaal, en spelde de rest anders dan dezelfde post
elders deed. buildHashtagList en hashtagTags zijn daarvoor van
ActivityPubService naar ap-core verhuisd, want music/ mag daar niet uit
importeren.

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

File:

• src/services/music/index.js (modified) (3 diffs)

src/services/music/index.js

@@ -14 +14 @@
 
 import db from '../../config/database.js';
-import { AP_CONTEXT, PUBLIC, actorId, noteId, safeUrl, guessMediaType, normalizeTags, tagParts } from '../ap-core.js';
+import { AP_CONTEXT, PUBLIC, actorId, noteId, safeUrl, guessMediaType, buildHashtagList } from '../ap-core.js';
 import { afleidenUitInsluitingen, ingeslotenPlaylists } from '../../assets/js/shared/post-music-type.js';
 
@@ -377 +377 @@
   }
 
-  const tags = hashtagsVanPost(base, post.tags);
+  // Dezelfde lijst als de Note: het tagveld EN de hashtags uit het lijf, waarbij
+  // de geschreven vorm voorgaat. Een eigen lijst hier zou de tags uit de tekst
+  // missen en de rest anders spellen dan dezelfde post elders doet.
+  const tags = buildHashtagList(base, post.tags, post.content);
   if (tags.length) obj.tag = tags;
 
@@ -386 +389 @@
   obj.context = noteId(base, post.id);
   return obj;
-}
-
-/**
- * De tags van een post als AS2 Hashtags -- zelfde vorm als buildHashtagList.
- *
- * Met normalizeTags en niet met een eigen split: het veld staat als JSON-array
- * in de database, en op komma's splitsen leverde live `#["Doen we Niet"` op.
- * Dat is het soort fout dat niet omvalt maar onzin uitlevert.
- */
-function hashtagsVanPost(base, tagsField) {
-  const uit = [], gezien = new Set();
-  for (const t of normalizeTags(tagsField)) {
-    const p = tagParts(t);
-    if (!p || gezien.has(p.slug)) continue;
-    gezien.add(p.slug);
-    uit.push({ type: 'Hashtag', href: `${base}/tag/${encodeURIComponent(p.slug)}`, name: '#' + p.label });
-  }
-  return uit;
 }