GUI: Enable clearing/saving results also if there are only hidden res…

…ults. This commit fixes a bug in my earlier commit that only checked visible results.
citypw · Jun 20, 2009 · 2adfd36 · 2adfd36
1 parent bf84ab4
commit 2adfd36
Show file tree

Hide file tree

Showing 5 changed files with 24 additions and 1 deletion.
diff --git a/gui/mainwindow.cpp b/gui/mainwindow.cpp
@@ -351,7 +351,7 @@ QStringList MainWindow::RemoveUnacceptedFiles(const QStringList &list)
 void MainWindow::CheckDone()
 {
     EnableCheckButtons(true);
-    if (mResults.HasVisibleResults())
+    if (mResults.HasResults())
     {
         mActionClearResults.setEnabled(true);
         mActionSave.setEnabled(true);

diff --git a/gui/resultstree.cpp b/gui/resultstree.cpp
@@ -778,3 +778,8 @@ bool ResultsTree::HasVisibleResults() const
 {
     return mVisibleErrors;
 }
+
+bool ResultsTree::HasResults() const
+{
+    return mModel.rowCount() > 0;
+}
diff --git a/gui/resultstree.h b/gui/resultstree.h
@@ -100,6 +100,12 @@ class ResultsTree : public QTreeView
     */
     bool HasVisibleResults() const;
 
+    /**
+    * @brief Do we have results from check?
+    * @return true if there is at least one warning/error, hidden or visible.
+    */
+    bool HasResults() const;
+
 protected slots:
     /**
     * @brief Slot to quickstart an error with default application

diff --git a/gui/resultsview.cpp b/gui/resultsview.cpp
@@ -170,3 +170,8 @@ bool ResultsView::HasVisibleResults() const
 {
     return mTree->HasVisibleResults();
 }
+
+bool ResultsView::HasResults() const
+{
+    return mTree->HasResults();
+}
diff --git a/gui/resultsview.h b/gui/resultsview.h
@@ -97,6 +97,13 @@ class ResultsView : public QWidget
     */
     bool HasVisibleResults() const;
 
+    /**
+    * @brief Do we have results from check?
+    *
+    * @return true if there is at least one warning/error, hidden or visible.
+    */
+    bool HasResults() const;
+
 signals:
 
     /**