Fix issue with not all connections being deleted when node deleted.

[Teknologicus]

First if statement fixes issue with not all connections to node being deleted when node is deleted. Second if statement prevents redundant DeleteConnection() calls from occurring.

[rokups]

This does not fix the issue though. Also checks are redundant because conceptually connection should not exist with only one end connected. The only exception is connection that is being created, but user does not deal with that. Anyhow, issue lies somewhere else. Do you plan to revisit and update this PR?

[Teknologicus]

I'll retract the pull request and issue a new pull request if I find a more elegant/correct solution. Thank you for your support and ImNodes!

…

On 8/19/19 7:09 AM, Rokas Kupstys wrote: This does not fix the issue though. Also checks are redundant because conceptually connection should not exist with only one end connected. The only exception is connection that is being created, but user does not deal with that. Anyhow, issue lies somewhere else. Do you plan to revisit and update this PR? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#9?email_source=notifications&email_token=ACS7Q5OWCI3FNU2ZA44EER3QFKSRLA5CNFSM4IMO7BR2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4TB7LI#issuecomment-522592173>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ACS7Q5MHJK35KJIVSNWWYLDQFKSRLANCNFSM4IMO7BRQ>.