Fix T56707: Assert changing grease pencil modes
authorCampbell Barton <ideasman42@gmail.com>
Fri, 7 Sep 2018 03:35:47 +0000 (13:35 +1000)
committerCampbell Barton <ideasman42@gmail.com>
Fri, 7 Sep 2018 03:41:31 +0000 (13:41 +1000)
source/blender/editors/gpencil/gpencil_edit.c
source/blender/editors/gpencil/gpencil_paint.c

index 48fd5736a132740cedd2323c6ad7fd611a970cb0..116c16c6ea4c685afdd60743f5e0f2994492ff6b 100644 (file)
@@ -73,6 +73,8 @@
 
 #include "WM_api.h"
 #include "WM_types.h"
+#include "WM_message.h"
+#include "WM_toolsystem.h"
 
 #include "RNA_access.h"
 #include "RNA_define.h"
@@ -113,8 +115,10 @@ static bool gpencil_editmode_toggle_poll(bContext *C)
 static int gpencil_editmode_toggle_exec(bContext *C, wmOperator *op)
 {
        const int back = RNA_boolean_get(op->ptr, "back");
-       Depsgraph *depsgraph = CTX_data_depsgraph(C);                                      \
-               bGPdata *gpd = ED_gpencil_data_get_active(C);
+
+       struct wmMsgBus *mbus = CTX_wm_message_bus(C);
+       Depsgraph *depsgraph = CTX_data_depsgraph(C);
+       bGPdata *gpd = ED_gpencil_data_get_active(C);
        bool is_object = false;
        short mode;
        /* if using a gpencil object, use this datablock */
@@ -161,6 +165,13 @@ static int gpencil_editmode_toggle_exec(bContext *C, wmOperator *op)
        WM_event_add_notifier(C, NC_GPENCIL | ND_GPENCIL_EDITMODE, NULL);
        WM_event_add_notifier(C, NC_SCENE | ND_MODE, NULL);
 
+       if (is_object) {
+               WM_msg_publish_rna_prop(mbus, &ob->id, ob, Object, mode);
+       }
+       if (G.background == false) {
+               WM_toolsystem_update_from_context_view3d(C);
+       }
+
        return OPERATOR_FINISHED;
 }
 
@@ -201,6 +212,7 @@ static int gpencil_paintmode_toggle_exec(bContext *C, wmOperator *op)
 {
        const bool back = RNA_boolean_get(op->ptr, "back");
 
+       struct wmMsgBus *mbus = CTX_wm_message_bus(C);
        bGPdata *gpd = ED_gpencil_data_get_active(C);
        ToolSettings *ts = CTX_data_tool_settings(C);
 
@@ -250,6 +262,13 @@ static int gpencil_paintmode_toggle_exec(bContext *C, wmOperator *op)
        WM_event_add_notifier(C, NC_GPENCIL | ND_DATA | ND_GPENCIL_EDITMODE, NULL);
        WM_event_add_notifier(C, NC_SCENE | ND_MODE, NULL);
 
+       if (is_object) {
+               WM_msg_publish_rna_prop(mbus, &ob->id, ob, Object, mode);
+       }
+       if (G.background == false) {
+               WM_toolsystem_update_from_context_view3d(C);
+       }
+
        return OPERATOR_FINISHED;
 }
 
@@ -290,6 +309,7 @@ static int gpencil_sculptmode_toggle_exec(bContext *C, wmOperator *op)
 {
        const bool back = RNA_boolean_get(op->ptr, "back");
 
+       struct wmMsgBus *mbus = CTX_wm_message_bus(C);
        bGPdata *gpd = ED_gpencil_data_get_active(C);
        bool is_object = false;
        short mode;
@@ -330,6 +350,13 @@ static int gpencil_sculptmode_toggle_exec(bContext *C, wmOperator *op)
        WM_event_add_notifier(C, NC_GPENCIL | ND_DATA | ND_GPENCIL_EDITMODE, NULL);
        WM_event_add_notifier(C, NC_SCENE | ND_MODE, NULL);
 
+       if (is_object) {
+               WM_msg_publish_rna_prop(mbus, &ob->id, ob, Object, mode);
+       }
+       if (G.background == false) {
+               WM_toolsystem_update_from_context_view3d(C);
+       }
+
        return OPERATOR_FINISHED;
 }
 
@@ -370,6 +397,7 @@ static int gpencil_weightmode_toggle_exec(bContext *C, wmOperator *op)
 {
        const bool back = RNA_boolean_get(op->ptr, "back");
 
+       struct wmMsgBus *mbus = CTX_wm_message_bus(C);
        bGPdata *gpd = ED_gpencil_data_get_active(C);
        bool is_object = false;
        short mode;
@@ -410,6 +438,13 @@ static int gpencil_weightmode_toggle_exec(bContext *C, wmOperator *op)
        WM_event_add_notifier(C, NC_GPENCIL | ND_DATA | ND_GPENCIL_EDITMODE, NULL);
        WM_event_add_notifier(C, NC_SCENE | ND_MODE, NULL);
 
+       if (is_object) {
+               WM_msg_publish_rna_prop(mbus, &ob->id, ob, Object, mode);
+       }
+       if (G.background == false) {
+               WM_toolsystem_update_from_context_view3d(C);
+       }
+
        return OPERATOR_FINISHED;
 }
 
index c1fc47d74fc466b8b3678802d34f8d4e62d73ae8..da03da7b40e3cf133de25e8a8fc83ed30ffed6c4 100644 (file)
@@ -2713,6 +2713,7 @@ static int gpencil_draw_invoke(bContext *C, wmOperator *op, const wmEvent *event
        if (p->sa->spacetype == SPACE_VIEW3D) {
                Object *ob = CTX_data_active_object(C);
                if (ob && (ob->type == OB_GPENCIL) && ((p->gpd->flag & GP_DATA_STROKE_PAINTMODE) == 0)) {
+                       /* FIXME: use the mode switching operator, this misses notifiers, messages. */
                        /* Just set paintmode flag... */
                        p->gpd->flag |= GP_DATA_STROKE_PAINTMODE;
                        /* disable other GP modes */