From de9448a65655451e1310947955dd6b341479dc23 Mon Sep 17 00:00:00 2001 From: Kitsune Ral Date: Wed, 8 Mar 2017 20:22:44 +0900 Subject: Connection::callApi<>, a factory for all API call jobs This call (intended to only be used within the lib) creates a job object of the passed type and passes a ConnectionData pointer to its constructor. This allows to avoid making a switchboard of methods on Connection for the whole API, leaving only those that naturally belong there - e.g. joinRoom() or sync() - and moving, e.g., postMessage() to where it belongs - Room. PostMessageJob and RoomMessagesJob were updated along the way, to unbind from the Room class (which they really don't and won't need). --- jobs/postmessagejob.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'jobs/postmessagejob.h') diff --git a/jobs/postmessagejob.h b/jobs/postmessagejob.h index 2e1989fd..eba3da51 100644 --- a/jobs/postmessagejob.h +++ b/jobs/postmessagejob.h @@ -22,11 +22,11 @@ namespace QMatrixClient { - class Room; class PostMessageJob: public BaseJob { public: - PostMessageJob(ConnectionData* connection, Room* room, QString type, QString message); + PostMessageJob(ConnectionData* connection, QString roomId, + QString type, QString message); virtual ~PostMessageJob(); //bool success(); -- cgit v1.2.3