Merge pull request #5350 from DonLakeFlyer/LogExtension

Change to log file extension handling

src/AnalyzeView/LogDownloadController.cc

@@ -596,9 +596,9 @@ LogDownloadController::_prepareLogDownload()
     _downloadData = new LogDownloadData(entry);
     _downloadData->filename = QString("log_") + QString::number(entry->id()) + "_" + ftime;
     if (_vehicle->firmwareType() == MAV_AUTOPILOT_PX4) {
-
-        // This is a stopgap and should be removed once log file types are properly supported by the log download protocol
-        if (_vehicle->parameterManager()->getParameter(FactSystem::defaultComponentId, "SYS_LOGGER")->rawValue().toInt() == 0) {
+        QString loggerParam("SYS_LOGGER");
+        if (_vehicle->parameterManager()->parameterExists(FactSystem::defaultComponentId, loggerParam) &&
+                _vehicle->parameterManager()->getParameter(FactSystem::defaultComponentId, loggerParam)->rawValue().toInt() == 0) {
             _downloadData->filename += ".px4log";
         } else {
             _downloadData->filename += ".ulg";