Changeset 1d1fdc9 in Klonkt for src/routes

Timestamp:

09/04/2026 12:17:49 PM (4 days ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

6d7ea52

Parents:

9946a68

Message:

Tijdstempels: de posts-kant ook, en de tijdbom ontmanteld (shaer-a937)

Correctie op mijn eigen sluitreden bij 9946a68: ik schreef daar dat de
posts-sorteringen door isoSql lopen, en dat was niet zo -- ik had ze juist
overgeslagen omdat ze vandaag nog niet scheef staan.

Twee dingen alsnog:

De TIJDBOM uit de bead. Scheduler zette published_at op CURRENT_TIMESTAMP,
dus zodra iemand een post inplant en die vanzelf publiceert, staat er SQL-
notatie tussen de ISO's en sorteert het hele archief door elkaar. Die schrijft
nu ISO, net als de andere zes schrijfwegen.

En het VANGNET voor wat er al staat: de zeven posts-sorteringen (outbox,
backfill, featured tweemaal, muziek tweemaal, beheeroverzicht) lopen nu ook
door isoSql. Half normaliseren is de verwarrendste toestand -- dan hangt het
per scherm af of een gemengde kolom goed valt.

Onderweg gecontroleerd dat elke gebruiker van isoSql en NU_ISO ze ook
importeert: admin.js gebruikte hem zonder import, en dat was pas omgevallen
op het moment dat iemand het beheeroverzicht opende. Alle aangeraakte
bestanden door node --check.

Volle suite 1255 groen.

Location:

src/routes

Files:

• activitypub.js (modified) (2 diffs)
• admin.js (modified) (2 diffs)

src/routes/activitypub.js

@@ -14 +14 @@
 import express from 'express';
 import { readFileSync } from 'fs';
-import db from '../config/database.js';
+import db, { isoSql } from '../config/database.js';
 import AP from '../services/ActivityPubService.js';
 import { apReadLimiter, apInboxLimiter } from '../middleware/rate-limit.js';
@@ -1176 +1176 @@
        AND IFNULL(ap_visibility, 'public') IN ('public', 'quiet')
        AND pinned IS NOT NULL AND pinned > 0
-     ORDER BY pinned DESC, COALESCE(published_at, created_at) ASC LIMIT 20`
+     ORDER BY pinned DESC, ${isoSql('COALESCE(published_at, created_at)')} ASC LIMIT 20`
   ).all(site.id);
   AP.sendAP(res, AP.buildFeatured(baseUrl(req), site, posts, { page: paginaNr(req) }));

src/routes/admin.js

@@ -6 +6 @@
 
 import express from 'express';
-import db from '../config/database.js';
+import db, { isoSql } from '../config/database.js';
 import { renderPage } from '../middleware/render.js';
 import { requireAuth } from '../middleware/auth.js';
@@ -22 +22 @@
     SELECT slug, title, status, published_at, created_at, updated_at
     FROM posts WHERE site_id = ?
-    ORDER BY (status != 'published') DESC, COALESCE(updated_at, published_at, created_at) DESC
+    ORDER BY (status != 'published') DESC, ${isoSql('COALESCE(updated_at, published_at, created_at)')} DESC
     LIMIT ?
   `).all(siteId, limit).map((p) => ({