Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove log warnings #218

Merged
merged 1 commit into from
Oct 25, 2020
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
28 changes: 0 additions & 28 deletions bimmer_connected/state.py
Original file line number Diff line number Diff line change
Expand Up @@ -132,7 +132,6 @@ def _parse_datetime(date_str: str) -> datetime.datetime:
@backend_parameter
def timestamp(self) -> datetime.datetime:
"""Get the timestamp when the data was recorded."""
_LOGGER.warning("Funcion deprecated use state.vehicle_status.timestamp instead")
return self.vehicle_status.timestamp

@property
Expand All @@ -143,7 +142,6 @@ def gps_position(self) -> (float, float):
Returns a tuple of (latitude, longitude).
This only provides data, if the vehicle tracking is enabled!
"""
_LOGGER.warning("Funcion deprecated use state.vehicle_status.gps_position instead")
return self.vehicle_status.gps_position

@property
Expand All @@ -153,7 +151,6 @@ def is_vehicle_tracking_enabled(self) -> bool:

The server return "OK" if tracking is enabled and "DRIVER_DISABLED" if it is disabled in the vehicle.
"""
_LOGGER.warning("Funcion deprecated use state.vehicle_status.is_vehicle_tracking_enabled instead")
return bool(self.vehicle_status.is_vehicle_tracking_enabled)

@property
Expand All @@ -163,7 +160,6 @@ def mileage(self) -> int:

Returns a tuple of (value, unit_of_measurement)
"""
_LOGGER.warning("Funcion deprecated use state.vehicle_status.mileage instead")
return self.vehicle_status.mileage

@property
Expand All @@ -173,7 +169,6 @@ def remaining_range_fuel(self) -> int:

Returns a tuple of (value, unit_of_measurement)
"""
_LOGGER.warning("Funcion deprecated use state.vehicle_status.remaining_range_fuel instead")
return self.vehicle_status.remaining_range_fuel

@property
Expand All @@ -183,91 +178,78 @@ def remaining_fuel(self) -> int:

Returns a tuple of (value, unit_of_measurement)
"""
_LOGGER.warning("Funcion deprecated use state.vehicle_status.remaining_fuel instead")
return self.vehicle_status.remaining_fuel

@property
@backend_parameter
def lids(self) -> List[Lid]:
"""Get all lids (doors+hatch+trunk) of the car."""
_LOGGER.warning("Funcion deprecated use state.vehicle_status.lids instead")
return self.vehicle_status.lids

@property
@backend_parameter
def open_lids(self) -> List[Lid]:
"""Get all open lids of the car."""
_LOGGER.warning("Funcion deprecated use state.vehicle_status.open_lids instead")
return self.vehicle_status.open_lids

@property
@backend_parameter
def all_lids_closed(self) -> bool:
"""Check if all lids are closed."""
_LOGGER.warning("Funcion deprecated use state.vehicle_status.all_lids_closed instead")
return self.vehicle_status.all_lids_closed

@property
@backend_parameter
def windows(self) -> List[Window]:
"""Get all windows (doors+sun roof) of the car."""
_LOGGER.warning("Funcion deprecated use state.vehicle_status.windows instead")
return self.vehicle_status.windows

@property
@backend_parameter
def open_windows(self) -> List[Window]:
"""Get all open windows of the car."""
_LOGGER.warning("Funcion deprecated use state.vehicle_status.open_windows instead")
return self.vehicle_status.open_windows

@property
@backend_parameter
def all_windows_closed(self) -> bool:
"""Check if all windows are closed."""
_LOGGER.warning("Funcion deprecated use state.vehicle_status.all_windows_closed instead")
return self.vehicle_status.all_windows_closed

@property
@backend_parameter
def door_lock_state(self) -> LockState:
"""Get state of the door locks."""
_LOGGER.warning("Funcion deprecated use state.vehicle_status.all_windows_closed instead")
return self.vehicle_status.door_lock_state

@property
@backend_parameter
def last_update_reason(self) -> str:
"""The reason for the last state update"""
_LOGGER.warning("Funcion deprecated use state.vehicle_status.last_update_reason instead")
return self.vehicle_status.last_update_reason

@property
@backend_parameter
def last_charging_end_result(self) -> str:
"""Get the last charging end result"""
_LOGGER.warning("Funcion deprecated use state.vehicle_status.last_charging_end_result instead")
return self.vehicle_status.last_charging_end_result

@property
@backend_parameter
def connection_status(self) -> str:
"""Get status of the connection"""
_LOGGER.warning("Funcion deprecated use state.vehicle_status.connection_status instead")
return self.vehicle_status.connection_status

@property
@backend_parameter
def condition_based_services(self) -> List[ConditionBasedServiceReport]:
"""Get status of the condition based services."""
_LOGGER.warning("Funcion deprecated use state.vehicle_status.condition_based_services instead")
return self.vehicle_status.condition_based_services

@property
@backend_parameter
def are_all_cbs_ok(self) -> bool:
"""Check if the status of all condition based services is "OK"."""
_LOGGER.warning("Funcion deprecated use state.vehicle_status.are_all_cbs_ok instead")
return bool(self.vehicle_status.are_all_cbs_ok)

@property
Expand All @@ -277,7 +259,6 @@ def parking_lights(self) -> ParkingLightState:

:returns None if status is unknown.
"""
_LOGGER.warning("Funcion deprecated use state.vehicle_status.parking_lights instead")
return self.vehicle_status.parking_lights

