[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [56183] trunk/blender/source/blender/ editors/space_node: Fix #34846, Node->Ungroup menu entry is broken.
Lukas Toenne
lukas.toenne at googlemail.com
Sat Apr 20 18:50:05 CEST 2013
Revision: 56183
http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=56183
Author: lukastoenne
Date: 2013-04-20 16:50:05 +0000 (Sat, 20 Apr 2013)
Log Message:
-----------
Fix #34846, Node->Ungroup menu entry is broken. The standard node group operators now don't require the node_type string property any more. They are limited to the Shader, Compositing and Texture node tree types and will pass through for other (pynodes) tree types. Associated node group types are hardcoded.
The original rationale for adding the node_type property was to allow node group operators work generically on any node group type automatically. The problem is that detecting the appropriate node group type and node tree type to use for a group depends on using a node base type. Due to the fact that RNA does not allow multiple inheritance (mixin classes) this is impossible to achieve if node types also have to use a base type such as ShaderNode, CompositingNode or TextureNode.
The idea is now to just "make it work" by limiting the node group operators to the standard tree types. For future pynodes we can implement these operators nicely in Python, which will allow pynodes to use mixin base classes or derive their own operator types and re-use the same keymapping.
Modified Paths:
--------------
trunk/blender/source/blender/editors/space_node/node_group.c
trunk/blender/source/blender/editors/space_node/node_ops.c
Modified: trunk/blender/source/blender/editors/space_node/node_group.c
===================================================================
--- trunk/blender/source/blender/editors/space_node/node_group.c 2013-04-20 16:49:02 UTC (rev 56182)
+++ trunk/blender/source/blender/editors/space_node/node_group.c 2013-04-20 16:50:05 UTC (rev 56183)
@@ -69,67 +69,41 @@
#include "NOD_common.h"
#include "NOD_socket.h"
-/* define common group node operator properties */
-static void node_group_operator_properties(wmOperatorType *ot)
+static int node_group_operator_poll(bContext *C)
{
- /* NB: not using an enum here, because that it would have to use an item callback and thus require
- * copying of identifier strings anyway. node_type is not a user option, just a way of allowing
- * node group operators to work on different types of group nodes.
- */
- RNA_def_string(ot->srna, "node_type", "", 0, "Node Type", "Group node type the operator works on");
+ if (ED_operator_node_active(C)) {
+ SpaceNode *snode = CTX_wm_space_node(C);
+
+ /* Group operators only defined for standard node tree types.
+ * Disabled otherwise to allow pynodes define their own operators
+ * with same keymap.
+ */
+ if (STREQ(snode->tree_idname, "ShaderNodeTree")
+ || STREQ(snode->tree_idname, "CompositorNodeTree")
+ || STREQ(snode->tree_idname, "TextureNodeTree"))
+ return true;
+ }
+ return false;
}
-/* Internal poll functions so group node operators can work with different group node types.
- * This checks the operator node type property and looks up the respective types.
- * If this function returns FALSE the operator should return PASS_THROUGH to allow other variants.
- */
-static int node_group_operator_check_type(bContext *C, wmOperator *op, char **r_node_idname, char **r_ntree_idname)
+static const char *group_ntree_idname(bContext *C)
{
SpaceNode *snode = CTX_wm_space_node(C);
- bNodeTree *ntree = snode->edittree;
- PropertyRNA *ntype_prop = RNA_struct_find_property(op->ptr, "node_type");
- char *node_idname, *ntree_idname;
- bNodeType *ntype;
- bNodeTreeType *ntreetype;
+ return snode->tree_idname;
+}
+
+static const char *group_node_idname(bContext *C)
+{
+ SpaceNode *snode = CTX_wm_space_node(C);
- if (!RNA_property_is_set(op->ptr, ntype_prop)) {
- BKE_report(op->reports, RPT_ERROR, "Group node type not set");
- return FALSE;
- }
+ if (STREQ(snode->tree_idname, "ShaderNodeTree"))
+ return "ShaderNodeGroup";
+ else if (STREQ(snode->tree_idname, "CompositorNodeTree"))
+ return "CompositorNodeGroup";
+ else if (STREQ(snode->tree_idname, "TextureNodeTree"))
+ return "TextureNodeGroup";
- node_idname = RNA_property_string_get_alloc(op->ptr, ntype_prop, NULL, 0, NULL);
- ntype = nodeTypeFind(node_idname);
- if (!ntype) {
- BKE_reportf(op->reports, RPT_ERROR, "Group node type %s undefined", node_idname);
- MEM_freeN(node_idname);
- return FALSE;
- }
-
- if (!ntype->poll(ntype, ntree)) {
- MEM_freeN(node_idname);
- return FALSE;
- }
-
- ntree_idname = BLI_strdup(ntype->group_tree_idname);
- ntreetype = ntreeTypeFind(ntree_idname);
- if (!ntreetype) {
- BKE_reportf(op->reports, RPT_ERROR, "Group node tree type %s undefined", ntree_idname);
- MEM_freeN(node_idname);
- MEM_freeN(ntree_idname);
- return FALSE;
- }
-
- if (r_node_idname)
- *r_node_idname = node_idname;
- else
- MEM_freeN(node_idname);
-
- if (r_ntree_idname)
- *r_ntree_idname = ntree_idname;
- else
- MEM_freeN(ntree_idname);
-
- return TRUE;
+ return "";
}
static bNode *node_group_get_active(bContext *C, const char *node_idname)
@@ -148,17 +122,13 @@
static int node_group_edit_exec(bContext *C, wmOperator *op)
{
SpaceNode *snode = CTX_wm_space_node(C);
- char *node_idname;
+ const char *node_idname = group_node_idname(C);
bNode *gnode;
int exit = RNA_boolean_get(op->ptr, "exit");
ED_preview_kill_jobs(C);
- if (!node_group_operator_check_type(C, op, &node_idname, NULL))
- return OPERATOR_PASS_THROUGH;
-
gnode = node_group_get_active(C, node_idname);
- MEM_freeN(node_idname);
if (gnode && !exit) {
bNodeTree *ngroup = (bNodeTree *)gnode->id;
@@ -180,14 +150,10 @@
static int node_group_edit_invoke(bContext *C, wmOperator *op, const wmEvent *UNUSED(event))
{
- char *node_idname;
+ const char *node_idname = group_node_idname(C);
bNode *gnode;
- if (!node_group_operator_check_type(C, op, &node_idname, NULL))
- return OPERATOR_PASS_THROUGH;
-
gnode = node_group_get_active(C, node_idname);
- MEM_freeN(node_idname);
if (gnode && gnode->id && gnode->id->lib) {
WM_operator_confirm_message(C, op, "Make group local?");
@@ -207,12 +173,11 @@
/* api callbacks */
ot->invoke = node_group_edit_invoke;
ot->exec = node_group_edit_exec;
- ot->poll = ED_operator_node_active;
+ ot->poll = node_group_operator_poll;
/* flags */
ot->flag = OPTYPE_REGISTER | OPTYPE_UNDO;
- node_group_operator_properties(ot);
RNA_def_boolean(ot->srna, "exit", FALSE, "Exit", "");
}
@@ -386,16 +351,12 @@
static int node_group_ungroup_exec(bContext *C, wmOperator *op)
{
SpaceNode *snode = CTX_wm_space_node(C);
- char *node_idname;
+ const char *node_idname = group_node_idname(C);
bNode *gnode;
ED_preview_kill_jobs(C);
- if (!node_group_operator_check_type(C, op, &node_idname, NULL))
- return OPERATOR_PASS_THROUGH;
-
gnode = node_group_get_active(C, node_idname);
- MEM_freeN(node_idname);
if (!gnode)
return OPERATOR_CANCELLED;
@@ -422,12 +383,10 @@
/* api callbacks */
ot->exec = node_group_ungroup_exec;
- ot->poll = ED_operator_node_active;
+ ot->poll = node_group_operator_poll;
/* flags */
ot->flag = OPTYPE_REGISTER | OPTYPE_UNDO;
-
- node_group_operator_properties(ot);
}
/* ******************** Separate operator ********************** */
@@ -603,22 +562,18 @@
return OPERATOR_FINISHED;
}
-static int node_group_separate_invoke(bContext *C, wmOperator *op, const wmEvent *UNUSED(event))
+static int node_group_separate_invoke(bContext *C, wmOperator *UNUSED(op), const wmEvent *UNUSED(event))
{
- if (!node_group_operator_check_type(C, op, NULL, NULL))
- return OPERATOR_PASS_THROUGH;
- else {
- uiPopupMenu *pup = uiPupMenuBegin(C, CTX_IFACE_(BLF_I18NCONTEXT_OPERATOR_DEFAULT, "Separate"), ICON_NONE);
- uiLayout *layout = uiPupMenuLayout(pup);
-
- uiLayoutSetOperatorContext(layout, WM_OP_EXEC_DEFAULT);
- uiItemEnumO(layout, "NODE_OT_group_separate", NULL, 0, "type", NODE_GS_COPY);
- uiItemEnumO(layout, "NODE_OT_group_separate", NULL, 0, "type", NODE_GS_MOVE);
-
- uiPupMenuEnd(C, pup);
-
- return OPERATOR_CANCELLED;
- }
+ uiPopupMenu *pup = uiPupMenuBegin(C, CTX_IFACE_(BLF_I18NCONTEXT_OPERATOR_DEFAULT, "Separate"), ICON_NONE);
+ uiLayout *layout = uiPupMenuLayout(pup);
+
+ uiLayoutSetOperatorContext(layout, WM_OP_EXEC_DEFAULT);
+ uiItemEnumO(layout, "NODE_OT_group_separate", NULL, 0, "type", NODE_GS_COPY);
+ uiItemEnumO(layout, "NODE_OT_group_separate", NULL, 0, "type", NODE_GS_MOVE);
+
+ uiPupMenuEnd(C, pup);
+
+ return OPERATOR_CANCELLED;
}
void NODE_OT_group_separate(wmOperatorType *ot)
@@ -631,12 +586,11 @@
/* api callbacks */
ot->invoke = node_group_separate_invoke;
ot->exec = node_group_separate_exec;
- ot->poll = ED_operator_node_active;
+ ot->poll = node_group_operator_poll;
/* flags */
ot->flag = OPTYPE_REGISTER | OPTYPE_UNDO;
- node_group_operator_properties(ot);
RNA_def_enum(ot->srna, "type", node_group_separate_types, NODE_GS_COPY, "Type", "");
}
@@ -957,24 +911,17 @@
{
SpaceNode *snode = CTX_wm_space_node(C);
bNodeTree *ntree = snode->edittree;
- char *node_idname, *ntree_idname;
+ const char *ntree_idname = group_ntree_idname(C);
+ const char *node_idname = group_node_idname(C);
bNodeTree *ngroup;
bNode *gnode;
ED_preview_kill_jobs(C);
- if (!node_group_operator_check_type(C, op, &node_idname, &ntree_idname))
- return OPERATOR_PASS_THROUGH;
-
- if (!node_group_make_test_selected(ntree, NULL, ntree_idname, op->reports)) {
- MEM_freeN(node_idname);
- MEM_freeN(ntree_idname);
+ if (!node_group_make_test_selected(ntree, NULL, ntree_idname, op->reports))
return OPERATOR_CANCELLED;
- }
gnode = node_group_make_from_selected(C, ntree, node_idname, ntree_idname);
- MEM_freeN(node_idname);
- MEM_freeN(ntree_idname);
if (gnode) {
ngroup = (bNodeTree *)gnode->id;
@@ -1003,12 +950,10 @@
/* api callbacks */
ot->exec = node_group_make_exec;
- ot->poll = ED_operator_node_active;
+ ot->poll = node_group_operator_poll;
/* flags */
ot->flag = OPTYPE_REGISTER | OPTYPE_UNDO;
-
- node_group_operator_properties(ot);
}
/* ****************** Group Insert operator ******************* */
@@ -1018,16 +963,12 @@
SpaceNode *snode = CTX_wm_space_node(C);
bNodeTree *ntree = snode->edittree;
bNodeTree *ngroup;
- char *node_idname;
+ const char *node_idname = group_node_idname(C);
bNode *gnode;
ED_preview_kill_jobs(C);
- if (!node_group_operator_check_type(C, op, &node_idname, NULL))
- return OPERATOR_PASS_THROUGH;
-
gnode = node_group_get_active(C, node_idname);
- MEM_freeN(node_idname);
if (!gnode || !gnode->id)
return OPERATOR_CANCELLED;
@@ -1059,10 +1000,8 @@
/* api callbacks */
ot->exec = node_group_insert_exec;
- ot->poll = ED_operator_node_active;
+ ot->poll = node_group_operator_poll;
/* flags */
ot->flag = OPTYPE_REGISTER | OPTYPE_UNDO;
-
- node_group_operator_properties(ot);
}
Modified: trunk/blender/source/blender/editors/space_node/node_ops.c
===================================================================
--- trunk/blender/source/blender/editors/space_node/node_ops.c 2013-04-20 16:49:02 UTC (rev 56182)
+++ trunk/blender/source/blender/editors/space_node/node_ops.c 2013-04-20 16:50:05 UTC (rev 56183)
@@ -200,29 +200,6 @@
}
}
-/* register group operators for a specific group node type */
-static void node_group_operators(wmKeyMap *keymap, const char *node_type)
-{
- wmKeyMapItem *kmi;
-
- kmi = WM_keymap_add_item(keymap, "NODE_OT_group_make", GKEY, KM_PRESS, KM_CTRL, 0);
- RNA_string_set(kmi->ptr, "node_type", node_type);
-
- kmi = WM_keymap_add_item(keymap, "NODE_OT_group_ungroup", GKEY, KM_PRESS, KM_ALT, 0);
- RNA_string_set(kmi->ptr, "node_type", node_type);
-
- kmi = WM_keymap_add_item(keymap, "NODE_OT_group_separate", PKEY, KM_PRESS, 0, 0);
- RNA_string_set(kmi->ptr, "node_type", node_type);
-
- kmi = WM_keymap_add_item(keymap, "NODE_OT_group_edit", TABKEY, KM_PRESS, 0, 0);
- RNA_string_set(kmi->ptr, "node_type", node_type);
@@ Diff output truncated at 10240 characters. @@
More information about the Bf-blender-cvs
mailing list