Merge pull request #1927 from glensc/lists-plugin-load

Refactor: Enable TraktListsPlugin only if two other plugins need it
Taxel · Apr 30, 2024 · 3201c2b · 3201c2b
2 parents 9e77979 + 2a19119
commit 3201c2b
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 6 deletions.
diff --git a/plextraktsync/config/SyncConfig.py b/plextraktsync/config/SyncConfig.py
@@ -80,6 +80,13 @@ def sync_wl(self):
     def sync_liked_lists(self):
         return self.trakt_to_plex["liked_lists"]
 
+    @cached_property
+    def sync_watchlists(self):
+        return any([
+            self.plex_to_trakt["watchlist"],
+            self.trakt_to_plex["watchlist"],
+        ])
+
     @cached_property
     def need_library_walk(self):
         return any([

diff --git a/plextraktsync/sync/TraktListsPlugin.py b/plextraktsync/sync/TraktListsPlugin.py
@@ -21,8 +21,12 @@ def __init__(self):
 
     @staticmethod
     def enabled(config):
-        # Check for need is performed in init()
-        return True
+        return any([
+            # LikedListsPlugin
+            config.sync_liked_lists,
+            # WatchListPlugin
+            config.sync_watchlists,
+        ])
 
     @classmethod
     def factory(cls, sync):

diff --git a/plextraktsync/sync/WatchListPlugin.py b/plextraktsync/sync/WatchListPlugin.py
@@ -24,10 +24,7 @@ def __init__(self, config: SyncConfig, plex: PlexApi, trakt: TraktApi):
 
     @staticmethod
     def enabled(config: SyncConfig):
-        return any([
-            config.plex_to_trakt["watchlist"],
-            config.trakt_to_plex["watchlist"],
-        ])
+        return config.sync_watchlists
 
     @classmethod
     def factory(cls, sync: Sync):