Actually delete BaseJob::Query

It was a tiny wrapper around QUrlQuery to facilitate creation from
an initializer list - however, Mustache templates long changed to
not actually used that additional constructor.

2 files changed, 3 insertions, 17 deletions

diff --git a/lib/jobs/basejob.cpp b/lib/jobs/basejob.cpp
index c27c6a89..9a7b9b5e 100644
--- a/lib/jobs/basejob.cpp
+++ b/lib/jobs/basejob.cpp
@@ -175,11 +175,11 @@ public:
 
 BaseJob::BaseJob(HttpVerb verb, const QString& name, const QString& endpoint,
                  bool needsToken)
-    : BaseJob(verb, name, endpoint, Query {}, Data {}, needsToken)
+    : BaseJob(verb, name, endpoint, QUrlQuery {}, Data {}, needsToken)
 {}
 
 BaseJob::BaseJob(HttpVerb verb, const QString& name, const QString& endpoint,
-                 const Query& query, Data&& data, bool needsToken)
+                 const QUrlQuery &query, Data&& data, bool needsToken)
     : d(new Private(verb, endpoint, query, std::move(data), needsToken))
 {
     setObjectName(name);
diff --git a/lib/jobs/basejob.h b/lib/jobs/basejob.h
index ca91a781..d33d542e 100644
--- a/lib/jobs/basejob.h
+++ b/lib/jobs/basejob.h
@@ -72,20 +72,6 @@ public:
     };
     Q_ENUM(StatusCode)
 
-    /**
-     * A simple wrapper around QUrlQuery that allows its creation from
-     * a list of string pairs
-     */
-    class Query : public QUrlQuery {
-    public:
-        using QUrlQuery::QUrlQuery;
-        Query() = default;
-        Query(const std::initializer_list<QPair<QString, QString>>& l)
-        {
-            setQueryItems(l);
-        }
-    };
-
     using Data = RequestData;
 
     /*!
@@ -139,7 +125,7 @@ public:
     BaseJob(HttpVerb verb, const QString& name, const QString& endpoint,
             bool needsToken = true);
     BaseJob(HttpVerb verb, const QString& name, const QString& endpoint,
-            const Query& query, Data&& data = {}, bool needsToken = true);
+            const QUrlQuery& query, Data&& data = {}, bool needsToken = true);
 
     QUrl requestUrl() const;
     bool isBackground() const;