Changeset 861c228 in Klonkt for src/services/MigrationService.js

Timestamp:

08/14/2026 01:43:59 AM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

dc802b4

Parents:

9f9b45f

Message:

Gebakken links wijzen voortaan naar hier, in beide routes

Robin: de gebakken tracklinks moeten de nieuwe links worden. Terecht, en ik had
ze te makkelijk afgedaan als "geen plaatje, dus niet mijn probleem". Het zijn
tijdbommen: buildNote zet ​https://oud/<slug>#track-<id> en ?fc=2 in de content,
en zodra dat domein opgezegd wordt staan er dode links in je eigen berichten.

Ze kunnen nu ook echt om, en dat is te danken aan "altijd behouden" van
vanmiddag: slug en track-id zijn hier hetzelfde, dus alleen de host hoeft eraf
en het fragment klopt vanzelf.

DE REGEL: alleen ombuigen voor een slug die hier ECHT bestaat. Wijst een link
naar iets dat we niet hebben, dan blijft hij naar de oude site wijzen. Een
verwijzing naar een pagina die er nog is, is beter dan een 404 op de nieuwe.

Pas aan het EIND van de ronde, want dan pas weten we welke slugs binnen zijn.
Bij een verse ingest zou de helft van de links nog niet te beoordelen zijn.

EN IN BEIDE ROUTES ("het moet wel gebeuren bij migratie direct ook"). Een
zip-import is net zo goed een verhuizing. Daar draait het binnen dezelfde
transactie: half bijgetrokken is erger dan niet.

Changed files:
src/services/MigrationService.js

• postLinksBijtrekken(), geexporteerd zodat de zip hem ook kan draaien
• aangeroepen zodra alle berichten binnen zijn

src/services/ArchiveImportService.js

• dezelfde stap aan het eind van de import, in de transactie
• rapportveld linksBijgetrokken

src/views/pages/admin-migrate.ejs

• in het import- en het ophaalverslag

src/services/i18n.js

• mig.r_links_fixed in nl/en/de

test/fep1580-migration.test.js

• fragment en querystring blijven heel, een onbekende slug blijft met rust

test/archive-audio.test.js

• dezelfde eis voor de zip-route

remarks: /media, /audio en /ap worden overgeslagen; die lopen via hun eigen weg.
Suite 1000 groen, tegenproef op allebei de routes.

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

File:

• src/services/MigrationService.js (modified) (2 diffs)

src/services/MigrationService.js

@@ -239 +239 @@
 
 const AFBEELDING = /^image\//i;
+
+/**
+ * Links naar de BRONPOSTS ombuigen naar hier.
+ *
+ * De gebakken content zit vol met https://oud/<slug>#track-<id> en
+ * https://oud/<slug>?fc=2: de "luister op"-links die buildNote maakt. Die
+ * blijven naar de oude site wijzen, en dat is een tijdbom, want zodra dat
+ * domein opgezegd wordt zijn het dode links in je eigen berichten.
+ *
+ * Kan pas als ALLE posts binnen zijn, en alleen voor een slug die hier echt
+ * bestaat. Een link naar iets dat we niet hebben laten we met rust: dan is een
+ * verwijzing naar de oude site nog altijd beter dan een 404 op de nieuwe.
+ *
+ * De #track-<id>-fragmenten kloppen vanzelf, want sinds "altijd behouden" is
+ * dat id hier hetzelfde.
+ */
+export function postLinksBijtrekken(site, bronOrigin, rapport = {}) {
+  if (!bronOrigin) return 0;
+  let n = 0;
+  const rijen = db.prepare('SELECT id, content FROM posts WHERE site_id = ? AND content LIKE ?')
+    .all(site.id, `%${bronOrigin}/%`);
+  if (!rijen.length) return 0;
+  const heeftSlug = db.prepare('SELECT 1 FROM posts WHERE site_id = ? AND slug = ?');
+  const upd = db.prepare('UPDATE posts SET content = ? WHERE id = ?');
+  const patroon = new RegExp(`${bronOrigin.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')}/([A-Za-z0-9._~-]+)`, 'g');
+  for (const r of rijen) {
+    let inhoud = String(r.content || '');
+    let raak = false;
+    for (const m of [...new Set([...inhoud.matchAll(patroon)].map((x) => x[1]))]) {
+      // media en audio lopen via hun eigen weg; hier gaat het om postpagina's
+      if (m === 'media' || m === 'audio' || m === 'ap') continue;
+      if (!heeftSlug.get(site.id, m)) continue;
+      inhoud = inhoud.split(`${bronOrigin}/${m}`).join(`/${m}`);
+      raak = true;
+    }
+    if (raak) { upd.run(inhoud, r.id); n++; }
+  }
+  if (n) {
+    rapport.linksBijgetrokken = n;
+    console.log('[FEP-1580] postlinks bijgetrokken in', n, 'bericht(en)');
+  }
+  return n;
+}
 
 /**
@@ -770 +813 @@
     }
 
+    // Postlinks eerst: pas nu zijn ALLE berichten binnen, dus pas nu weten we
+    // welke slugs hier bestaan.
+    postLinksBijtrekken(site, bronOrigin, rapport);
+
     // ── De verwijzingen in de tekst bijtrekken ────────────────────
     //