refactor(SceneController): rename SceneMgr to SceneController

SceneController.js

@@ -1,7 +1,7 @@
 // Settings
 const ITEMPREFIX = "System_StatefulScene";
 const DEBOUNCETIME = 3000;
-const LOGGER = "org.openhab.js.SceneMgr";
+const LOGGER = "org.openhab.js.SceneController";
 
 // Logging
 const log = Java.type('org.slf4j.LoggerFactory').getLogger(LOGGER);
@@ -13,19 +13,14 @@ const tm = new TimerMgr();
 const { ruleRegistry } = require('@runtime/RuleSupport');
 
 // Functions
-class SceneMgr {
+class SceneController {
 
     constructor() {
-        // Log the initialization message for Scene Manager.
+        // Log the initialization message for SceneController.
-        log.info('Initialization of SceneMgr');
+        log.info('Initialization of SceneController');
 
         // Initialize the scenes map by reducing the array of scenes into an object with unique IDs as keys.
-        this.scenes = this.initializeScenes();
+        this.scenes = this.getScenes().reduce((scenes, scene) => {
-    }
-
-    // Helper method to initialize scenes using reduce.
-    initializeScenes() {
-        return this.getScenes().reduce((scenes, scene) => {
             const sceneUID = scene.getUID();
             scenes[sceneUID] = new Scene(scene);
             return scenes;
@@ -264,11 +259,11 @@ class Scene {
 }
 
 // Load script
-const sm = new SceneMgr();
+const sm = new SceneController();
 
 // Unload script
 require('@runtime').lifecycleTracker.addDisposeHook(() => {
-    log.info('Deinitialization of SceneMgr');
+    log.info('Deinitialization of SceneController');
 
     sm.purgeUnusedSceneSwitchItems();
 

utils/TimerMgr.js

@@ -11,8 +11,7 @@ class TimerMgr {
 
     create(id, timeout, func, ...params) {
         if (this.hasTimer(id)) {
-            log.error(`Timer with id ${id} already exists`);
+            this.cancel(id);
-            return;
         }
         log.debug("Create timer with id " + id);
         this.#timers[id] = actions.ScriptExecution.createTimer(id, timeout, func, ...params);