hotfix: Reorder topology loading in scenario data to prevent overwrites by event handlers.

Author: FerTV

nebula/frontend/static/js/deployment/scenario.js

@@ -1,8 +1,8 @@
 // Scenario Management Module
-const ScenarioManager = (function() {
+const ScenarioManager = (function () {
     let scenariosList = [];
     let actual_scenario = 0;
-    let physical_ips     = [];
+    let physical_ips = [];
 
     // Initialize scenarios from session storage
     function initializeScenarios() {
@@ -151,28 +151,7 @@ const ScenarioManager = (function() {
         document.getElementById("rounds").value = scenario.rounds;
 
         // Load topology
-        if (scenario.nodes && scenario.nodes_graph) {
-            const topologyData = {
-                nodes: Object.values(scenario.nodes),
-                links: []
-            };
-
-            // Reconstruct links from the nodes' neighbors
-            topologyData.nodes.forEach(node => {
-                if (node.neighbors) {
-                    node.neighbors.forEach(neighborId => {
-                        topologyData.links.push({
-                            source: node.id,
-                            target: neighborId
-                        });
-                    });
-                }
-            });
 
-            window.TopologyManager.setData(topologyData);
-        } else {
-            window.TopologyManager.generatePredefinedTopology();
-        }
 
         /*  if "physical" assign the IPs again*/
         if (scenario.physical_ips) {
@@ -247,6 +226,30 @@ const ScenarioManager = (function() {
         document.getElementById("federationArchitecture").dispatchEvent(new Event('change'));
         document.getElementById("datasetSelect").dispatchEvent(new Event('change'));
         document.getElementById("iidSelect").dispatchEvent(new Event('change'));
+
+        // Load topology last to avoid overwrites by event handlers
+        if (scenario.nodes && scenario.nodes_graph) {
+            const topologyData = {
+                nodes: Object.values(scenario.nodes),
+                links: []
+            };
+
+            // Reconstruct links from the nodes' neighbors
+            topologyData.nodes.forEach(node => {
+                if (node.neighbors) {
+                    node.neighbors.forEach(neighborId => {
+                        topologyData.links.push({
+                            source: node.id,
+                            target: neighborId
+                        });
+                    });
+                }
+            });
+
+            window.TopologyManager.setData(topologyData);
+        } else {
+            window.TopologyManager.generatePredefinedTopology();
+        }
     }
 
     function saveScenario() {
@@ -353,16 +356,16 @@ const ScenarioManager = (function() {
     function setPhysicalIPs(ipList = []) {
         physical_ips = [...ipList];
     }
- 
+
     function setActualScenario(index) {
         actual_scenario = index;
         if (scenariosList[index]) {
             // Clear the current graph
             window.TopologyManager.clearGraph();
-            
+
             // Load new scenario data
             loadScenarioData(scenariosList[index]);
-            
+
             // If physical deployment, set physical IPs
             if (scenariosList[index].deployment === 'physical' && scenariosList[index].physical_ips) {
                 window.TopologyManager.setPhysicalIPs(scenariosList[index].physical_ips);