From b8b605cff0d54d8b5570c4fabb617d74dc72813c Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sat, 14 Feb 2026 13:30:42 +0000
Subject: [PATCH] Improve accessibility and file handling in CRUD

Co-authored-by: naielv <109038805+naielv@users.noreply.github.com>
---
 public_html/entreaulas/alumnos.php | 14 ++++++++------
 public_html/entreaulas/aulario.php |  2 +-
 2 files changed, 9 insertions(+), 7 deletions(-)

diff --git a/public_html/entreaulas/alumnos.php b/public_html/entreaulas/alumnos.php
index 8acbbf7..b27073a 100644
--- a/public_html/entreaulas/alumnos.php
+++ b/public_html/entreaulas/alumnos.php
@@ -82,8 +82,9 @@ switch ($_GET["form"] ?? '') {
             // Validate image
             $image_info = getimagesize($tmp_name);
             if ($image_info !== false) {
-                move_uploaded_file($tmp_name, $photo_path);
-                chmod($photo_path, 0644);
+                if (move_uploaded_file($tmp_name, $photo_path)) {
+                    chmod($photo_path, 0644);
+                }
             }
         }
         
@@ -133,8 +134,9 @@ switch ($_GET["form"] ?? '') {
             // Validate image
             $image_info = getimagesize($tmp_name);
             if ($image_info !== false) {
-                move_uploaded_file($tmp_name, $photo_path);
-                chmod($photo_path, 0644);
+                if (move_uploaded_file($tmp_name, $photo_path)) {
+                    chmod($photo_path, 0644);
+                }
             }
         }
         
@@ -223,7 +225,7 @@ switch ($_GET["action"] ?? '') {
                     <?php if ($photo_exists): ?>
                         <div class="mb-2">
                             <img src="_filefetch.php?type=alumno_photo&alumno=<?= urlencode($nombre) ?>&centro=<?= urlencode($centro_id) ?>&aulario=<?= urlencode($aulario_id) ?>" 
-                                 alt="<?= htmlspecialchars($nombre) ?>" 
+                                 alt="Foto de <?= htmlspecialchars($nombre) ?>" 
                                  style="max-width: 200px; max-height: 200px; border: 2px solid #ddd; border-radius: 10px;">
                         </div>
                     <?php else: ?>
@@ -311,7 +313,7 @@ switch ($_GET["action"] ?? '') {
                             <td>
                                 <?php if ($photo_exists): ?>
                                     <img src="_filefetch.php?type=alumno_photo&alumno=<?= urlencode($nombre) ?>&centro=<?= urlencode($centro_id) ?>&aulario=<?= urlencode($aulario_id) ?>" 
-                                         alt="<?= htmlspecialchars($nombre) ?>" 
+                                         alt="Foto de <?= htmlspecialchars($nombre) ?>" 
                                          style="width: 50px; height: 50px; object-fit: cover; border-radius: 5px;">
                                 <?php else: ?>
                                     <div style="width: 50px; height: 50px; background: #f0f0f0; display: flex; align-items: center; justify-content: center; border-radius: 5px; border: 2px dashed #ccc;">
diff --git a/public_html/entreaulas/aulario.php b/public_html/entreaulas/aulario.php
index 6f34a2a..1d502bc 100644
--- a/public_html/entreaulas/aulario.php
+++ b/public_html/entreaulas/aulario.php
@@ -22,7 +22,7 @@ $aulario = json_decode(file_get_contents("/DATA/entreaulas/Centros/$centro_id/Au
     </a>
     <?php if (in_array("entreaulas:docente", $_SESSION["auth_data"]["permissions"] ?? [])): ?>
         <a href="/entreaulas/alumnos.php?aulario=<?= urlencode($aulario_id) ?>" class="btn btn-info grid-item">
-            <img src="/static/arasaac/alumnos.png" height="125" style="background: white; padding: 5px; border-radius: 10px;">
+            <img src="/static/arasaac/alumnos.png" height="125" style="background: white; padding: 5px; border-radius: 10px;" alt="Icono de gestión de alumnos">
             <br>
             Gestión de Alumnos
         </a>