jeans/frontend
1
0
Fork 0
mirror of https://github.com/home-assistant/frontend.git synced 2025-09-21 02:49:44 +00:00
Code Issues Packages Projects Releases Wiki Activity

make sure we can do bulk ops for non-unique id entities

Browse Source
This commit is contained in:
J. Nick Koston
2025-09-12 09:45:26 -05:00
parent 1e573bb358
commit e46e502db3
1 changed files with 15 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

23
src/panels/config/entities/ha-config-entities.ts
View File

@@ -590,6 +590,18 @@ export class HaConfigEntities extends SubscribeMixin(LitElement) {
})
);
private _hasNonUniqueIdEntities = memoize(
(selected: string[], filteredEntities: EntityRow[]) => {
// Create a Set of readonly entity IDs for O(1) lookup
const readonlyEntityIds = new Set(
filteredEntities
.filter((e) => e.readonly === true)
.map((e) => e.entity_id)
);
return selected.some((entityId) => readonlyEntityIds.has(entityId));
}
);
private _filteredEntitiesAndDomains = memoize(
(
localize: LocalizeFunc,
@@ -884,15 +896,10 @@ export class HaConfigEntities extends SubscribeMixin(LitElement) {
[...filteredDomains][0]
);
// Check if any selected entities are without unique IDs
// Create a Set of readonly entity IDs for O(1) lookup
const readonlyEntityIds = new Set(
// Check if any selected entities are without unique IDs (memoized for performance)
const hasNonUniqueIdEntities = this._hasNonUniqueIdEntities(
this._selected,
filteredEntities
.filter((e) => e.readonly === true)
.map((e) => e.entity_id)
);
const hasNonUniqueIdEntities = this._selected.some((entityId) =>
readonlyEntityIds.has(entityId)
);
// Helper to render menu items that can be disabled for non-unique ID entities