christian/openhab-automation-js
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor(TimerMgr): streamline timer identification and logging

Browse Source
This commit is contained in:
Christian Weimann
2025-01-10 06:08:04 +01:00
parent 33e5781ee8
commit 47d6b80511
1 changed files with 9 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
utils/TimerMgr.js
View File

@@ -18,23 +18,23 @@ class TimerMgr {
this.#timers[id] = actions.ScriptExecution.createTimer(id, timeout, func); this.#timers[id] = actions.ScriptExecution.createTimer(id, timeout, func);
} }
cancel(identifier) { cancel(id) {
// Return if no timer with the respactive identifier is available // Return if no timer with the respactive id is available
if (!this.#timers.hasOwnProperty(identifier)) { if (!this.hasTimer(id)) {
log.debug(`No timer with identifier ${identifier} available to cancel`); log.warn(`No timer with id ${id} available to cancel`);
return false; return false;
} }
// Check if timer is active // Check if timer is active
if (!this.#timers[identifier].isActive()) { if (!this.#timers[id].isActive()) {
log.debug(`Timer with identifier ${identifier} not running. Cancel anyway`); log.debug(`Timer with id ${id} not running. Cancel anyway`);
} else { } else {
log.debug(`Cancel timer with identifier ${identifier}`); log.debug(`Cancel timer with identifier ${id}`);
} }
// Cancel timer // Cancel timer
this.#timers[identifier].cancel(); this.#timers[id].cancel();
delete this.#timers[identifier]; delete this.#timers[id];
} }
hasTimer(id) { hasTimer(id) {