Changes in src/routes/admin-settings.js [63edb0d:0091cb7] in Klonkt

File:

• src/routes/admin-settings.js (modified) (4 diffs)

src/routes/admin-settings.js

@@ -2 +2 @@
  * Admin: globale instellingen.
  *  - tenancy-modus (Solo/Hub)
- *  - hub-branding (naam/tagline/intro/hero van de generieke hub-hoofdpagina)
+ *  - hub-branding (naam/tagline/intro van de generieke hub-hoofdpagina)
  *
  * GET  /admin/settings   -> toon huidige instellingen
- * POST /admin/settings   -> sla op (god-only). Accepteert nu ook een geuploade
- *                           hero-afbeelding (multipart); een upload wint van het
- *                           URL-tekstveld. Zonder upload blijft het URL-veld leidend.
+ * POST /admin/settings   -> sla op (god-only)
  *
  * De hub-pagina is generiek (van geen enkele user); deze branding leeft in
@@ -14 +12 @@
 
 import express from 'express';
-import path from 'path';
-import fs from 'fs';
-import { fileURLToPath } from 'url';
-import multer from 'multer';
-import { v4 as uuid } from 'uuid';
 import { renderPage } from '../middleware/render.js';
 import { requireGod } from '../middleware/auth.js';
@@ -24 +17 @@
 
 const router = express.Router();
-
-// Hero dark-overlay: percentage 0-100 (0 = geen overlay, 100 = volledig zwart).
-// Default 45 = de oude hardgecodeerde waarde, zodat bestaande hubs niet wijzigen.
-function clampOverlay(raw) {
-  const v = parseInt(raw, 10);
-  return Number.isFinite(v) ? Math.max(0, Math.min(100, v)) : 45;
-}
-
-const __dirname = path.dirname(fileURLToPath(import.meta.url));
-// Hero-uploads landen in storage/media/hero → bereikbaar als /media/hero/<file>
-// (de /media static handler serveert storage/media). Zelfde model als avatars.
-const HERO_DIR = path.resolve(
-  process.env.HERO_PATH || path.join(__dirname, '..', '..', 'storage', 'media', 'hero')
-);
-fs.mkdirSync(HERO_DIR, { recursive: true });
-
-// Alleen raster-formaten voor de upload. SVG mag bewust NIET via upload (raw
-// SVG kan script bevatten → opgeslagen-XSS bij direct openen); een SVG-hero kan
-// nog steeds via het URL-veld (zoals de meegeleverde demo-placeholder).
-const ALLOWED_HERO_EXT = new Set(['.jpg', '.jpeg', '.png', '.webp', '.gif']);
-const MAX_HERO_BYTES = 5 * 1024 * 1024;
-
-const heroUpload = multer({
-  storage: multer.diskStorage({
-    destination: (req, file, cb) => cb(null, HERO_DIR),
-    filename: (req, file, cb) => {
-      const ext = path.extname(file.originalname).toLowerCase();
-      cb(null, `${uuid()}${ext}`);
-    },
-  }),
-  limits: { fileSize: MAX_HERO_BYTES },
-  fileFilter: (req, file, cb) => {
-    const ext = path.extname(file.originalname).toLowerCase();
-    if (!ALLOWED_HERO_EXT.has(ext)) {
-      return cb(new Error('Hero-afbeelding moet jpg/png/webp/gif zijn'));
-    }
-    cb(null, true);
-  },
-});
 
 router.get('/', requireGod, (req, res) => {
@@ -73 +27 @@
     hubIntro: getSetting('hub_intro') || '',
     hubHeroImage: getSetting('hub_hero_image') || '',
-    hubHeroOverlay: clampOverlay(getSetting('hub_hero_overlay')),
     success: req.query.success || null,
-    error: req.query.error || null,
   });
 });
 
 router.post('/', requireGod, (req, res) => {
-  // multer.single verwerkt multipart (hub-branding form). Bij een gewone
-  // urlencoded POST (tenancy-form) doet multer niets en blijft req.body intact.
-  heroUpload.single('hub_hero_file')(req, res, (err) => {
-    if (err) {
-      return res.redirect('/admin/settings?error=' + encodeURIComponent(err.message));
-    }
-
-    if (typeof req.body.tenancy !== 'undefined') {
-      setTenancy(req.body.tenancy === 'hub' ? 'hub' : 'solo');
-    }
-    if (typeof req.body.hub_title !== 'undefined') {
-      setSetting('hub_title', (req.body.hub_title || '').toString().slice(0, 80).trim());
-    }
-    if (typeof req.body.hub_tagline !== 'undefined') {
-      setSetting('hub_tagline', (req.body.hub_tagline || '').toString().slice(0, 120).trim());
-    }
-    if (typeof req.body.hub_intro !== 'undefined') {
-      setSetting('hub_intro', (req.body.hub_intro || '').toString().slice(0, 400).trim());
-    }
-
-    // Hero: een geüploade afbeelding wint; anders het URL-tekstveld.
-    if (req.file) {
-      const newUrl = `/media/hero/${req.file.filename}`;
-      // Ruim een vorige geüploade hero op (alleen als die uit onze hero-map kwam).
-      const old = getSetting('hub_hero_image') || '';
-      if (old.startsWith('/media/hero/')) {
-        try { fs.unlinkSync(path.join(HERO_DIR, path.basename(old))); } catch {}
-      }
-      setSetting('hub_hero_image', newUrl);
-    } else if (typeof req.body.hub_hero_image !== 'undefined') {
-      setSetting('hub_hero_image', (req.body.hub_hero_image || '').toString().slice(0, 300).trim());
-    }
-
-    if (typeof req.body.hub_hero_overlay !== 'undefined') {
-      setSetting('hub_hero_overlay', String(clampOverlay(req.body.hub_hero_overlay)));
-    }
-
-    res.redirect('/admin/settings?success=' + encodeURIComponent('Opgeslagen'));
-  });
+  if (typeof req.body.tenancy !== 'undefined') {
+    setTenancy(req.body.tenancy); // valideert naar solo | hub | circle
+  }
+  if (typeof req.body.hub_title !== 'undefined') {
+    setSetting('hub_title', (req.body.hub_title || '').toString().slice(0, 80).trim());
+  }
+  if (typeof req.body.hub_tagline !== 'undefined') {
+    setSetting('hub_tagline', (req.body.hub_tagline || '').toString().slice(0, 120).trim());
+  }
+  if (typeof req.body.hub_intro !== 'undefined') {
+    setSetting('hub_intro', (req.body.hub_intro || '').toString().slice(0, 400).trim());
+  }
+  if (typeof req.body.hub_hero_image !== 'undefined') {
+    setSetting('hub_hero_image', (req.body.hub_hero_image || '').toString().slice(0, 300).trim());
+  }
+  res.redirect('/admin/settings?success=' + encodeURIComponent('Opgeslagen'));
 });