Changeset 861c228 in Klonkt for src/services/ArchiveImportService.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/ArchiveImportService.js (modified) (2 diffs)

src/services/ArchiveImportService.js

@@ -256 +256 @@
     posts: 0, overgeslagen: 0, overschreven: 0,
     replies: 0, media: 0, mediaMissing: 0, gemist: [], waarschuwingen: [],
-    tracks: 0, tracksMissing: 0, playlists: 0,
+    tracks: 0, tracksMissing: 0, playlists: 0, linksBijgetrokken: 0,
   };
 
@@ -505 +505 @@
       }
     }
+
+    // Links naar de BRONPOSTS ombuigen naar hier. Ook bij een zip-import: een
+    // verhuizing is een verhuizing, en een bericht dat naar het oude domein
+    // linkt wordt een dode link zodra dat domein opgezegd wordt. Binnen dezelfde
+    // transactie, want half bijgetrokken is erger dan niet.
+    //
+    // Pas hier, aan het eind: nu staan alle berichten er, dus nu weten we welke
+    // slugs bestaan. Een link naar iets dat we niet hebben blijft met rust.
+    if (manifest.origin && manifest.origin !== eigenOrigin) {
+      Migration.postLinksBijtrekken(site, String(manifest.origin).replace(/\/+$/, ''), rapport);
+    }
   })();