Fix modules to reload previously active module rather than load it (which will make no change due to cacheing)

Author: acockburn

Diff for: appdaemon/appdaemon.py

@@ -1799,7 +1799,12 @@ def read_app(self, file, reload=False):
             app = self.get_app_from_file(file)
             if app is not None:
                 self.log("INFO", "Loading App Module: {}".format(file))
-                self.modules[module_name] = importlib.import_module(module_name)
+                if module_name not in self.modules:
+                    self.modules[module_name] = importlib.import_module(module_name)
+                else:
+                    # We previously imported it so we need to reload to pick up any potential changes
+                    importlib.reload(self.modules[module_name])
+
             elif "global_modules" in self.app_config and module_name in self.app_config["global_modules"]:
                 self.log("INFO", "Loading Global Module: {}".format(file))
                 self.modules[module_name] = importlib.import_module(module_name)

Diff for: docs/HISTORY.rst

@@ -11,7 +11,8 @@ Change Log
 **Fixes**
 
 - Fixed incorrect service call in ``set_value()``
-- Enforce domain name in rss feed target
+- Enforce domain name in rss feed target to avoid issues with other functions
+- Previously deleted modules will now be correctly reloaded to reflect changes
 
 **Breaking Changes**