From 18be5f1773acf2311af0e71a50cdd93aa041d073 Mon Sep 17 00:00:00 2001 From: moklick Date: Mon, 2 May 2022 14:57:38 +0200 Subject: [PATCH] chore(warnings): only print for node_env=development --- src/container/EdgeRenderer/MarkerSymbols.tsx | 7 ++++--- src/container/EdgeRenderer/index.tsx | 14 +++++++++----- src/container/NodeRenderer/index.tsx | 1 + src/hooks/useResizeHandler.ts | 1 + src/store/utils.ts | 1 + src/utils/graph.ts | 3 +++ 6 files changed, 19 insertions(+), 8 deletions(-) diff --git a/src/container/EdgeRenderer/MarkerSymbols.tsx b/src/container/EdgeRenderer/MarkerSymbols.tsx index 2b9461f0..2a452495 100644 --- a/src/container/EdgeRenderer/MarkerSymbols.tsx +++ b/src/container/EdgeRenderer/MarkerSymbols.tsx @@ -38,13 +38,14 @@ export function useMarkerSymbol(type: MarkerType) { const symbol = useMemo(() => { const symbolExists = MarkerSymbols.hasOwnProperty(type); - if (process.env.NODE_ENV === 'development') { - if (!symbolExists) { + if (!symbolExists) { + // @ts-ignore + if (process.env.NODE_ENV === 'development') { console.warn( `[React Flow]: Marker type "${type}" doesn't exist. Help: https://reactflow.dev/error-decoder#900` ); - return () => null; } + return () => null; } return MarkerSymbols[type]; diff --git a/src/container/EdgeRenderer/index.tsx b/src/container/EdgeRenderer/index.tsx index b17e1e2c..3805f689 100644 --- a/src/container/EdgeRenderer/index.tsx +++ b/src/container/EdgeRenderer/index.tsx @@ -107,20 +107,24 @@ const EdgeRenderer = (props: EdgeRendererProps) => { const sourcePosition = sourceHandle?.position || Position.Bottom; const targetPosition = targetHandle?.position || Position.Top; - if (process.env.NODE_ENV === 'development') { - if (!sourceHandle) { + if (!sourceHandle) { + // @ts-ignore + if (process.env.NODE_ENV === 'development') { console.warn( `[React Flow]: Couldn't create edge for source handle id: ${edge.sourceHandle}; edge id: ${edge.id}. Help: https://reactflow.dev/error-decoder#800` ); - return null; } + return null; + } - if (!targetHandle) { + if (!targetHandle) { + // @ts-ignore + if (process.env.NODE_ENV === 'development') { console.warn( `[React Flow]: Couldn't create edge for target handle id: ${edge.targetHandle}; edge id: ${edge.id}. Help: https://reactflow.dev/error-decoder#800` ); - return null; } + return null; } const { sourceX, sourceY, targetX, targetY } = getEdgePositions( diff --git a/src/container/NodeRenderer/index.tsx b/src/container/NodeRenderer/index.tsx index bc5d5dbb..3d7135c4 100644 --- a/src/container/NodeRenderer/index.tsx +++ b/src/container/NodeRenderer/index.tsx @@ -70,6 +70,7 @@ const NodeRenderer = (props: NodeRendererProps) => { {nodes.map((node) => { const nodeType = node.type || 'default'; + // @ts-ignore if (process.env.NODE_ENV === 'development') { if (!props.nodeTypes[nodeType]) { console.warn( diff --git a/src/hooks/useResizeHandler.ts b/src/hooks/useResizeHandler.ts index 72670a59..608d2b43 100644 --- a/src/hooks/useResizeHandler.ts +++ b/src/hooks/useResizeHandler.ts @@ -16,6 +16,7 @@ function useResizeHandler(rendererNode: MutableRefObject) const size = getDimensions(rendererNode.current); + // @ts-ignore if (process.env.NODE_ENV === 'development') { if (size.height === 0 || size.width === 0) { console.warn( diff --git a/src/store/utils.ts b/src/store/utils.ts index 61db3f4b..e5a149f6 100644 --- a/src/store/utils.ts +++ b/src/store/utils.ts @@ -141,6 +141,7 @@ export function createPositionChange({ ] : currentExtent; } else { + // @ts-ignore if (process.env.NODE_ENV === 'development') { console.warn( '[React Flow]: Only child nodes can use a parent extent. Help: https://reactflow.dev/error-decoder#500' diff --git a/src/utils/graph.ts b/src/utils/graph.ts index 4c6c5ba4..99a75240 100644 --- a/src/utils/graph.ts +++ b/src/utils/graph.ts @@ -57,6 +57,7 @@ const connectionExists = (edge: Edge, edges: Edge[]) => { export const addEdge = (edgeParams: Edge | Connection, edges: Edge[]): Edge[] => { if (!edgeParams.source || !edgeParams.target) { + // @ts-ignore if (process.env.NODE_ENV === 'development') { console.warn( "[React Flow]: Can't create edge. An edge needs a source and a target. Help: https://reactflow.dev/error-decoder#600" @@ -84,6 +85,7 @@ export const addEdge = (edgeParams: Edge | Connection, edges: Edge[]): Edge[] => export const updateEdge = (oldEdge: Edge, newConnection: Connection, edges: Edge[]): Edge[] => { if (!newConnection.source || !newConnection.target) { + // @ts-ignore if (process.env.NODE_ENV === 'development') { console.warn( "[React Flow]: Can't create a new edge. An edge needs a source and a target. Help: https://reactflow.dev/error-decoder#600" @@ -95,6 +97,7 @@ export const updateEdge = (oldEdge: Edge, newConnection: Connection, edges: Edge const foundEdge = edges.find((e) => e.id === oldEdge.id) as Edge; if (!foundEdge) { + // @ts-ignore if (process.env.NODE_ENV === 'development') { console.warn( `[React Flow]: The old edge with id=${oldEdge.id} does not exist. Help: https://reactflow.dev/error-decoder#700`