https://bugs.kde.org/show_bug.cgi?id=411760

Wolfgang Bauer <wba...@tmo.at> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |wba...@tmo.at
     Ever confirmed|0                           |1
             Status|REPORTED                    |CONFIRMED

--- Comment #1 from Wolfgang Bauer <wba...@tmo.at> ---
I can reproduce it here.

It's caused by a bug in the function that generates the name for the mood file
(introduced when porting it from KUrl to QUrl):
QUrl 
SqlCollectionLocation::moodFile( const QUrl &track ) const
{
    QUrl moodPath = track;
    moodPath = moodPath.adjusted(QUrl::RemoveFilename);
    moodPath.setPath(moodPath.path() +  '.' + moodPath.fileName().replace(
QRegExp( "(\\.\\w{2,5})$" ), ".mood" ) );
    return moodPath;
}

This wants to replace "filename.ext" with ".filename.mood" in the Url, but it
already removed the filename from the path in the line before, so the actual
result is "/path/to/folder/." (as the fileName() is an empty string) which is
the containing folder itself.
And when Amarok tries to delete the mood file corresponding to the music file
you delete afterwards, the whole folder gets deleted instead.

Proposed fix:
https://phabricator.kde.org/D24040

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to