From 033ebc6861e6e6e40d728c3bb3f2a40058df3aeb Mon Sep 17 00:00:00 2001
From: adevopg <adevopg@gmail.com>
Date: Tue, 14 Jul 2026 22:41:58 +0000
Subject: [PATCH] =?UTF-8?q?Fix:=20tooltips=20de=20wowhead=20en=20el=20idio?=
 =?UTF-8?q?ma=20de=20la=20web=20(no=20siempre=20ingl=C3=A9s)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

El script de tooltips prioriza el atributo data-wowhead sobre el subdominio del
href, y su parámetro `domain` codifica idioma + versión: el 1.er segmento va al
mapa de locales ({es:6,...}) y el resto a la versión (wotlk=WRATH). Antes se
enviaba `domain=wotlk` (sin idioma) → locale 0 = inglés siempre.

Ahora `wowheadData(type,id,locale)` genera `domain=es.wotlk` en español (locale 6)
y `domain=wotlk` en inglés. Verificado: el endpoint de datos con locale 6 devuelve
los nombres en español.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
---
 .../app/[locale]/quest-character/page.tsx     |  6 ++---
 web-next/components/RecruitPanel.tsx          |  2 +-
 web-next/components/WowheadLink.tsx           |  2 +-
 web-next/lib/wowhead.ts                       | 24 ++++++++++++++-----
 4 files changed, 23 insertions(+), 11 deletions(-)

diff --git a/web-next/app/[locale]/quest-character/page.tsx b/web-next/app/[locale]/quest-character/page.tsx
index 602d004..270dcb3 100644
--- a/web-next/app/[locale]/quest-character/page.tsx
+++ b/web-next/app/[locale]/quest-character/page.tsx
@@ -66,7 +66,7 @@ export default async function QuestCharacterPage({ params }: { params: Promise<{
                     {o.link ? (
                       <a
                         href={wowheadUrl(o.link.type as WowheadType, o.link.id, locale)}
-                        data-wowhead={wowheadData(o.link.type as WowheadType, o.link.id)}
+                        data-wowhead={wowheadData(o.link.type as WowheadType, o.link.id, locale)}
                         target="_blank"
                         rel="noopener noreferrer"
                         className="icontinyl"
@@ -96,8 +96,8 @@ export default async function QuestCharacterPage({ params }: { params: Promise<{
         <ul>
           <li>
             {t.rich('quest.hodir', {
-              hodir: (c) => <a href={wowheadUrl('faction', 1119, locale)} data-wowhead={wowheadData('faction', 1119)} target="_blank" rel="noopener noreferrer">{c}</a>,
-              lille: (c) => <a href={wowheadUrl('npc', 32540, locale)} data-wowhead={wowheadData('npc', 32540)} target="_blank" rel="noopener noreferrer">{c}</a>,
+              hodir: (c) => <a href={wowheadUrl('faction', 1119, locale)} data-wowhead={wowheadData('faction', 1119, locale)} target="_blank" rel="noopener noreferrer">{c}</a>,
+              lille: (c) => <a href={wowheadUrl('npc', 32540, locale)} data-wowhead={wowheadData('npc', 32540, locale)} target="_blank" rel="noopener noreferrer">{c}</a>,
             })}
           </li>
         </ul>
diff --git a/web-next/components/RecruitPanel.tsx b/web-next/components/RecruitPanel.tsx
index 834ed43..33a7757 100644
--- a/web-next/components/RecruitPanel.tsx
+++ b/web-next/components/RecruitPanel.tsx
@@ -78,7 +78,7 @@ export function RecruitPanel({
       <>
         <a
           href={wowheadUrl('item', r.item_id, locale)}
-          data-wowhead={wowheadData('item', r.item_id)}
+          data-wowhead={wowheadData('item', r.item_id, locale)}
           target="_blank"
           rel="noopener noreferrer"
           className={`icontinyl ${quality || ''}`}
diff --git a/web-next/components/WowheadLink.tsx b/web-next/components/WowheadLink.tsx
index 8804af1..3f5bedf 100644
--- a/web-next/components/WowheadLink.tsx
+++ b/web-next/components/WowheadLink.tsx
@@ -25,7 +25,7 @@ export function WowheadLink({
   return (
     <a
       href={wowheadUrl(type, id, locale)}
-      data-wowhead={wowheadData(type, id)}
+      data-wowhead={wowheadData(type, id, locale)}
       target="_blank"
       rel="noopener noreferrer"
       className={className}
diff --git a/web-next/lib/wowhead.ts b/web-next/lib/wowhead.ts
index f7fb52b..36fdd36 100644
--- a/web-next/lib/wowhead.ts
+++ b/web-next/lib/wowhead.ts
@@ -43,10 +43,22 @@ export function wowheadIcon(name: string, size: IconSize = 'large'): string {
   return `https://wow.zamimg.com/images/wow/icons/${size}/${name}.${ext}`
 }
 
-/**
- * Valor del atributo `data-wowhead`, para que el script de tooltips lo reconozca
- * con la rama WotLK aunque el href no baste (iconos, celdas sin enlace, etc.).
- */
-export function wowheadData(type: WowheadType, id: number | string): string {
-  return `${type}=${id}&domain=wotlk`
+// Código de idioma de wowhead para el parámetro `domain` del tooltip. El inglés
+// es la base (sin prefijo). El script mapea el 1.er segmento a locale
+// ({es:6,fr:2,de:3,...}) y el resto a la versión del juego (wotlk = WRATH).
+const LOCALE_WH_CODE: Record<string, string> = { es: 'es' }
+
+function wowheadDomain(locale: string): string {
+  const code = LOCALE_WH_CODE[locale]
+  return code ? `${code}.wotlk` : 'wotlk'
+}
+
+/**
+ * Valor del atributo `data-wowhead`, para que el tooltip salga en la rama WotLK
+ * y EN EL IDIOMA de la web. El `domain` lleva el idioma (p.ej. `es.wotlk`) porque
+ * el script prioriza este atributo sobre el subdominio del href; sin el idioma
+ * aquí, el tooltip saldría en inglés aunque el enlace sea es.wowhead.com.
+ */
+export function wowheadData(type: WowheadType, id: number | string, locale: string): string {
+  return `${type}=${id}&domain=${wowheadDomain(locale)}`
 }