Merging r50149 through r50156 from trunk into soc-2011-tomato
[blender.git] / source / blender / makesrna / intern / rna_sequencer.c
index fd8195005aaa06dd3deb5b3e4da0c169a15fd6e1..bdee89308cfa4fbd06b2b69d62e42444002700b1 100644 (file)
@@ -30,6 +30,7 @@
 
 #include "RNA_access.h"
 #include "RNA_define.h"
+#include "RNA_enum_types.h"
 
 #include "rna_internal.h"
 
@@ -58,11 +59,24 @@ typedef struct EffectInfo {
        int supports_mask;
 } EffectInfo;
 
+EnumPropertyItem sequence_modifier_type_items[] = {
+       {seqModifierType_ColorBalance, "COLOR_BALANCE", ICON_NONE, "Color Balance", ""},
+       {seqModifierType_Curves, "CURVES", ICON_NONE, "Curves", ""},
+       {seqModifierType_HueCorrect, "HUE_CORRECT", ICON_NONE, "Hue Correct", ""},
+       {0, NULL, 0, NULL, NULL}
+};
+
 #ifdef RNA_RUNTIME
 
+#include "BKE_report.h"
+
+#include "WM_api.h"
+#include "WM_types.h"
+
 typedef struct SequenceSearchData {
        Sequence *seq;
        void *data;
+       SequenceModifierData *smd;
 } SequenceSearchData;
 
 /* build a temp reference to the parent */
@@ -646,7 +660,7 @@ static void rna_Sequence_update_reopen_files(Main *UNUSED(bmain), Scene *UNUSED(
        Scene *scene = (Scene *) ptr->id.data;
        Editing *ed = BKE_sequencer_editing_get(scene, FALSE);
 
-       BKE_sequencer_free_imbuf(scene, &ed->seqbase, FALSE, FALSE);
+       BKE_sequencer_free_imbuf(scene, &ed->seqbase, FALSE);
 
        if (RNA_struct_is_a(ptr->type, &RNA_SoundSequence))
                BKE_sequencer_update_sound_bounds(scene, ptr->data);
@@ -730,6 +744,7 @@ static int colbalance_seq_cmp_cb(Sequence *seq, void *arg_pt)
 
        if (seq->strip && seq->strip->color_balance == data->data) {
                data->seq = seq;
+               data->smd = NULL;
                return -1; /* done so bail out */
        }
 
@@ -742,6 +757,7 @@ static int colbalance_seq_cmp_cb(Sequence *seq, void *arg_pt)
 
                                if (&cbmd->color_balance == data->data) {
                                        data->seq = seq;
+                                       data->smd = smd;
                                        return -1; /* done so bail out */
                                }
                        }
@@ -751,27 +767,39 @@ static int colbalance_seq_cmp_cb(Sequence *seq, void *arg_pt)
        return 1;
 }
 
-static Sequence *sequence_get_by_colorbalance(Editing *ed, StripColorBalance *cb)
+static Sequence *sequence_get_by_colorbalance(Editing *ed, StripColorBalance *cb, SequenceModifierData **smd_r)
 {
        SequenceSearchData data;
 
        data.seq = NULL;
+       data.smd = NULL;
        data.data = cb;
 
        /* irritating we need to search for our sequence! */
        BKE_sequencer_base_recursive_apply(&ed->seqbase, colbalance_seq_cmp_cb, &data);
 
+       *smd_r = data.smd;
+
        return data.seq;
 }
 
 static char *rna_SequenceColorBalance_path(PointerRNA *ptr)
 {
        Scene *scene = ptr->id.data;
+       SequenceModifierData *smd;
        Editing *ed = BKE_sequencer_editing_get(scene, FALSE);
-       Sequence *seq = sequence_get_by_colorbalance(ed, ptr->data);
+       Sequence *seq = sequence_get_by_colorbalance(ed, ptr->data, &smd);
 
-       if (seq && seq->name + 2)
-               return BLI_sprintfN("sequence_editor.sequences_all[\"%s\"].color_balance", seq->name + 2);
+       if (seq && seq->name + 2) {
+               if (!smd) {
+                       /* path to old filter color balance */
+                       return BLI_sprintfN("sequence_editor.sequences_all[\"%s\"].color_balance", seq->name + 2);
+               }
+               else {
+                       /* path to modifier */
+                       return BLI_sprintfN("sequence_editor.sequences_all[\"%s\"].modifiers[\"%s\"].color_balance", seq->name + 2, smd->name);
+               }
+       }
        else
                return BLI_strdup("");
 }
