From e0dd5ca3557070328eb1cf16fb8a8a021f562aa3 Mon Sep 17 00:00:00 2001
From: Robert Griebl <robert@griebl.org>
Date: Sun, 7 Sep 2025 18:16:48 +0200
Subject: [PATCH] Cleanup online-state handling

---
 src/common/config.cpp      | 16 ----------------
 src/common/config.h        |  3 ---
 src/common/onlinestate.cpp |  3 ++-
 3 files changed, 2 insertions(+), 20 deletions(-)

diff --git a/src/common/config.cpp b/src/common/config.cpp
index 3f6aad4d..adc6b11b 100644
--- a/src/common/config.cpp
+++ b/src/common/config.cpp
@@ -616,22 +616,6 @@ void Config::setBrickLinkPassword(const QString &pass, bool doNotSave)
     }
 }
 
-bool Config::onlineStatus() const
-{
-    return value(u"Internet/Online"_qs, true).toBool();
-}
-
-void Config::setOnlineStatus(bool b)
-{
-    bool ob = onlineStatus();
-
-    if (b != ob) {
-        setValue(u"Internet/Online"_qs, b);
-
-        emit onlineStatusChanged(b);
-    }
-}
-
 Config::PartOutMode Config::partOutMode() const
 {
     int v = value(u"General/PartOutMode"_qs).toInt();
diff --git a/src/common/config.h b/src/common/config.h
index f3ccb575..94e76602 100755
--- a/src/common/config.h
+++ b/src/common/config.h
@@ -76,8 +76,6 @@ public:
     void setShowInputErrors(bool b);
     bool showDifferenceIndicators() const;
     void setShowDifferenceIndicators(bool b);
-    bool onlineStatus() const;
-    void setOnlineStatus(bool b);
 
     enum class PartOutMode {
         Ask,
@@ -184,7 +182,6 @@ signals:
     void showDifferenceIndicatorsChanged(bool b);
     void visualChangesMarkModifiedChanged(bool b);
     void updateIntervalsChanged(const QMap<QByteArray, int> &intervals);
-    void onlineStatusChanged(bool b);
     void toolBarSizeChanged(Config::UISize iconSize);
     void iconSizePercentChanged(int p);
     void fontSizePercentChanged(int p);
diff --git a/src/common/onlinestate.cpp b/src/common/onlinestate.cpp
index 8f7e2784..befca08c 100755
--- a/src/common/onlinestate.cpp
+++ b/src/common/onlinestate.cpp
@@ -34,7 +34,8 @@ OnlineState::OnlineState(QObject *parent)
 
         connect(qni, &QNetworkInformation::reachabilityChanged,
                 this, [this](QNetworkInformation::Reachability r) {
-            auto online = (r == QNetworkInformation::Reachability::Online);
+            auto online = (r == QNetworkInformation::Reachability::Online)
+                || (r == QNetworkInformation::Reachability::Unknown);
             if (online != m_online) {
                 m_online = online;
                 emit onlineStateChanged(m_online);