added TODO + formatting

python_apps/media-monitor2/media/saas/airtimeinstance.py

@@ -50,15 +50,19 @@ class AirtimeInstance(object):
     def mm_config(self):
         return MMConfig(self.config_paths['media_monitor'])
 
-    @LazyProperty
+    # NOTE to future code monkeys:
-    def owner(self):
+    # I'm well aware that I'm using the shitty service locator pattern
-        return Owner()
+    # instead of normal constructor injection as I should be. The reason
+    # for this is that I found these issues a little too close to the
+    # end of my tenure. It's highly recommended to rewrite this crap
+    # using proper constructor injection if you ever have the time
 
     @LazyProperty
-    def event_registry(self):
+    def owner(self): return Owner()
-        return EventRegistry()
 
     @LazyProperty
-    def file_mediator(self):
+    def event_registry(self): return EventRegistry()
-        return FileMediator()
+
+    @LazyProperty
+    def file_mediator(self): return FileMediator()