Fix T94827: Group Input/Output cannot connect to custom sockets
Caused by rBa5c59fb90ef9. Since Group Input and Output sockets happen to be of type `SOCK_CUSTOM` [and since rBa5c59fb90ef9 custom py defined sockets are too :)] a check introduced in rB513066e8ad6f that prevents connections for `SOCK_CUSTOM` triggered. Now refine the check, so it specifically looks for NODE_GROUP_INPUT / NODE_GROUP_OUTPUT, too (this keeps the intention intact to not connect group inputs to group outputs and vice versa, but allows custom py defined sockets to connect again) and put it in new utility function. Maniphest Tasks: T94827 Differential Revision: https://developer.blender.org/D13817
This commit is contained in:
parent
e57365a70b
commit
85df7036f7
Notes:
blender-bot
2023-02-14 08:42:53 +01:00
Referenced by issue #94827, Regression: Group Input/Output cannot connect to some custom sockets Referenced by issue #93479, 3.0 Potential candidates for corrective releases
|
@ -420,6 +420,11 @@ void BKE_node_tree_unlink_id(ID *id, struct bNodeTree *ntree)
|
|||
/** \name Node #GROUP_INPUT / #GROUP_OUTPUT
|
||||
* \{ */
|
||||
|
||||
static bool is_group_extension_socket(const bNode *node, const bNodeSocket *socket)
|
||||
{
|
||||
return socket->type == SOCK_CUSTOM && ELEM(node->type, NODE_GROUP_OUTPUT, NODE_GROUP_INPUT);
|
||||
}
|
||||
|
||||
static void node_group_input_init(bNodeTree *ntree, bNode *node)
|
||||
{
|
||||
node_group_input_update(ntree, node);
|
||||
|
@ -471,7 +476,7 @@ void node_group_input_update(bNodeTree *ntree, bNode *node)
|
|||
* This could be improved by choosing the "best" type among all links,
|
||||
* whatever that means.
|
||||
*/
|
||||
if (link->tosock->type != SOCK_CUSTOM) {
|
||||
if (!is_group_extension_socket(link->tonode, link->tosock)) {
|
||||
exposelink = link;
|
||||
break;
|
||||
}
|
||||
|
@ -568,7 +573,7 @@ void node_group_output_update(bNodeTree *ntree, bNode *node)
|
|||
* This could be improved by choosing the "best" type among all links,
|
||||
* whatever that means.
|
||||
*/
|
||||
if (link->fromsock->type != SOCK_CUSTOM) {
|
||||
if (!is_group_extension_socket(link->fromnode, link->fromsock)) {
|
||||
exposelink = link;
|
||||
break;
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue