Bug 368051 - Transcoded clip is not added to the right folder
Summary: Transcoded clip is not added to the right folder
Status: RESOLVED FIXED
Alias: None
Product: kdenlive
Classification: Applications
Component: User Interface & Miscellaneous (other bugs)
Version First Reported In: 16.08.0
Platform: Ubuntu Linux
: NOR minor
Target Milestone: ---
Assignee: Jean-Baptiste Mardelle
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-08-31 12:25 UTC by aslanex
Modified: 2016-09-04 10:44 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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.