Bug 356536

Summary:	Copying effect causes timeline-project inconsistency
Product:	[Applications] kdenlive	Reporter:	qubodup
Component:	Video Effects & Transitions	Assignee:	Vincent PINON <vpinon>
Status:	RESOLVED FIXED
Severity:	crash	CC:	jb
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	Compiled Sources
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:

Description qubodup 2015-12-12 03:18:01 UTC

kdenlive-git v15.11.90-76-gcb3c173 master
catalyst-libgl 15.11-1
ffmpeg 1:2.8.3-1
ffmpeg-compat 1:0.10.16-4
libx264 2:148.20150725-1
mlt-git r4145.v0.9.2.367.g0de91fe-1
mlt-python-bindings 0.9.8-3
movit-git r713.20c2ca7-1
qt5-base 5.5.1-3
x264 2:148.20150725-1
xorg-server 1.16.4-1

Copying the speed effect and then moving another clip seems to damage the copied-from clip's position in the UI while the project remembers it (if you save and reload it).

Steps to reproduce:
1. Start Kdenlive
2. Import a video
3. Drag video to timeline
4. Cut the video twice
5. Drag "Speed" effect to first clip, change to 1000%
6. Copy the clip
7. Rightclick->Paste the effect to the second clip
8. Leftclick-Drag the third clip

Expected result:
1. Third clip moves

Actual result:
1. Second and third clip move
2. After moving, the second clip can't be moved
3a. If you use undo quickly, sometimes Kdenlive will crash.
3b. If you save the project, close Kdenlive and re-open it, the second clip will jump back to its position

Videos:
https://youtu.be/qXUjeJ1pZfw
https://youtu.be/woR-yeBEH0k

This might be related to https://bugs.kde.org/show_bug.cgi?id=355003 and/or https://bugs.kde.org/show_bug.cgi?id=348148 .

Comment 1 qubodup 2015-12-12 03:22:12 UTC

The second upload was unsuccessful, here is the video that I wanted to link second:
https://youtu.be/yECM0Sr6uuA

Comment 2 qubodup 2015-12-13 05:29:49 UTC

I discovered this is not specific to the speed effect. It can be reproduced with the fade from black effect for example:
1. Start Kdenlive
2. Add a video
3. Put video in timeline
4. Cut video in two pieces
5. Add fade effect to first clip
6. Copy first clip
7. Paste effects to second clip AND DO NOTHING ELSE
8. Drag the second clip somewhere else

At this point, the first clip starts "following" weirdly.

Workaround is to click somewhere else before starting to drag the second clip.

New video:
https://youtu.be/S6BN7acoQAQ

Comment 3 Jean-Baptiste Mardelle 2015-12-13 20:52:31 UTC

Git commit ce0475749015de3707b06077fb05abb613eb14ce by Jean-Baptiste Mardelle.
Committed on 13/12/2015 at 20:52.
Pushed by mardelle into branch 'master'.

Fix broken move in timeline after using context menu

M  +1    -1    src/project/jobs/proxyclipjob.cpp
M  +10   -13   src/timeline/customtrackview.cpp

http://commits.kde.org/kdenlive/ce0475749015de3707b06077fb05abb613eb14ce

Comment 4 qubodup 2015-12-13 21:09:09 UTC

Resolution confirmed working. Thank you very much!

Video:
https://youtu.be/SFD5p3TRy1E

Comment 5 Jean-Baptiste Mardelle 2015-12-14 19:36:20 UTC

Git commit fa4a2bdc6329da2aabfe6e8425076765c8105c22 by Jean-Baptiste Mardelle.
Committed on 14/12/2015 at 19:29.
Pushed by mardelle into branch 'Applications/15.12'.

Fix broken move in timeline after using context menu

M  +5    -0    src/project/jobs/proxyclipjob.cpp
M  +10   -13   src/timeline/customtrackview.cpp

http://commits.kde.org/kdenlive/fa4a2bdc6329da2aabfe6e8425076765c8105c22