Merge pull request #8879 from DonLakeFlyer/ManifestJson

Fix manifest load bug

Merge pull request #8879 from DonLakeFlyer/ManifestJson
Fix manifest load bug
c72b9e2e · Don Gagne · GitHub · 14796b39 · f960ce23 · c72b9e2e
Unverified Commit c72b9e2e authored Jul 03, 2020 by Don Gagne Committed by GitHub Jul 03, 2020
--- a/src/VehicleSetup/FirmwareUpgradeController.cc
+++ b/src/VehicleSetup/FirmwareUpgradeController.cc
@@ -901,7 +901,7 @@ void FirmwareUpgradeController::_ardupilotManifestDownloadFinished(QString remot
    QString         errorString;
    QJsonDocument   doc;
-    if (JsonHelper::isJsonFile(jsonFileName, doc, errorString)) {
+    if (!JsonHelper::isJsonFile(jsonFileName, doc, errorString)) {
        qCWarning(FirmwareUpgradeLog) << "Json file read failed" << errorString;
        return;
    }