Added support for custom RNA properties on Bones, only worked for
[blender.git] / source / blender / makesrna / intern / rna_armature.c
index d49e5d147149abbedddfc0ac770e7cf96bf2a865..889f23e0d391eaf5250cf363f2370c84c00f54ab 100644 (file)
 #include "rna_internal.h"
 
 #include "DNA_armature_types.h"
+#include "DNA_object_types.h"
 #include "DNA_scene_types.h"
 
+#include "WM_api.h"
+#include "WM_types.h"
+
 #ifdef RNA_RUNTIME
 
-static void rna_Bone_layer_set(PointerRNA *ptr, const int *values)
+#include "BLI_arithb.h"
+
+#include "BKE_context.h"
+#include "BKE_depsgraph.h"
+#include "BKE_idprop.h"
+#include "BKE_main.h"
+
+#include "ED_armature.h"
+
+static void rna_Armature_update_data(bContext *C, PointerRNA *ptr)
+{
+       ID *id= ptr->id.data;
+
+       DAG_id_flush_update(id, OB_RECALC_DATA);
+       WM_event_add_notifier(C, NC_GEOM|ND_DATA, id);
+       //WM_event_add_notifier(C, NC_OBJECT|ND_POSE, NULL);
+}
+
+static void rna_Armature_redraw_data(bContext *C, PointerRNA *ptr)
+{
+       ID *id= ptr->id.data;
+
+       WM_event_add_notifier(C, NC_GEOM|ND_DATA, id);
+}
+
+static char *rna_Bone_path(PointerRNA *ptr)
+{
+       return BLI_sprintfN("bones[\"%s\"]", ((Bone*)ptr->data)->name);
+}
+
+static IDProperty *rna_Bone_idproperties(PointerRNA *ptr, int create)
+{
+       Bone *bone= ptr->data;
+
+       if(create && !bone->prop) {
+               IDPropertyTemplate val = {0};
+               bone->prop= IDP_New(IDP_GROUP, val, "RNA_Bone ID properties");
+       }
+
+       return bone->prop;
+}
+
+static IDProperty *rna_EditBone_idproperties(PointerRNA *ptr, int create)
+{
+       EditBone *ebone= ptr->data;
+
+       if(create && !ebone->prop) {
+               IDPropertyTemplate val = {0};
+               ebone->prop= IDP_New(IDP_GROUP, val, "RNA_EditBone ID properties");
+       }
+
+       return ebone->prop;
+}
+
+static void rna_bone_layer_set(short *layer, const int *values)
 {
-       Bone *bone= (Bone*)ptr->data;
        int i, tot= 0;
 
        /* ensure we always have some layer selected */
-       for(i=0; i<20; i++)
+       for(i=0; i<16; i++)
                if(values[i])
                        tot++;
        
        if(tot==0)
                return;
 
-       for(i=0; i<20; i++) {
-               if(values[i]) bone->layer |= (1<<i);
-               else bone->layer &= ~(1<<i);
+       for(i=0; i<16; i++) {
+               if(values[i]) *layer |= (1<<i);
+               else *layer &= ~(1<<i);
        }
 }
 
+static void rna_Bone_layer_set(PointerRNA *ptr, const int *values)
+{
+       Bone *bone= (Bone*)ptr->data;
+       rna_bone_layer_set(&bone->layer, values);
+}
+
 static void rna_Armature_layer_set(PointerRNA *ptr, const int *values)
 {
        bArmature *arm= (bArmature*)ptr->data;
@@ -100,285 +163,619 @@ static void rna_Armature_path_end_frame_set(PointerRNA *ptr, int value)
        data->pathef= value;
 }
 
-#else
-
-// err... bones should not be directly edited (only editbones should be...)
-static void rna_def_bone(BlenderRNA *brna)
+static void rna_EditBone_name_set(PointerRNA *ptr, const char *value)
 {
-       StructRNA *srna;
-       PropertyRNA *prop;
+       bArmature *arm= (bArmature*)ptr->id.data;
+       EditBone *ebone= (EditBone*)ptr->data;
+       char oldname[32], newname[32];
        
-       srna= RNA_def_struct(brna, "Bone", NULL);
-       RNA_def_struct_ui_text(srna, "Bone", "Bone in an Armature datablock.");
-       RNA_def_struct_ui_icon(srna, ICON_BONE_DATA);
+       /* need to be on the stack */
+       BLI_strncpy(newname, value, 32);
+       BLI_strncpy(oldname, ebone->name, 32);
        
-       /* pointers/collections */
-               /* parent (pointer) */
-       prop= RNA_def_property(srna, "parent", PROP_POINTER, PROP_NONE);
-       RNA_def_property_struct_type(prop, "Bone");
-       RNA_def_property_pointer_sdna(prop, NULL, "parent");
-       RNA_def_property_ui_text(prop, "Parent", "Parent bone (in same Armature).");
+       ED_armature_bone_rename(arm, oldname, newname);
+}
+
+static void rna_Bone_name_set(PointerRNA *ptr, const char *value)
+{
+       bArmature *arm= (bArmature*)ptr->id.data;
+       Bone *bone= (Bone*)ptr->data;
+       char oldname[32], newname[32];
        
-               /* children (collection) */
-       prop= RNA_def_property(srna, "children", PROP_COLLECTION, PROP_NONE);
-       RNA_def_property_collection_sdna(prop, NULL, "childbase", NULL);
-       RNA_def_property_struct_type(prop, "Bone");
-       RNA_def_property_ui_text(prop, "Children", "Bones which are children of this bone");
+       /* need to be on the stack */
+       BLI_strncpy(newname, value, 32);
+       BLI_strncpy(oldname, bone->name, 32);
        
+       ED_armature_bone_rename(arm, oldname, newname);
+}
+
+static void rna_EditBone_layer_get(PointerRNA *ptr, int values[16])
+{
+       EditBone *data= (EditBone*)(ptr->data);
+       values[0]= ((data->layer & (1<<0)) != 0);
+       values[1]= ((data->layer & (1<<1)) != 0);
+       values[2]= ((data->layer & (1<<2)) != 0);
+       values[3]= ((data->layer & (1<<3)) != 0);
+       values[4]= ((data->layer & (1<<4)) != 0);
+       values[5]= ((data->layer & (1<<5)) != 0);
+       values[6]= ((data->layer & (1<<6)) != 0);
+       values[7]= ((data->layer & (1<<7)) != 0);
+       values[8]= ((data->layer & (1<<8)) != 0);
+       values[9]= ((data->layer & (1<<9)) != 0);
+       values[10]= ((data->layer & (1<<10)) != 0);
+       values[11]= ((data->layer & (1<<11)) != 0);
+       values[12]= ((data->layer & (1<<12)) != 0);
+       values[13]= ((data->layer & (1<<13)) != 0);
+       values[14]= ((data->layer & (1<<14)) != 0);
+       values[15]= ((data->layer & (1<<15)) != 0);
+}
+
+static void rna_EditBone_layer_set(PointerRNA *ptr, const int values[16])
+{
+       EditBone *data= (EditBone*)(ptr->data);
+       rna_bone_layer_set(&data->layer, values);
+}
+
+static void rna_EditBone_connected_check(EditBone *ebone)
+{
+       if(ebone->parent) {
+               if(ebone->flag & BONE_CONNECTED) {
+                       /* Attach this bone to its parent */
+                       VECCOPY(ebone->head, ebone->parent->tail);
+
+                       if(ebone->flag & BONE_ROOTSEL)
+                               ebone->parent->flag |= BONE_TIPSEL;
+               }
+               else if(!(ebone->parent->flag & BONE_ROOTSEL)) {
+                       ebone->parent->flag &= ~BONE_TIPSEL;
+               }
+       }
+}
+
+static void rna_EditBone_connected_set(PointerRNA *ptr, int value)
+{
+       EditBone *ebone= (EditBone*)(ptr->data);
+
+       if(value) ebone->flag |= BONE_CONNECTED;
+       else ebone->flag &= ~BONE_CONNECTED;
+
+       rna_EditBone_connected_check(ebone);
+}
+
+static PointerRNA rna_EditBone_parent_get(PointerRNA *ptr)
+{
+       EditBone *data= (EditBone*)(ptr->data);
+       return rna_pointer_inherit_refine(ptr, &RNA_EditBone, data->parent);
+}
+
+static void rna_EditBone_parent_set(PointerRNA *ptr, PointerRNA value)
+{
+       EditBone *ebone= (EditBone*)(ptr->data);
+       EditBone *pbone, *parbone= (EditBone*)value.data;
+
+       /* within same armature */
+       if(value.id.data != ptr->id.data)
+               return;
+
+       if(parbone == NULL) {
+               if(ebone->parent && !(ebone->parent->flag & BONE_ROOTSEL))
+                       ebone->parent->flag &= ~BONE_TIPSEL;
+
+               ebone->parent = NULL;
+               ebone->flag &= ~BONE_CONNECTED;
+       }
+       else {
+               /* make sure this is a valid child */
+               if(parbone == ebone)
+                       return;
+                       
+               for(pbone= parbone->parent; pbone; pbone=pbone->parent)
+                       if(pbone == ebone)
+                               return;
+
+               ebone->parent = parbone;
+               rna_EditBone_connected_check(ebone);
+       }
+}
+
+static void rna_Armature_editbone_transform_update(bContext *C, PointerRNA *ptr)
+{
+       bArmature *arm= (bArmature*)ptr->id.data;
+       EditBone *ebone= (EditBone*)ptr->data;
+       EditBone *child, *eboflip;
+       
+       /* update our parent */
+       if(ebone->parent && ebone->flag & BONE_CONNECTED)
+               VECCOPY(ebone->parent->tail, ebone->head)
+
+       /* update our children if necessary */
+       for(child = arm->edbo->first; child; child=child->next)
+               if(child->parent == ebone && (child->flag & BONE_CONNECTED))
+                       VECCOPY(child->head, ebone->tail);
+
+       if(arm->flag & ARM_MIRROR_EDIT) {
+               eboflip= ED_armature_bone_get_mirrored(arm->edbo, ebone);
+
+               if(eboflip) {
+                       eboflip->roll= -ebone->roll;
+
+                       eboflip->head[0]= -ebone->head[0];
+                       eboflip->tail[0]= -ebone->tail[0];
+                       
+                       /* update our parent */
+                       if(eboflip->parent && eboflip->flag & BONE_CONNECTED)
+                               VECCOPY(eboflip->parent->tail, eboflip->head);
+                       
+                       /* update our children if necessary */
+                       for(child = arm->edbo->first; child; child=child->next)
+                               if(child->parent == eboflip && (child->flag & BONE_CONNECTED))
+                                       VECCOPY (child->head, eboflip->tail);
+               }
+       }
+
+       rna_Armature_update_data(C, ptr);
+}
+
+static void rna_Armature_bones_next(CollectionPropertyIterator *iter)
+{
+       ListBaseIterator *internal= iter->internal;
+       Bone *bone= (Bone*)internal->link;
+
+       if(bone->childbase.first)
+               internal->link= (Link*)bone->childbase.first;
+       else if(bone->next)
+               internal->link= (Link*)bone->next;
+       else {
+               internal->link= NULL;
+
+               do {
+                       bone= bone->parent;
+                       if(bone && bone->next) {
+                               internal->link= (Link*)bone->next;
+                               break;
+                       }
+               } while(bone);
+       }
+
+       iter->valid= (internal->link != NULL);
+}
+
+#else
+
+static void rna_def_bone_common(StructRNA *srna, int editbone)
+{
+       PropertyRNA *prop;
+
        /* strings */
        prop= RNA_def_property(srna, "name", PROP_STRING, PROP_NONE);
-       RNA_def_property_clear_flag(prop, PROP_EDITABLE); /* must be unique */
+       RNA_def_property_string_sdna(prop, NULL, "name");
        RNA_def_property_ui_text(prop, "Name", "");
        RNA_def_struct_name_property(srna, prop);
-       
+       if(editbone) RNA_def_property_string_funcs(prop, NULL, NULL, "rna_EditBone_name_set");
+       else RNA_def_property_string_funcs(prop, NULL, NULL, "rna_Bone_name_set");
+       RNA_def_property_update(prop, 0, "rna_Armature_redraw_data");
+
        /* flags */
-               /* layer */
-       prop= RNA_def_property(srna, "layer", PROP_BOOLEAN, PROP_NONE);
+       prop= RNA_def_property(srna, "layer", PROP_BOOLEAN, PROP_LAYER_MEMBER);
        RNA_def_property_boolean_sdna(prop, NULL, "layer", 1);
        RNA_def_property_array(prop, 16);
-       RNA_def_property_ui_text(prop, "Bone Layers", "Layers bone exists in");
-       RNA_def_property_boolean_funcs(prop, NULL, "rna_Bone_layer_set");
+       if(editbone) RNA_def_property_boolean_funcs(prop, "rna_EditBone_layer_get", "rna_EditBone_layer_set");
+       else RNA_def_property_boolean_funcs(prop, NULL, "rna_Bone_layer_set");
+       RNA_def_property_ui_text(prop, "Layers", "Layers bone exists in");
+       RNA_def_property_update(prop, 0, "rna_Armature_redraw_data");
 
-               /* flag */
-       prop= RNA_def_property(srna, "selected", PROP_BOOLEAN, PROP_NONE);
-       RNA_def_property_boolean_sdna(prop, NULL, "flag", BONE_SELECTED);
-       RNA_def_property_ui_text(prop, "Selected", "");
-       
-       prop= RNA_def_property(srna, "head_selected", PROP_BOOLEAN, PROP_NONE);
-       RNA_def_property_boolean_sdna(prop, NULL, "flag", BONE_ROOTSEL);
-       RNA_def_property_ui_text(prop, "Head Selected", "");
-       
-       prop= RNA_def_property(srna, "tail_selected", PROP_BOOLEAN, PROP_NONE);
-       RNA_def_property_boolean_sdna(prop, NULL, "flag", BONE_TIPSEL);
-       RNA_def_property_ui_text(prop, "Tail Selected", "");
-       
        prop= RNA_def_property(srna, "connected", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_sdna(prop, NULL, "flag", BONE_CONNECTED);
+       if(editbone) RNA_def_property_boolean_funcs(prop, NULL, "rna_EditBone_connected_set");
+       else RNA_def_property_clear_flag(prop, PROP_EDITABLE);
        RNA_def_property_ui_text(prop, "Connected", "When bone has a parent, bone's head is struck to the parent's tail.");
-       
-               // XXX should we define this in PoseChannel wrapping code instead? but PoseChannels directly get some of their flags from here...
-       prop= RNA_def_property(srna, "pose_channel_hidden", PROP_BOOLEAN, PROP_NONE);
-       RNA_def_property_boolean_sdna(prop, NULL, "flag", BONE_HIDDEN_P);
-       RNA_def_property_ui_text(prop, "Pose Channel Hidden", "Bone is not visible when it is not in Edit Mode (i.e. in Object or Pose Modes).");
+       RNA_def_property_update(prop, 0, "rna_Armature_update_data");
        
        prop= RNA_def_property(srna, "active", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_sdna(prop, NULL, "flag", BONE_ACTIVE);
        RNA_def_property_ui_text(prop, "Active", "Bone was the last bone clicked on (most operations are applied to only this bone)");
+       RNA_def_property_update(prop, 0, "rna_Armature_redraw_data");
        
        prop= RNA_def_property(srna, "hinge", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_negative_sdna(prop, NULL, "flag", BONE_HINGE);
        RNA_def_property_ui_text(prop, "Inherit Rotation", "Bone doesn't inherit rotation or scale from parent bone.");
-       
-       prop= RNA_def_property(srna, "editmode_hidden", PROP_BOOLEAN, PROP_NONE);
-       RNA_def_property_boolean_sdna(prop, NULL, "flag", BONE_HIDDEN_A);
-       RNA_def_property_ui_text(prop, "Edit Mode Hidden", "Bone is not visible when in Edit Mode");
+       RNA_def_property_update(prop, 0, "rna_Armature_update_data");
        
        prop= RNA_def_property(srna, "multiply_vertexgroup_with_envelope", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_sdna(prop, NULL, "flag", BONE_MULT_VG_ENV);
        RNA_def_property_ui_text(prop, "Multiply Vertex Group with Envelope", "When deforming bone, multiply effects of Vertex Group weights with Envelope influence.");
+       RNA_def_property_update(prop, 0, "rna_Armature_update_data");
        
        prop= RNA_def_property(srna, "deform", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_negative_sdna(prop, NULL, "flag", BONE_NO_DEFORM);
        RNA_def_property_ui_text(prop, "Deform", "Bone does not deform any geometry.");
+       RNA_def_property_update(prop, 0, "rna_Armature_update_data");
        
        prop= RNA_def_property(srna, "inherit_scale", PROP_BOOLEAN, PROP_NONE);
-       RNA_def_property_boolean_negative_sdna(prop, NULL, "flag", BONE_NO_SCALE);
        RNA_def_property_ui_text(prop, "Inherit Scale", "Bone inherits scaling from parent bone.");
+       RNA_def_property_boolean_negative_sdna(prop, NULL, "flag", BONE_NO_SCALE);
+       RNA_def_property_update(prop, 0, "rna_Armature_update_data");
        
        prop= RNA_def_property(srna, "draw_wire", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_sdna(prop, NULL, "flag", BONE_DRAWWIRE);
        RNA_def_property_ui_text(prop, "Draw Wire", "Bone is always drawn as Wireframe regardless of viewport draw mode. Useful for non-obstructive custom bone shapes.");
+       RNA_def_property_update(prop, 0, "rna_Armature_redraw_data");
        
        prop= RNA_def_property(srna, "cyclic_offset", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_negative_sdna(prop, NULL, "flag", BONE_NO_CYCLICOFFSET);
        RNA_def_property_ui_text(prop, "Cyclic Offset", "When bone doesn't have a parent, it receives cyclic offset effects.");
+       RNA_def_property_update(prop, 0, "rna_Armature_update_data");
        
-       prop= RNA_def_property(srna, "editmode_locked", PROP_BOOLEAN, PROP_NONE);
-       RNA_def_property_boolean_sdna(prop, NULL, "flag", BONE_EDITMODE_LOCKED);
-       RNA_def_property_ui_text(prop, "Edit Mode Locked", "Bone is not able to be transformed when in Edit Mode.");
-       
+       prop= RNA_def_property(srna, "selectable", PROP_BOOLEAN, PROP_NONE);
+       RNA_def_property_boolean_negative_sdna(prop, NULL, "flag", BONE_UNSELECTABLE);
+       RNA_def_property_ui_text(prop, "Selectable", "Bone is able to be selected");
+       RNA_def_property_update(prop, 0, "rna_Armature_redraw_data");
+
        /* Number values */
                /* envelope deform settings */
        prop= RNA_def_property(srna, "envelope_distance", PROP_FLOAT, PROP_NONE);
        RNA_def_property_float_sdna(prop, NULL, "dist");
        RNA_def_property_range(prop, 0.0f, 1000.0f);
        RNA_def_property_ui_text(prop, "Envelope Deform Distance", "Bone deformation distance (for Envelope deform only).");
+       RNA_def_property_update(prop, 0, "rna_Armature_update_data");
        
        prop= RNA_def_property(srna, "envelope_weight", PROP_FLOAT, PROP_NONE);
        RNA_def_property_float_sdna(prop, NULL, "weight");
        RNA_def_property_range(prop, 0.0f, 1000.0f);
        RNA_def_property_ui_text(prop, "Envelope Deform Weight", "Bone deformation weight (for Envelope deform only).");
+       RNA_def_property_update(prop, 0, "rna_Armature_update_data");
        
-       prop= RNA_def_property(srna, "radius_head", PROP_FLOAT, PROP_NONE);
+       prop= RNA_def_property(srna, "head_radius", PROP_FLOAT, PROP_NONE);
+       if(editbone) RNA_def_property_update(prop, 0, "rna_Armature_editbone_transform_update");
        RNA_def_property_float_sdna(prop, NULL, "rad_head");
        //RNA_def_property_range(prop, 0, 1000);  // XXX range is 0 to lim, where lim= 10000.0f*MAX2(1.0, view3d->grid);
-       RNA_def_property_ui_text(prop, "Envelope Radius Head", "Radius of head of bone (for Envelope deform only).");
+       RNA_def_property_ui_text(prop, "Envelope Head Radius", "Radius of head of bone (for Envelope deform only).");
        
-       prop= RNA_def_property(srna, "radius_tail", PROP_FLOAT, PROP_NONE);
+       prop= RNA_def_property(srna, "tail_radius", PROP_FLOAT, PROP_NONE);
+       if(editbone) RNA_def_property_update(prop, 0, "rna_Armature_editbone_transform_update");
        RNA_def_property_float_sdna(prop, NULL, "rad_tail");
        //RNA_def_property_range(prop, 0, 1000);  // XXX range is 0 to lim, where lim= 10000.0f*MAX2(1.0, view3d->grid);
-       RNA_def_property_ui_text(prop, "Envelope Radius Tail", "Radius of tail of bone (for Envelope deform only).");
+       RNA_def_property_ui_text(prop, "Envelope Tail Radius", "Radius of tail of bone (for Envelope deform only).");
        
                /* b-bones deform settings */
        prop= RNA_def_property(srna, "bbone_segments", PROP_INT, PROP_NONE);
        RNA_def_property_int_sdna(prop, NULL, "segments");
        RNA_def_property_range(prop, 1, 32);
        RNA_def_property_ui_text(prop, "B-Bone Segments", "Number of subdivisions of bone (for B-Bones only).");
+       RNA_def_property_update(prop, 0, "rna_Armature_update_data");
        
        prop= RNA_def_property(srna, "bbone_in", PROP_FLOAT, PROP_NONE);
        RNA_def_property_float_sdna(prop, NULL, "ease1");
        RNA_def_property_range(prop, 0.0f, 2.0f);
        RNA_def_property_ui_text(prop, "B-Bone Ease In", "Length of first Bezier Handle (for B-Bones only).");
+       RNA_def_property_update(prop, 0, "rna_Armature_update_data");
        
        prop= RNA_def_property(srna, "bbone_out", PROP_FLOAT, PROP_NONE);
        RNA_def_property_float_sdna(prop, NULL, "ease2");
        RNA_def_property_range(prop, 0.0f, 2.0f);
        RNA_def_property_ui_text(prop, "B-Bone Ease Out", "Length of second Bezier Handle (for B-Bones only).");
+       RNA_def_property_update(prop, 0, "rna_Armature_update_data");
+}
+
+// err... bones should not be directly edited (only editbones should be...)
+static void rna_def_bone(BlenderRNA *brna)
+{
+       StructRNA *srna;
+       PropertyRNA *prop;
+       
+       srna= RNA_def_struct(brna, "Bone", NULL);
+       RNA_def_struct_ui_text(srna, "Bone", "Bone in an Armature datablock.");
+       RNA_def_struct_ui_icon(srna, ICON_BONE_DATA);
+       RNA_def_struct_path_func(srna, "rna_Bone_path");
+       RNA_def_struct_idproperties_func(srna, "rna_Bone_idproperties");
+       
+       /* pointers/collections */
+               /* parent (pointer) */
+       prop= RNA_def_property(srna, "parent", PROP_POINTER, PROP_NONE);
+       RNA_def_property_struct_type(prop, "Bone");
+       RNA_def_property_pointer_sdna(prop, NULL, "parent");
+       RNA_def_property_ui_text(prop, "Parent", "Parent bone (in same Armature).");
+       RNA_def_property_update(prop, 0, "rna_Armature_redraw_data");
+       
+               /* children (collection) */
+       prop= RNA_def_property(srna, "children", PROP_COLLECTION, PROP_NONE);
+       RNA_def_property_collection_sdna(prop, NULL, "childbase", NULL);
+       RNA_def_property_struct_type(prop, "Bone");
+       RNA_def_property_ui_text(prop, "Children", "Bones which are children of this bone");
+
+       rna_def_bone_common(srna, 0);
+
+               // XXX should we define this in PoseChannel wrapping code instead? but PoseChannels directly get some of their flags from here...
+       prop= RNA_def_property(srna, "hidden", PROP_BOOLEAN, PROP_NONE);
+       RNA_def_property_boolean_sdna(prop, NULL, "flag", BONE_HIDDEN_P);
+       RNA_def_property_ui_text(prop, "Hidden", "Bone is not visible when it is not in Edit Mode (i.e. in Object or Pose Modes).");
+       RNA_def_property_update(prop, 0, "rna_Armature_redraw_data");
+
+       prop= RNA_def_property(srna, "selected", PROP_BOOLEAN, PROP_NONE);
+       RNA_def_property_boolean_sdna(prop, NULL, "flag", BONE_SELECTED);
+       RNA_def_property_ui_text(prop, "Selected", "");
+       RNA_def_property_update(prop, 0, "rna_Armature_redraw_data");
+
+       /* XXX better matrix descriptions possible (Arystan) */
+       prop= RNA_def_property(srna, "matrix", PROP_FLOAT, PROP_MATRIX);
+       RNA_def_property_float_sdna(prop, NULL, "bone_mat");
+       RNA_def_property_array(prop, 9);
+       RNA_def_property_ui_text(prop, "Bone Matrix", "3x3 bone matrix.");
+
+       prop= RNA_def_property(srna, "armature_matrix", PROP_FLOAT, PROP_MATRIX);
+       RNA_def_property_float_sdna(prop, NULL, "arm_mat");
+       RNA_def_property_array(prop, 16);
+       RNA_def_property_ui_text(prop, "Bone Armature-Relative Matrix", "4x4 bone matrix relative to armature.");
+
+       prop= RNA_def_property(srna, "tail", PROP_FLOAT, PROP_TRANSLATION);
+       RNA_def_property_float_sdna(prop, NULL, "tail");
+       RNA_def_property_array(prop, 3);
+       RNA_def_property_ui_text(prop, "Tail", "Location of tail end of the bone.");
+
+       prop= RNA_def_property(srna, "armature_tail", PROP_FLOAT, PROP_TRANSLATION);
+       RNA_def_property_float_sdna(prop, NULL, "arm_tail");
+       RNA_def_property_array(prop, 3);
+       RNA_def_property_ui_text(prop, "Armature-Relative Tail", "Location of tail end of the bone relative to armature.");
+
+       prop= RNA_def_property(srna, "head", PROP_FLOAT, PROP_TRANSLATION);
+       RNA_def_property_float_sdna(prop, NULL, "head");
+       RNA_def_property_array(prop, 3);
+       RNA_def_property_ui_text(prop, "Head", "Location of head end of the bone.");
+
+       prop= RNA_def_property(srna, "armature_head", PROP_FLOAT, PROP_TRANSLATION);
+       RNA_def_property_float_sdna(prop, NULL, "arm_head");
+       RNA_def_property_array(prop, 3);
+       RNA_def_property_ui_text(prop, "Armature-Relative Head", "Location of head end of the bone relative to armature.");
+}
+
+static void rna_def_edit_bone(BlenderRNA *brna)
+{
+       StructRNA *srna;
+       PropertyRNA *prop;
        
-               /* editmode bone coordinates */
-       // XXX not sure if we want to wrap these here... besides, changing these requires changing the matrix?
-       prop= RNA_def_property(srna, "head", PROP_FLOAT, PROP_VECTOR);
-       RNA_def_property_ui_text(prop, "Bone Head Location", "In Edit Mode, the location of the 'head' of the bone.");
+       srna= RNA_def_struct(brna, "EditBone", NULL);
+       RNA_def_struct_sdna(srna, "EditBone");
+       RNA_def_struct_idproperties_func(srna, "rna_Bone_idproperties");
+       RNA_def_struct_ui_text(srna, "Edit Bone", "Editmode bone in an Armature datablock.");
+       RNA_def_struct_ui_icon(srna, ICON_BONE_DATA);
        
-       prop= RNA_def_property(srna, "tail", PROP_FLOAT, PROP_VECTOR);
-       RNA_def_property_ui_text(prop, "Bone Tail Location", "In Edit Mode, the location of the 'head' of the bone.");
+       RNA_define_verify_sdna(0); // not in sdna
+
+       prop= RNA_def_property(srna, "parent", PROP_POINTER, PROP_NONE);
+       RNA_def_property_struct_type(prop, "EditBone");
+       RNA_def_property_pointer_funcs(prop, "rna_EditBone_parent_get", "rna_EditBone_parent_set", NULL);
+       RNA_def_property_flag(prop, PROP_EDITABLE);
+       RNA_def_property_ui_text(prop, "Parent", "Parent edit bone (in same Armature).");
+       RNA_def_property_update(prop, 0, "rna_Armature_redraw_data");
        
        prop= RNA_def_property(srna, "roll", PROP_FLOAT, PROP_NONE);
-       RNA_def_property_range(prop, 0.0f, 2.0f);
-       RNA_def_property_ui_text(prop, "Bone Roll", "In Edit Mode, the 'roll' (i.e. rotation around the bone vector, equivalent to local Y-axis rotation).");
+       RNA_def_property_float_sdna(prop, NULL, "roll");
+       RNA_def_property_ui_text(prop, "Roll", "Bone rotation around head-tail axis.");
+       RNA_def_property_update(prop, 0, "rna_Armature_editbone_transform_update");
+
+       prop= RNA_def_property(srna, "head", PROP_FLOAT, PROP_TRANSLATION);
+       RNA_def_property_float_sdna(prop, NULL, "head");
+       RNA_def_property_array(prop, 3);
+       RNA_def_property_ui_text(prop, "Head", "Location of head end of the bone.");
+       RNA_def_property_update(prop, 0, "rna_Armature_editbone_transform_update");
+
+       prop= RNA_def_property(srna, "tail", PROP_FLOAT, PROP_TRANSLATION);
+       RNA_def_property_float_sdna(prop, NULL, "tail");
+       RNA_def_property_array(prop, 3);
+       RNA_def_property_ui_text(prop, "Tail", "Location of tail end of the bone.");
+       RNA_def_property_update(prop, 0, "rna_Armature_editbone_transform_update");
+
+       rna_def_bone_common(srna, 1);
+
+       prop= RNA_def_property(srna, "hidden", PROP_BOOLEAN, PROP_NONE);
+       RNA_def_property_boolean_sdna(prop, NULL, "flag", BONE_HIDDEN_A);
+       RNA_def_property_ui_text(prop, "Hidden", "Bone is not visible when in Edit Mode");
+       RNA_def_property_update(prop, 0, "rna_Armature_redraw_data");
+
+       prop= RNA_def_property(srna, "locked", PROP_BOOLEAN, PROP_NONE);
+       RNA_def_property_boolean_sdna(prop, NULL, "flag", BONE_EDITMODE_LOCKED);
+       RNA_def_property_ui_text(prop, "Locked", "Bone is not able to be transformed when in Edit Mode.");
+       RNA_def_property_update(prop, 0, "rna_Armature_redraw_data");
+
+       prop= RNA_def_property(srna, "head_selected", PROP_BOOLEAN, PROP_NONE);
+       RNA_def_property_boolean_sdna(prop, NULL, "flag", BONE_ROOTSEL);
+       RNA_def_property_ui_text(prop, "Head Selected", "");
+       RNA_def_property_update(prop, 0, "rna_Armature_redraw_data");
+       
+       prop= RNA_def_property(srna, "tail_selected", PROP_BOOLEAN, PROP_NONE);
+       RNA_def_property_boolean_sdna(prop, NULL, "flag", BONE_TIPSEL);
+       RNA_def_property_ui_text(prop, "Tail Selected", "");
+       RNA_def_property_update(prop, 0, "rna_Armature_redraw_data");
+
+       RNA_define_verify_sdna(1);
 }
 
-void rna_def_armature(BlenderRNA *brna)
+static void rna_def_armature(BlenderRNA *brna)
 {
        StructRNA *srna;
        PropertyRNA *prop;
        
        static EnumPropertyItem prop_drawtype_items[] = {
-               {ARM_OCTA, "OCTAHEDRAL", "Octahedral", "Draw bones as octahedral shape (default)."},
-               {ARM_LINE, "STICK", "Stick", "Draw bones as simple 2D lines with dots."},
-               {ARM_B_BONE, "BBONE", "B-Bone", "Draw bones as boxes, showing subdivision and B-Splines"},
-               {ARM_ENVELOPE, "ENVELOPE", "Envelope", "Draw bones as extruded spheres, showing defomation influence volume."},
-               {0, NULL, NULL, NULL}};
+               {ARM_OCTA, "OCTAHEDRAL", 0, "Octahedral", "Display bones as octahedral shape (default)."},
+               {ARM_LINE, "STICK", 0, "Stick", "Display bones as simple 2D lines with dots."},
+               {ARM_B_BONE, "BBONE", 0, "B-Bone", "Display bones as boxes, showing subdivision and B-Splines"},
+               {ARM_ENVELOPE, "ENVELOPE", 0, "Envelope", "Display bones as extruded spheres, showing defomation influence volume."},
+               {0, NULL, 0, NULL, NULL}};
        static EnumPropertyItem prop_ghost_type_items[] = {
-               {ARM_GHOST_CUR, "CURRENT_FRAME", "Around Current Frame", "Draw Ghosts of poses within a fixed number of frames around the current frame."},
-               {ARM_GHOST_RANGE, "RANGE", "In Range", "Draw Ghosts of poses within specified range."},
-               {ARM_GHOST_KEYS, "KEYS", "On Keyframes", "Draw Ghosts of poses on Keyframes."},
-               {0, NULL, NULL, NULL}};
+               {ARM_GHOST_CUR, "CURRENT_FRAME", 0, "Around Frame", "Display Ghosts of poses within a fixed number of frames around the current frame."},
+               {ARM_GHOST_RANGE, "RANGE", 0, "In Range", "Display Ghosts of poses within specified range."},
+               {ARM_GHOST_KEYS, "KEYS", 0, "On Keyframes", "Display Ghosts of poses on Keyframes."},
+               {0, NULL, 0, NULL, NULL}};
+       static const EnumPropertyItem prop_paths_type_items[]= {
+               {ARM_PATH_ACFRA, "CURRENT_FRAME", 0, "Around Frame", "Display Paths of poses within a fixed number of frames around the current frame."},
+               {0, "RANGE", 0, "In Range", "Display Paths of poses within specified range."},
+               {0, NULL, 0, NULL, NULL}};
+       static const EnumPropertyItem prop_paths_location_items[]= {
+               {ARM_PATH_HEADS, "HEADS", 0, "Heads", "Calculate bone paths from heads"},
+               {0, "TAILS", 0, "Tails", "Calculate bone paths from tails"},
+               {0, NULL, 0, NULL, NULL}};
+       static const EnumPropertyItem prop_pose_position_items[]= {
+               {0, "POSE_POSITION", 0, "Pose Position", "Show armature in posed state."},
+               {ARM_RESTPOS, "REST_POSITION", 0, "Rest Position", "Show Armature in binding pose state. No posing possible."},
+               {0, NULL, 0, NULL, NULL}};
        
        srna= RNA_def_struct(brna, "Armature", "ID");
        RNA_def_struct_ui_text(srna, "Armature", "Armature datablock containing a hierarchy of bones, usually used for rigging characters.");
        RNA_def_struct_ui_icon(srna, ICON_ARMATURE_DATA);
-       
        RNA_def_struct_sdna(srna, "bArmature");
        
+       /* Animation Data */
+       rna_def_animdata_common(srna);
+       
        /* Collections */
        prop= RNA_def_property(srna, "bones", PROP_COLLECTION, PROP_NONE);
        RNA_def_property_collection_sdna(prop, NULL, "bonebase", NULL);
+       RNA_def_property_collection_funcs(prop, 0, "rna_Armature_bones_next", 0, 0, 0, 0, 0, 0, 0);
        RNA_def_property_struct_type(prop, "Bone");
        RNA_def_property_ui_text(prop, "Bones", "");
+
+       prop= RNA_def_property(srna, "edit_bones", PROP_COLLECTION, PROP_NONE);
+       RNA_def_property_collection_sdna(prop, NULL, "edbo", NULL);
+       RNA_def_property_struct_type(prop, "EditBone");
+       RNA_def_property_ui_text(prop, "Edit Bones", "");
        
        /* Enum values */
+//     prop= RNA_def_property(srna, "rest_position", PROP_BOOLEAN, PROP_NONE);
+//     RNA_def_property_boolean_sdna(prop, NULL, "flag", ARM_RESTPOS);
+//     RNA_def_property_ui_text(prop, "Rest Position", "Show Armature in Rest Position. No posing possible.");
+//     RNA_def_property_update(prop, 0, "rna_Armature_update_data");
+       
+       prop= RNA_def_property(srna, "pose_position", PROP_ENUM, PROP_NONE);
+       RNA_def_property_enum_bitflag_sdna(prop, NULL, "flag");
+       RNA_def_property_enum_items(prop, prop_pose_position_items);
+       RNA_def_property_ui_text(prop, "Pose Position", "Show armature in binding pose or final posed state.");
+       RNA_def_property_update(prop, 0, "rna_Armature_update_data");
+       
        prop= RNA_def_property(srna, "drawtype", PROP_ENUM, PROP_NONE);
        RNA_def_property_enum_items(prop, prop_drawtype_items);
        RNA_def_property_ui_text(prop, "Draw Type", "");
+       RNA_def_property_update(prop, 0, "rna_Armature_redraw_data");
        
        prop= RNA_def_property(srna, "ghost_type", PROP_ENUM, PROP_NONE);
        RNA_def_property_enum_sdna(prop, NULL, "ghosttype");
        RNA_def_property_enum_items(prop, prop_ghost_type_items);
-       RNA_def_property_ui_text(prop, "Ghost Drawing", "Method of Onion-skinning for active Action");
+       RNA_def_property_ui_text(prop, "Ghost Type", "Method of Onion-skinning for active Action");
+       RNA_def_property_update(prop, 0, "rna_Armature_redraw_data");
+       
+       prop= RNA_def_property(srna, "paths_type", PROP_ENUM, PROP_NONE);
+       RNA_def_property_enum_bitflag_sdna(prop, NULL, "pathflag");
+       RNA_def_property_enum_items(prop, prop_paths_type_items);
+       RNA_def_property_ui_text(prop, "Paths Type", "Type of range to show for Bone Paths");
+       RNA_def_property_update(prop, 0, "rna_Armature_redraw_data");
+       
+       prop= RNA_def_property(srna, "paths_location", PROP_ENUM, PROP_NONE);
+       RNA_def_property_enum_bitflag_sdna(prop, NULL, "pathflag");
+       RNA_def_property_enum_items(prop, prop_paths_location_items);
+       RNA_def_property_ui_text(prop, "Paths Location", "When calculating Bone Paths, use Head or Tips");
+       RNA_def_property_update(prop, 0, "rna_Armature_redraw_data");
        
        /* Boolean values */
                /* layer */
-       prop= RNA_def_property(srna, "layer", PROP_BOOLEAN, PROP_NONE);
+       prop= RNA_def_property(srna, "layer", PROP_BOOLEAN, PROP_LAYER_MEMBER);
        RNA_def_property_boolean_sdna(prop, NULL, "layer", 1);
        RNA_def_property_array(prop, 16);
        RNA_def_property_ui_text(prop, "Visible Layers", "Armature layer visibility.");
        RNA_def_property_boolean_funcs(prop, NULL, "rna_Armature_layer_set");
+       RNA_def_property_update(prop, NC_OBJECT|ND_POSE, "rna_Armature_redraw_data");
+       RNA_def_property_flag(prop, PROP_LIB_EXCEPTION);
        
                /* layer protection */
-       prop= RNA_def_property(srna, "layer_protection", PROP_BOOLEAN, PROP_NONE);
+       prop= RNA_def_property(srna, "layer_protection", PROP_BOOLEAN, PROP_LAYER);
        RNA_def_property_boolean_sdna(prop, NULL, "layer_protected", 1);
        RNA_def_property_array(prop, 16);
        RNA_def_property_ui_text(prop, "Layer Proxy Protection", "Protected layers in Proxy Instances are restored to Proxy settings on file reload and undo.");        
+       RNA_def_property_update(prop, 0, "rna_Armature_redraw_data");
                
                /* flag */
-       prop= RNA_def_property(srna, "rest_position", PROP_BOOLEAN, PROP_NONE);
-       RNA_def_property_boolean_sdna(prop, NULL, "flag", ARM_RESTPOS);
-       RNA_def_property_ui_text(prop, "Rest Position", "Show Armature in Rest Position. No posing possible.");
+       
        
        prop= RNA_def_property(srna, "draw_axes", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_sdna(prop, NULL, "flag", ARM_DRAWAXES);
        RNA_def_property_ui_text(prop, "Draw Axes", "Draw bone axes.");
+       RNA_def_property_update(prop, 0, "rna_Armature_redraw_data");
        
        prop= RNA_def_property(srna, "draw_names", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_sdna(prop, NULL, "flag", ARM_DRAWNAMES);
        RNA_def_property_ui_text(prop, "Draw Names", "Draw bone names.");
+       RNA_def_property_update(prop, 0, "rna_Armature_redraw_data");
        
        prop= RNA_def_property(srna, "delay_deform", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_sdna(prop, NULL, "flag", ARM_DELAYDEFORM);
        RNA_def_property_ui_text(prop, "Delay Deform", "Don't deform children when manipulating bones in Pose Mode");
+       RNA_def_property_update(prop, 0, "rna_Armature_update_data");
        
        prop= RNA_def_property(srna, "x_axis_mirror", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_sdna(prop, NULL, "flag", ARM_MIRROR_EDIT);
        RNA_def_property_ui_text(prop, "X-Axis Mirror", "Apply changes to matching bone on opposite side of X-Axis.");
+       RNA_def_property_update(prop, 0, "rna_Armature_redraw_data");
        
        prop= RNA_def_property(srna, "auto_ik", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_sdna(prop, NULL, "flag", ARM_AUTO_IK);
        RNA_def_property_ui_text(prop, "Auto IK", "Add temporaral IK constraints while grabbing bones in Pose Mode.");
+       RNA_def_property_update(prop, 0, "rna_Armature_redraw_data");
        
        prop= RNA_def_property(srna, "draw_custom_bone_shapes", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_negative_sdna(prop, NULL, "flag", ARM_NO_CUSTOM);
        RNA_def_property_ui_text(prop, "Draw Custom Bone Shapes", "Draw bones with their custom shapes.");
+       RNA_def_property_update(prop, 0, "rna_Armature_redraw_data");
        
        prop= RNA_def_property(srna, "draw_group_colors", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_sdna(prop, NULL, "flag", ARM_COL_CUSTOM);
        RNA_def_property_ui_text(prop, "Draw Bone Group Colors", "Draw bone group colors.");
+       RNA_def_property_update(prop, 0, "rna_Armature_redraw_data");
        
        prop= RNA_def_property(srna, "ghost_only_selected", PROP_BOOLEAN, PROP_NONE);
-       RNA_def_property_boolean_negative_sdna(prop, NULL, "flag", ARM_GHOST_ONLYSEL);
-       RNA_def_property_ui_text(prop, "Draw Ghosts on Selected Keyframes Only", "");
+       RNA_def_property_boolean_sdna(prop, NULL, "flag", ARM_GHOST_ONLYSEL);
+       RNA_def_property_ui_text(prop, "Draw Ghosts on Selected Bones Only", "");
+       RNA_def_property_update(prop, 0, "rna_Armature_redraw_data");
        
                /* deformflag */
        prop= RNA_def_property(srna, "deform_vertexgroups", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_sdna(prop, NULL, "deformflag", ARM_DEF_VGROUP);
        RNA_def_property_ui_text(prop, "Deform Vertex Groups", "Enable Vertex Groups when defining deform");
+       RNA_def_property_update(prop, 0, "rna_Armature_update_data");
        
        prop= RNA_def_property(srna, "deform_envelope", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_sdna(prop, NULL, "deformflag", ARM_DEF_ENVELOPE);
        RNA_def_property_ui_text(prop, "Deform Envelopes", "Enable Bone Envelopes when defining deform");
+       RNA_def_property_update(prop, 0, "rna_Armature_update_data");
        
        prop= RNA_def_property(srna, "deform_quaternion", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_sdna(prop, NULL, "deformflag", ARM_DEF_QUATERNION);
        RNA_def_property_ui_text(prop, "Use Dual Quaternion Deformation", "Enable deform rotation with Quaternions");
+       RNA_def_property_update(prop, 0, "rna_Armature_update_data");
        
        prop= RNA_def_property(srna, "deform_bbone_rest", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_sdna(prop, NULL, "deformflag", ARM_DEF_B_BONE_REST);
        RNA_def_property_ui_text(prop, "B-Bones Deform in Rest Position", "Make B-Bones deform already in Rest Position");
+       RNA_def_property_update(prop, 0, "rna_Armature_update_data");
        
        //prop= RNA_def_property(srna, "deform_invert_vertexgroups", PROP_BOOLEAN, PROP_NONE);
        //RNA_def_property_boolean_negative_sdna(prop, NULL, "deformflag", ARM_DEF_INVERT_VGROUP);
        //RNA_def_property_ui_text(prop, "Invert Vertex Group Influence", "Invert Vertex Group influence (only for Modifiers)");
+       //RNA_def_property_update(prop, 0, "rna_Armature_update_data");
        
                /* pathflag */
        prop= RNA_def_property(srna, "paths_show_frame_numbers", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_sdna(prop, NULL, "pathflag", ARM_PATH_FNUMS);
-       RNA_def_property_ui_text(prop, "Bone Paths Show Frame Numbers", "When drawing Armature in Pose Mode, show frame numbers on Bone Paths");
+       RNA_def_property_ui_text(prop, "Paths Show Frame Numbers", "When drawing Armature in Pose Mode, show frame numbers on Bone Paths");
+       RNA_def_property_update(prop, 0, "rna_Armature_redraw_data");
        
        prop= RNA_def_property(srna, "paths_highlight_keyframes", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_sdna(prop, NULL, "pathflag", ARM_PATH_KFRAS);
-       RNA_def_property_ui_text(prop, "Bone Paths Highlight Keyframes", "When drawing Armature in Pose Mode, emphasize position of keyframes on Bone Paths");
+       RNA_def_property_ui_text(prop, "Paths Highlight Keyframes", "When drawing Armature in Pose Mode, emphasize position of keyframes on Bone Paths");
+       RNA_def_property_update(prop, 0, "rna_Armature_redraw_data");
        
        prop= RNA_def_property(srna, "paths_show_keyframe_numbers", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_sdna(prop, NULL, "pathflag", ARM_PATH_KFNOS);
-       RNA_def_property_ui_text(prop, "Bone Paths Show Keyframe Numbers", "When drawing Armature in Pose Mode, show frame numbers of Keyframes on Bone Paths");
-       
-       prop= RNA_def_property(srna, "paths_show_around_current_frame", PROP_BOOLEAN, PROP_NONE);
-       RNA_def_property_boolean_sdna(prop, NULL, "pathflag", ARM_PATH_ACFRA);
-       RNA_def_property_ui_text(prop, "Bone Paths Around Current Frame", "When drawing Armature in Pose Mode, only show section of Bone Paths that falls around current frame");
+       RNA_def_property_ui_text(prop, "Paths Show Keyframe Numbers", "When drawing Armature in Pose Mode, show frame numbers of Keyframes on Bone Paths");
+       RNA_def_property_update(prop, 0, "rna_Armature_redraw_data");
        
-       prop= RNA_def_property(srna, "paths_calculate_head_positions", PROP_BOOLEAN, PROP_NONE);
-       RNA_def_property_boolean_sdna(prop, NULL, "pathflag", ARM_PATH_HEADS);
-       RNA_def_property_ui_text(prop, "Bone Paths Use Heads", "When calculating Bone Paths, use Head locations instead of Tips");
        
        /* Number fields */
                /* ghost/onionskining settings */
@@ -386,53 +783,63 @@ void rna_def_armature(BlenderRNA *brna)
        RNA_def_property_int_sdna(prop, NULL, "ghostep");
        RNA_def_property_range(prop, 0, 30);
        RNA_def_property_ui_text(prop, "Ghosting Step", "Number of frame steps on either side of current frame to show as ghosts (only for 'Around Current Frame' Onion-skining method).");
+       RNA_def_property_update(prop, 0, "rna_Armature_redraw_data");
        
        prop= RNA_def_property(srna, "ghost_size", PROP_INT, PROP_NONE);
        RNA_def_property_int_sdna(prop, NULL, "ghostsize");
        RNA_def_property_range(prop, 1, 20);
        RNA_def_property_ui_text(prop, "Ghosting Frame Step", "Frame step for Ghosts (not for 'On Keyframes' Onion-skining method).");
+       RNA_def_property_update(prop, 0, "rna_Armature_redraw_data");
        
-       prop= RNA_def_property(srna, "ghost_start_frame", PROP_INT, PROP_NONE);
+       prop= RNA_def_property(srna, "ghost_start_frame", PROP_INT, PROP_TIME);
        RNA_def_property_int_sdna(prop, NULL, "ghostsf");
        RNA_def_property_int_funcs(prop, NULL, "rna_Armature_ghost_start_frame_set", NULL);
        RNA_def_property_ui_text(prop, "Ghosting Start Frame", "Starting frame of range of Ghosts to display (not for 'Around Current Frame' Onion-skinning method).");
+       RNA_def_property_update(prop, 0, "rna_Armature_redraw_data");
        
-       prop= RNA_def_property(srna, "ghost_end_frame", PROP_INT, PROP_NONE);
+       prop= RNA_def_property(srna, "ghost_end_frame", PROP_INT, PROP_TIME);
        RNA_def_property_int_sdna(prop, NULL, "ghostef");
        RNA_def_property_int_funcs(prop, NULL, "rna_Armature_ghost_end_frame_set", NULL);
        RNA_def_property_ui_text(prop, "Ghosting End Frame", "End frame of range of Ghosts to display (not for 'Around Current Frame' Onion-skinning method).");
+       RNA_def_property_update(prop, 0, "rna_Armature_redraw_data");
        
                /* bone path settings */
        prop= RNA_def_property(srna, "path_size", PROP_INT, PROP_NONE);
        RNA_def_property_int_sdna(prop, NULL, "pathsize");
        RNA_def_property_range(prop, 1, 100);
-       RNA_def_property_ui_text(prop, "Bone Paths Frame Step", "Number of frames between 'dots' on Bone Paths (when drawing).");
+       RNA_def_property_ui_text(prop, "Paths Frame Step", "Number of frames between 'dots' on Bone Paths (when drawing).");
+       RNA_def_property_update(prop, 0, "rna_Armature_redraw_data");
        
-       prop= RNA_def_property(srna, "path_start_frame", PROP_INT, PROP_NONE);
+       prop= RNA_def_property(srna, "path_start_frame", PROP_INT, PROP_TIME);
        RNA_def_property_int_sdna(prop, NULL, "pathsf");
        RNA_def_property_int_funcs(prop, NULL, "rna_Armature_path_start_frame_set", NULL);
-       RNA_def_property_ui_text(prop, "Bone Paths Calculation Start Frame", "Starting frame of range of frames to use for Bone Path calculations.");
+       RNA_def_property_ui_text(prop, "Paths Calculation Start Frame", "Starting frame of range of frames to use for Bone Path calculations.");
+       RNA_def_property_update(prop, 0, "rna_Armature_update_data");
        
-       prop= RNA_def_property(srna, "path_end_frame", PROP_INT, PROP_NONE);
+       prop= RNA_def_property(srna, "path_end_frame", PROP_INT, PROP_TIME);
        RNA_def_property_int_sdna(prop, NULL, "pathef");
        RNA_def_property_int_funcs(prop, NULL, "rna_Armature_path_end_frame_set", NULL);
-       RNA_def_property_ui_text(prop, "Bone Paths Calculation End Frame", "End frame of range of frames to use for Bone Path calculations.");
+       RNA_def_property_ui_text(prop, "Paths Calculation End Frame", "End frame of range of frames to use for Bone Path calculations.");
+       RNA_def_property_update(prop, 0, "rna_Armature_update_data");
        
        prop= RNA_def_property(srna, "path_before_current", PROP_INT, PROP_NONE);
        RNA_def_property_int_sdna(prop, NULL, "pathbc");
        RNA_def_property_range(prop, 1, MAXFRAMEF/2);
-       RNA_def_property_ui_text(prop, "Bone Paths Frames Before Current", "Number of frames before current frame to show on Bone Paths (only for 'Around Current' option).");
+       RNA_def_property_ui_text(prop, "Paths Frames Before Current", "Number of frames before current frame to show on Bone Paths (only for 'Around Current' option).");
+       RNA_def_property_update(prop, 0, "rna_Armature_update_data");
        
        prop= RNA_def_property(srna, "path_after_current", PROP_INT, PROP_NONE);
        RNA_def_property_int_sdna(prop, NULL, "pathac");
        RNA_def_property_range(prop, 1, MAXFRAMEF/2);
-       RNA_def_property_ui_text(prop, "Bone Paths Frames After Current", "Number of frames after current frame to show on Bone Paths (only for 'Around Current' option).");
+       RNA_def_property_ui_text(prop, "Paths Frames After Current", "Number of frames after current frame to show on Bone Paths (only for 'Around Current' option).");
+       RNA_def_property_update(prop, 0, "rna_Armature_update_data");
 }
 
 void RNA_def_armature(BlenderRNA *brna)
 {
        rna_def_armature(brna);
        rna_def_bone(brna);
+       rna_def_edit_bone(brna);
 }
 
 #endif