Legend.Node in Graphin package assumes bindType is Node in click handler. Throws `item.getEdges` is not a function error on item click.

msnyder-msft commented 1 year ago

Describe the bug

When you use the Legend component from the graphin library (not the graphin-components package) and you pass bindType='edge' it throws an exception onClick because the click handler assumes the legend items are related to a node. You can see the bug here in the Legend/Node.tsx file.

// from graphin/src/components/Legend/Node.tsx#L50
const handleClick = (option: OptionType) => {
    const nodes = dataMap.get(checkedValue.value);
    ...
    // @ts-ignore
    nodes.forEach((node: any) => {
      graph.setItemState(node.id, 'active', checkedValue.checked);
      graph.setItemState(node.id, 'inactive', !checkedValue.checked);
      const { id } = node;
      const item = graph.findById(id) **as INode**;
      const edges = item.getEdges();
      edges.forEach(edge => {
        graph.setItemState(edge, 'normal', checkedValue.checked);
        graph.setItemState(edge, 'inactive', !checkedValue.checked);
      })
    });

    onChange(checkedValue, result);
  };

The assumption that the item is an INode is the problem here since an IEdge would not have the getEdges function. It results in an error like this TypeError: item.getEdges is not a function