Show results on error. Refs #6202

mantidproject · Apr 9, 2013 · 4dcb51c · 4dcb51c
1 parent 37842da
commit 4dcb51c
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 2 deletions.
diff --git a/Code/Mantid/MantidPlot/src/ApplicationWindow.cpp b/Code/Mantid/MantidPlot/src/ApplicationWindow.cpp
@@ -284,6 +284,7 @@ void ApplicationWindow::init(bool factorySettings, const QStringList& args)
   qRegisterMetaType<Message>("Message"); // Required to use it in signals-slots
   resultsLog = new MantidQt::API::MessageDisplay(MessageDisplay::EnableLogLevelControl, logWindow);
   logWindow->setWidget(resultsLog);
+  connect(resultsLog, SIGNAL(errorReceived(const QString &)), logWindow, SLOT(show()));
 
   // Start Mantid
   // Set the Paraview path BEFORE libaries are loaded. Doing it here prevents

diff --git a/Code/Mantid/MantidQt/API/inc/MantidQtAPI/MessageDisplay.h b/Code/Mantid/MantidQt/API/inc/MantidQtAPI/MessageDisplay.h
@@ -40,7 +40,7 @@ namespace MantidQt
       /// Controls whether the display is allowed to set the log levels
       enum LogLevelControl {
         EnableLogLevelControl = 0,
-        DisableLogLevelControl = 1
+        DisableLogLevelControl
       };
 
       /// Default constructor
@@ -52,6 +52,10 @@ namespace MantidQt
       // Setup logging framework connections
       void attachLoggingChannel();
 
+    signals:
+      /// Indicate that a message of error or higher has been received.
+      void errorReceived(const QString & text);
+
     public slots:
       /// Convenience method for appending message at fatal level
       void appendFatal(const QString & text);

diff --git a/Code/Mantid/MantidQt/API/src/MessageDisplay.cpp b/Code/Mantid/MantidQt/API/src/MessageDisplay.cpp
@@ -125,14 +125,16 @@ namespace MantidQt
       appendText(msg.text());
       //set the colour back to the default (black) for historical reasons
       setTextColor(Message::Priority::PRIO_INFORMATION);
+      if(msg.priority() <= Message::Priority::PRIO_ERROR ) emit errorReceived(msg.text());
     }
 
     /**
      * @param msg Replace the current contents with this message
      */
     void MessageDisplay::replace(const Message & msg)
     {
-      m_textDisplay->setText(msg.text());
+      clear();
+      append(msg.text());
     }
 
     /**