fix: corregir repeticion del nombre de aula en detalles de reserva y selector

CHANGELOG_SESSION.md

@@ -307,6 +307,19 @@
   * **Navegación:** Acceso directo a *Comisiones y Cursadas* disponible en la barra superior (menús *Schedule* y *Management*).
   * **Modales interactivos:** Creación de comisión (`#modalAddCommission`), edición (`#modalEditCommission`), configuración de aula virtual (`#modalVirtualLink`), programación de hitos (`#modalAddMilestone`) y eliminación segura (`#deleteCommissionModal`).
 
+---
+
+## 19. Corrección de Redundancia en Nombre de Aula en Formulario y Resumen de Reserva
+* **Desplegable de Selección de Aula ([app/routes/schedule.py](file:///c:/Workspace/admin-edu-space/app/routes/schedule.py)):**
+  * Se corrigió la tupla de opciones en `form.classroom_id.choices`. Anteriormente concatenaba `c.code_display` y `c.name` (ambas propiedades computaban `"Edificio Central-Aula 204"`), provocando el texto repetido `Edificio Central-Aula 204 - Edificio Central-Aula 204 (Cap: 35)`.
+  * Se simplificó a `f'{c.building} - {c.code} (Cap: {c.capacity_display})'`, mostrando limpiamente `Edificio Central - Aula 204 (Cap: 35)`.
+* **Cuadro de Resumen en Tiempo Real ([app/templates/schedule/add.html](file:///c:/Workspace/admin-edu-space/app/templates/schedule/add.html)):**
+  * Se corrigió la función JavaScript `updateSummary()` que concatenaba `${code_display} - ${name}`.
+  * Ahora renderiza de forma limpia y directa `${classroomData[classroomId].code_display} (${capLabel}: ${classroomData[classroomId].capacity})`.
+* **Formato de Códigos de Comisión ([app/models/subject.py](file:///c:/Workspace/admin-edu-space/app/models/subject.py)):**
+  * Se optimizó `Commission.get_full_code()` para no repetir el año cuando ya viene incluido en el string del semestre (ej: `2026-2`).
+
+
 
 
 

app/models/subject.py

@@ -98,7 +98,20 @@ class Commission(db.Model):
     
     def get_full_code(self):
         """Get full commission code"""
-        return f"{self.subject.code if self.subject else 'N/A'}-{self.code}-{self.semester.upper()}{self.year if self.year else ''}"
+        subj = self.subject.code if self.subject else 'N/A'
+        code = self.code or ''
+        sem = str(self.semester or '').strip()
+        yr = str(self.year or '').strip()
+
+        if yr and yr in sem:
+            return f"{subj}-{code}-{sem}"
+        elif sem and yr:
+            return f"{subj}-{code}-{sem}{yr}"
+        elif sem:
+            return f"{subj}-{code}-{sem}"
+        elif yr:
+            return f"{subj}-{code}-{yr}"
+        return f"{subj}-{code}"
     
     def __repr__(self):
         return f'<Commission {self.get_full_code()}>'

app/routes/schedule.py

@@ -410,7 +410,7 @@ def add_reservation():
     commissions = Commission.query.filter_by(active=True).all()
     
     # Populate form choices
-    form.classroom_id.choices = [(c.id, f'{c.code_display} - {c.name} (Cap: {c.capacity})') for c in classrooms]
+    form.classroom_id.choices = [(c.id, f'{c.building} - {c.code} (Cap: {c.capacity_display})') for c in classrooms]
     form.commission_id.choices = [(c.id, f'{c.get_full_code()} - {c.subject.name if c.subject else "No Subject"} (Students: {c.max_students})') for c in commissions]
     
     if request.method == 'POST':

app/templates/schedule/add.html

@@ -185,9 +185,8 @@
 {
     {% for classroom in classroom_list %}
     "{{ classroom.id }}": {
-        "capacity": {{ classroom.capacity }},
-        "code_display": {{ classroom.code_display|tojson }},
-        "name": {{ classroom.name|tojson }}
+        "capacity": {{ classroom.capacity_display|tojson }},
+        "code_display": {{ (classroom.building ~ ' - ' ~ classroom.code)|tojson }}
     }{% if not loop.last %},{% endif %}
     {% endfor %}
 }
@@ -240,7 +239,7 @@ $(document).ready(function() {
         // Update classroom summary
         if (classroomId && classroomData[classroomId]) {
             const capLabel = i18n.capacity || 'Capacidad';
-            $('#summary-classroom').text(`${classroomData[classroomId].code_display} - ${classroomData[classroomId].name} (${capLabel}: ${classroomData[classroomId].capacity})`);
+            $('#summary-classroom').text(`${classroomData[classroomId].code_display} (${capLabel}: ${classroomData[classroomId].capacity})`);
         } else {
             $('#summary-classroom').text(i18n.not_selected || 'No seleccionada');
         }