[kget] [Bug 390612] kget directory not writable

[Wolfgang Bauer]

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

Wolfgang Bauer <wbauer at tmo.at> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|CONFIRMED                   |RESOLVED
      Latest Commit|                            |https://commits.kde.org/kge
                   |                            |t/ed694ff5f5c8418a93d59bae6
                   |                            |b68e11b9816c1f3
         Resolution|---                         |FIXED
   Version Fixed In|                            |17.12.3

--- Comment #5 from Wolfgang Bauer <wbauer at tmo.at> ---
Git commit ed694ff5f5c8418a93d59bae6b68e11b9816c1f3 by Wolfgang Bauer.
Committed on 05/03/2018 at 14:28.
Pushed by wbauer into branch 'Applications/17.12'.

Fix destDir handling in KGet::addTransfer()

destDir is supposed to be a local path, so treat it as such and use
QUrl::fromLocalFile() to convert it to QUrl.
And use QUrl::path() to get the directory as QString, QUrl::toString()
returns an Url including "file://".

Also, adjust KGet::redownloadTransfer() this change, it should actually
pass a path as destDir (without "file://"), and avoid an unnecessary
QUrl->QString->QUrl conversion for the src whilst at it.

This fixes spurious "Directory is not writable" errors when trying to
add a download via the clipboard "Auto-Paste" feature or DBUS.
FIXED-IN: 17.12.3

M  +8    -9    core/kget.cpp

https://commits.kde.org/kget/ed694ff5f5c8418a93d59bae6b68e11b9816c1f3

-- 
You are receiving this mail because:
You are the assignee for the bug.