From ba2751141455c2060cacb59794c30f27906fb7b9 Mon Sep 17 00:00:00 2001
From: Lizandro Guarnizo <77708265+lizandrogd@users.noreply.github.com>
Date: Wed, 24 Jun 2026 08:52:26 -0500
Subject: [PATCH] fix(sidebar): null-check en route de children para evitar
 TypeError en PHP 8.2

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 modules/turnero/views/recepcion.php | 2 +-
 shared/components/sidebar.php       | 7 ++++---
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/modules/turnero/views/recepcion.php b/modules/turnero/views/recepcion.php
index dfa13e8..5c08542 100644
--- a/modules/turnero/views/recepcion.php
+++ b/modules/turnero/views/recepcion.php
@@ -78,7 +78,7 @@ $adminNombre = $_SESSION['admin_user']['full_name'] ?? $_SESSION['admin_user']['
     <title><?= htmlspecialchars($deskNombre ?? 'Recepción') ?> — Turnero</title>
     <link href="https://cdn.jsdelivr.net/npm/bootstrap@5.3.0/dist/css/bootstrap.min.css" rel="stylesheet">
     <link href="https://cdnjs.cloudflare.com/ajax/libs/font-awesome/6.4.0/css/all.min.css" rel="stylesheet">
-    <link href="<?= BASE_URL ?>assets/css/styles.css?v=12" rel="stylesheet">
+    <link href="<?= BASE_URL ?>assets/css/styles.css?v=14" rel="stylesheet">
     <style>
         /* ── Layout de dos columnas ── */
         .rec-layout {
diff --git a/shared/components/sidebar.php b/shared/components/sidebar.php
index f6e8816..d078b33 100644
--- a/shared/components/sidebar.php
+++ b/shared/components/sidebar.php
@@ -177,7 +177,7 @@ if (class_exists('ModuleRegistry', false)) {
                         // Sub-acordeón para links con hijos
                         $subId    = $catId . '-sub-' . $slug . '-' . $lnkIdx;
                         $hasActive = false;
-                        foreach ($children as $ch) { if ($_isActive($ch['route']) !== '') { $hasActive = true; break; } }
+                        foreach ($children as $ch) { if (!empty($ch['route']) && $_isActive($ch['route']) !== '') { $hasActive = true; break; } }
 ?>
                 <li class="sb-sub-header<?= $hasActive ? ' has-active' : '' ?>"
                     onclick="sbToggleSub('<?= $subId ?>',this,event)">
@@ -188,8 +188,9 @@ if (class_exists('ModuleRegistry', false)) {
                 <li class="sb-sub-group<?= $hasActive ? '' : ' collapsed' ?>" id="<?= $subId ?>">
                     <ul>
 <?php           foreach ($children as $ch):
-                    $href   = $_base . $ch['route'];
-                    $active = $_isActive($ch['route']);
+                    $chRoute = $ch['route'] ?? '';
+                    $href    = $chRoute ? $_base . $chRoute : '#';
+                    $active  = $chRoute ? $_isActive($chRoute) : '';
 ?>
                         <li><a href="<?= htmlspecialchars($href, ENT_QUOTES, 'UTF-8') ?>"
                                class="nav-link nav-link-sub<?= $active ?>">