Fix losing links when importing a copy of links into a subflow

[GogoVega]

• [x] Bugfix (non-breaking change which fixes an issue)
• [ ] New feature (non-breaking change which adds functionality)

Proposed changes

Fixes #4749.

Fixes node_map does not contain as key the new id of a copied node.

It's new_nodes which contains the new ids. At the end of updating the new nodes, they are added to node_map which explains why they are in the image.

Checklist

• [x] I have read the contribution guidelines
• [ ] For non-bugfix PRs, I have discussed this change on the forum/slack team.
• [x] I have run npm run test to verify the unit tests pass
• [ ] I have added suitable unit tests to cover the new/changed functionality

[GogoVega]

Definitely not my PR 😅

What properties can have def.type outside config node select input (and links)?

[knolleary]

@GogoVega in theory any property can have the type property if it contains references to other nodes.

The Catch/Status/Complete nodes have it - https://github.com/node-red/node-red/blob/master/packages/node_modules/%40node-red/nodes/core/common/25-catch.html#L34