diff --git a/editor/js/ui/deploy.js b/editor/js/ui/deploy.js index 6cba93fae..d2c413521 100644 --- a/editor/js/ui/deploy.js +++ b/editor/js/ui/deploy.js @@ -212,18 +212,7 @@ RED.deploy = (function() { tabLabel = tab.label; } } - var label = ""; - if (typeof node._def.label == "function") { - try { - label = node._def.label.call(node); - } catch(err) { - console.log("Definition error: "+node_def.type+".label",err); - label = node_def.type; - } - } else { - label = node._def.label; - } - label = label || node.id; + var label = RED.utils.getNodeLabel(node,node.id); return {tab:tabLabel,type:node.type,label:label}; } function sortNodeInfo(A,B) { diff --git a/editor/js/ui/editor.js b/editor/js/ui/editor.js index de1e691dd..1310f6cbb 100644 --- a/editor/js/ui/editor.js +++ b/editor/js/ui/editor.js @@ -265,17 +265,8 @@ RED.editor = (function() { var configNode = RED.nodes.node(node[property]); var node_def = RED.nodes.getType(type); - if (configNode && node_def.label) { - if (typeof node_def.label == "function") { - try { - label = node_def.label.call(configNode); - } catch(err) { - console.log("Definition error: "+node_def.type+".label",err); - label = node_def.type; - } - } else { - label = node_def.label; - } + if (configNode) { + label = RED.utils.getNodeLabel(configNode,configNode.id); } input.val(label); } @@ -1207,17 +1198,7 @@ RED.editor = (function() { RED.nodes.eachConfig(function(config) { if (config.type == type && (!config.z || config.z === activeWorkspace.id)) { - var label = ""; - if (typeof node_def.label == "function") { - try { - label = node_def.label.call(config); - } catch(err) { - console.log("Definition error: "+node_def.type+".label",err); - label = node_def.type; - } - } else { - label = node_def.label; - } + var label = RED.utils.getNodeLabel(config,config.id); config.__label__ = label; configNodes.push(config); } diff --git a/editor/js/ui/search.js b/editor/js/ui/search.js index 18565fd13..693154436 100644 --- a/editor/js/ui/search.js +++ b/editor/js/ui/search.js @@ -27,19 +27,11 @@ RED.search = (function() { var results = []; function indexNode(n) { - var l = ""; - if (n._def && n._def.label) { - l = n._def.label; - try { - l = (typeof l === "function" ? l.call(n) : l); - if (l) { - l = (""+l).toLowerCase(); - index[l] = index[l] || {}; - index[l][n.id] = {node:n,label:l} - } - } catch(err) { - console.log("Definition error: "+n.type+".label",err); - } + var l = RED.utils.getNodeLabel(n); + if (l) { + l = (""+l).toLowerCase(); + index[l] = index[l] || {}; + index[l][n.id] = {node:n,label:l} } l = l||n.label||n.name||n.id||""; diff --git a/editor/js/ui/tab-config.js b/editor/js/ui/tab-config.js index 5041d4433..9a4619b28 100644 --- a/editor/js/ui/tab-config.js +++ b/editor/js/ui/tab-config.js @@ -131,19 +131,7 @@ RED.sidebar.config = (function() { } else { var currentType = ""; nodes.forEach(function(node) { - var label = ""; - if (typeof node._def.label == "function") { - try { - label = node._def.label.call(node); - } catch(err) { - console.log("Definition error: "+node._def.type+".label",err); - label = node._def.type; - } - - } else { - label = node._def.label; - } - label = label || node.id; + var label = RED.utils.getNodeLabel(node,node.id); if (node.type != currentType) { $('