fixed bug where logger stopped logging, because of calling cbpi.run() twice

modules/core/basetypes.py

@@ -65,6 +65,7 @@ class Sensor(Base):
 
     def update_value(self, value):
         self.value = value
+        self.__logger.info("Updated value for sensor [%s] with value [%s].", self.id, value)
         self.cbpi.sensor.write_log(self.id, value)
         self.cbpi.emit("SENSOR_UPDATE", id=self.id, value=value)
         self.cbpi.ws_emit("SENSOR_UPDATE", self.cbpi.cache["sensors"][self.id])

modules/core/core.py

@@ -205,9 +205,11 @@ class SensorCore(object):
         self.logger.debug(self.cbpi)
 
     def write_log(self, id, value, prefix="sensor"):
+        self.logger.info("Write Log")
         filename = "./logs/%s_%s.log" % (prefix, str(id))
         formatted_time = strftime("%Y-%m-%d %H:%M:%S", localtime())
         msg = str(formatted_time) + "," + str(value) + "\n"
+        self.logger.info("Write to log [%s]", msg)
 
         with open(filename, "a") as file:
             file.write(msg)
@@ -246,7 +248,7 @@ class BrewingCore(object):
             # Start controller
             if kettle.logic is not None:
                 cfg = kettle.config.copy()
-                cfg.update(dict(api=cbpi, kettle_id=kettle.id, heater=kettle.heater, sensor=kettle.sensor))
+                cfg.update(dict(api=self.cbpi, kettle_id=kettle.id, heater=kettle.heater, sensor=kettle.sensor))
                 instance = self.get_controller(kettle.logic).get("class")(**cfg)
                 instance.init()
                 kettle.instance = instance
@@ -256,13 +258,13 @@ class BrewingCore(object):
 
                 t = self.cbpi._socketio.start_background_task(target=run, instance=instance)
             kettle.state = not kettle.state
-            self.cbpi.ws_emit("UPDATE_KETTLE", cbpi.cache.get("kettle").get(id))
+            self.cbpi.ws_emit("UPDATE_KETTLE", self.cbpi.cache.get("kettle").get(id))
             self.cbpi.emit("KETTLE_CONTROLLER_STARTED", id=id)
         else:
             # Stop controller
             kettle.instance.stop()
             kettle.state = not kettle.state
-            self.cbpi.ws_emit("UPDATE_KETTLE", cbpi.cache.get("kettle").get(id))
+            self.cbpi.ws_emit("UPDATE_KETTLE", self.cbpi.cache.get("kettle").get(id))
             self.cbpi.emit("KETTLE_CONTROLLER_STOPPED", id=id)
 
 
@@ -285,6 +287,7 @@ class CraftBeerPI(object):
         #logging.basicConfig(filename='./logs/app.log', level=logging.INFO, format=FORMAT)
         logging.config.dictConfig(yaml.load(open('./config/logger.yaml', 'r')))
         self.logger = logging.getLogger(__name__)
+        self.logger.info("Logger got initialized.")
 
         self.cache["messages"] = []
         self.cache["version"] = "3.1"
@@ -399,6 +402,5 @@ class CraftBeerPI(object):
                 self.logger.error(e)
                 self.notify("Failed to load plugin %s " % filename, str(e), type="danger", timeout=None)
 
-
 cbpi = CraftBeerPI()
-addon = cbpi.addon
+addon = cbpi.addon

run.py

@@ -1,12 +1,6 @@
 #!/usr/bin/env python
 
 from modules.core.core import *
-
-cbpi = CraftBeerPI()
-
-addon = cbpi.addon
-
-
 from modules.core.db_migrate import *
 from modules.buzzer import *
 from modules.config import *