Notify user of vehicle boot errors

src/FactSystem/ParameterLoader.cc

@@ -29,6 +29,7 @@
 #include "QGCLoggingCategory.h"
 #include "QGCApplication.h"
 #include "QGCMessageBox.h"
+#include "UASMessageHandler.h"
 
 #include <QFile>
 #include <QDebug>
@@ -752,14 +753,37 @@ void ParameterLoader::_checkInitialLoadComplete(void)
         }
     }
     
+    // Check for any errors during vehicle boot
+    
+    UASMessageHandler* msgHandler = UASMessageHandler::instance();
+    if (msgHandler->getErrorCount()) {
+        QString errors;
+        
+        msgHandler->lockAccess();
+        foreach (UASMessage* msg, msgHandler->messages()) {
+            if (msg->severityIsError()) {
+                errors += msg->getText();
+                errors += "\n";
+            }
+        }
+        msgHandler->unlockAccess();
+        
+        QGCMessageBox::critical("Vehicle startup errors",
+                                QString("Errors were detected during vehicle startup:\n"
+                                        "%1"
+                                        "You should resolve these prior to flight.").arg(errors));
+    }
+    
+    // Warn of parameter load failure
+    
     if (initialLoadFailures) {
         QGCMessageBox::critical("Parameter Load Failure",
-                                QString("QGroundControl was unable to retrieve the full set of parameters from the vehicle. "
-                                        "This will cause QGroundControl to be unable to display it's full user interface. "
-                                        "This usually indicates an error in the vehicle's firmware. "
-                                        "Please upgrade your firmware to the latest version if possible. "
-                                        "If that doesn't work, notify the firmware developers of this error. "
-                                        "The following parameter indices could not be loaded after the maximum number of retries: %1.").arg(indexList));
+                                "QGroundControl was unable to retrieve the full set of parameters from the vehicle. "
+                                "This will cause QGroundControl to be unable to display it's full user interface. "
+                                "If you are using modified firmware, you may need to resolve any vehicle startup errors to resolve the issue. "
+                                "If you are using standard firmware, you may need to upgrade to a newer version to resolve the issue.");
+        qCWarning(ParameterLoaderLog) << "The following parameter indices could not be loaded after the maximum number of retries: " << indexList;
+
     } else {
         // No failed parameters, ok to signal ready
         _parametersReady = true;
@@ -767,4 +791,4 @@ void ParameterLoader::_checkInitialLoadComplete(void)
         _setupGroupMap();
         emit parametersReady();
     }
-}
\ No newline at end of file
+}

src/uas/UASMessageHandler.cc

@@ -38,6 +38,19 @@ UASMessage::UASMessage(int componentid, int severity, QString text)
     _text     = text;
 }
 
+bool UASMessage::severityIsError()
+{
+    switch (_severity) {
+        case MAV_SEVERITY_EMERGENCY:
+        case MAV_SEVERITY_ALERT:
+        case MAV_SEVERITY_CRITICAL:
+        case MAV_SEVERITY_ERROR:
+            return true;
+        default:
+            return false;
+    }
+}
+
 IMPLEMENT_QGC_SINGLETON(UASMessageHandler, UASMessageHandler)
 
 UASMessageHandler::UASMessageHandler(QObject *parent)

src/uas/UASMessageHandler.h

@@ -63,6 +63,11 @@ public:
      * @brief Get (html) formatted text (in the form: "[11:44:21.137 - COMP:50] Info: [pm] sending list")
      */
     QString getFormatedText()   { return _formatedText; }
+    /**
+     * @return true: This message is a of a severity which is considered an error
+     */
+    bool severityIsError();
+    
 private:
     UASMessage(int componentid, int severity, QString text);
     void _setFormatedText(const QString formatedText) { _formatedText = formatedText; }