Skip to content

Commit

Permalink
Explicitly pass in the config_entry in israel_rail coordinator (#138132)
Browse files Browse the repository at this point in the history
explicitly pass in the config_entry in coordinator
  • Loading branch information
mib1185 authored Feb 9, 2025
1 parent 52363d5 commit 733d9de
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 9 deletions.
8 changes: 2 additions & 6 deletions homeassistant/components/israel_rail/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,23 +4,19 @@

from israelrailapi import TrainSchedule

from homeassistant.config_entries import ConfigEntry
from homeassistant.const import Platform
from homeassistant.core import HomeAssistant
from homeassistant.exceptions import ConfigEntryNotReady

from .const import CONF_DESTINATION, CONF_START, DOMAIN
from .coordinator import IsraelRailDataUpdateCoordinator
from .coordinator import IsraelRailConfigEntry, IsraelRailDataUpdateCoordinator

_LOGGER = logging.getLogger(__name__)


PLATFORMS: list[Platform] = [Platform.SENSOR]


type IsraelRailConfigEntry = ConfigEntry[IsraelRailDataUpdateCoordinator]


async def async_setup_entry(hass: HomeAssistant, entry: IsraelRailConfigEntry) -> bool:
"""Set up Israel rail from a config entry."""
config = entry.data
Expand All @@ -43,7 +39,7 @@ async def async_setup_entry(hass: HomeAssistant, entry: IsraelRailConfigEntry) -
) from e

israel_rail_coordinator = IsraelRailDataUpdateCoordinator(
hass, train_schedule, start, destination
hass, entry, train_schedule, start, destination
)
await israel_rail_coordinator.async_config_entry_first_refresh()
entry.runtime_data = israel_rail_coordinator
Expand Down
7 changes: 6 additions & 1 deletion homeassistant/components/israel_rail/coordinator.py
Original file line number Diff line number Diff line change
Expand Up @@ -38,14 +38,18 @@ def departure_time(train_route: TrainRoute) -> datetime | None:
return start_datetime.astimezone() if start_datetime else None


type IsraelRailConfigEntry = ConfigEntry[IsraelRailDataUpdateCoordinator]


class IsraelRailDataUpdateCoordinator(DataUpdateCoordinator[list[DataConnection]]):
"""A IsraelRail Data Update Coordinator."""

config_entry: ConfigEntry
config_entry: IsraelRailConfigEntry

def __init__(
self,
hass: HomeAssistant,
config_entry: IsraelRailConfigEntry,
train_schedule: TrainSchedule,
start: str,
destination: str,
Expand All @@ -54,6 +58,7 @@ def __init__(
super().__init__(
hass,
_LOGGER,
config_entry=config_entry,
name=DOMAIN,
update_interval=DEFAULT_SCAN_INTERVAL,
)
Expand Down
7 changes: 5 additions & 2 deletions homeassistant/components/israel_rail/sensor.py
Original file line number Diff line number Diff line change
Expand Up @@ -19,9 +19,12 @@
from homeassistant.helpers.typing import StateType
from homeassistant.helpers.update_coordinator import CoordinatorEntity

from . import IsraelRailConfigEntry
from .const import ATTRIBUTION, DEPARTURES_COUNT, DOMAIN
from .coordinator import DataConnection, IsraelRailDataUpdateCoordinator
from .coordinator import (
DataConnection,
IsraelRailConfigEntry,
IsraelRailDataUpdateCoordinator,
)

_LOGGER = logging.getLogger(__name__)

Expand Down

0 comments on commit 733d9de

Please sign in to comment.