From f7eef285dd1ad5ea9485cee912ab90bde70bc34a Mon Sep 17 00:00:00 2001
From: Boernsman <bernhard.trinnes@nymea.io>
Date: Thu, 22 Oct 2020 11:25:54 +0200
Subject: [PATCH] improved discovery error repsonse

---
 aqi/integrationpluginaqi.cpp                               | 7 ++++++-
 .../57d69b76-4d2d-41ec-bef6-949a79ffbe6b-de.ts             | 5 +++++
 .../57d69b76-4d2d-41ec-bef6-949a79ffbe6b-en_US.ts          | 5 +++++
 3 files changed, 16 insertions(+), 1 deletion(-)

diff --git a/aqi/integrationpluginaqi.cpp b/aqi/integrationpluginaqi.cpp
index 04bfa517..a4283a3e 100644
--- a/aqi/integrationpluginaqi.cpp
+++ b/aqi/integrationpluginaqi.cpp
@@ -248,7 +248,12 @@ void IntegrationPluginAqi::onPluginTimer()
 
 void IntegrationPluginAqi::onRequestExecuted(QUuid requestId, bool success)
 {
-    qCDebug(dcAirQualityIndex()) << "Request executd, requestId:" << requestId << "Success:" << success << "is an async request:" << m_asyncRequests.contains(requestId);
+    qCDebug(dcAirQualityIndex()) << "Request executed, requestId:" << requestId << "Success:" << success << "is an async request:" << m_asyncRequests.contains(requestId);
+    if (m_asyncDiscovery.contains(requestId) && !success) {
+        ThingDiscoveryInfo *info = m_asyncDiscovery.take(requestId);
+        info->finish(Thing::ThingErrorHardwareNotAvailable, QT_TR_NOOP("Air quality index server not available, please check your internet connection."));
+    }
+
     if (m_asyncRequests.contains(requestId)) {
 
         Thing *thing = myThings().findById(m_asyncRequests.value(requestId));
diff --git a/aqi/translations/57d69b76-4d2d-41ec-bef6-949a79ffbe6b-de.ts b/aqi/translations/57d69b76-4d2d-41ec-bef6-949a79ffbe6b-de.ts
index 9e4cb02e..52781677 100644
--- a/aqi/translations/57d69b76-4d2d-41ec-bef6-949a79ffbe6b-de.ts
+++ b/aqi/translations/57d69b76-4d2d-41ec-bef6-949a79ffbe6b-de.ts
@@ -272,5 +272,10 @@ The name of the StateType ({c4366608-2511-428b-964e-2ad9e37f8f3c}) of ThingClass
         <source>Everyone should avoid all outdoor exertion</source>
         <translation>Jeder sollte jede Anstrengung im Freien vermeiden.</translation>
     </message>
+    <message>
+        <location filename="../integrationpluginaqi.cpp" line="254"/>
+        <source>Air quality index server not available, please check your internet connection.</source>
+        <translation>Air quality index Server nicht verfügbar, bitte überprüfe die Internetverbindung.</translation>
+    </message>
 </context>
 </TS>
diff --git a/aqi/translations/57d69b76-4d2d-41ec-bef6-949a79ffbe6b-en_US.ts b/aqi/translations/57d69b76-4d2d-41ec-bef6-949a79ffbe6b-en_US.ts
index 53ea90f7..97117c6f 100644
--- a/aqi/translations/57d69b76-4d2d-41ec-bef6-949a79ffbe6b-en_US.ts
+++ b/aqi/translations/57d69b76-4d2d-41ec-bef6-949a79ffbe6b-en_US.ts
@@ -272,5 +272,10 @@ The name of the StateType ({c4366608-2511-428b-964e-2ad9e37f8f3c}) of ThingClass
         <source>Everyone should avoid all outdoor exertion</source>
         <translation type="unfinished"></translation>
     </message>
+    <message>
+        <location filename="../integrationpluginaqi.cpp" line="254"/>
+        <source>Air quality index server not available, please check your internet connection.</source>
+        <translation type="unfinished"></translation>
+    </message>
 </context>
 </TS>