[Bf-blender-cvs] [ac16d56aa85] master: Fix T66390: Update missing when snaping strips.

[Richard Antalik]

Commit: ac16d56aa8548f53436e8eaa87c1df32b1708c14
Author: Richard Antalik
Date:   Thu Jun 18 04:26:41 2020 +0200
Branches: master
https://developer.blender.org/rBac16d56aa8548f53436e8eaa87c1df32b1708c14

Fix T66390: Update missing when snaping strips.

Effect strips bound recalculation was mixed with overlap handling, which
caused, that effects wasn't handled.

In some cases there may be problem with order of strips in seqbase. We should
traverse hierarchy instead. This is design issue that applies to all
operators, and should be fixed separately.

Reviewed By: brecht

Differential Revision: https://developer.blender.org/D7880

===================================================================

M	source/blender/editors/space_sequencer/sequencer_edit.c

===================================================================

diff --git a/source/blender/editors/space_sequencer/sequencer_edit.c b/source/blender/editors/space_sequencer/sequencer_edit.c
index 9311cbed265..4a8bea11b41 100644
--- a/source/blender/editors/space_sequencer/sequencer_edit.c
+++ b/source/blender/editors/space_sequencer/sequencer_edit.c
@@ -1403,7 +1403,11 @@ static int sequencer_snap_exec(bContext *C, wmOperator *op)
         BKE_sequence_base_shuffle(ed->seqbasep, seq, scene);
       }
     }
-    else if (seq->type & SEQ_TYPE_EFFECT) {
+  }
+
+  /* Recalculate bounds of effect strips. */
+  for (seq = ed->seqbasep->first; seq; seq = seq->next) {
+    if (seq->type & SEQ_TYPE_EFFECT) {
       if (seq->seq1 && (seq->seq1->flag & SELECT)) {
         BKE_sequence_calc(scene, seq);
       }