diff --git a/web-next/lib/forum-sanitize.ts b/web-next/lib/forum-sanitize.ts index b2d822c..b057ea3 100644 --- a/web-next/lib/forum-sanitize.ts +++ b/web-next/lib/forum-sanitize.ts @@ -1,5 +1,25 @@ import sanitizeHtml from 'sanitize-html' +// Tipos de entidad de wowhead que aceptan tooltip (para normalizar el enlace). +const WOWHEAD_TYPES = 'item|spell|quest|npc|achievement|object|faction|currency|itemset|title' + +/** + * Normaliza un enlace de wowhead para que el tooltip salga en la rama WotLK. + * - Mantiene el subdominio (www=inglés, es=español…), que es lo que fija el idioma + * del tooltip cuando no hay data-wowhead. + * - Fuerza el segmento `/wotlk/` (si el usuario pega un enlace retail, el tooltip + * saldría de retail; así siempre es WotLK). + * Devuelve el href tal cual si no reconoce un `=`. + */ +function normalizeWowheadHref(href: string): string { + const m = /^https?:\/\/([a-z]+)\.wowhead\.com\/.*?\b(?:wotlk\/)?(?:(item|spell|quest|npc|achievement|object|faction|currency|itemset|title)=(\d+))/i.exec( + href, + ) + if (!m) return href + const [, sub, type, id] = m + return `https://${sub}.wowhead.com/wotlk/${type.toLowerCase()}=${id}` +} + // Misma allowlist que forum/sanitize.py (nh3) para el HTML de los mensajes. export function cleanPostHtml(html: string): string { if (!html) return '' @@ -13,16 +33,26 @@ export function cleanPostHtml(html: string): string { 'table', 'thead', 'tbody', 'tr', 'th', 'td', ], allowedAttributes: { - a: ['href', 'title', 'target'], + // data-wowhead + class dejan que los enlaces de wowhead muestren tooltip, + // color por calidad e icono (los pinta el script global tooltips.js del layout). + a: ['href', 'title', 'target', 'data-wowhead', 'class'], img: ['src', 'alt', 'title', 'width', 'height'], - span: ['style'], + span: ['style', 'data-wowhead'], div: ['style'], td: ['colspan', 'rowspan'], th: ['colspan', 'rowspan'], }, allowedSchemes: ['http', 'https', 'mailto'], transformTags: { - a: sanitizeHtml.simpleTransform('a', { rel: 'noopener noreferrer nofollow' }), + a: (tagName, attribs) => { + const out: Record = { ...attribs, rel: 'noopener noreferrer nofollow' } + if (out.href && /\.wowhead\.com\//i.test(out.href)) { + out.href = normalizeWowheadHref(out.href) + // abrir en pestaña nueva, como los enlaces de wowhead del resto del sitio + out.target = '_blank' + } + return { tagName, attribs: out } + }, }, }) }