Changeset fc664ce in Klonkt for src/routes/posts.js

Timestamp:

08/14/2026 02:21:29 AM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

f704237

Parents:

e9128bc

Message:

Je volglijst verhuist mee naar de migrate-pagina

Robin stuurde een schermafbeelding van het blok op Connect: "this is still very
ugly". Terecht. "Kies je gedownloade CSV-bestand:" stond naast zijn invoerveld,
"Of plak" werd naar rechts geduwd en "de lijst hier:" viel eronder, met een
textarea van vijf tekens breed ertussen.

De reden is simpel: er was geen enkele opmaak voor .tl-move. Geen display,
geen breedte, niets. De labels waren dus inline en de tekst brak om de velden
heen.

Niet ter plekke bijgepoetst maar verplaatst, want dit is een migratie-optie en
die horen sinds vanmiddag bij elkaar. Op /admin/migrate erft het blok de
kaartopmaak die de andere secties al hadden: labels op eigen regels, invoervelden
over de volle breedte.

Gemeten in de browser: label display block, textarea 408px in een kaart van
455px. Op Connect blijft niets achter.

Changed files:
src/views/pages/admin-migrate.ejs

• de volglijst als echte sectie, met downloadknop, bestand, plakveld

src/views/pages/connect.ejs

• het ongestileerde <details>-blok eruit

src/routes/posts.js

• /news/following/import mag terugsturen naar een eigen pad

src/services/i18n.js

• mig.follows_note in nl/en/de

remarks: de sectie hergebruikt de bestaande tl.move_*-teksten, dus er hoefde
niets opnieuw vertaald. Suite 1005 groen.

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

File:

• src/routes/posts.js (modified) (2 diffs)

src/routes/posts.js

@@ -1348 +1348 @@
     ? req.file.buffer.toString('utf8').replace(/^/, '')   // BOM eraf; Excel zet die erin
     : ((req.body && req.body.csv) || '');
-  if (!site || !String(csv).trim()) return res.redirect('/connect?error=' + encodeURIComponent('Geen lijst ontvangen'));
+  // Terug naar waar je vandaan kwam. Sinds 14-8 staat dit formulier op
+  // /admin/migrate (Robin: alle migratie-opties bij elkaar); terugspringen naar
+  // Connect is dan desorienterend. Alleen een eigen pad, geen open redirect.
+  const terug = /^\/[A-Za-z0-9/_-]*$/.test(String(req.body.next || '')) ? String(req.body.next) : '/connect';
+  if (!site || !String(csv).trim()) return res.redirect(terug + '?error=' + encodeURIComponent('Geen lijst ontvangen'));
 
   const { importFollowing } = await import('../services/ArchiveImportService.js');
@@ -1370 +1374 @@
   }
   // Terug naar /connect: daar staat het blok, /following is de oude pagina.
-  res.redirect('/connect?' + (r.mislukt.length ? 'error=' : 'success=') + encodeURIComponent(delen.join(', ')));
+  res.redirect(terug + '?' + (r.mislukt.length ? 'error=' : 'success=') + encodeURIComponent(delen.join(', ')));
 });