Remove unnecessary DOMAIN alias in components (#145791)

This commit is contained in:
epenet 2025-05-28 14:04:35 +02:00 committed by GitHub
parent f59001d45f
commit 6b28af8282
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
9 changed files with 26 additions and 28 deletions

View File

@ -15,7 +15,7 @@ from homeassistant.helpers.entity_platform import (
) )
from . import AgentDVRConfigEntry from . import AgentDVRConfigEntry
from .const import ATTRIBUTION, CAMERA_SCAN_INTERVAL_SECS, DOMAIN as AGENT_DOMAIN from .const import ATTRIBUTION, CAMERA_SCAN_INTERVAL_SECS, DOMAIN
SCAN_INTERVAL = timedelta(seconds=CAMERA_SCAN_INTERVAL_SECS) SCAN_INTERVAL = timedelta(seconds=CAMERA_SCAN_INTERVAL_SECS)
@ -82,7 +82,7 @@ class AgentCamera(MjpegCamera):
still_image_url=f"{device.client._server_url}{device.still_image_url}&size={device.mjpegStreamWidth}x{device.mjpegStreamHeight}", # noqa: SLF001 still_image_url=f"{device.client._server_url}{device.still_image_url}&size={device.mjpegStreamWidth}x{device.mjpegStreamHeight}", # noqa: SLF001
) )
self._attr_device_info = DeviceInfo( self._attr_device_info = DeviceInfo(
identifiers={(AGENT_DOMAIN, self.unique_id)}, identifiers={(DOMAIN, self.unique_id)},
manufacturer="Agent", manufacturer="Agent",
model="Camera", model="Camera",
name=f"{device.client.name} {device.name}", name=f"{device.client.name} {device.name}",

View File

@ -11,7 +11,7 @@ from homeassistant.helpers import device_registry as dr
from homeassistant.helpers.device_registry import CONNECTION_NETWORK_MAC, format_mac from homeassistant.helpers.device_registry import CONNECTION_NETWORK_MAC, format_mac
from homeassistant.helpers.dispatcher import async_dispatcher_send from homeassistant.helpers.dispatcher import async_dispatcher_send
from ..const import ATTR_MANUFACTURER, DOMAIN as AXIS_DOMAIN from ..const import ATTR_MANUFACTURER, DOMAIN
from .config import AxisConfig from .config import AxisConfig
from .entity_loader import AxisEntityLoader from .entity_loader import AxisEntityLoader
from .event_source import AxisEventSource from .event_source import AxisEventSource
@ -79,7 +79,7 @@ class AxisHub:
config_entry_id=self.config.entry.entry_id, config_entry_id=self.config.entry.entry_id,
configuration_url=self.api.config.url, configuration_url=self.api.config.url,
connections={(CONNECTION_NETWORK_MAC, self.unique_id)}, connections={(CONNECTION_NETWORK_MAC, self.unique_id)},
identifiers={(AXIS_DOMAIN, self.unique_id)}, identifiers={(DOMAIN, self.unique_id)},
manufacturer=ATTR_MANUFACTURER, manufacturer=ATTR_MANUFACTURER,
model=f"{self.config.model} {self.product_type}", model=f"{self.config.model} {self.product_type}",
name=self.config.name, name=self.config.name,

View File

@ -9,7 +9,7 @@ from homeassistant.const import ATTR_DEVICE_ID, CONF_EVENT, CONF_ID
from homeassistant.core import Event, HomeAssistant, callback from homeassistant.core import Event, HomeAssistant, callback
from homeassistant.helpers import device_registry as dr from homeassistant.helpers import device_registry as dr
from .const import CONF_GESTURE, DOMAIN as DECONZ_DOMAIN from .const import CONF_GESTURE, DOMAIN
from .deconz_event import CONF_DECONZ_ALARM_EVENT, CONF_DECONZ_EVENT from .deconz_event import CONF_DECONZ_ALARM_EVENT, CONF_DECONZ_EVENT
from .device_trigger import ( from .device_trigger import (
CONF_BOTH_BUTTONS, CONF_BOTH_BUTTONS,
@ -200,6 +200,6 @@ def async_describe_events(
} }
async_describe_event( async_describe_event(
DECONZ_DOMAIN, CONF_DECONZ_ALARM_EVENT, async_describe_deconz_alarm_event DOMAIN, CONF_DECONZ_ALARM_EVENT, async_describe_deconz_alarm_event
) )
async_describe_event(DECONZ_DOMAIN, CONF_DECONZ_EVENT, async_describe_deconz_event) async_describe_event(DOMAIN, CONF_DECONZ_EVENT, async_describe_deconz_event)

View File

@ -55,7 +55,7 @@ from homeassistant.helpers.issue_registry import (
) )
from homeassistant.helpers.typing import ConfigType, DiscoveryInfoType, StateType from homeassistant.helpers.typing import ConfigType, DiscoveryInfoType, StateType
from .const import CONF_IGNORE_NON_NUMERIC, DOMAIN as GROUP_DOMAIN from .const import CONF_IGNORE_NON_NUMERIC, DOMAIN
from .entity import GroupEntity from .entity import GroupEntity
DEFAULT_NAME = "Sensor Group" DEFAULT_NAME = "Sensor Group"
@ -509,7 +509,7 @@ class SensorGroup(GroupEntity, SensorEntity):
return state_classes[0] return state_classes[0]
async_create_issue( async_create_issue(
self.hass, self.hass,
GROUP_DOMAIN, DOMAIN,
f"{self.entity_id}_state_classes_not_matching", f"{self.entity_id}_state_classes_not_matching",
is_fixable=False, is_fixable=False,
is_persistent=False, is_persistent=False,
@ -566,7 +566,7 @@ class SensorGroup(GroupEntity, SensorEntity):
return device_classes[0] return device_classes[0]
async_create_issue( async_create_issue(
self.hass, self.hass,
GROUP_DOMAIN, DOMAIN,
f"{self.entity_id}_device_classes_not_matching", f"{self.entity_id}_device_classes_not_matching",
is_fixable=False, is_fixable=False,
is_persistent=False, is_persistent=False,
@ -654,7 +654,7 @@ class SensorGroup(GroupEntity, SensorEntity):
if device_class: if device_class:
async_create_issue( async_create_issue(
self.hass, self.hass,
GROUP_DOMAIN, DOMAIN,
f"{self.entity_id}_uoms_not_matching_device_class", f"{self.entity_id}_uoms_not_matching_device_class",
is_fixable=False, is_fixable=False,
is_persistent=False, is_persistent=False,
@ -670,7 +670,7 @@ class SensorGroup(GroupEntity, SensorEntity):
else: else:
async_create_issue( async_create_issue(
self.hass, self.hass,
GROUP_DOMAIN, DOMAIN,
f"{self.entity_id}_uoms_not_matching_no_device_class", f"{self.entity_id}_uoms_not_matching_no_device_class",
is_fixable=False, is_fixable=False,
is_persistent=False, is_persistent=False,

View File

@ -18,7 +18,7 @@ from homeassistant.helpers import config_validation as cv, entity_platform
from homeassistant.helpers.entity_platform import AddConfigEntryEntitiesCallback from homeassistant.helpers.entity_platform import AddConfigEntryEntitiesCallback
from . import NukiEntryData from . import NukiEntryData
from .const import ATTR_ENABLE, ATTR_UNLATCH, DOMAIN as NUKI_DOMAIN, ERROR_STATES from .const import ATTR_ENABLE, ATTR_UNLATCH, DOMAIN, ERROR_STATES
from .entity import NukiEntity from .entity import NukiEntity
from .helpers import CannotConnect from .helpers import CannotConnect
@ -29,7 +29,7 @@ async def async_setup_entry(
async_add_entities: AddConfigEntryEntitiesCallback, async_add_entities: AddConfigEntryEntitiesCallback,
) -> None: ) -> None:
"""Set up the Nuki lock platform.""" """Set up the Nuki lock platform."""
entry_data: NukiEntryData = hass.data[NUKI_DOMAIN][entry.entry_id] entry_data: NukiEntryData = hass.data[DOMAIN][entry.entry_id]
coordinator = entry_data.coordinator coordinator = entry_data.coordinator
entities: list[NukiDeviceEntity] = [ entities: list[NukiDeviceEntity] = [

View File

@ -26,14 +26,14 @@ from homeassistant.helpers.entity_platform import AddEntitiesCallback
from homeassistant.helpers.event import async_track_state_change_event from homeassistant.helpers.event import async_track_state_change_event
from homeassistant.helpers.typing import ConfigType, DiscoveryInfoType from homeassistant.helpers.typing import ConfigType, DiscoveryInfoType
from .const import DOMAIN as SWITCH_DOMAIN from .const import DOMAIN
DEFAULT_NAME = "Light Switch" DEFAULT_NAME = "Light Switch"
PLATFORM_SCHEMA = LIGHT_PLATFORM_SCHEMA.extend( PLATFORM_SCHEMA = LIGHT_PLATFORM_SCHEMA.extend(
{ {
vol.Optional(CONF_NAME, default=DEFAULT_NAME): cv.string, vol.Optional(CONF_NAME, default=DEFAULT_NAME): cv.string,
vol.Required(CONF_ENTITY_ID): cv.entity_domain(SWITCH_DOMAIN), vol.Required(CONF_ENTITY_ID): cv.entity_domain(DOMAIN),
} }
) )
@ -76,7 +76,7 @@ class LightSwitch(LightEntity):
async def async_turn_on(self, **kwargs: Any) -> None: async def async_turn_on(self, **kwargs: Any) -> None:
"""Forward the turn_on command to the switch in this light switch.""" """Forward the turn_on command to the switch in this light switch."""
await self.hass.services.async_call( await self.hass.services.async_call(
SWITCH_DOMAIN, DOMAIN,
SERVICE_TURN_ON, SERVICE_TURN_ON,
{ATTR_ENTITY_ID: self._switch_entity_id}, {ATTR_ENTITY_ID: self._switch_entity_id},
blocking=True, blocking=True,
@ -86,7 +86,7 @@ class LightSwitch(LightEntity):
async def async_turn_off(self, **kwargs: Any) -> None: async def async_turn_off(self, **kwargs: Any) -> None:
"""Forward the turn_off command to the switch in this light switch.""" """Forward the turn_off command to the switch in this light switch."""
await self.hass.services.async_call( await self.hass.services.async_call(
SWITCH_DOMAIN, DOMAIN,
SERVICE_TURN_OFF, SERVICE_TURN_OFF,
{ATTR_ENTITY_ID: self._switch_entity_id}, {ATTR_ENTITY_ID: self._switch_entity_id},
blocking=True, blocking=True,

View File

@ -19,7 +19,7 @@ from homeassistant.helpers.device_registry import DeviceInfo
from homeassistant.helpers.entity import Entity, ToggleEntity from homeassistant.helpers.entity import Entity, ToggleEntity
from homeassistant.helpers.event import async_track_state_change_event from homeassistant.helpers.event import async_track_state_change_event
from .const import DOMAIN as SWITCH_AS_X_DOMAIN from .const import DOMAIN
class BaseEntity(Entity): class BaseEntity(Entity):
@ -61,7 +61,7 @@ class BaseEntity(Entity):
self._switch_entity_id = switch_entity_id self._switch_entity_id = switch_entity_id
self._is_new_entity = ( self._is_new_entity = (
registry.async_get_entity_id(domain, SWITCH_AS_X_DOMAIN, unique_id) is None registry.async_get_entity_id(domain, DOMAIN, unique_id) is None
) )
@callback @callback
@ -102,7 +102,7 @@ class BaseEntity(Entity):
if registry.async_get(self.entity_id) is not None: if registry.async_get(self.entity_id) is not None:
registry.async_update_entity_options( registry.async_update_entity_options(
self.entity_id, self.entity_id,
SWITCH_AS_X_DOMAIN, DOMAIN,
self.async_generate_entity_options(), self.async_generate_entity_options(),
) )

View File

@ -16,7 +16,7 @@ from homeassistant.helpers.device_registry import (
) )
from homeassistant.helpers.dispatcher import async_dispatcher_send from homeassistant.helpers.dispatcher import async_dispatcher_send
from ..const import ATTR_MANUFACTURER, CONF_SITE_ID, DOMAIN as UNIFI_DOMAIN, PLATFORMS from ..const import ATTR_MANUFACTURER, CONF_SITE_ID, DOMAIN, PLATFORMS
from .config import UnifiConfig from .config import UnifiConfig
from .entity_helper import UnifiEntityHelper from .entity_helper import UnifiEntityHelper
from .entity_loader import UnifiEntityLoader from .entity_loader import UnifiEntityLoader
@ -104,7 +104,7 @@ class UnifiHub:
return DeviceInfo( return DeviceInfo(
entry_type=DeviceEntryType.SERVICE, entry_type=DeviceEntryType.SERVICE,
identifiers={(UNIFI_DOMAIN, self.config.entry.unique_id)}, identifiers={(DOMAIN, self.config.entry.unique_id)},
manufacturer=ATTR_MANUFACTURER, manufacturer=ATTR_MANUFACTURER,
model="UniFi Network Application", model="UniFi Network Application",
name="UniFi Network", name="UniFi Network",

View File

@ -52,7 +52,7 @@ from homeassistant.helpers.device_registry import DeviceEntryType, DeviceInfo
from homeassistant.helpers.entity_platform import AddConfigEntryEntitiesCallback from homeassistant.helpers.entity_platform import AddConfigEntryEntitiesCallback
from . import UnifiConfigEntry from . import UnifiConfigEntry
from .const import ATTR_MANUFACTURER, DOMAIN as UNIFI_DOMAIN from .const import ATTR_MANUFACTURER, DOMAIN
from .entity import ( from .entity import (
HandlerT, HandlerT,
SubscriptionT, SubscriptionT,
@ -367,14 +367,12 @@ def async_update_unique_id(hass: HomeAssistant, config_entry: UnifiConfigEntry)
def update_unique_id(obj_id: str, type_name: str) -> None: def update_unique_id(obj_id: str, type_name: str) -> None:
"""Rework unique ID.""" """Rework unique ID."""
new_unique_id = f"{type_name}-{obj_id}" new_unique_id = f"{type_name}-{obj_id}"
if ent_reg.async_get_entity_id(SWITCH_DOMAIN, UNIFI_DOMAIN, new_unique_id): if ent_reg.async_get_entity_id(SWITCH_DOMAIN, DOMAIN, new_unique_id):
return return
prefix, _, suffix = obj_id.partition("_") prefix, _, suffix = obj_id.partition("_")
unique_id = f"{prefix}-{type_name}-{suffix}" unique_id = f"{prefix}-{type_name}-{suffix}"
if entity_id := ent_reg.async_get_entity_id( if entity_id := ent_reg.async_get_entity_id(SWITCH_DOMAIN, DOMAIN, unique_id):
SWITCH_DOMAIN, UNIFI_DOMAIN, unique_id
):
ent_reg.async_update_entity(entity_id, new_unique_id=new_unique_id) ent_reg.async_update_entity(entity_id, new_unique_id=new_unique_id)
for obj_id in hub.api.outlets: for obj_id in hub.api.outlets: