Changeset 3ca7bdf in Klonkt

Timestamp:

08/07/2026 07:04:08 PM (5 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

01fb44f

Parents:

a8364ee

git-author:

Robin <roboburr@…> (08/07/2026 07:04:04 PM)

git-committer:

Robin <roboburr@…> (08/07/2026 07:04:08 PM)

Message:

De editpagina laadt zijn editor weer, en opslaan zonder module wist niets meer

Robins test at een post op: /posts/:slug/edit rendert dezelfde template als
/posts/new, maar alleen de nieuw-route kreeg bij shaer-bqr zijn pageJs. De
editpagina had dus data-js="chrome" -- geen editor, geen mediaprompts, en
een verborgen contentveld dat leeg rendert. Opslaan schreef dat lege veld
naar de database: strike-homerun verloor zo om 18:54 zijn inhoud (testpost
van 18:12, niets ouds verloren).

Drie lagen, van symptoom naar vangrail:

1. de edit-route declareert pageJs 'post-edit playlist-editor'
2. het verborgen veld rendert de BESTAANDE inhoud: laadt de module niet (oude cache, js uit), dan is opslaan een no-op in plaats van een wisser
3. test/page-modules.test.js scheurt luid als een render van de editor zijn modules vergeet -- een brontekst-test, bewust, want het gedrag leeft in de browser en dit is de fout die twee keer beet

En dezelfde audit ving download: drie renders, en pageJs stond op het
FORMULIER in plaats van op ready -- de auto-start hoorde v66r shaer-bqr bij
de ready-tak en omzeilde nu de e-mailvraag. Verplaatst, en de module weigert
voortaan een lege fileUrl.

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

Files:

• src/assets/js/mod/download.js (modified) (1 diff)
• src/routes/download.js (modified) (2 diffs)
• src/routes/posts.js (modified) (1 diff)
• src/views/pages/post-edit.ejs (modified) (1 diff)
• test/page-modules.test.js (added)

src/assets/js/mod/download.js

@@ -13 +13 @@
 function run() {
         // Auto-start de download (zelfde-origin attachment-link).
-        setTimeout(function(){ try { window.location.href = (pageData().fileUrl || ''); } catch(e){} }, 600);
+        setTimeout(function(){ try { var u = pageData().fileUrl; if (u) window.location.href = u; } catch(e){} }, 600);
 }

src/routes/download.js

@@ -85 +85 @@
   const fan = req.session && req.session.user;
   renderPage(req, res, 'pages/download', {
-    pageJs: 'download',
     pageTitle: track.title + ' — download',
     bodyClass: 'on-download',
@@ -114 +113 @@
   req.session.dl[track.id] = Date.now();
   renderPage(req, res, 'pages/download', {
+    // De auto-start hoort ALLEEN bij ready: op het formulier zou hij de
+    // e-mailvraag omzeilen. Het script stond v66r shaer-bqr dan ook binnen
+    // de ready-tak van de template.
+    pageJs: 'download',
     pageTitle: track.title + ' — download', bodyClass: 'on-download',
     dlState: 'ready', dlTrack: track,

src/routes/posts.js

@@ -447 +447 @@
 
   renderPage(req, res, 'pages/post-edit', {
+    // Zelfde modules als de nieuw-route hierboven: zonder deze regel laadt de
+    // editor niet, en dan wist een opslag de post (shaer-5s1, de beet van 7-8).
+    pageJs: 'post-edit playlist-editor',
     post,
     isNew: false,

src/views/pages/post-edit.ejs

@@ -208 +208 @@
         <%# Hidden field is what actually submits to the server. The visible
             contenteditable's serialized HTML is copied here on submit. %>
-        <input type="hidden" name="content" id="content-hidden" value="">
+        <%# value = de BESTAANDE inhoud: laadt de module niet (oude cache, js uit),
+    dan slaat het formulier de post ongewijzigd op in plaats van hem te wissen. %>
+        <input type="hidden" name="content" id="content-hidden" value="<%= post.content || '' %>">
 
         <%# Initial content is injected as a JSON string in a script tag so we