Blender Git Commit Log

Git Commits -> Revision ef170c1

Revision ef170c1 by Lukas Toenne (master)
April 20, 2013, 16:50 (GMT)
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.

Commit Details:

Full Hash: ef170c16be8a8a228f3e0a06bca17cd7766a7ac1
SVN Revision: 56183
Parent Commit: 46b40e1
Lines Changed: +59, -138

2 Modified Paths:

/source/blender/editors/space_node/node_group.c (+51, -112) (Diff)
/source/blender/editors/space_node/node_ops.c (+8, -26) (Diff)
By: Miika HämäläinenLast update: Nov-07-2014 14:18MiikaHweb | 2003-2021