Changeset 3ca7bdf in Klonkt for src/routes/download.js

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@…>

File:

• src/routes/download.js (modified) (2 diffs)

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,