Merge pull request #1290 from ProcessMaker/bugfix/FOUR-7111

Bugfix/FOUR-7111: Previewing a screen that contains a nested screen and another control causes two 404 error calls
ProcessMaker · Nov 22, 2022 · d90fabf · d90fabf
2 parents 6c7dc49 + f83179a
commit d90fabf
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/src/DataProvider.js b/src/DataProvider.js
@@ -99,6 +99,10 @@ export default {
     });
   },
   getScreen(id, query = "") {
+    if (!id) {
+      return null;
+    }
+
     const cachedPromise = this.cachedScreenPromises.find(
       (item) => item.id === id && item.query === query
     );

diff --git a/src/ValidationsFactory.js b/src/ValidationsFactory.js
@@ -89,6 +89,9 @@ class FormNestedScreenValidations extends Validations {
   }
 
   async loadScreen(id) {
+    if (!id) {
+      return null;
+    }
     if (!globalObject['nestedScreens']) {
       globalObject['nestedScreens'] = {};
     }
@@ -99,7 +102,6 @@ class FormNestedScreenValidations extends Validations {
     globalObject.nestedScreens['id_' + id] = response.data.config;
     return response.data.config;
   }
-
 }
 
 /**