fixed bug in configprocessing and removed deprecated function

hoylabs · Sep 26, 2024 · 88852eb · 88852eb
1 parent cc601d2
commit 88852eb
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 19 deletions.
diff --git a/include/defaults.h b/include/defaults.h
@@ -160,8 +160,8 @@
 #define BATTERY_ZENDURE_MAX_SOC 100
 #define BATTERY_ZENDURE_BYPASS_MODE 0
 #define BATTERY_ZENDURE_MAX_OUTPUT 800
-#define BATTERY_ZENDURE_AUTO_SHUTDOWN 1
-#define BATTERY_ZENDURE_FORCE_LIMIT 0
+#define BATTERY_ZENDURE_AUTO_SHUTDOWN true
+#define BATTERY_ZENDURE_FORCE_LIMIT false
 #define BATTERY_ZENDURE_OUTPUT_LIMIT BATTERY_ZENDURE_MAX_OUTPUT
 
 #define HUAWEI_ENABLED false

diff --git a/src/Utils.cpp b/src/Utils.cpp
@@ -238,13 +238,8 @@ template std::optional<float> Utils::getNumericValueFromMqttPayload(char const*
 
 template<typename T>
 std::optional<T> Utils::getJsonElement(JsonObjectConst const root, char const* key, size_t nesting /* = 0*/) {
-    if (root.containsKey(key)){
-
-        auto item = root[key].as<JsonVariantConst>();
-
-        if (item.is<T>() && item.nesting() == nesting){
-            return item.as<T>();
-        }
+    if (!root[key].isNull() && root[key].is<T>() && root[key].nesting() == nesting){
+        return root[key].as<T>();
     }
     return std::nullopt;
 }

diff --git a/webapp/src/views/BatteryAdminView.vue b/webapp/src/views/BatteryAdminView.vue
@@ -252,16 +252,17 @@
                         :tooltip="$t('batteryadmin.ZendureForceLimitDescription')"
                     />
 
-                    <InputElement
-                        v-if="batteryConfigList.zendure_force_limit"
-                        :label="$t('batteryadmin.ZendureOutputLimit')"
-                        v-model="batteryConfigList.zendure_output_limit"
-                        type="number"
-                        min="0"
-                        max="1200"
-                        step="1"
-                        :postfix="$t('batteryadmin.Watt')"
-                    />
+                    <template v-if="batteryConfigList.zendure_force_limit">
+                        <InputElement
+                            :label="$t('batteryadmin.ZendureOutputLimit')"
+                            v-model="batteryConfigList.zendure_output_limit"
+                            type="number"
+                            min="0"
+                            max="1200"
+                            step="1"
+                            :postfix="$t('batteryadmin.Watt')"
+                        />
+                    </template>
 
                     <InputElement
                         :label="$t('batteryadmin.ZendureMinSoc')"