1 files changed, 13 insertions, 62 deletions

diff --git a/lib/csapi/presence.cpp b/lib/csapi/presence.cpp
index 024d7a34..58d0d157 100644
--- a/lib/csapi/presence.cpp
+++ b/lib/csapi/presence.cpp
@@ -4,82 +4,33 @@
 
 #include "presence.h"
 
-#include "converters.h"
-
 #include <QtCore/QStringBuilder>
 
-using namespace QMatrixClient;
-
-static const auto basePath = QStringLiteral("/_matrix/client/r0");
-
-static const auto SetPresenceJobName = QStringLiteral("SetPresenceJob");
+using namespace Quotient;
 
-SetPresenceJob::SetPresenceJob(const QString& userId, const QString& presence, const QString& statusMsg)
-    : BaseJob(HttpVerb::Put, SetPresenceJobName,
-        basePath % "/presence/" % userId % "/status")
+SetPresenceJob::SetPresenceJob(const QString& userId, const QString& presence,
+                               const QString& statusMsg)
+    : BaseJob(HttpVerb::Put, QStringLiteral("SetPresenceJob"),
+              QStringLiteral("/_matrix/client/r0") % "/presence/" % userId
+                  % "/status")
 {
     QJsonObject _data;
     addParam<>(_data, QStringLiteral("presence"), presence);
     addParam<IfNotEmpty>(_data, QStringLiteral("status_msg"), statusMsg);
-    setRequestData(_data);
+    setRequestData(std::move(_data));
 }
 
-class GetPresenceJob::Private
-{
-    public:
-        QString presence;
-        Omittable<int> lastActiveAgo;
-        QString statusMsg;
-        Omittable<bool> currentlyActive;
-};
-
 QUrl GetPresenceJob::makeRequestUrl(QUrl baseUrl, const QString& userId)
 {
     return BaseJob::makeRequestUrl(std::move(baseUrl),
-            basePath % "/presence/" % userId % "/status");
+                                   QStringLiteral("/_matrix/client/r0")
+                                       % "/presence/" % userId % "/status");
 }
 
-static const auto GetPresenceJobName = QStringLiteral("GetPresenceJob");
-
 GetPresenceJob::GetPresenceJob(const QString& userId)
-    : BaseJob(HttpVerb::Get, GetPresenceJobName,
-        basePath % "/presence/" % userId % "/status")
-    , d(new Private)
+    : BaseJob(HttpVerb::Get, QStringLiteral("GetPresenceJob"),
+              QStringLiteral("/_matrix/client/r0") % "/presence/" % userId
+                  % "/status")
 {
+    addExpectedKey("presence");
 }
-
-GetPresenceJob::~GetPresenceJob() = default;
-
-const QString& GetPresenceJob::presence() const
-{
-    return d->presence;
-}
-
-Omittable<int> GetPresenceJob::lastActiveAgo() const
-{
-    return d->lastActiveAgo;
-}
-
-const QString& GetPresenceJob::statusMsg() const
-{
-    return d->statusMsg;
-}
-
-Omittable<bool> GetPresenceJob::currentlyActive() const
-{
-    return d->currentlyActive;
-}
-
-BaseJob::Status GetPresenceJob::parseJson(const QJsonDocument& data)
-{
-    auto json = data.object();
-    if (!json.contains("presence"_ls))
-        return { JsonParseError,
-            "The key 'presence' not found in the response" };
-    fromJson(json.value("presence"_ls), d->presence);
-    fromJson(json.value("last_active_ago"_ls), d->lastActiveAgo);
-    fromJson(json.value("status_msg"_ls), d->statusMsg);
-    fromJson(json.value("currently_active"_ls), d->currentlyActive);
-    return Success;
-}
-