diff options
author | Hannah von Reth <hannah.vonreth@owncloud.com> | 2021-03-18 13:26:07 +0300 |
---|---|---|
committer | Hannah von Reth <vonreth@kde.org> | 2021-03-31 16:50:25 +0300 |
commit | 8c69bb70b3b0b357b29b0f45da5e8c91773696e6 (patch) | |
tree | 08c2f4eb4b0715cfd781d18a0c04177d90cb14dd /src/gui/socketapi | |
parent | 31c66f739d307152fba816313dde90dafccfdc64 (diff) |
Use uuid in socket api
(cherry picked from commit 86f86acc1860b308e947188d821081be6dcf92c4)
Diffstat (limited to 'src/gui/socketapi')
-rw-r--r-- | src/gui/socketapi/socketapi.cpp | 7 | ||||
-rw-r--r-- | src/gui/socketapi/socketuploadjob.cpp | 10 |
2 files changed, 12 insertions, 5 deletions
diff --git a/src/gui/socketapi/socketapi.cpp b/src/gui/socketapi/socketapi.cpp index cc253cbdc..6e2c96ca1 100644 --- a/src/gui/socketapi/socketapi.cpp +++ b/src/gui/socketapi/socketapi.cpp @@ -848,10 +848,11 @@ void SocketApi::command_V2_LIST_ACCOUNTS(const QSharedPointer<SocketApiJobV2> &j { QJsonArray out; for (auto acc : AccountManager::instance()->accounts()) { - // TODO: Use uuid once https://github.com/owncloud/client/pull/8397 is merged - out << QJsonObject({ { "name", acc->account()->displayName() }, { "id", acc->account()->id() } }); + out << QJsonObject({ { QStringLiteral("name"), acc->account()->displayName() }, + { QStringLiteral("id"), acc->account()->id() }, + { QStringLiteral("uuid"), acc->account()->uuid().toString(QUuid::WithoutBraces) } }); } - job->success({ { "accounts", out } }); + job->success({ { QStringLiteral("accounts"), out } }); } void SocketApi::command_V2_UPLOAD_FILES_FROM(const QSharedPointer<SocketApiJobV2> &job) const diff --git a/src/gui/socketapi/socketuploadjob.cpp b/src/gui/socketapi/socketuploadjob.cpp index 467a46973..2e1b4f1ca 100644 --- a/src/gui/socketapi/socketuploadjob.cpp +++ b/src/gui/socketapi/socketuploadjob.cpp @@ -37,9 +37,15 @@ SocketUploadJob::SocketUploadJob(const QSharedPointer<SocketApiJobV2> &job) } _pattern = job->arguments()[QLatin1String("pattern")].toString(); - // TODO: use uuid const auto accname = job->arguments()[QLatin1String("account")][QLatin1String("name")].toString(); - auto account = AccountManager::instance()->account(accname); + const auto accUUID = QUuid::fromString(job->arguments()[QLatin1String("account")][QLatin1String("uuid")].toString()); + AccountStatePtr account; + if (accUUID.isNull()) { + job->setWarning("Using the name as identifier is deprecated, please use the uuid"); + account = AccountManager::instance()->account(accname); + } else { + account = AccountManager::instance()->account(accUUID); + } if (!QFileInfo(_localPath).isAbsolute()) { job->failure(QStringLiteral("Local path must be a an absolute path")); |