Skip to content

Commit

Permalink
fix: Bind code optimisation/formatting (#23110)
Browse files Browse the repository at this point in the history
* Bind code optimization/formatting

* Feedback

* Fix coverage
  • Loading branch information
Nerivec committed Jun 23, 2024
1 parent 8877c9e commit 3d772b5
Show file tree
Hide file tree
Showing 4 changed files with 290 additions and 144 deletions.
4 changes: 2 additions & 2 deletions lib/extension/availability.ts
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ import debounce from 'debounce';
import bind from 'bind-decorator';
import * as zhc from 'zigbee-herdsman-converters';

const retrieveOnReconnect = [
const RETRIEVE_ON_RECONNECT: readonly {keys: string[], condition?: (state: KeyValue) => boolean}[] = [
{keys: ['state']},
{keys: ['brightness'], condition: (state: KeyValue): boolean => state.state === 'ON'},
{keys: ['color', 'color_temp'], condition: (state: KeyValue): boolean => state.state === 'ON'},
Expand Down Expand Up @@ -217,7 +217,7 @@ export default class Availability extends Extension {
logger.debug(`Retrieving state of '${device.name}' after reconnect`);

// Color and color temperature converters do both, only needs to be called once.
for (const item of retrieveOnReconnect) {
for (const item of RETRIEVE_ON_RECONNECT) {
if (item.condition && this.state.get(device) && !item.condition(this.state.get(device))) {
continue;
}
Expand Down
Loading

0 comments on commit 3d772b5

Please sign in to comment.