fix for possible race condition in setting edit mode

alan-turing-institute · Apr 25, 2022 · 988f39f · 988f39f
1 parent 48e0bc5
commit 988f39f
Showing 1 changed file with 15 additions and 7 deletions.
diff --git a/frontend/src/components/CaseContainer.js b/frontend/src/components/CaseContainer.js
@@ -66,7 +66,7 @@ class CaseContainer extends Component {
       headers: { "Content-Type": "application/json" },
       body: JSON.stringify(changeObj),
     };
-    fetch(backendURL, requestOptions);
+    return fetch(backendURL, requestOptions);
   }
 
   deleteCurrentCase() {
@@ -158,7 +158,7 @@ class CaseContainer extends Component {
     this.hideViewLayer();
     this.hideEditLayer();
     this.hideCreateLayer();
-    this.fetchData(this.state.id);
+    return this.fetchData(this.state.id);
   }
 
   showViewLayer(e) {
@@ -372,25 +372,33 @@ class CaseContainer extends Component {
 
   enableEditing() {
     if (!this.state.assurance_case.lock_uuid) {
-      this.submitCaseChange("lock_uuid", this.state.session_id);
+      this.submitCaseChange("lock_uuid", this.state.session_id).then(
+        (response) => {
+          this.updateView();
+        }
+      );
     } else if (this.state.assurance_case.lock_uuid !== this.state.session_id) {
       // override!
       if (
         window.confirm(
           "Are you sure?  You might be overwriting someone's work..."
         )
       ) {
-        this.submitCaseChange("lock_uuid", this.state.session_id);
+        this.submitCaseChange("lock_uuid", this.state.session_id).then(
+          (response) => {
+            this.updateView();
+          }
+        );
       }
     }
-    this.updateView();
   }
 
   disableEditing() {
     if (this.state.assurance_case.lock_uuid) {
-      this.submitCaseChange("lock_uuid", null);
+      this.submitCaseChange("lock_uuid", null).then((response) => {
+        this.updateView();
+      });
     }
-    this.updateView();
   }
 
   inEditMode() {