fix(studio): don't allow same name in two nodes

botpress · Dec 4, 2019 · be48b7e · be48b7e
1 parent 3a06e25
commit be48b7e
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 2 deletions.
diff --git a/src/bp/ui-studio/src/web/views/FlowBuilder/nodeProps/SkillCallNode.jsx b/src/bp/ui-studio/src/web/views/FlowBuilder/nodeProps/SkillCallNode.jsx
@@ -10,7 +10,13 @@ const style = require('./style.scss')
 
 export default class SkillCallNodePropertiesPanel extends Component {
   renameNode = text => {
-    if (text && text !== this.props.node.name) {
+    if (text) {
+      let nodes = this.props.flow.nodes;
+      for (let i = 0; i < nodes.length; i++) {
+        if (text === nodes[i].name) {
+          return;
+        }
+      }
       this.props.updateNode({ name: text })
     }
   }

diff --git a/src/bp/ui-studio/src/web/views/FlowBuilder/nodeProps/StandardNode.jsx b/src/bp/ui-studio/src/web/views/FlowBuilder/nodeProps/StandardNode.jsx
@@ -11,7 +11,13 @@ const style = require('./style.scss')
 
 export default class StandardNodePropertiesPanel extends Component {
   renameNode = text => {
-    if (text && text !== this.props.node.name) {
+    if (text) {
+      let nodes = this.props.flow.nodes;
+      for (let i = 0; i < nodes.length; i++) {
+        if (text === nodes[i].name) {
+          return;
+        }
+      }
       this.props.updateNode({ name: text })
     }
   }