From 7529b9252c68aed73ae658853b13dfd85c976f07 Mon Sep 17 00:00:00 2001 From: epenet <6771947+epenet@users.noreply.github.com> Date: Fri, 3 Apr 2026 10:00:55 +0200 Subject: [PATCH] Remove unnecessary None checks in Renault numbers and binary sensors (#167271) --- homeassistant/components/renault/binary_sensor.py | 11 ++++------- homeassistant/components/renault/number.py | 6 ++---- 2 files changed, 6 insertions(+), 11 deletions(-) diff --git a/homeassistant/components/renault/binary_sensor.py b/homeassistant/components/renault/binary_sensor.py index 5e4f08e9d5c7..702e23d1489e 100644 --- a/homeassistant/components/renault/binary_sensor.py +++ b/homeassistant/components/renault/binary_sensor.py @@ -84,15 +84,12 @@ class RenaultBinarySensor( def _plugged_in_value_lambda(self: RenaultBinarySensor) -> bool | None: """Return true if the vehicle is plugged in.""" - - data = self.coordinator.data - plug_status = data.get_plug_status() if data else None - - if plug_status is not None: + if (plug_status := self.coordinator.data.get_plug_status()) is not None: return plug_status == PlugState.PLUGGED - charging_status = data.get_charging_status() if data else None - if charging_status is not None and charging_status in _PLUG_FROM_CHARGE_STATUS: + if ( + charging_status := self.coordinator.data.get_charging_status() + ) is not None and charging_status in _PLUG_FROM_CHARGE_STATUS: return True return None diff --git a/homeassistant/components/renault/number.py b/homeassistant/components/renault/number.py index 555bb9b9e72b..b487eedd3dd7 100644 --- a/homeassistant/components/renault/number.py +++ b/homeassistant/components/renault/number.py @@ -43,9 +43,7 @@ async def _set_charge_limit_min(entity: RenaultNumberEntity, value: float) -> No The target SOC is required to set the minimum SOC, so we need to fetch it first. """ - if (data := entity.coordinator.data) is None or ( - target_soc := data.socTarget - ) is None: + if (target_soc := entity.coordinator.data.socTarget) is None: raise ServiceValidationError( translation_domain=DOMAIN, translation_key="battery_soc_unavailable", @@ -58,7 +56,7 @@ async def _set_charge_limit_target(entity: RenaultNumberEntity, value: float) -> The minimum SOC is required to set the target SOC, so we need to fetch it first. """ - if (data := entity.coordinator.data) is None or (min_soc := data.socMin) is None: + if (min_soc := entity.coordinator.data.socMin) is None: raise ServiceValidationError( translation_domain=DOMAIN, translation_key="battery_soc_unavailable",