From 9a8377f93ab772d9aafb5ceae44896c599bcc371 Mon Sep 17 00:00:00 2001
From: Michael Zanetti <michael_zanetti@gmx.net>
Date: Tue, 20 Dec 2022 23:35:39 +0100
Subject: [PATCH] Mennekes: Reject wallboxes where the firmware is too old

---
 mennekes/integrationpluginmennekes.cpp          | 10 ++++++++++
 ...7c3c65c-a0cc-4ab1-90d8-4ad05bfcdc38-en_US.ts | 17 +++++++++++------
 2 files changed, 21 insertions(+), 6 deletions(-)

diff --git a/mennekes/integrationpluginmennekes.cpp b/mennekes/integrationpluginmennekes.cpp
index 3e21175..832cc9d 100644
--- a/mennekes/integrationpluginmennekes.cpp
+++ b/mennekes/integrationpluginmennekes.cpp
@@ -389,6 +389,16 @@ void IntegrationPluginMennekes::setupAmtronECUConnection(ThingSetupInfo *info)
         }
 
         qCDebug(dcMennekes()) << "Connection init finished successfully" << amtronECUConnection;
+
+        QString minimumVersion = "5.22";
+        if (!ensureAmtronECUVersion(amtronECUConnection, minimumVersion)) {
+            qCWarning(dcMennekes()) << "Firmware version too old:" << QByteArray::fromHex(QByteArray::number(amtronECUConnection->firmwareVersion(), 16)) << "Minimum required:" << minimumVersion;
+            hardwareManager()->networkDeviceDiscovery()->unregisterMonitor(monitor);
+            amtronECUConnection->deleteLater();
+            info->finish(Thing::ThingErrorHardwareFailure, QT_TR_NOOP("The firmware version of this wallbox is too old. Please upgrade the firmware to at least version 5.22."));
+            return;
+        }
+
         m_amtronECUConnections.insert(thing, amtronECUConnection);
         info->finish(Thing::ThingErrorNoError);
 
diff --git a/mennekes/translations/c7c3c65c-a0cc-4ab1-90d8-4ad05bfcdc38-en_US.ts b/mennekes/translations/c7c3c65c-a0cc-4ab1-90d8-4ad05bfcdc38-en_US.ts
index 1ae03de..3179782 100644
--- a/mennekes/translations/c7c3c65c-a0cc-4ab1-90d8-4ad05bfcdc38-en_US.ts
+++ b/mennekes/translations/c7c3c65c-a0cc-4ab1-90d8-4ad05bfcdc38-en_US.ts
@@ -9,23 +9,28 @@
         <translation type="unfinished"></translation>
     </message>
     <message>
-        <location filename="../integrationpluginmennekes.cpp" line="127"/>
-        <location filename="../integrationpluginmennekes.cpp" line="177"/>
+        <location filename="../integrationpluginmennekes.cpp" line="133"/>
+        <location filename="../integrationpluginmennekes.cpp" line="183"/>
         <source>The MAC address is not known. Please reconfigure the thing.</source>
         <translation type="unfinished"></translation>
     </message>
     <message>
-        <location filename="../integrationpluginmennekes.cpp" line="138"/>
-        <location filename="../integrationpluginmennekes.cpp" line="188"/>
+        <location filename="../integrationpluginmennekes.cpp" line="144"/>
+        <location filename="../integrationpluginmennekes.cpp" line="194"/>
         <source>The host address is not known yet. Trying later again.</source>
         <translation type="unfinished"></translation>
     </message>
     <message>
-        <location filename="../integrationpluginmennekes.cpp" line="372"/>
-        <location filename="../integrationpluginmennekes.cpp" line="497"/>
+        <location filename="../integrationpluginmennekes.cpp" line="387"/>
+        <location filename="../integrationpluginmennekes.cpp" line="515"/>
         <source>Error communicating with the wallbox.</source>
         <translation type="unfinished"></translation>
     </message>
+    <message>
+        <location filename="../integrationpluginmennekes.cpp" line="398"/>
+        <source>The firmware version of this wallbox is too old. Please upgrade the firmware to at least version 5.22.</source>
+        <translation type="unfinished"></translation>
+    </message>
 </context>
 <context>
     <name>Mennekes</name>