Changeset 8b65e67 in Klonkt

Timestamp:

08/18/2026 06:47:58 PM (3 weeks ago)

Author:

Bart <bart@…>

Branches:

main

Children:

f5cf299

Parents:

b1b7652

git-author:

Bart <bart@…> (08/17/2026 07:37:17 PM)

git-committer:

Bart <bart@…> (08/18/2026 06:47:58 PM)

Message:

Een gesloten bericht houdt zijn plek, en zijn tekst blijft thuis

Tweede stap voor de tijdlijn (Barts idee, naar bartoverkamp.nl): hele berichten
in de stroom in plaats van kaartjes. Een poort kan dan geen pagina meer zijn --
een betaald of fan-bericht moet ter plekke zeggen wat er is, met behoud van zijn
plek. Weglaten zou erger zijn: dan zien een supporter en een vreemde een andere
tijdlijn met andere buren, en klopt "vorige/volgende" voor niemand.

postEntry() is met opzet een functie en geen sjabloonconditie. Het lijf wordt
alleen GERENDERD als het ook getoond mag worden, dus bij een dichte poort reist
de tekst niet mee. Een <% if %> in een sjabloon krijgt hem wel in het model,
en dan staat er één vergeten conditie -- of één partial die net iets anders
samenstelt -- tussen de betaalmuur en de tekst erachter.

partials/post-body.ejs kiest alleen de vorm en gebruikt de woorden van de
bestaande poorten (pgate./fgate.), zodat ze niet uiteen gaan lopen na de eerste
tekstwijziging. Het ontgrendelen zelf blijft op de eigen pagina, want daar zit
de passkey-code; één knop ernaartoe is eerlijker dan een halve poort.

Vijf toetsen erbij, waaronder dat er bij een dichte poort niets van de tweede
alinea in het model zit.

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

Files:

• src/services/PostAccessService.js (modified) (1 diff)
• src/views/partials/post-body.ejs (added)
• test/post-access.test.js (modified) (1 diff)

src/services/PostAccessService.js

@@ -69 +69 @@
 export function canReadBody(post, viewer) { return postAccess(post, viewer) === 'full'; }
 
-export default { postAccess, canReadBody, paidTeaser };
+/**
+ * Eén bericht, klaar voor de tijdlijn.
+ *
+ * DE HELE REDEN DAT DIT EEN FUNCTIE IS: het lijf wordt alleen GERENDERD als het
+ * ook getoond mag worden. Een sjabloon dat zelf `<% if (access === 'full') %>`
+ * doet krijgt de volledige tekst tóch mee in het model, en dan is het één
+ * vergeten conditie -- of één `?partial=1` die net iets anders samenstelt --
+ * tussen een betaalmuur en de tekst erachter. Hier komt hij niet eens mee.
+ *
+ * `renderBody` wordt ingespoten (posts.js levert renderPostBodyHtml), zodat
+ * deze module niets van routes of sjablonen hoeft te weten.
+ */
+export function postEntry(post, viewer, { renderBody } = {}) {
+  const access = postAccess(post, viewer);
+  const full = access === 'full';
+  return {
+    slug: post && post.slug,
+    title: (post && post.title) || '',
+    access,
+    // Alleen bij een gesloten poort een teaser, en nooit allebei: de teaser is
+    // de vervanging van het lijf, niet een voorproefje ernaast.
+    teaser: full || access === 'forbidden' ? null : paidTeaser(post),
+    content_html: full && typeof renderBody === 'function' ? renderBody(post) : null,
+  };
+}
+
+export default { postAccess, canReadBody, paidTeaser, postEntry };

test/post-access.test.js

@@ -96 +96 @@
   assert.equal(paidTeaser(post({ excerpt: 'zelf geschreven' })), 'zelf geschreven');
 });
+
+// ── Wat de tijdlijn krijgt ────────────────────────────────────────────────
+// De tijdlijn toont hele berichten, dus een gesloten bericht moet zijn plek
+// houden ZONDER dat de tekst meereist. Dat is geen sjabloonkwestie: wat niet
+// gerenderd wordt, kan ook niet per ongeluk getoond worden.
+
+const { postEntry } = await import('../src/services/PostAccessService.js');
+const renderBody = (p) => `<div>GEHEIM: ${p.content}</div>`;
+
+test('een open bericht krijgt zijn lijf', () => {
+  const e = postEntry(post(), { user: VREEMDE, site: SITE }, { renderBody });
+  assert.equal(e.access, 'full');
+  assert.match(e.content_html, /GEHEIM/);
+  assert.equal(e.teaser, null, 'geen teaser NAAST het lijf');
+});
+
+test('een betaald bericht reist zonder tekst', () => {
+  const e = postEntry(post({ paid: 1 }), { user: VREEMDE, site: SITE }, { renderBody });
+  assert.equal(e.access, 'paid');
+  assert.equal(e.content_html, null, 'het lijf wordt niet eens gerenderd');
+  assert.match(e.teaser, /eerste alinea/);
+  assert.ok(!JSON.stringify(e).includes('tweede alinea'), 'en niets erachter lekt mee');
+});
+
+test('een fan-bericht ook niet', () => {
+  const e = postEntry(post({ fan_only: 1 }), { user: VREEMDE, site: SITE }, { renderBody });
+  assert.equal(e.access, 'fan');
+  assert.equal(e.content_html, null);
+});
+
+test('een concept levert helemaal niets op', () => {
+  const e = postEntry(post({ status: 'draft' }), { user: VREEMDE, site: SITE }, { renderBody });
+  assert.equal(e.access, 'forbidden');
+  assert.equal(e.content_html, null);
+  assert.equal(e.teaser, null, 'ook geen teaser: over een concept valt niets te zeggen');
+});
+
+test('zonder renderer nog steeds geen lijf, en geen fout', () => {
+  const e = postEntry(post(), { user: VREEMDE, site: SITE });
+  assert.equal(e.access, 'full');
+  assert.equal(e.content_html, null);
+});