Review Request 126233: Fix extract to desktop:/ by using a StatJob to resolve to local URL
Eike Hein
hein at kde.org
Thu Dec 3 18:12:23 UTC 2015
> On Dec. 3, 2015, 5:52 p.m., Elvis Angelaccio wrote:
> > Thanks for working on this!
> > This patch does fix bug 294904 [1], so you could even commit it to Applications/15.12 branch.
> >
> > [1]: https://bugs.kde.org/show_bug.cgi?id=294904
Will do~
- Eike
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126233/#review89091
-----------------------------------------------------------
On Dec. 3, 2015, 5:29 p.m., Eike Hein wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126233/
> -----------------------------------------------------------
>
> (Updated Dec. 3, 2015, 5:29 p.m.)
>
>
> Review request for Plasma and Elvis Angelaccio.
>
>
> Repository: ark
>
>
> Description
> -------
>
> Ark's D-Bus service only supports extraction to local paths, which breaks dropping files onto a Folder View using desktop:/. Ideally Ark would use KIO properly (difficult though since not all the backend tools ultimately support it, etc.), but in lieu of that I think it's best to resolve to a local path whenever possible on the Ark side, since otherwise it has to be done in all the callers (aside from FV we also have Dolphin).
>
> See also https://mail.kde.org/pipermail/plasma-devel/2015-December/046881.html
>
>
> Diffs
> -----
>
> part/part.cpp 240b628
>
> Diff: https://git.reviewboard.kde.org/r/126233/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Eike Hein
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20151203/b729ffe6/attachment.html>
More information about the Plasma-devel
mailing list