Refactor init for initialize backwards-compat.

Author: NeonDaniel

ovos_workshop/skills/common_play.py

@@ -50,9 +50,7 @@ def ...
     vocab for starting playback is needed.
     """
 
-    def __init__(self, name=None, bus=None, **kwargs):
-        OVOSSkill.__init__(self, name, bus, **kwargs)
-        # NOTE: derived skills will likely want to override this list
+    def __init__(self, name=None, bus=None, **kwargs):        # NOTE: derived skills will likely want to override this list
         self.supported_media = [MediaType.GENERIC,
                                 MediaType.AUDIO]
         self._search_handlers = []  # added via decorators
@@ -67,6 +65,7 @@ def __init__(self, name=None, bus=None, **kwargs):
         self._playing = Event()
         # TODO replace with new default
         self.skill_icon = "https://github.com/OpenVoiceOS/ovos-ocp-audio-plugin/raw/master/ovos_plugin_common_play/ocp/res/ui/images/ocp.png"
+        OVOSSkill.__init__(self, name, bus, **kwargs)
 
     def bind(self, bus):
         """Overrides the normal bind method.

ovos_workshop/skills/common_query_skill.py

@@ -57,7 +57,6 @@ class CommonQuerySkill(OVOSSkill):
     """
 
     def __init__(self, name=None, bus=None, **kwargs):
-        OVOSSkill.__init__(self, name, bus, **kwargs)
         noise_words_filepath = f"text/{self.lang}/noise_words.list"
         default_res = f"{dirname(dirname(__file__))}/res/text/{self.lang}/noise_words.list"
         noise_words_filename = resolve_resource_file(noise_words_filepath) or \
@@ -75,6 +74,7 @@ def __init__(self, name=None, bus=None, **kwargs):
             CQSMatchLevel.CATEGORY: 0.6,
             CQSMatchLevel.GENERAL: 0.5
         }
+        OVOSSkill.__init__(self, name, bus, **kwargs)
 
     @property
     def translated_noise_words(self):

ovos_workshop/skills/fallback.py

@@ -100,12 +100,12 @@ class FallbackSkillV1(_MetaFB, metaclass=_MutableFallback):
     wrapper_map: List[Tuple[callable, callable]] = []  # [(handler, wrapper)]
 
     def __init__(self, name=None, bus=None, use_settings=True, **kwargs):
-        super().__init__(name, bus, use_settings, **kwargs)
         #  list of fallback handlers registered by this instance
         self.instance_fallback_handlers = []
 
         # "skill_id": priority (int)  overrides
         self.fallback_config = self.config_core["skills"].get("fallbacks", {})
+        super().__init__(name, bus, use_settings, **kwargs)
 
     @classmethod
     def make_intent_failure_handler(cls, bus: MessageBusClient):