From 28bb07d415e88d3051ac18390b1953f9d949e06c Mon Sep 17 00:00:00 2001 From: Michael Maier Date: Sun, 28 Nov 2021 15:35:56 +0100 Subject: [PATCH] fix: check if `node.prop` is defined --- src/dagre-wrapper/nodes.js | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/src/dagre-wrapper/nodes.js b/src/dagre-wrapper/nodes.js index b4f9d6946..40444f921 100644 --- a/src/dagre-wrapper/nodes.js +++ b/src/dagre-wrapper/nodes.js @@ -325,14 +325,16 @@ const rect = (parent, node) => { .attr('width', totalWidth) .attr('height', totalHeight); - const propKeys = new Set(Object.keys(node.props)); - if (node.props?.borders) { - applyNodePropertyBorders(rect, node.props.borders, totalWidth, totalHeight); - propKeys.delete('borders'); + if (node.props) { + const propKeys = new Set(Object.keys(node.props)); + if (node.props.borders) { + applyNodePropertyBorders(rect, node.props.borders, totalWidth, totalHeight); + propKeys.delete('borders'); + } + propKeys.forEach((propKey) => { + log.warn(`Unknown node property ${propKey}`); + }); } - propKeys.forEach((propKey) => { - log.warn(`Unknown node property ${propKey}`); - }); updateNodeBounds(node, rect);