GraphEdit. Bad key name in get_connection_list() dictionary

[Lexpartizan]

This is a small problem that can hardly be called a error. More like inattention. When we use the connect_node(from, from_slot, to,to_slot) or disconnect_node(from,from_slot to,to_slot) functions in such graphedit, we use from_slot and to_slot . get_connection_list () return array structure of the form {from_port: 0, from: "graphnode name 0", to_port: 1, to: "graphnode name 1" }.
In one case, we refer to ports, and in the other to slots, although they are the same entity.

I suggest changing the key names in the get_connection_list () function to from_slot and to_slot to avoid confusion.
I think that with the transition to the new major version 4.0, you can fix this and break compatibility.
Or duplicate this keys to from_slot and to_slot to maintain compatibility.

It might have been worth adding to the proposal, but I don't think it's a suggestion, it's a correction of an error that was made inattentively. So I write about it here.

[mrushyendra]

Related to #40043

[Lexpartizan]

@mrushyendra
Thank you, I didn't expect that these are really different concepts. This can lead to confusion.
#37227

[umarcor]

@Lexpartizan, @mrushyendra can this issue be considered to be fixed by #40043 and, thus, to be a dup of #37227?

[Lexpartizan]

@umarcor, Yes, i think so.

[umarcor]

@Lexpartizan thanks!
@mrushyendra, I guess it's sensible to make it explicit in the first comment of #37227.

[akien-mga]

Closing as duplicate of #37227 then.