D15787: [cli] Always take working dir into account when parsing url

Aleix Pol Gonzalez noreply at phabricator.kde.org
Thu Sep 27 02:19:40 BST 2018


apol accepted this revision.
apol added inline comments.
This revision is now accepted and ready to land.

INLINE COMMENTS

> kdeconnect-cli.cpp:151
>              QList<QUrl> urls;
> -            QUrl url = QUrl::fromUserInput(parser.value(QStringLiteral("share")));
> +            QUrl url = QUrl::fromUserInput(parser.value(QStringLiteral("share")), QDir::currentPath());
>              urls.append(url);

like I said on the other, add `QUrl::AssumeLocalFile`.

REPOSITORY
  R224 KDE Connect

BRANCH
  parseurl

REVISION DETAIL
  https://phabricator.kde.org/D15787

To: nicolasfella, #kde_connect, apol
Cc: apol, kdeconnect, wistak, dvalencia, rmenezes, julioc, Leptopoda, timothyc, jdvr, yannux, Danial0_0, johnq, Pitel, adeen-s, SemperPeritus, ndavis, daniel.z.tg, jeanv, seebauer, bugzy, MayeulC, menasshock, tctara
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20180927/23d8be36/attachment.html>


More information about the KDEConnect mailing list