Changeset 812fea9 in Klonkt for src/services/ArchiveExportService.js

Timestamp:

08/06/2026 02:20:19 PM (5 weeks ago)

Author:

roboburr <roboburr@…>

Branches:

main

Children:

7aa3140

Parents:

58cfe5f

git-author:

Robin <roboburr@…> (08/06/2026 02:20:09 PM)

git-committer:

roboburr <roboburr@…> (08/06/2026 02:20:19 PM)

Message:

Elke bijlage zegt waar hij voor was (shaer-pmr, vervolg)

Gevonden bij een oefenherstel: ik vergeleek dertien kolommen, noemde het een
schone rondgang, en vergeleek daarna ALLE kolommen. Toen bleken er drie dingen
weg te vallen.

COVER de bytes zaten in het archief, maar nergens stond dat ze de

cover waren. cover_image_url kwam als null terug, dus de post
stond zonder cover.

C2S-BIJLAGE idem: het audiobestand en zijn poster zaten erin, maar

c2s_attachments werd niet hersteld, dus de speler verdween.

SUB-SECONDE de import zette tijdstempels om naar SQL-notatie en verloor de

milliseconden. Twee posts in dezelfde seconde staan dan in
willekeurige volgorde.

Elke bijlage draagt nu een shaer:role -- cover, coverVideo, inline, c2s, poster,
track -- en een poster wijst met shaer:posterFor naar de bijlage waar hij bij
hoort. De importer bouwt cover_image_url, cover_video_url en c2s_attachments
daaruit terug. Een rol is geen versiering: zonder rol heeft het archief wel de
bytes maar niet het feit dat ze de cover WAREN, en dat zie je pas als je alle
kolommen vergelijkt.

Tijdstempels gaan nu ongewijzigd de database in. Klonkt schrijft zelf in twee
spellingen; het archief normaliseert naar ISO. Het MOMENT overleeft de rondgang,
de spelling niet -- nu ook zo gedocumenteerd in plaats van stilzwijgend.

Rondgang op echte beta-data: geen verschil meer over 29 kolommen, met de tijden
op moment vergeleken in plaats van op tekst. 21 tests. Suite 514/514.

File:

• src/services/ArchiveExportService.js (modified) (4 diffs)

src/services/ArchiveExportService.js

@@ -81 +81 @@
   const uit = [];
   const zie = new Set();
-  const voegToe = (url, name) => {
+  const voegToe = (url, name, rol, extra = {}) => {
     const u = String(url || '').trim();
     if (!u || zie.has(u)) return;
     zie.add(u);
-    uit.push({ url: u, name: name || null });
+    uit.push({ url: u, name: name || null, rol, ...extra });
   };
-  voegToe(post.cover_image_url, post.cover_alt);
-  voegToe(post.cover_video_url, post.cover_alt);
-  for (const m of String(post.content || '').matchAll(/<img[^>]+src=["']([^"']+)["'][^>]*>/gi)) voegToe(m[1]);
+  // De ROL is niet decoratief. Zonder rol staat er in het archief wel een
+  // bestand, maar niet dat het de cover was of bij de speler hoorde -- en dan
+  // komt de post na een herstel zonder cover en zonder speler terug. Gevonden
+  // door bij de oefenherstel ALLE kolommen te vergelijken in plaats van een
+  // handjevol.
+  voegToe(post.cover_image_url, post.cover_alt, 'cover');
+  voegToe(post.cover_video_url, post.cover_alt, 'coverVideo');
+  for (const m of String(post.content || '').matchAll(/<img[^>]+src=["']([^"']+)["'][^>]*>/gi)) voegToe(m[1], null, 'inline');
   try {
     for (const a of JSON.parse(post.c2s_attachments || '[]')) {
-      voegToe(a && a.url, a && a.name);
+      voegToe(a && a.url, a && a.name, 'c2s');
       // Een audio-bijlage draagt een poster (de omslag die de speler toont). Die
       // staat in een eigen veld en zou anders stil wegvallen -- op beta viel dat
       // pas op bij de export van echte data.
-      voegToe(a && a.poster, a && a.name ? `${a.name} (poster)` : null);
+      voegToe(a && a.poster, a && a.name ? `${a.name} (poster)` : null, 'poster', { posterFor: a && a.url });
     }
   } catch { /* kapotte kolom blokkeert de export niet */ }
@@ -103 +108 @@
     try {
       const t = db.prepare('SELECT t.title, m.storage_path FROM audio_tracks t LEFT JOIN media m ON m.id = t.media_id WHERE t.id = ?').get(m[1]);
-      if (t && t.storage_path) voegToe(`/media/${path.relative(path.resolve(MEDIA_ROOT), path.resolve(t.storage_path))}`, t.title);
+      if (t && t.storage_path) voegToe(`/media/${path.relative(path.resolve(MEDIA_ROOT), path.resolve(t.storage_path))}`, t.title, 'track');
     } catch { /* geen audio-tabellen: niets te doen */ }
   }
@@ -251 +256 @@
           'shaer:originalUrl': /^https?:/i.test(ref.url) ? ref.url : `${origin}${ref.url}`,
           'shaer:sha256': hash,
+          'shaer:role': ref.rol,
+          'shaer:posterFor': ref.posterFor || undefined,
         });
       } else {
@@ -261 +268 @@
           type: as2TypeOf(mime), mediaType: mime, name: ref.name || undefined,
           url: orig, 'shaer:availability': 'missing', 'shaer:originalUrl': orig,
+          'shaer:role': ref.rol,
+          'shaer:posterFor': ref.posterFor || undefined,
         });
       }