Removing UI before creating new one - prevents race cond. Fixes #359 (#…

…360)
deepforge-dev · Jun 24, 2016 · 68e4e06 · 68e4e06
1 parent 4e03997
commit 68e4e06
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/src/common/globals.js b/src/common/globals.js
@@ -102,6 +102,10 @@ define([
         DeepForge.places = {};
 
         // Create a territory
+        if (placesTerritoryId) {
+            client.removeUI(placesTerritoryId);
+        }
+
         territory[CONSTANTS.PROJECT_ROOT_ID] = {children: 1};
         placesTerritoryId = client.addUI(null, updateDeepForgePlaces);
 
@@ -126,6 +130,7 @@ define([
 
         // Remove the territory
         client.removeUI(placesTerritoryId);
+        placesTerritoryId = null;
     };
 
     // Add DeepForge action primitives