bruns added inline comments. INLINE COMMENTS
> query.cpp:317 > +{ > + const QString jsonQuery(QStringLiteral("{\"dayFilter\": 0,\ > + \"monthFilter\": 0, \ dayFilter/monthFilter/yearFilter are pointless, as they just default, but have to be parsed in `fromJSON`. > query.cpp:329 > + if (path == QLatin1String("/documents")) { > + return jsonQueryForType(QStringLiteral("Document")); > + } else if (path.endsWith(QLatin1String("/images"))) { can be replaced by `QStringLiteral("{\"type\":[\"Document\"]}")`, dito below. REPOSITORY R293 Baloo BRANCH extend_query_url REVISION DETAIL https://phabricator.kde.org/D24433 To: iasensio, #baloo, meven, ngraham, astippich Cc: bruns, kde-frameworks-devel, #dolphin, hurikhan77, lots0logs, LeGast00n, fbampaloukas, GB_2, domson, ashaposhnikov, michaelh, astippich, spoorun, ngraham, abrahams