Changeset d7e72b8 in Klonkt for src/routes

Timestamp:

08/09/2026 07:47:48 AM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

b406d25

Parents:

c832064

Message:

Audio is geen keuze meer: Album en Playlist staan in de balk (shaer-cyg)

De posttype-balk had post/foto/video/audio. Audio verdwijnt als keuze -- alles
wat muziek is landt voortaan op album of playlist, en dat maakt de keuze
betekenisvol in plaats van cosmetisch. Album en Playlist delen het muziekpaneel,
want het verschil zit in de playlist en niet in wat je uploadt.

De knop Audio blijft alleen staan zolang een post nog type=audio IS. Anders zou
het openen van een oude post hem stilzwijgend van type veranderen, en dat is
precies het soort verlies dat niemand ziet gebeuren.

De lijst van geldige types stond op DRIE plaatsen los van elkaar: twee keer in
routes/posts.js en een keer in routes/types.js. Dat viel niet op zolang ze
gelijk waren, maar het faalt stil: kent het opslaan 'playlist' niet, dan wordt
de post zonder melding een gewone post en is de keuze verdwenen in plaats van
geweigerd. Nu een lijst, in config/post-types.js, die ook de editor voedt.

En de migratie: scripts/backfill-post-types.mjs zet bestaande audio-posts om via
postMusicType. Proefdraai is de standaard, --doen schrijft. Wat de afleiding
niet tot album of playlist maakt blijft met rust en wordt apart gemeld -- die
verdienen een blik en geen gok.

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

Location:

src/routes

Files:

• posts.js (modified) (5 diffs)
• types.js (modified) (1 diff)

src/routes/posts.js

@@ -6 +6 @@
 import ejs from 'ejs';
 import db from '../config/database.js';
+import { POST_TYPES, KEUZE_TYPES } from '../config/post-types.js';
 import { requireAuth, requireSiteManager, isViewer } from '../middleware/auth.js';
 import { renderPage } from '../middleware/render.js';
@@ -265 +266 @@
     },
     isNew: true,
+    keuzeTypes: KEUZE_TYPES,
     pageTitle: 'New post',
     bodyClass: 'on-special',
@@ -359 +361 @@
   finalSlug = uniqueSlug(site.id, finalSlug);
 
-  const validTypes = new Set(['post', 'foto', 'video', 'audio']);
-  const finalType = validTypes.has(type) ? type : 'post';
+  const finalType = POST_TYPES.has(type) ? type : 'post';
   const pollJson = parsePollForm(req.body);   // AS2 Question definition, or null
   const postId = uuid();
@@ -454 +455 @@
     post,
     isNew: false,
+    keuzeTypes: KEUZE_TYPES,
     pollLocked,
     fediOpenAudio: postAudioFediOpen(site.id, post.content),
@@ -486 +488 @@
   const newSlug = req.body.slug;
   const action = req.body.action || 'save';
-  const validTypes = new Set(['post', 'foto', 'video', 'audio']);
-  const finalType = validTypes.has(type) ? type : (post.type || 'post');
+  const finalType = POST_TYPES.has(type) ? type : (post.type || 'post');
 
   // A poll that has already received votes is frozen (you can still edit the surrounding

src/routes/types.js

@@ -8 +8 @@
 import express from 'express';
 import db from '../config/database.js';
+import { POST_TYPES } from '../config/post-types.js';
 import { renderPage } from '../middleware/render.js';
 
 const router = express.Router();
-const VALID_TYPES = new Set(['post', 'foto', 'video', 'audio']);
+// Een lijst, drie gebruikers: zie config/post-types.js.
+const VALID_TYPES = POST_TYPES;
 
 router.get('/:type', (req, res) => {