Bug 313003

Summary: Copy/Move to collection doesn't pick up tags and appends .MetaProxy::Track to each output filename
Product: [Applications] amarok Reporter: Thierry Göckel <thierry>
Component: File BrowserAssignee: Amarok Developers <amarok-bugs-dist>
Status: RESOLVED FIXED    
Severity: major CC: matej
Priority: NOR Keywords: regression, release_blocker
Version: 2.6-git   
Target Milestone: 2.7   
Platform: Debian unstable   
OS: Linux   
URL: http://i.imgur.com/0Fwcj.png
Latest Commit: Version Fixed In: 2.7

Description Thierry Göckel 2013-01-10 10:09:47 UTC
On most folders and files I try to use 'copy to collection' (with advanced output options) with, the problem described in the summary and shown in the screenshot happens. It doesn't happen with files actually already _in_ the collection.

Cheers!

Reproducible: Always

Steps to Reproduce:
1. Go (inside Amarok) to some folder outside the collection and check if tags are in fact valid
2. Choose Copy to collection from Context Menu
3. Take a look at preview
Actual Results:  
See screenshot.

Expected Results:  
Paths generated from files' tags.

git describe: v2.6.90-79-g852a71c
Comment 1 Matěj Laitl 2013-01-10 10:16:11 UTC
Oh, my bad, this seems to be a bug that I introduced. Lemme have a look at it and thanks for testing current git and reporting back.
Comment 2 Matěj Laitl 2013-01-11 16:38:41 UTC
Git commit d8aa75f0f5ba607c7db14a5b1ddea6c17beeb966 by Matěj Laitl.
Committed on 11/01/2013 at 17:33.
Pushed by laitl into branch 'master'.

DirectoryLoader, FileView: fix broken Copy/Move to collection

I accidentally broke this in commit 7f1f9f606ad1af5b3d1.

This doesn't go into ChangeLog, because the bug was introduced after
2.7 Beta was released. Thierry, please test that this commit fixes
your problem.
FIXED-IN: 2.7

M  +17   -4    src/DirectoryLoader.cpp
M  +13   -1    src/DirectoryLoader.h
M  +4    -2    src/browsers/filebrowser/FileView.cpp

http://commits.kde.org/amarok/d8aa75f0f5ba607c7db14a5b1ddea6c17beeb966
Comment 3 Thierry Göckel 2013-01-12 11:26:45 UTC
Thanks Matěj, it's working great again!

Cheers