Fix T68112: Align View on object add is broken
authorSergey Sharybin <sergey.vfx@gmail.com>
Fri, 2 Aug 2019 13:13:59 +0000 (15:13 +0200)
committerSergey Sharybin <sergey.vfx@gmail.com>
Fri, 2 Aug 2019 13:15:41 +0000 (15:15 +0200)
No need to use BKE_object_where_is_calc() in this case: there is no
parenting or constraint system involved on object add.

So simply use direct object matrix calculation from it's local
transform. No need in dependency graph either in this case.

source/blender/editors/curve/editfont.c
source/blender/editors/include/ED_object.h
source/blender/editors/object/object_add.c

index c912cea5937af281018e17b86793b362f0a8de8b..f0c1abff2017cdbf53e410df31ce45084a220682 100644 (file)
@@ -606,7 +606,7 @@ static void txt_add_object(bContext *C, TextLine *firstline, int totline, const
   base = view_layer->basact;
 
   /* seems to assume view align ? TODO - look into this, could be an operator option */
-  ED_object_base_init_transform(C, base, NULL, rot);
+  ED_object_base_init_transform_on_add(base->object, NULL, rot);
 
   BKE_object_where_is_calc(depsgraph, scene, obedit);
 
index 3a74bdc59ff7b36df0501c2215fab7b343e299b4..7b9a96e4d07be1af1cb433a880d5c6771d6a43ea 100644 (file)
@@ -197,10 +197,9 @@ void ED_object_sculptmode_exit(struct bContext *C, struct Depsgraph *depsgraph);
 void ED_object_location_from_view(struct bContext *C, float loc[3]);
 void ED_object_rotation_from_quat(float rot[3], const float quat[4], const char align_axis);
 void ED_object_rotation_from_view(struct bContext *C, float rot[3], const char align_axis);
-void ED_object_base_init_transform(struct bContext *C,
-                                   struct Base *base,
-                                   const float loc[3],
-                                   const float rot[3]);
+void ED_object_base_init_transform_on_add(struct Object *obejct,
+                                          const float loc[3],
+                                          const float rot[3]);
 float ED_object_new_primitive_matrix(struct bContext *C,
                                      struct Object *editob,
                                      const float loc[3],
index 137af917f9aa8f2a11948451a444a59b8a3246a1..7e031866dec99f0ea2e26cb4b4378d9d26b48b4f 100644 (file)
@@ -232,29 +232,17 @@ void ED_object_rotation_from_view(bContext *C, float rot[3], const char align_ax
   }
 }
 
-void ED_object_base_init_transform(bContext *C, Base *base, const float loc[3], const float rot[3])
+void ED_object_base_init_transform_on_add(Object *object, const float loc[3], const float rot[3])
 {
-  Object *ob = base->object;
-  Scene *scene = CTX_data_scene(C);
-  Depsgraph *depsgraph = CTX_data_ensure_evaluated_depsgraph(C);
-
-  if (!scene) {
-    return;
-  }
-
   if (loc) {
-    copy_v3_v3(ob->loc, loc);
+    copy_v3_v3(object->loc, loc);
   }
 
   if (rot) {
-    copy_v3_v3(ob->rot, rot);
+    copy_v3_v3(object->rot, rot);
   }
 
-  Scene *scene_eval = DEG_get_evaluated_scene(depsgraph);
-  Object *object_eval = DEG_get_evaluated_object(depsgraph, ob);
-  BKE_object_transform_copy(object_eval, ob);
-  BKE_object_where_is_calc(depsgraph, scene_eval, object_eval);
-  BKE_object_transform_copy(ob, object_eval);
+  BKE_object_to_mat4(object, object->obmat);
 }
 
 /* Uses context to figure out transform for primitive.
@@ -494,7 +482,7 @@ Object *ED_object_add_type(bContext *C,
   ED_object_base_activate(C, view_layer->basact);
 
   /* more editor stuff */
-  ED_object_base_init_transform(C, view_layer->basact, loc, rot);
+  ED_object_base_init_transform_on_add(ob, loc, rot);
 
   /* TODO(sergey): This is weird to manually tag objects for update, better to
    * use DEG_id_tag_update here perhaps.