@property
Expand All @@ -287,14 +268,12 @@ def are_parking_lights_on(self) -> bool:

:returns None if status is unknown.
"""
_LOGGER.warning("Funcion deprecated use state.vehicle_status.are_parking_lights_on instead")
return bool(self.vehicle_status.are_parking_lights_on)

@property
@backend_parameter
def remaining_range_electric(self) -> int:
"""Remaining range on battery, in kilometers."""
_LOGGER.warning("Funcion deprecated use state.vehicle_status.remaining_range_electric instead")
return self.vehicle_status.remaining_range_electric

@property
Expand All @@ -304,47 +283,40 @@ def remaining_range_total(self) -> int:

That is electrical range + fuel range.
"""
_LOGGER.warning("Funcion deprecated use state.vehicle_status.remaining_range_total instead")
return self.vehicle_status.remaining_range_total

@property
@backend_parameter
def max_range_electric(self) -> int:
""" This can change with driving style and temperature in kilometers."""
_LOGGER.warning("Funcion deprecated use state.vehicle_status.max_range_electric instead")
return self.vehicle_status.max_range_electric

@property
@backend_parameter
def charging_status(self) -> ChargingState:
"""Charging state of the vehicle."""
_LOGGER.warning("Funcion deprecated use state.vehicle_status.charging_status instead")
return self.vehicle_status.charging_status

@property
@backend_parameter
def charging_time_remaining(self) -> datetime.timedelta:
"""Get the remaining charging time."""
_LOGGER.warning("Funcion deprecated use state.vehicle_status.charging_time_remaining instead")
return self.vehicle_status.charging_time_remaining

@property
@backend_parameter
def charging_level_hv(self) -> int:
"""State of charge of the high voltage battery in percent."""
_LOGGER.warning("Funcion deprecated use state.vehicle_status.charging_level_hv instead")
return self.vehicle_status.charging_level_hv

@property
@backend_parameter
def check_control_messages(self) -> List[CheckControlMessage]:
"""List of check control messages."""
_LOGGER.warning("Funcion deprecated use state.vehicle_status.check_control_messages instead")
return self.vehicle_status.check_control_messages

@property
@backend_parameter
def has_check_control_messages(self) -> bool:
"""Return true if any check control message is present."""
_LOGGER.warning("Funcion deprecated use state.vehicle_status.has_check_control_messages instead")
return bool(self.vehicle_status.has_check_control_messages)