diff options
author | Hannah von Reth <hannah.vonreth@owncloud.com> | 2022-07-01 15:16:22 +0300 |
---|---|---|
committer | Hannah von Reth <vonreth@kde.org> | 2022-07-05 09:54:53 +0300 |
commit | 9d9003496ff6c0550a3aea266ae44484a917ce4f (patch) | |
tree | 10b521edbdac66e748c057722704dd4bbbc58768 /src/gui | |
parent | e60f5f2867545ae3af4c2bdf233e14569361e975 (diff) |
Use const
Diffstat (limited to 'src/gui')
-rw-r--r-- | src/gui/socketapi/socketapi.cpp | 22 |
1 files changed, 11 insertions, 11 deletions
diff --git a/src/gui/socketapi/socketapi.cpp b/src/gui/socketapi/socketapi.cpp index 3321d8951..dc983bfad 100644 --- a/src/gui/socketapi/socketapi.cpp +++ b/src/gui/socketapi/socketapi.cpp @@ -1013,15 +1013,15 @@ void SocketApi::command_GET_MENU_ITEMS(const QString &argument, OCC::SocketListe // Some options only show for single files if (files.size() == 1) { - FileData fileData = FileData::get(files.first()); - auto record = fileData.journalRecord(); - bool isOnTheServer = record.isValid(); - auto flagString = isOnTheServer ? QLatin1String("::") : QLatin1String(":d:"); + const FileData fileData = FileData::get(files.first()); + const auto record = fileData.journalRecord(); + const bool isOnTheServer = record.isValid(); + const auto flagString = isOnTheServer ? QLatin1String("::") : QLatin1String(":d:"); if (fileData.folder && fileData.folder->accountState()->isConnected()) { sendSharingContextMenuOptions(fileData, listener); - auto &capabilities = folder->accountState()->account()->capabilities(); + const auto &capabilities = folder->accountState()->account()->capabilities(); if (capabilities.privateLinkPropertyAvailable()) { listener->sendMessage(QLatin1String("MENU_ITEM:OPEN_PRIVATE_LINK") + flagString + tr("Open in browser")); } @@ -1034,16 +1034,16 @@ void SocketApi::command_GET_MENU_ITEMS(const QString &argument, OCC::SocketListe } // Conflict files get conflict resolution actions - bool isConflict = Utility::isConflictFile(fileData.folderRelativePath); + const bool isConflict = Utility::isConflictFile(fileData.folderRelativePath); if (isConflict || !isOnTheServer) { // Check whether this new file is in a read-only directory - QFileInfo fileInfo(fileData.localPath); - auto parentDir = fileData.parentFolder(); - auto parentRecord = parentDir.journalRecord(); - bool canAddToDir = parentRecord._remotePerm.isNull() + const QFileInfo fileInfo(fileData.localPath); + const auto parentDir = fileData.parentFolder(); + const auto parentRecord = parentDir.journalRecord(); + const bool canAddToDir = parentRecord._remotePerm.isNull() || (fileInfo.isFile() && !parentRecord._remotePerm.hasPermission(RemotePermissions::CanAddFile)) || (fileInfo.isDir() && !parentRecord._remotePerm.hasPermission(RemotePermissions::CanAddSubDirectories)); - bool canChangeFile = + const bool canChangeFile = !isOnTheServer || (record._remotePerm.hasPermission(RemotePermissions::CanDelete) && record._remotePerm.hasPermission(RemotePermissions::CanMove) |