From: Claudio Cambra <claudio.cambra@nextcloud.com>
Date: Fri, 30 Sep 2022 23:13:41 +0000 (+0200)
Subject: Use [[nodiscard]] more broadly in methods returning values, and respect it elsewhere
X-Git-Tag: archive/raspbian/3.16.7-1_deb13u1+rpi1~1^2~12^2~11^2~169^2~21
X-Git-Url: https://dgit.raspbian.org/?a=commitdiff_plain;h=de0831c2975f6cb27ec80f4ed2fd7c73c908964b;p=nextcloud-desktop.git

Use [[nodiscard]] more broadly in methods returning values, and respect it elsewhere

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

diff --git a/src/gui/fileactivitylistmodel.h b/src/gui/fileactivitylistmodel.h
index 4134dc0c7..a11197405 100644
--- a/src/gui/fileactivitylistmodel.h
+++ b/src/gui/fileactivitylistmodel.h
@@ -27,7 +27,7 @@ class FileActivityListModel : public ActivityListModel
 public:
     explicit FileActivityListModel(QObject *parent = nullptr);
 
-    QString localPath() const;
+    [[nodiscard]] QString localPath() const;
 
 signals:
     void localPathChanged();
diff --git a/src/gui/filedetails/filedetails.cpp b/src/gui/filedetails/filedetails.cpp
index b7fa88a18..2f0e29a10 100644
--- a/src/gui/filedetails/filedetails.cpp
+++ b/src/gui/filedetails/filedetails.cpp
@@ -19,6 +19,8 @@
 
 namespace OCC {
 
+Q_LOGGING_CATEGORY(lcFileDetails, "nextcloud.gui.filedetails", QtInfoMsg)
+
 FileDetails::FileDetails(QObject *parent)
     : QObject(parent)
 {
@@ -61,7 +63,11 @@ void FileDetails::setLocalPath(const QString &localPath)
     const auto folder = FolderMan::instance()->folderForPath(_localPath);
     const auto file = _localPath.mid(folder->cleanPath().length() + 1);
 
-    folder->journalDb()->getFileRecord(file, &_fileRecord);
+    if (!folder->journalDb()->getFileRecord(file, &_fileRecord)) {
+        qCWarning(lcFileDetails) << "Invalid file record for path:"
+                                 << _localPath
+                                 << "will not load file details.";
+    }
 
     _filelockState = _fileRecord._lockstate;
     updateLockExpireString();
diff --git a/src/gui/filedetails/filedetails.h b/src/gui/filedetails/filedetails.h
index 7b3384709..3a5d1e529 100644
--- a/src/gui/filedetails/filedetails.h
+++ b/src/gui/filedetails/filedetails.h
@@ -37,13 +37,13 @@ class FileDetails : public QObject
 public:
     explicit FileDetails(QObject *parent = nullptr);
 
-    QString localPath() const;
-    QString name() const;
-    QString sizeString() const;
-    QString lastChangedString() const;
-    QString iconUrl() const;
-    QString lockExpireString() const;
-    bool isFolder() const;
+    [[nodiscard]] QString localPath() const;
+    [[nodiscard]] QString name() const;
+    [[nodiscard]] QString sizeString() const;
+    [[nodiscard]] QString lastChangedString() const;
+    [[nodiscard]] QString iconUrl() const;
+    [[nodiscard]] QString lockExpireString() const;
+    [[nodiscard]] bool isFolder() const;
 
 public slots:
     void setLocalPath(const QString &localPath);
diff --git a/src/gui/filedetails/shareemodel.h b/src/gui/filedetails/shareemodel.h
index 954f10ad7..8029ee765 100644
--- a/src/gui/filedetails/shareemodel.h
+++ b/src/gui/filedetails/shareemodel.h
@@ -51,15 +51,15 @@ public:
 
     using ShareeSet = QVector<ShareePtr>; // FIXME: make it a QSet<Sharee> when Sharee can be compared
 
-    int rowCount(const QModelIndex &parent = QModelIndex()) const override;
-    QHash<int, QByteArray> roleNames() const override;
-    QVariant data(const QModelIndex &index, const int role) const override;
+    [[nodiscard]] int rowCount(const QModelIndex &parent = QModelIndex()) const override;
+    [[nodiscard]] QHash<int, QByteArray> roleNames() const override;
+    [[nodiscard]] QVariant data(const QModelIndex &index, const int role) const override;
 
-    AccountState *accountState() const;
-    bool shareItemIsFolder() const;
-    QString searchString() const;
-    bool fetchOngoing() const;
-    LookupMode lookupMode() const;
+    [[nodiscard]] AccountState *accountState() const;
+    [[nodiscard]] bool shareItemIsFolder() const;
+    [[nodiscard]] QString searchString() const;
+    [[nodiscard]] bool fetchOngoing() const;
+    [[nodiscard]] LookupMode lookupMode() const;
 
 signals:
     void accountStateChanged();
@@ -83,7 +83,7 @@ private slots:
      void shareesFetched(const QJsonDocument &reply);
 
 private:
-    ShareePtr parseSharee(const QJsonObject &data) const;
+    [[nodiscard]] ShareePtr parseSharee(const QJsonObject &data) const;
 
     QTimer _userStoppedTypingTimer;
 
diff --git a/src/gui/filedetails/sharemodel.h b/src/gui/filedetails/sharemodel.h
index 4328aeff1..5262bdf4b 100644
--- a/src/gui/filedetails/sharemodel.h
+++ b/src/gui/filedetails/sharemodel.h
@@ -78,21 +78,21 @@ public:
 
     explicit ShareModel(QObject *parent = nullptr);
 
-    QVariant data(const QModelIndex &index, const int role) const override;
-    int rowCount(const QModelIndex &parent = QModelIndex()) const override;
-    QHash<int, QByteArray> roleNames() const override;
+    [[nodiscard]] QVariant data(const QModelIndex &index, const int role) const override;
+    [[nodiscard]] int rowCount(const QModelIndex &parent = QModelIndex()) const override;
+    [[nodiscard]] QHash<int, QByteArray> roleNames() const override;
 
-    AccountState *accountState() const;
-    QString localPath() const;
+    [[nodiscard]] AccountState *accountState() const;
+    [[nodiscard]] QString localPath() const;
 
-    bool accountConnected() const;
-    bool sharingEnabled() const;
-    bool publicLinkSharesEnabled() const;
-    bool userGroupSharingEnabled() const;
-    bool canShare() const;
+    [[nodiscard]] bool accountConnected() const;
+    [[nodiscard]] bool sharingEnabled() const;
+    [[nodiscard]] bool publicLinkSharesEnabled() const;
+    [[nodiscard]] bool userGroupSharingEnabled() const;
+    [[nodiscard]] bool canShare() const;
 
-    bool fetchOngoing() const;
-    bool hasInitialShareFetchCompleted() const;
+    [[nodiscard]] bool fetchOngoing() const;
+    [[nodiscard]] bool hasInitialShareFetchCompleted() const;
 
 signals:
     void localPathChanged();
@@ -167,11 +167,11 @@ private slots:
     void slotShareExpireDateSet(const QString &shareId);
 
 private:
-    QString displayStringForShare(const SharePtr &share) const;
-    QString iconUrlForShare(const SharePtr &share) const;
-    QString avatarUrlForShare(const SharePtr &share) const;
-    long long enforcedMaxExpireDateForShare(const SharePtr &share) const;
-    bool expireDateEnforcedForShare(const SharePtr &share) const;
+    [[nodiscard]] QString displayStringForShare(const SharePtr &share) const;
+    [[nodiscard]] QString iconUrlForShare(const SharePtr &share) const;
+    [[nodiscard]] QString avatarUrlForShare(const SharePtr &share) const;
+    [[nodiscard]] long long enforcedMaxExpireDateForShare(const SharePtr &share) const;
+    [[nodiscard]] bool expireDateEnforcedForShare(const SharePtr &share) const;
 
     bool _fetchOngoing = false;
     bool _hasInitialShareFetchCompleted = false;
diff --git a/src/gui/filedetails/sortedsharemodel.h b/src/gui/filedetails/sortedsharemodel.h
index c5cdbb975..a9acc1d58 100644
--- a/src/gui/filedetails/sortedsharemodel.h
+++ b/src/gui/filedetails/sortedsharemodel.h
@@ -27,7 +27,7 @@ class SortedShareModel : public QSortFilterProxyModel
 public:
     explicit SortedShareModel(QObject *parent = nullptr);
 
-    ShareModel *shareModel() const;
+    [[nodiscard]] ShareModel *shareModel() const;
 
 signals:
     void shareModelChanged();
@@ -36,7 +36,7 @@ public slots:
     void setShareModel(ShareModel *shareModel);
 
 protected:
-    bool lessThan(const QModelIndex &sourceLeft, const QModelIndex &sourceRight) const override;
+    [[nodiscard]] bool lessThan(const QModelIndex &sourceLeft, const QModelIndex &sourceRight) const override;
 
 private slots:
     void sortModel();