Merge pull request #3921 from Imat00/bug/importer-rework-FortiManager-Autodiscovery

tpurschke · web-flow · commit 08dc12124c6b · 2025-11-15T08:28:39.000+01:00
fix/importer-rework forti manager autodiscovery #3448
diff --git a/roles/ui/files/FWO.UI/Shared/AutoDiscovery.razor b/roles/ui/files/FWO.UI/Shared/AutoDiscovery.razor
@@ -624,8 +624,9 @@
             {
                 return;
             }
-            Management mgmtToIgnore = managementsToDisplay[managementsToDisplay.FindIndex(x => x.Id == discoveredDevice.Management.Id)];
-            if(mgmtToIgnore.Devices.FirstOrDefault(y => y.Relevant == true) == null)
+            int index = managementsToDisplay.FindIndex(x => x.Id == discoveredDevice.Management.Id);
+            Management? mgmtToIgnore = index >= 0 ? managementsToDisplay[index] : null;
+            if (mgmtToIgnore != null &&  mgmtToIgnore.Devices.FirstOrDefault(y => y.Relevant == true) == null)
             {
                 managementsToDisplay.Remove(mgmtToIgnore);
                 if(managementsToDisplay.Count == 0)

Original file line number	Diff line number	Diff line change
`@@ -624,8 +624,9 @@`
`624`	`624`	`{`
`625`	`625`	`return;`
`626`	`626`	`}`
`627`		`- Management mgmtToIgnore = managementsToDisplay[managementsToDisplay.FindIndex(x => x.Id == discoveredDevice.Management.Id)];`
`628`		`- if(mgmtToIgnore.Devices.FirstOrDefault(y => y.Relevant == true) == null)`
	`627`	`+ int index = managementsToDisplay.FindIndex(x => x.Id == discoveredDevice.Management.Id);`
	`628`	`+ Management? mgmtToIgnore = index >= 0 ? managementsToDisplay[index] : null;`
	`629`	`+ if (mgmtToIgnore != null && mgmtToIgnore.Devices.FirstOrDefault(y => y.Relevant == true) == null)`
`629`	`630`	`{`
`630`	`631`	`managementsToDisplay.Remove(mgmtToIgnore);`
`631`	`632`	`if(managementsToDisplay.Count == 0)`