From 3ca9eb60c364e596ce6b168d39712f2f08a3073b Mon Sep 17 00:00:00 2001
From: Prophet731 <Prophet731@users.noreply.github.com>
Date: Thu, 19 Mar 2026 14:18:12 -0500
Subject: [PATCH] fix: force generic icon on 'Other' category even when API
 provides linux_logo.png

VirtFusion API returns an 'Other' category with icon=linux_logo.png by default.
Null out the icon in groupOsTemplates() so the JS SVG fallback renders instead.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
---
 modules/servers/VirtFusionDirect/lib/Module.php | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/modules/servers/VirtFusionDirect/lib/Module.php b/modules/servers/VirtFusionDirect/lib/Module.php
index d68af3e..24b2cb4 100644
--- a/modules/servers/VirtFusionDirect/lib/Module.php
+++ b/modules/servers/VirtFusionDirect/lib/Module.php
@@ -303,9 +303,10 @@ class Module
             if (count($catTemplates) <= 1) {
                 $otherTemplates = array_merge($otherTemplates, $catTemplates);
             } else {
+                $catName = $osCategory['name'] ?? 'Unknown';
                 $categories[] = [
-                    'name' => $esc($osCategory['name'] ?? 'Unknown'),
-                    'icon' => $osCategory['icon'] ?? null,
+                    'name' => $esc($catName),
+                    'icon' => ($catName === 'Other') ? null : ($osCategory['icon'] ?? null),
                     'templates' => $catTemplates,
                 ];
             }