From e0fa2dec3de9122932d6931f50c8f7bbe1efb7c0 Mon Sep 17 00:00:00 2001
From: "Joseph.Rawlings" <root0091@gmail.com>
Date: Mon, 1 Dec 2025 17:33:07 -0500
Subject: [PATCH] refactor: update Proxmox layout to include 'firewall' in
 advanced view checks and ensure consistent handling of advanced views

---
 .../Servers/Proxmox/views/components/layout.blade.php     | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/extensions/Servers/Proxmox/views/components/layout.blade.php b/extensions/Servers/Proxmox/views/components/layout.blade.php
index 335d3b4e3..11c88df05 100644
--- a/extensions/Servers/Proxmox/views/components/layout.blade.php
+++ b/extensions/Servers/Proxmox/views/components/layout.blade.php
@@ -17,7 +17,7 @@
 
         <div class="flex flex-col flex-1" x-data="{
             isAdvancedView: localStorage.getItem('proxmox-view-mode') === 'advanced',
-            isAdvancedOnlyView: {{ in_array($view, ['boot-order', 'cloud-init', 'display', 'system', 'subdomain', 'statistics', 'help']) ? 'true' : 'false' }},
+            isAdvancedOnlyView: {{ in_array($view, ['boot-order', 'cloud-init', 'display', 'system', 'subdomain', 'statistics', 'help', 'firewall']) ? 'true' : 'false' }},
             toggleView() {
                 if (!this.isAdvancedOnlyView) {
                     this.isAdvancedView = !this.isAdvancedView;
@@ -33,7 +33,7 @@
                 isAdvancedView = false;
             }">
             @php
-                $advancedOnlyViews = ['boot-order', 'cloud-init', 'display', 'system', 'subdomain', 'statistics', 'help'];
+                $advancedOnlyViews = ['boot-order', 'cloud-init', 'display', 'system', 'subdomain', 'statistics', 'help', 'firewall'];
                 $isAdvancedOnlyView = in_array($view, $advancedOnlyViews);
             @endphp
             
@@ -153,7 +153,7 @@
                     ['label' => 'Overview', 'view' => 'overview', 'route' => route('proxmox.server.overview', ['server' => $server->id])],
                     ['label' => 'Statistics', 'view' => 'statistics', 'route' => route('proxmox.server.statistics', ['server' => $server->id]), 'advanced' => true],
                     ['label' => 'Boot Settings', 'view' => 'boot-order', 'route' => route('proxmox.server.boot-order', ['server' => $server->id]), 'advanced' => true],
-                    ['label' => 'Firewall', 'view' => 'firewall', 'route' => route('proxmox.server.firewall', ['server' => $server->id])],
+                    ['label' => 'Firewall', 'view' => 'firewall', 'route' => route('proxmox.server.firewall', ['server' => $server->id]), 'advanced' => true],
                     ['label' => 'Network', 'view' => 'network', 'route' => route('proxmox.server.network', ['server' => $server->id])],
                     ['label' => 'Subdomain', 'view' => 'subdomain', 'route' => route('proxmox.server.subdomain', ['server' => $server->id]), 'advanced' => true],
                     ['label' => 'Cloud-init', 'view' => 'cloud-init', 'route' => route('proxmox.server.cloud-init', ['server' => $server->id]), 'advanced' => true],
@@ -275,7 +275,7 @@
                 <div class="lg:hidden" x-data="{ 
                     open: false, 
                     isAdvancedView: localStorage.getItem('proxmox-view-mode') === 'advanced',
-                    isAdvancedOnlyView: {{ in_array($view, ['boot-order', 'cloud-init', 'display', 'system', 'subdomain', 'statistics', 'help']) ? 'true' : 'false' }},
+                    isAdvancedOnlyView: {{ in_array($view, ['boot-order', 'cloud-init', 'display', 'system', 'subdomain', 'statistics', 'help', 'firewall']) ? 'true' : 'false' }},
                     toggleView() {
                         if (!this.isAdvancedOnlyView) {
                             this.isAdvancedView = !this.isAdvancedView;