Refactor error handling in YamtrackImporter

Author: FuzzyGrim

src/integrations/imports/yamtrack.py

@@ -9,6 +9,7 @@
 import app
 from app.models import MediaTypes, Sources
 from app.providers import services
+from app.templatetags import app_tags
 from integrations.imports import helpers
 from integrations.imports.helpers import MediaImportError, MediaImportUnexpectedError
 
@@ -65,11 +66,16 @@ def import_data(self):
         for row in reader:
             try:
                 self._process_row(row)
-            except Exception as error:
-                error_msg = f"Error processing entry: {row.get('title', 'Unknown')} ({row.get('media_type', 'Unknown')}): {error}"
+            except services.ProviderAPIError as error:
+                error_msg = (
+                    f"Error processing entry with ID {row['media_id']} "
+                    f"({app_tags.media_type_readable(row['media_type'])}): {error}"
+                )
                 self.warnings.append(error_msg)
-                logger.error(error_msg, exc_info=True)
                 continue
+            except Exception as error:
+                error_msg = f"Error processing entry: {row}"
+                raise MediaImportUnexpectedError(error_msg) from error
 
         helpers.cleanup_existing_media(self.to_delete, self.user)
         helpers.bulk_create_media(self.bulk_media, self.user)
@@ -159,18 +165,12 @@ def _handle_missing_metadata(self, row, media_type, season_number, episode_numbe
         if row["source"] == Sources.MANUAL.value and row["image"] == "":
             row["image"] = settings.IMG_NONE
         else:
-            try:
-                metadata = services.get_media_metadata(
-                    media_type,
-                    row["media_id"],
-                    row["source"],
-                    [season_number],
-                    episode_number,
-                )
-                row["title"] = metadata["title"]
-                row["image"] = metadata["image"]
-            except services.ProviderAPIError as e:
-                self.warnings.append(
-                    f"Failed to fetch metadata for {row['media_id']}: {e!s}",
-                )
-                raise
+            metadata = services.get_media_metadata(
+                media_type,
+                row["media_id"],
+                row["source"],
+                [season_number],
+                episode_number,
+            )
+            row["title"] = metadata["title"]
+            row["image"] = metadata["image"]