Bug: SingleEffectManager should run effect module we supply in the initializer

Author: einarf

demosys/effects/managers.py

@@ -66,14 +66,14 @@ def pre_load(self):
         """
         effect_list = [cfg.cls() for name, cfg in effects.effects.items()]
 
-        # If an effect want't specified in the initializer we grab the first one
-        if not self.active_effect:
-            self.active_effect = effect_list[0]
-        else:
-            # Otherwise we look for an effect with the name
+        # If an effect was specified in the initializer, find it
+        if self.effect_module:
             for effect in effect_list:
                 if effect.name == self.effect_module:
                     self.active_effect = effect
+        else:
+            # Otherwise we look just grab the first effect
+            self.active_effect = effect_list[0]
 
         if not self.active_effect:
             print("Cannot find effect '{}'".format(self.active_effect))
@@ -90,7 +90,8 @@ def draw(self, time, frametime, target):
         self.active_effect.draw(time, frametime, target)
 
     def key_event(self, key, scancode, action, mods):
-        print("SingleEffectManager:key_event", key, scancode, action, mods)
+        # print("SingleEffectManager:key_event", key, scancode, action, mods)
+        pass
 
 
 class TrackerEffectManager(BaseEffectManger):