D24433: Move URL parsing methods from kioslave to query object

Stefan BrĂ¼ns noreply at phabricator.kde.org
Fri Dec 13 01:12:36 GMT 2019


bruns added inline comments.

INLINE COMMENTS

> query.cpp:342
> +        QString jsonString = urlQuery.queryItemValue(QStringLiteral("json"), QUrl::FullyDecoded);
> +        q = Query::fromJSON(jsonString.toUtf8());
> +    } else if (urlQuery.hasQueryItem(QStringLiteral("query"))) {

just `return Query::fromJSON(jsonString.toUtf8());`

> query.cpp:343
> +        q = Query::fromJSON(jsonString.toUtf8());
> +    } else if (urlQuery.hasQueryItem(QStringLiteral("query"))) {
> +        QString queryString = urlQuery.queryItemValue(QStringLiteral("query"), QUrl::FullyDecoded);

remove the `else`

> query.cpp:345
> +        QString queryString = urlQuery.queryItemValue(QStringLiteral("query"), QUrl::FullyDecoded);
> +        q.setSearchString(queryString);
> +    } else {

just `return q;`

> query.cpp:346
> +        q.setSearchString(queryString);
> +    } else {
> +        const QString jsonString = jsonQueryFromUrl(url);

remove `else`

> query.cpp:349
> +        if (!jsonString.isEmpty()) {
> +            q = Query::fromJSON(jsonString.toUtf8());
> +        }

just `return Query::fromJSON(jsonString.toUtf8());`

REPOSITORY
  R293 Baloo

BRANCH
  extend_query_url

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

To: iasensio, #baloo, meven, ngraham, astippich
Cc: broulik, bruns, kde-frameworks-devel, #dolphin, hurikhan77, lots0logs, LeGast00n, fbampaloukas, GB_2, domson, ashaposhnikov, michaelh, astippich, spoorun, ngraham, abrahams
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20191213/c426e0a2/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list