Merge master into Beta (#2049)

* unlock chargepoint with vehicle id (MAC) (#1889) * Update version 2.1.5-Patch.2 * build UI (#2009) * Update for Polestar auth change:client id and optional acceptance of terms and conditions * add terser * Added const for client_id * Added blank line for Flake check * fix heartbeat internal chargepoint (#2013) * Build Display Theme: Cards * Wiki (#2014) * Wiki * typos Wiki * clear browser console at midnight * Build Display Theme: Cards * fix SolarEdge synergy units (#2026) * reset boot_done before shutdown (#2027) * fix solaredge synergy units (#2030) * build * Satellit: Fix telnet (#2032) * Update version 2.1.6-RC.2 * fix soc error and scheduled charging (#2038) * Update version 2.1.6 * fix disable after unplugging (#2043) * fix disable after unplugging * Wiki * fix * Update version 2.1.6-Patch.1 --------- Co-authored-by: PK <pk.hh8@online.de> Co-authored-by: Lutz Bender <github@lutz-bender.de> Co-authored-by: benderl <benderl@users.noreply.github.com>
openWB · Dec 3, 2024 · 645aba2 · 645aba2
1 parent a589276
commit 645aba2
Show file tree

Hide file tree

Showing 5 changed files with 19 additions and 15 deletions.
diff --git a/docs/Identifikation.md b/docs/Identifikation.md
@@ -1,4 +1,4 @@
-Die openWB bietet die Möglichkeit, den Ladepunkt vor dem Laden zu entsperren und/oder das Fahrzeug zuzuordnen, welches geladen wird. Es gibt zwei grundlegende Konzepte, die für sich oder in Kombination genutzt werden können: Das Entsperren eines Ladepunkts und das Zuordnen eines Fahrzeugs.
+Mit den verschiedenen Identifikations-Möglichkeiten kannst du die openWB grundsätzlich vor unbefugtem Laden schützen oder fahrzeugbasierte Funktionen nutzen. Es gibt zwei grundlegende Konzepte: Das Entsperren eines Ladepunkts und das Zuordnen eines Fahrzeugs.
 
 Die Identifikation erfolgt über
 

diff --git a/packages/control/chargepoint/chargepoint.py b/packages/control/chargepoint/chargepoint.py
@@ -164,16 +164,21 @@ def _is_autolock_inactive(self) -> Tuple[bool, Optional[str]]:
         return state, message
 
     def _is_manual_lock_inactive(self) -> Tuple[bool, Optional[str]]:
-        if (self.data.set.manual_lock is False or
-                (self.data.get.rfid or
-                 self.data.get.vehicle_id or
-                 self.data.set.rfid) in self.template.data.valid_tags):
+        # Die Pro schickt je nach Timing auch nach Abstecken noch ein paar Zyklen den Tag. Dann darf der Ladepunkt
+        # nicht wieder entsperrt werden.
+        if (self.data.get.rfid or
+            self.data.get.vehicle_id or
+                self.data.set.rfid) in self.template.data.valid_tags:
             Pub().pub(f"openWB/set/chargepoint/{self.num}/set/manual_lock", False)
-            charging_possible = True
-            message = None
-        else:
+        elif self.template.data.disable_after_unplug and self.data.get.plug_state is False:
+            Pub().pub(f"openWB/set/chargepoint/{self.num}/set/manual_lock", True)
+
+        if self.data.set.manual_lock:
             charging_possible = False
             message = "Keine Ladung, da der Ladepunkt gesperrt ist."
+        else:
+            charging_possible = True
+            message = None
         return charging_possible, message
 
     def _is_ev_plugged(self) -> Tuple[bool, Optional[str]]:
@@ -231,6 +236,7 @@ def _process_charge_stop(self) -> None:
                 if self.template.data.disable_after_unplug:
                     self.data.set.manual_lock = True
                     Pub().pub("openWB/set/chargepoint/"+str(self.num)+"/set/manual_lock", True)
+                    log.debug("/set/manual_lock True")
                 # Ev wurde noch nicht aktualisiert.
                 chargelog.save_and_reset_data(self, data.data.ev_data["ev"+str(self.data.set.charging_ev_prev)])
                 self.data.set.charging_ev_prev = -1

diff --git a/packages/control/ev.py b/packages/control/ev.py
@@ -833,7 +833,7 @@ def calculate_duration(self,
                            charging_type: str,
                            ev_template: EvTemplate) -> Tuple[float, float]:
         if plan.limit.selected == "soc":
-            if soc:
+            if soc is not None:
                 missing_amount = ((plan.limit.soc_scheduled - soc) / 100) * battery_capacity
             else:
                 raise ValueError("Um Zielladen mit SoC-Ziel nutzen zu können, bitte ein SoC-Modul konfigurieren.")

diff --git a/packages/helpermodules/setdata.py b/packages/helpermodules/setdata.py
@@ -623,20 +623,18 @@ def process_chargepoint_get_topics(self, msg):
             self._validate_value(msg, int, [(0, 2)])
         elif "/get/evse_current" in msg.topic:
             self._validate_value(msg, float, [(0, 0), (6, 32), (600, 3200)])
+        elif ("/get/error_timestamp" in msg.topic or
+                "/get/rfid_timestamp" in msg.topic):
+            self._validate_value(msg, float)
         elif ("/get/fault_str" in msg.topic or
                 "/get/state_str" in msg.topic or
                 "/get/heartbeat" in msg.topic or
                 "/get/rfid" in msg.topic or
                 "/get/vehicle_id" in msg.topic or
                 "/get/serial_number" in msg.topic):
             self._validate_value(msg, str)
-        elif ("/get/error_timestamp" in msg.topic or
-                "/get/rfid_timestamp" in msg.topic):
-            self._validate_value(msg, float)
         elif ("/get/soc" in msg.topic):
             self._validate_value(msg, float, [(0, 100)])
-        elif "/get/rfid_timestamp" in msg.topic:
-            self._validate_value(msg, float)
         elif "/get/simulation" in msg.topic:
             self._validate_value(msg, "json")
         else:

diff --git a/web/version b/web/version
@@ -1 +1 @@
-2.1.6-RC.2
+2.1.6-Patch.1