reapack

commit 8684faf8b9760237ce9632987f7446e97b4c3865
parent 055f3fb32ae89312b28499a57e275b08f19f7c06
Author: cfillion <cfillion@users.noreply.github.com>
Date:   Thu, 30 Mar 2017 19:43:50 -0400

report: don't show "All Done" message when there are only errors

Diffstat:
M
src/report.cpp
 | 
11
++++++++---
M
src/report.hpp
 | 
2
+-

2 files changed, 9 insertions(+), 4 deletions(-)
diff --git a/src/report.cpp b/src/report.cpp
@@ -38,9 +38,6 @@ void Report::onInit()
 
   fillReport();
 
-  const auto_string &str = make_autostring(m_stream.str());
-  SetDlgItemText(handle(), IDC_REPORT, str.c_str());
-
   SetFocus(getControl(IDOK));
 }
 
@@ -93,6 +90,14 @@ void Report::fillReport()
 
   if(removals)
     printRemovals();
+
+  if(installs + updates + removals == 0) {
+    SetDlgItemText(handle(), IDC_LABEL,
+      AUTO_STR("Oops! The following error(s) occured:"));
+  }
+
+  const auto_string &str = make_autostring(m_stream.str());
+  SetDlgItemText(handle(), IDC_REPORT, str.c_str());
 }
 
 void Report::printInstalls()
diff --git a/src/report.hpp b/src/report.hpp
@@ -34,7 +34,7 @@ public:
 protected:
   void onInit() override;
 
-  virtual void fillReport();
+  void fillReport();
 
   void printHeader(const char *);