Freie Links je Server statt fester Felder
connect_url und mods_url waren zwei Spalten fuer zwei Zwecke — und beim naechsten Zweck waere eine dritte dazugekommen. Jetzt gibt es eine Liste: Beschriftung plus Adresse, so viele wie noetig. Regeln, TeamSpeak, Shop, Bewerbung. Als JSON-Spalte, nicht als eigene Tabelle: es haengt an genau einem Server, wird immer komplett gelesen und immer komplett geschrieben. Gelesen wird es als Liste, geschrieben darf es als Liste hereinkommen — better-sqlite3 kann kein Array binden und wuerde sonst werfen. Kaputtes JSON ergibt eine leere Liste statt eines Absturzes im Monitor. Discord nimmt fuenf Knoepfe je Reihe; ab jetzt werden sie umgebrochen, bei zwei Reihen ist Schluss. Die Grenze steht im Code und nicht im Fehler beim Senden, den niemand sieht. Zu lange Beschriftungen werden auf 80 Zeichen gekuerzt, halbe Zeilen fliegen raus statt als leerer Knopf zu erscheinen. Was nicht http(s) ist — ts3server://, steam:// —, wird wie bisher ein Codeblock im Embed zum Kopieren. Dabei umbenannt: das Embed-Feld fuer einen nicht klickbaren Connect-Link hiess "Connect", der Knopf daneben "Verbinden". Jetzt heisst beides gleich. Geprueft: Umbruch bei zwoelf Links, Reihenfolge mit Connect und Mods vorn, halbe Zeilen, ueberlange Beschriftung, nicht klickbare Schemata, fehlendes und unbrauchbares links-Feld, dazu Speichern, Aendern und kaputtes JSON in der Datenbank. Panel und /server-Seite im laufenden Frontend angesehen. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
@@ -226,6 +226,7 @@ export function registerApiRoutes(app, client) {
|
||||
address: s.address || null,
|
||||
connect_url: s.connect_url || null,
|
||||
mods_url: s.mods_url || null,
|
||||
links: s.links ?? [],
|
||||
online: r?.online ?? null, // null = noch nie gecheckt
|
||||
players: r?.players ?? null,
|
||||
max: r?.max ?? null,
|
||||
@@ -1609,9 +1610,28 @@ ${rssItems}
|
||||
mods_url: /^https?:\/\/.+/i.test(String(body.mods_url ?? '').trim())
|
||||
? String(body.mods_url).trim().slice(0, 300)
|
||||
: '',
|
||||
links: freieLinks(body.links),
|
||||
};
|
||||
}
|
||||
|
||||
/**
|
||||
* Freie Links säubern.
|
||||
*
|
||||
* Zeilen ohne Beschriftung oder ohne Adresse fliegen raus statt als leerer
|
||||
* Knopf im Discord zu landen. Zehn ist die Grenze, weil Discord fünf
|
||||
* Knöpfe je Reihe nimmt und zwei Reihen unter einem Embed genug sind.
|
||||
*/
|
||||
function freieLinks(roh) {
|
||||
if (!Array.isArray(roh)) return [];
|
||||
return roh
|
||||
.map((l) => ({
|
||||
label: String(l?.label ?? '').trim().slice(0, 80),
|
||||
url: String(l?.url ?? '').trim().slice(0, 400),
|
||||
}))
|
||||
.filter((l) => l.label && l.url)
|
||||
.slice(0, 10);
|
||||
}
|
||||
|
||||
app.get('/api/gameservers', async (request, reply) => {
|
||||
if (requireScope(request, reply, 'server')) return;
|
||||
return { servers: listGameservers() };
|
||||
|
||||
Reference in New Issue
Block a user