Corregido cambiar avatar en cabecera

controladores/ControladorUsuario.php

@@ -55,10 +55,11 @@ switch ($_POST['orden']) {
 
         if ($resultado instanceof Usuario) {
 
-            // === AÑADIR ESTAS LÍNEAS PARA GUARDAR EN SESIÓN ===
+            // === GUARDAR EN SESIÓN (incluido avatar)===
             $_SESSION['idUsuario'] = $resultado->getIdUsuario();
             $_SESSION['nombreUsuario'] = $resultado->getNombreUsuario();
             $_SESSION['email'] = $resultado->getEmail();
+            $_SESSION['avatar'] = $resultado->getAvatar(); 
             // =================================================
 
             echo json_encode([
@@ -83,6 +84,7 @@ switch ($_POST['orden']) {
         unset($_SESSION['idUsuario']);
         unset($_SESSION['email']);
         unset($_SESSION['nombreUsuario']);
+        unset($_SESSION['avatar']); 
 
         // Destruimos la sesión
         session_destroy();
@@ -92,7 +94,6 @@ switch ($_POST['orden']) {
         ]);
 
         break;
-
     case 'mostrarEstadisticas':
 
         $usuario = Usuario::buscarPorId($_SESSION['idUsuario']);
@@ -237,16 +238,17 @@ switch ($_POST['orden']) {
     break;
 
     case 'comprobarSesion':
-    if (isset($_SESSION['idUsuario']) && !empty($_SESSION['idUsuario'])) {
+        if (isset($_SESSION['idUsuario']) && !empty($_SESSION['idUsuario'])) {
-        echo json_encode([
+            echo json_encode([
-            'activa' => true,
+                'activa' => true,
-            'usuario' => [
+                'usuario' => [
-                'idUsuario' => $_SESSION['idUsuario'],
+                    'idUsuario' => $_SESSION['idUsuario'],
-                'nombreUsuario' => $_SESSION['nombreUsuario'] ?? ''
+                    'nombreUsuario' => $_SESSION['nombreUsuario'] ?? '',
-            ]
+                    'avatar' => $_SESSION['avatar'] ?? '' 
-        ]);
+                ]
-    } else {
+            ]);
-        echo json_encode(['activa' => false]);
+        } else {
-    }
+            echo json_encode(['activa' => false]);
-    break;
+        }
+        break;
 }

js/cabecera.js

@@ -26,7 +26,7 @@ $(document).ready(function () {
 });
 
 /**
- * Comprueba con el servidor si existe una sesión PHP activa.
+ * Comprueba con el servidor si existe una sesión PHP
  */
 function comprobarEstadoSesion() {
     $.ajax({
@@ -37,7 +37,6 @@ function comprobarEstadoSesion() {
         success: function (respuesta) {
             if (respuesta.activa) {
                 renderizarCabeceraConSesion(respuesta.usuario);
-                // Si existe el contenedor del botón Jugar en index, lo mostramos
                 $('#colBotonJugar').removeClass('d-none');
             } else {
                 renderizarCabeceraSinSesion();
@@ -79,13 +78,8 @@ function ejecutarLogin() {
                 $('#modalLogin').modal('hide');
                 $('#formLogin')[0].reset();
 
-                renderizarCabeceraConSesion({
+                // Cargar datos de la sesión actualizados 
-                    idUsuario: respuesta.idUsuario,
+                comprobarEstadoSesion();
-                    nombreUsuario: respuesta.nombreUsuario,
-                    avatar: respuesta.avatar
-                });
-
-                $('#colBotonJugar').removeClass('d-none');
             }
         },
         error: function () {
@@ -132,7 +126,7 @@ function ejecutarRegistro() {
                 $('#modalRegistro').modal('hide');
                 $('#formRegistro')[0].reset();
 
-                // Autologin
+                // Autologin tras registrarse
                 $('#loginEmail').val(email);
                 $('#loginPassword').val(pass);
                 ejecutarLogin();
@@ -165,15 +159,21 @@ function cerrarSesion() {
 }
 
 /**
- * Renderiza la cabecera cuando HAY sesión iniciada (Menú Perfil + Avatar)
+ * Renderiza la cabecera cuando HAY sesión iniciada 
  */
 function renderizarCabeceraConSesion(usuario) {
-    let avatarImg = usuario.avatar ? usuario.avatar : 'img/avatars/defaultProfile.jpeg';
+    //Mete el avatar en cabecera
+    let avatarImg = (usuario.avatar && usuario.avatar !== '') 
+        ? usuario.avatar 
+        : 'img/avatars/defaultProfile.jpeg';
+
+    // Refrescar avatar si se cambia
+    let avatarConCacheBuster = avatarImg + '?v=' + new Date().getTime();
 
     let html = `
         <div class="dropdown">
             <button class="btn btn-outline-light dropdown-toggle d-flex align-items-center gap-2 rounded-pill px-3 py-1 fw-bold" type="button" id="userMenu" data-bs-toggle="dropdown" aria-expanded="false">
-                <img src="${avatarImg}" alt="Avatar" class="rounded-circle border" style="width: 32px; height: 32px; object-fit: cover;">
+                <img src="${avatarConCacheBuster}" alt="Avatar" class="rounded-circle border" style="width: 32px; height: 32px; object-fit: cover;">
                 <span>${escapeHtml(usuario.nombreUsuario)}</span>
             </button>
             <ul class="dropdown-menu dropdown-menu-end shadow border-0 mt-2" aria-labelledby="userMenu">
@@ -195,13 +195,11 @@ function renderizarCabeceraConSesion(usuario) {
 }
 
 /**
- * Renderiza los botones originales:
+ * Renderiza los botones en la cabecera cuando NO hay sesión (Iniciar sesión azul)
- * - "Iniciar Sesión" (Rojo -> btn-danger)
- * - "Registrarse" (Verde -> btn-success)
  */
 function renderizarCabeceraSinSesion() {
     let html = `
-        <button class="btn btn-danger rounded-pill px-4 fw-bold shadow-sm" data-bs-toggle="modal" data-bs-target="#modalLogin">
+        <button class="btn btn-primary rounded-pill px-4 fw-bold shadow-sm" data-bs-toggle="modal" data-bs-target="#modalLogin">
             <i class="fa-solid fa-right-to-bracket me-1"></i> Iniciar sesión
         </button>
         <button class="btn btn-success rounded-pill px-4 fw-bold shadow-sm" data-bs-toggle="modal" data-bs-target="#modalRegistro">

js/perfil.js

@@ -26,6 +26,10 @@ $.ajax({
 function cambiarNombre() {
   let nuevoNombre = $("#nuevoNombre").val().trim();
 
+  if (nuevoNombre === "") {
+    return;
+  }
+
   $.ajax({
     method: "POST",
     url: "controladores/controladorUsuario.php",
@@ -42,8 +46,13 @@ function cambiarNombre() {
       // Actualizamos el nombre mostrado en la página
       $("#nombre").text(respuesta.nombreUsuario);
 
-      // Limpiamos el input
+      
       $("#nuevoNombre").val("");
+
+    
+      if (typeof comprobarEstadoSesion === "function") {
+        comprobarEstadoSesion();
+      }
     }
   });
 }
@@ -74,14 +83,11 @@ $("#inputFoto").change(function () {
   // Comprobamos que sea una imagen
   if (!archivo.type.startsWith("image/")) {
     alert("El archivo seleccionado no es una imagen.");
-
     $("#inputFoto").val("");
-
     return;
   }
 
   let datos = new FormData();
-
   datos.append("avatar", archivo);
   datos.append("orden", "cambiarAvatar");
 
@@ -97,18 +103,23 @@ $("#inputFoto").change(function () {
     if (respuesta.error) {
       alert(respuesta.error);
     } else {
-      // Actualizamos la imagen sin recargar la página
+      // Actualizamos la imagen en la página de perfil sin recargar
       $("#fotoPerfil").attr(
         "src",
-        respuesta.avatar + "?t=" + new Date().getTime(),
+        respuesta.avatar + "?t=" + new Date().getTime()
       );
+
+      // Actualizamos también el avatar superior en la cabecera 
+      if (typeof comprobarEstadoSesion === "function") {
+        comprobarEstadoSesion();
+      }
     }
   });
 });
 
 // =========================
 // RECOGER CATEGORIAS
-// ====
+// =========================
 
 $("#modalCrearPregunta").on("show.bs.modal", function () {
   const selectCategoria = $("#idCategoria");
@@ -141,16 +152,14 @@ $("#modalCrearPregunta").on("show.bs.modal", function () {
           $("<option>", {
             value: categoria.idCategoria,
             text: categoria.nombreCategoria,
-          }),
+          })
         );
       });
     },
 
     error: function (xhr, status, error) {
       console.error("Error al cargar las categorías:", error);
-
       console.error("Respuesta del servidor:", xhr.responseText);
-
       alert("No se han podido cargar las categorías.");
     },
   });
@@ -158,7 +167,7 @@ $("#modalCrearPregunta").on("show.bs.modal", function () {
 
 // =========================
 // CREAR PREGUNTA
-// ====
+// =========================
 
 $("#formCrearPregunta").on("submit", function (e) {
   e.preventDefault();
@@ -188,24 +197,15 @@ $("#formCrearPregunta").on("submit", function (e) {
 
   $.ajax({
     url: "controladores/controladorPregunta.php",
-
     type: "POST",
-
     dataType: "json",
-
     data: {
       orden: "crearPregunta",
-
       idCategoria: $("#idCategoria").val(),
-
       nombrePregunta: pregunta,
-
       respuestaCorrecta: respuestaCorrecta,
-
       respuestaIncorrecta1: respuestaIncorrecta1,
-
       respuestaIncorrecta2: respuestaIncorrecta2,
-
       respuestaIncorrecta3: respuestaIncorrecta3,
     },
 
@@ -214,10 +214,8 @@ $("#formCrearPregunta").on("submit", function (e) {
 
       if (respuesta.error) {
         alert(respuesta.error);
-
         boton.prop("disabled", false);
         boton.text("Crear pregunta");
-
         return;
       }
 
@@ -233,21 +231,16 @@ $("#formCrearPregunta").on("submit", function (e) {
         // Volvemos a activar el botón
         boton.prop("disabled", false);
         boton.text("Crear pregunta");
-
-        // Si tienes que actualizar la lista de preguntas:
-        // cargarPreguntas();
       }
     },
 
     error: function (xhr, status, error) {
       console.error("Error al crear la pregunta:", error);
-
       console.error("Respuesta del servidor:", xhr.responseText);
-
       alert("No se ha podido crear la pregunta.");
 
       boton.prop("disabled", false);
       boton.text("Crear pregunta");
     },
   });
-});
+});