Simplify check for file record sharing permission

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>

diff --git a/src/gui/filedetails/sharemodel.cpp b/src/gui/filedetails/sharemodel.cpp
index aa406cdd0543ce47ff1cf37b711fe4a35705292c..8c3735f288b57d591f43cc9820b792e479cddae8 100644 (file)
--- a/src/gui/filedetails/sharemodel.cpp
+++ b/src/gui/filedetails/sharemodel.cpp
@@ -240,12 +240,13 @@ void ShareModel::updateData()
     SyncJournalFileRecord fileRecord;
     auto resharingAllowed = true; // lets assume the good
 
-    if(_folder->journalDb()->getFileRecord(relPath, &fileRecord) && fileRecord.isValid()) {
-        if (!fileRecord._remotePerm.isNull() &&
-            !fileRecord._remotePerm.hasPermission(RemotePermissions::CanReshare)) {
+    if(_folder->journalDb()->getFileRecord(relPath, &fileRecord) &&
+       fileRecord.isValid() &&
+       !fileRecord._remotePerm.isNull() &&
+       !fileRecord._remotePerm.hasPermission(RemotePermissions::CanReshare)) {
 
-            resharingAllowed = false;
-        }
+        qCInfo(lcShareModel) << "File record says resharing not allowed";
+        resharingAllowed = false;
     }
 
     _maxSharingPermissions = resharingAllowed ? SharePermissions(_accountState->account()->capabilities().shareDefaultPermissions()) : SharePermissions({});