From 48130243ed99fcfdad24379c5d0d076909bb859c Mon Sep 17 00:00:00 2001
From: sawinglogz <3787776-sawinglogz@users.noreply.gitlab.com>
Date: Wed, 3 Nov 2021 16:06:54 -0400
Subject: [PATCH] Fix minor PRS1 regression introduced in 8fd2840.

Complete failures were incorrectly being treated as fully up to date.
---
 oscar/SleepLib/loader_plugins/prs1_loader.cpp | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/oscar/SleepLib/loader_plugins/prs1_loader.cpp b/oscar/SleepLib/loader_plugins/prs1_loader.cpp
index 830d9b84..08ccb5d4 100644
--- a/oscar/SleepLib/loader_plugins/prs1_loader.cpp
+++ b/oscar/SleepLib/loader_plugins/prs1_loader.cpp
@@ -779,17 +779,24 @@ int PRS1Loader::Open(const QString & selectedPath)
     // TODO: Loaders should return the set of machines during detection, so that Open() will
     // open a unique device, instead of surprising the user.
     int c = 0;
+    bool failures = false;
     for (auto & machinePath : machines) {
         if (m_ctx == nullptr) {
             qWarning() << "PRS1Loader::Open() called without a valid m_ctx object present";
             return 0;
         }
         int imported = OpenMachine(machinePath);
-        if (imported > 0) {  // don't let errors < 0 suppress subsequent successes
+        if (imported >= 0) {  // don't let errors < 0 suppress subsequent successes
             c += imported;
+        } else {
+            failures = true;
         }
         m_ctx->FlushUnexpectedMessages();
     }
+    if (c == 0 && failures) {
+        // report an error when there were failures and no successess
+        c = -1;
+    }
     return c;
 }