@@ -780,9 +808,10 @@ static void rna_SequenceColorBalance_update(Main *UNUSED(bmain), Scene *UNUSED(s
 {
        Scene *scene = (Scene *) ptr->id.data;
        Editing *ed = BKE_sequencer_editing_get(scene, FALSE);
-       Sequence *seq = sequence_get_by_colorbalance(ed, ptr->data);
+       SequenceModifierData *smd;
+       Sequence *seq = sequence_get_by_colorbalance(ed, ptr->data, &smd);
 
-       if (seq->strip->color_balance == ptr->data)
+       if (smd == NULL)
                BKE_sequence_invalidate_cache(scene, seq);
        else
                BKE_sequence_invalidate_cache_for_modifier(scene, seq);
@@ -930,8 +959,8 @@ static void rna_SequenceModifier_name_set(PointerRNA *ptr, const char *value)
        if (adt) {
                char path[1024];
 
-               BLI_snprintf(path, sizeof(path), "sequence_editor.sequences_all[\"%s\"].modifiers", seq->name);
-               BKE_animdata_fix_paths_rename(&scene->id, adt, NULL, path, oldname, smd->name + 2, 0, 0, 1);
+               BLI_snprintf(path, sizeof(path), "sequence_editor.sequences_all[\"%s\"].modifiers", seq->name + 2);
+               BKE_animdata_fix_paths_rename(&scene->id, adt, NULL, path, oldname, smd->name, 0, 0, 1);
        }
 }
 
@@ -961,6 +990,52 @@ static int rna_SequenceModifier_otherSequence_poll(PointerRNA *ptr, PointerRNA v
        return TRUE;
 }
 
+static SequenceModifierData *rna_Sequence_modifier_new(Sequence *seq, bContext *C, ReportList *reports, const char *name, int type)
+{
+       if (!BKE_sequence_supports_modifiers(seq)) {
+               BKE_report(reports, RPT_ERROR, "Sequence type does not support modifiers");
+
+               return NULL;
+       }
+       else {
+               Scene *scene = CTX_data_scene(C);
+               SequenceModifierData *smd;
+
+               smd = BKE_sequence_modifier_new(seq, name, type);
+
+               BKE_sequence_invalidate_cache_for_modifier(scene, seq);
+
+               WM_main_add_notifier(NC_SCENE | ND_SEQUENCER, NULL);
+
+               return smd;
+       }
+}
+
+static void rna_Sequence_modifier_remove(Sequence *seq, bContext *C, ReportList *reports, SequenceModifierData *smd)
+{
+       Scene *scene = CTX_data_scene(C);
+
+       if (BKE_sequence_modifier_remove(seq, smd)) {
+               BKE_sequence_invalidate_cache_for_modifier(scene, seq);
+
+               WM_main_add_notifier(NC_SCENE | ND_SEQUENCER, NULL);
+       }
+       else {
+               BKE_report(reports, RPT_ERROR, "Modifier was not found in the stack");
+       }
+}
+
+static void rna_Sequence_modifier_clear(Sequence *seq, bContext *C)
+{
+       Scene *scene = CTX_data_scene(C);
+
+       BKE_sequence_modifier_clear(seq);
+
+       BKE_sequence_invalidate_cache_for_modifier(scene, seq);
+
+       WM_main_add_notifier(NC_SCENE | ND_SEQUENCER, NULL);
+}
+
 #else
 
 static void rna_def_strip_element(BlenderRNA *brna)
@@ -1180,6 +1255,8 @@ static void rna_def_color_balance(BlenderRNA *brna)
        RNA_def_property_ui_text(prop, "Saturation", "");
        RNA_def_property_update(prop, NC_SCENE | ND_SEQUENCER, "rna_Sequence_ColorBabalnce_update");
 #endif
+
+       RNA_def_struct_path_func(srna, "rna_SequenceColorBalance_path");
 }
 
 static void rna_def_strip_color_balance(BlenderRNA *brna)
@@ -1189,8 +1266,6 @@ static void rna_def_strip_color_balance(BlenderRNA *brna)
        srna = RNA_def_struct(brna, "SequenceColorBalance", "SequenceColorBalanceData");
        RNA_def_struct_ui_text(srna, "Sequence Color Balance", "Color balance parameters for a sequence strip");
        RNA_def_struct_sdna(srna, "StripColorBalance");
-
-       RNA_def_struct_path_func(srna, "rna_SequenceColorBalance_path");
 }
 
 EnumPropertyItem blend_mode_items[] = {
@@ -1210,12 +1285,39 @@ static void rna_def_sequence_modifiers(BlenderRNA *brna, PropertyRNA *cprop)
 {
        StructRNA *srna;
 
+       FunctionRNA *func;
+       PropertyRNA *parm;
+
        RNA_def_property_srna(cprop, "SequenceModifiers");
        srna = RNA_def_struct(brna, "SequenceModifiers", NULL);
        RNA_def_struct_sdna(srna, "Sequence");
        RNA_def_struct_ui_text(srna, "Strip Modifiers", "Collection of strip modifiers");
 
-       /* TODO: implement new/remove/clear methods for modifier stack */
+       /* add modifier */
+       func = RNA_def_function(srna, "new", "rna_Sequence_modifier_new");
+       RNA_def_function_flag(func, FUNC_USE_CONTEXT | FUNC_USE_REPORTS);
+       RNA_def_function_ui_description(func, "Add a new modifier");
+       parm = RNA_def_string(func, "name", "Name", 0, "", "New name for the modifier");
+       RNA_def_property_flag(parm, PROP_REQUIRED);
+       /* modifier to add */
+       parm = RNA_def_enum(func, "type", sequence_modifier_type_items, seqModifierType_ColorBalance, "", "Modifier type to add");
+       RNA_def_property_flag(parm, PROP_REQUIRED);
+       /* return type */
+       parm = RNA_def_pointer(func, "modifier", "SequenceModifier", "", "Newly created modifier");
+       RNA_def_function_return(func, parm);
+
+       /* remove modifier */
+       func = RNA_def_function(srna, "remove", "rna_Sequence_modifier_remove");
+       RNA_def_function_flag(func, FUNC_USE_CONTEXT | FUNC_USE_REPORTS);
+       RNA_def_function_ui_description(func, "Remove an existing modifier from the sequence");
+       /* modifier to remove */
+       parm = RNA_def_pointer(func, "modifier", "SequenceModifier", "", "Modifier to remove");
+       RNA_def_property_flag(parm, PROP_REQUIRED | PROP_NEVER_NULL);
+
+       /* clear all modifiers */
+       func = RNA_def_function(srna, "clear", "rna_Sequence_modifier_clear");
+       RNA_def_function_flag(func, FUNC_USE_CONTEXT);
+       RNA_def_function_ui_description(func, "Remove all modifiers from the sequence");
 }
 
 static void rna_def_sequence(BlenderRNA *brna)
@@ -2130,13 +2232,6 @@ static void rna_def_modifier(BlenderRNA *brna)
        StructRNA *srna;
        PropertyRNA *prop;
 
-       static EnumPropertyItem sequence_modifier_type_items[] = {
-               {seqModifierType_ColorBalance, "COLOR_BALANCE", ICON_NONE, "Color Balance", ""},
-               {seqModifierType_Curves, "CURVES", ICON_NONE, "Curves", ""},
-               {seqModifierType_HueCorrect,"HUE_CORRECT", ICON_NONE, "Hue Correct", ""},
-               {0, NULL, 0, NULL, NULL}
-       };
-
        static const EnumPropertyItem mask_input_type_items[] = {
                {SEQUENCE_MASK_INPUT_STRIP, "STRIP", 0, "Strip", "Use sequencer strip as mask input"},
                {SEQUENCE_MASK_INPUT_ID, "ID", 0, "Mask", "Use mask ID as mask input"},