From 9e6a8f823111d02b15a7885ce62a7dd549687f44 Mon Sep 17 00:00:00 2001
From: adevopg <adevopg@gmail.com>
Date: Tue, 14 Jul 2026 14:46:17 +0000
Subject: [PATCH] Quitar URLs javascript: (React 19 las bloquea)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

React 19 bloquea href="javascript:void(0)". SiteHeader: DESCONECTAR ahora enlaza
a /{locale}/log-out (ruta real), el menú hamburguesa y el logo pequeño usan
role/button o href real. PostActions (foro): editar/eliminar pasan a role=button.
Se elimina el logout por fetch del header (lo hace la ruta /log-out).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
---
 web-next/components/PostActions.tsx |  4 ++--
 web-next/components/SiteHeader.tsx  | 25 +++++++------------------
 2 files changed, 9 insertions(+), 20 deletions(-)

diff --git a/web-next/components/PostActions.tsx b/web-next/components/PostActions.tsx
index 54d9ff7..a531b9b 100644
--- a/web-next/components/PostActions.tsx
+++ b/web-next/components/PostActions.tsx
@@ -120,11 +120,11 @@ export function PostActions({
 
   return (
     <div className="separate small-font">
-      <a href="javascript:void(0);" onClick={() => setEditing(true)} className="second-brown">
+      <a role="button" tabIndex={0} onClick={() => setEditing(true)} className="second-brown" style={{ cursor: 'pointer' }}>
         {t('edit')}
       </a>
       {'  '}
-      <a href="javascript:void(0);" onClick={() => { if (!busy) remove() }} className="second-brown">
+      <a role="button" tabIndex={0} onClick={() => { if (!busy) remove() }} className="second-brown" style={{ cursor: 'pointer' }}>
         {t('delete')}
       </a>
       {error && <span className="red-info2"> {error}</span>}
diff --git a/web-next/components/SiteHeader.tsx b/web-next/components/SiteHeader.tsx
index f1fcca7..184c604 100644
--- a/web-next/components/SiteHeader.tsx
+++ b/web-next/components/SiteHeader.tsx
@@ -1,8 +1,8 @@
 'use client'
 
 import { useState } from 'react'
+import { useLocale } from 'next-intl'
 import { Link } from '@/i18n/navigation'
-import { useRouter } from '@/i18n/navigation'
 
 const SERVER_NAME = 'NovaWoW'
 
@@ -26,28 +26,19 @@ export function SiteHeader({
   realmHref: string
   playersHref: string
 }) {
-  const router = useRouter()
+  const locale = useLocale()
   const [open, setOpen] = useState(false)
 
-  async function logout(e: React.MouseEvent) {
-    e.preventDefault()
-    await fetch('/api/auth/logout', { method: 'POST', credentials: 'same-origin' })
-    router.push('/')
-    router.refresh()
-  }
-
   return (
     <header>
       <div className="nav-wrapper">
         <div className={open ? 'nav-bar responsive' : 'nav-bar'} id="nw-nav-bar">
           <a
-            href="javascript:void(0);"
+            role="button"
+            tabIndex={0}
             className={open ? 'icon change' : 'icon'}
             id="menu-icon"
-            onClick={(e) => {
-              e.preventDefault()
-              setOpen((v) => !v)
-            }}
+            onClick={() => setOpen((v) => !v)}
           >
             <div className="bar1"></div>
             <div className="bar2"></div>
@@ -129,9 +120,7 @@ export function SiteHeader({
                   <Link href="/my-account">MI CUENTA</Link>
                 </p>
                 <p>
-                  <a href="javascript:void(0);" onClick={logout}>
-                    DESCONECTAR
-                  </a>
+                  <a href={`/${locale}/log-out`}>DESCONECTAR</a>
                 </p>
               </div>
             </div>
@@ -152,7 +141,7 @@ export function SiteHeader({
           title={SERVER_NAME}
         />
       </a>
-      <a href="javascript:void(0);">
+      <a href={`/${locale}`}>
         <img
           className="nw-logo"
           src="/nw-themes/nw-ryu/nw-images/nw-logos/nw-logo.webp"