From b0980969302780b5cb9156c20b3231a6adf14d03 Mon Sep 17 00:00:00 2001
From: Yasen Pramatarov <yasen@lindeas.com>
Date: Tue, 15 Apr 2025 22:40:29 +0300
Subject: [PATCH] Uses $userId instead of session var

---
 app/templates/page-menu.php | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/app/templates/page-menu.php b/app/templates/page-menu.php
index 502dbfd..943db99 100644
--- a/app/templates/page-menu.php
+++ b/app/templates/page-menu.php
@@ -65,20 +65,20 @@
                     </a>
                     <div class="dropdown-menu dropdown-menu-right">
                         <h6 class="dropdown-header">system</h6>
-<?php   if ($userObject->hasRight($_SESSION['user_id'], 'view config file')) {?>
+<?php   if ($userObject->hasRight($userId, 'view config file')) {?>
                         <a class="dropdown-item" href="<?= htmlspecialchars($app_root) ?>?page=config">
                             <i class="fas fa-wrench"></i>Configuration
                         </a>
 <?php   } ?>
-<?php   if ($userObject->hasRight($_SESSION['user_id'], 'superuser') ||
-          $userObject->hasRight($_SESSION['user_id'], 'edit whitelist') ||
-          $userObject->hasRight($_SESSION['user_id'], 'edit blacklist') ||
-          $userObject->hasRight($_SESSION['user_id'], 'edit ratelimiting')) { ?>
+<?php   if ($userObject->hasRight($userId, 'superuser') ||
+          $userObject->hasRight($userId, 'edit whitelist') ||
+          $userObject->hasRight($userId, 'edit blacklist') ||
+          $userObject->hasRight($userId, 'edit ratelimiting')) { ?>
                         <a class="dropdown-item" href="<?= htmlspecialchars($app_root) ?>?page=security">
                             <i class="fas fa-shield-alt"></i>Security
                         </a>
 <?php   } ?>
-<?php   if ($userObject->hasRight($_SESSION['user_id'], 'view logs')) {?>
+<?php   if ($userObject->hasRight($userId, 'view app logs')) {?>
                         <a class="dropdown-item" href="<?= htmlspecialchars($app_root) ?>?page=logs">
                             <i class="fas fa-list"></i>Logs
                         </a>