Refactor: Move URL construction logic from use case to infrastructure layer

Co-authored-by: marceljungle <44347173+marceljungle@users.noreply.github.com>

Author: Copilot

red_plex/infrastructure/rest/gazelle/gazelle_api.py

@@ -378,6 +378,20 @@ def add_to_collage(self, collage_id: str,
             logger.error('Error adding groups %s to collage %s: %s', group_ids_str, collage_id, e)
             return None
 
+    def get_torrent_group_url(self, group_id: str) -> str:
+        """
+        Get the full URL for a torrent group page.
+        
+        Args:
+            group_id: The ID of the torrent group
+            
+        Returns:
+            Full URL to the torrent group page
+        """
+        # Get the base URL (remove '/ajax.php?action=' suffix to get clean base)
+        base_url = self.base_url.replace('/ajax.php?action=', '')
+        return f"{base_url}/torrents.php?id={group_id}"
+
     @staticmethod
     def _normalize_string(text: str) -> str:
         """

red_plex/use_case/show_missing/show_missing_use_case.py

@@ -81,9 +81,6 @@ def execute(self, collage_id: str) -> ShowMissingResponse:
                 missing_groups=[]
             )
 
-        # Get the base URL from config for links
-        base_url = self._get_base_url(site)
-
         # Fetch details for each missing group
         missing_groups = []
         for group_id in sorted(missing_group_ids):
@@ -92,7 +89,7 @@ def execute(self, collage_id: str) -> ShowMissingResponse:
                 if torrent_group:
                     artists = torrent_group.artists if torrent_group.artists else ["Unknown Artist"]
                     album_name = torrent_group.album_name or "Unknown Album"
-                    torrent_url = f"{base_url}/torrents.php?id={group_id}"
+                    torrent_url = self.gazelle_api.get_torrent_group_url(str(group_id))
 
                     missing_groups.append(MissingGroupInfo(
                         group_id=group_id,
@@ -106,7 +103,7 @@ def execute(self, collage_id: str) -> ShowMissingResponse:
                         group_id=group_id,
                         artist_names=["Unknown Artist"],
                         album_name="Details not available",
-                        torrent_url=f"{base_url}/torrents.php?id={group_id}"
+                        torrent_url=self.gazelle_api.get_torrent_group_url(str(group_id))
                     ))
             except Exception as e:  # pylint: disable=W0718
                 logger.warning("Failed to fetch details for group %s: %s", group_id, e)
@@ -115,7 +112,7 @@ def execute(self, collage_id: str) -> ShowMissingResponse:
                     group_id=group_id,
                     artist_names=["Unknown Artist"],
                     album_name="Error fetching details",
-                    torrent_url=f"{base_url}/torrents.php?id={group_id}"
+                    torrent_url=self.gazelle_api.get_torrent_group_url(str(group_id))
                 ))
 
         return ShowMissingResponse(
@@ -124,17 +121,3 @@ def execute(self, collage_id: str) -> ShowMissingResponse:
             site=site,
             missing_groups=missing_groups
         )
-
-    def _get_base_url(self, site: str) -> str:
-        """Get the base URL for the site from config."""
-        try:
-            # pylint: disable=C0415
-            from red_plex.infrastructure.config.config import load_config
-            config_data = load_config()
-            site_config = config_data.site_configurations.get(site.upper())
-            if site_config:
-                return site_config.base_url.rstrip('/')
-            return f"https://{site}.example.com"  # fallback
-        except Exception as e:  # pylint: disable=W0718
-            logger.warning("Failed to load config for URL construction: %s", e)
-            return f"https://{site}.example.com"  # fallback