Bug 368051

Summary: Transcoded clip is not added to the right folder
Product: [Applications] kdenlive Reporter: aslanex
Component: User Interface & MiscellaneousAssignee: Jean-Baptiste Mardelle <jb>
Status: RESOLVED FIXED    
Severity: minor CC: wegwerf-1-2-3
Priority: NOR    
Version First Reported In: 16.08.0   
Target Milestone: ---   
Platform: Ubuntu   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:

Description aslanex 2016-08-31 12:25:02 UTC 
I'm using folders in the project bin. When I set a clip to transcode and automatically add it to the project, it is added to the folder where cursor is (which clip is currently selected) in the moment when clip is added. I would expect it to be added to the folder where the old clip is.

Additionally, an option to automatically remove the old clip would be handy :)

Reproducible: Always

Steps to Reproduce:
1. Add a clip to a folder in the project bin
2. Set it to be transcoded and be added to the project
3. Move cursor out of the folder ­- select a clip outside the folder
4. Wait until the clip is transcoded
5. Clip is added, but not to the original folder
Comment 1 Jean-Baptiste Mardelle 2016-09-02 14:55:58 UTC 
Git commit 0a1f6ca62b2d3a0e5e5d536d9b2e8e8e7e0f2e17 by Jean-Baptiste Mardelle.
Committed on 02/09/2016 at 14:55.
Pushed by mardelle into branch 'Applications/16.08'.

Fix transcoded clips showing up in wrong folder

M  +28   -3    src/bin/bin.cpp
M  +4    -2    src/bin/bin.h
M  +1    -1    src/core.cpp
M  +5    -4    src/mainwindow.cpp
M  +1    -1    src/mainwindow.h
M  +3    -3    src/project/cliptranscode.cpp
M  +3    -2    src/project/cliptranscode.h
M  +2    -2    src/project/jobs/abstractclipjob.cpp
M  +3    -3    src/project/jobs/abstractclipjob.h
M  +5    -2    src/project/jobs/cutclipjob.cpp
M  +1    -1    src/project/jobs/filterjob.cpp
M  +2    -2    src/project/jobs/jobmanager.cpp
M  +1    -1    src/project/jobs/jobmanager.h

http://commits.kde.org/kdenlive/0a1f6ca62b2d3a0e5e5d536d9b2e8e8e7e0f2e17
Comment 2 Wegwerf 2016-09-02 17:43:49 UTC 
Great! This is a fix for something that has annoyed me many times while working in my daytime projects,  where I need to transcode a lot of screen recording footage. Thank you very much, Jean-Baptiste!
Comment 3 aslanex 2016-09-02 21:19:41 UTC 
Thanks for the fix. Should I mark it as resolved, or wait for the next version and check it, or should it be done by someone on a higher place?
Comment 4 Wegwerf 2016-09-02 21:36:35 UTC 
Can you try the testing ppa and then close if satisfyingly fixed? Otherwise I can test in a few days and close the report.
Comment 5 Wegwerf 2016-09-04 10:12:08 UTC 
Tested: works like a charm; thank you very much, Jean-Baptiste for fixing!
Comment 6 aslanex 2016-09-04 10:44:04 UTC 
Ah, sorry, I wasn't at home yesterday. Thanks for your time, developers.