[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [26763] trunk/blender/source/blender/ editors/animation/anim_filter.c: Committing a little note of a bug that isn 't quite so easy to fix yet with groups+visibility operator (anim channels) .
Joshua Leung
aligorith at gmail.com
Wed Feb 10 10:53:55 CET 2010
Revision: 26763
http://projects.blender.org/plugins/scmsvn/viewcvs.php?view=rev&root=bf-blender&revision=26763
Author: aligorith
Date: 2010-02-10 10:53:55 +0100 (Wed, 10 Feb 2010)
Log Message:
-----------
Committing a little note of a bug that isn't quite so easy to fix yet with groups+visibility operator (anim channels).
Modified Paths:
--------------
trunk/blender/source/blender/editors/animation/anim_filter.c
Modified: trunk/blender/source/blender/editors/animation/anim_filter.c
===================================================================
--- trunk/blender/source/blender/editors/animation/anim_filter.c 2010-02-10 09:30:22 UTC (rev 26762)
+++ trunk/blender/source/blender/editors/animation/anim_filter.c 2010-02-10 09:53:55 UTC (rev 26763)
@@ -969,12 +969,22 @@
*/
first_fcu = animdata_filter_fcurve_next(ads, agrp->channels.first, agrp, filter_gmode, owner_id);
+ /* Bug note:
+ * Selecting open group to toggle visbility of the group, where the F-Curves of the group are not suitable
+ * for inclusion due to their selection status (vs visibility status of bones/etc., as is usually the case),
+ * will not work, since the group gets skipped. However, fixing this can easily reintroduce the bugs whereby
+ * hidden groups (due to visibility status of bones/etc.) that were selected before becoming invisible, can
+ * easily get deleted accidentally as they'd be included in the list filtered for that purpose.
+ *
+ * So, for now, best solution is to just leave this note here, and hope to find a solution at a later date.
+ * -- Joshua Leung, 2010 Feb 10
+ */
if (first_fcu) {
/* add this group as a channel first */
if ((filter_mode & ANIMFILTER_CHANNELS) || !(filter_mode & ANIMFILTER_CURVESONLY)) {
/* filter selection of channel specially here again, since may be open and not subject to previous test */
if ( ANIMCHANNEL_SELOK(SEL_AGRP(agrp)) ) {
- ale= make_new_animlistelem(agrp, ANIMTYPE_GROUP, NULL, ANIMTYPE_NONE, owner_id);
+ ale= make_new_animlistelem(agrp, ANIMTYPE_GROUP, NULL, ANIMTYPE_NONE, owner_id);
if (ale) {
BLI_addtail(anim_data, ale);
items++;
More information about the Bf-blender-cvs
mailing list