[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [33930] trunk/blender/source/blender/ editors/transform/transform_conversions.c: Bugfix #21825

[Ton Roosendaal]

Revision: 33930
          http://projects.blender.org/plugins/scmsvn/viewcvs.php?view=rev&root=bf-blender&revision=33930
Author:   ton
Date:     2010-12-28 18:03:09 +0100 (Tue, 28 Dec 2010)

Log Message:
-----------
Bugfix #21825

Auto-IK: applying back the IK transform on bones fails for constrainted 
bones themselves. This would require the post-transform of IK to be
converted to a pre-transform for the constraint. That's too much of
a headache for bugfixing days.

To keep the autoIK feature work it now stops IK chains at constrainted
bones. Added todo item for a matrix wizzard to fix this one day.

(What would be really cool is a matrix-api for armature to easily 
get all the required pre- and post transforms and apply. There's a lot
of duplicate exception handling for hinge bones for example.) 

Modified Paths:
--------------
    trunk/blender/source/blender/editors/transform/transform_conversions.c

Modified: trunk/blender/source/blender/editors/transform/transform_conversions.c
===================================================================
--- trunk/blender/source/blender/editors/transform/transform_conversions.c	2010-12-28 16:02:47 UTC (rev 33929)
+++ trunk/blender/source/blender/editors/transform/transform_conversions.c	2010-12-28 17:03:09 UTC (rev 33930)
@@ -875,6 +875,12 @@
 			pchan = pchan->parent;
 		else
 			pchan = NULL;
+		
+		/* but, constrainted bones can't get auto-ik transform applied, exclude these */
+		if(pchan && pchan->constraints.first) {
+			data->rootbone--;
+			pchan= NULL;
+		}
 	}
 
 	/* make a copy of maximum chain-length */