Only return warning if page itemtype (v2) is not implemented (Fixes: #362)

Author: tehkillerbee

tidalapi/page.py

@@ -18,6 +18,7 @@
 """
 
 import copy
+import logging
 from dataclasses import dataclass
 from typing import (
     TYPE_CHECKING,
@@ -65,6 +66,8 @@
 
 AllCategoriesV2 = Union[PageCategoriesV2]
 
+log = logging.getLogger(__name__)
+
 
 class Page:
     """
@@ -337,13 +340,16 @@ def __init__(self, session: "Session"):
         self.items: List[Any] = []
 
     def parse(self, json_obj: "JsonObj"):
-        self.items = [self.get_item(item) for item in json_obj["items"]]
+        self.items = [
+            self.get_item(item) for item in json_obj["items"] if item is not None
+        ]
         return self
 
     def get_item(self, json_obj: "JsonObj") -> Any:
         item_type = json_obj.get("type")
         if item_type not in self.item_types:
-            raise NotImplementedError(f"Item type '{item_type}' not implemented")
+            log.warning(f"Item type '{item_type}' not implemented")
+            return None
 
         return self.item_types[item_type](json_obj["data"])
 

tidalapi/user.py

@@ -594,8 +594,9 @@ def get_artists_count(
         self,
     ) -> int:
         """Get the total number of artists in the user's collection.
-        This performs a minimal API request (limit=1) to fetch metadata about
-        the artists without retrieving all of them. The API response contains
+
+        This performs a minimal API request (limit=1) to fetch metadata about the
+        artists without retrieving all of them. The API response contains
         'totalNumberOfItems', which represents the total items (artists) available.
         :return: The number of items available.
         """
@@ -652,9 +653,10 @@ def get_albums_count(
         self,
     ) -> int:
         """Get the total number of albums in the user's collection.
-        This performs a minimal API request (limit=1) to fetch metadata about
-        the albums without retrieving all of them. The API response contains
-        'totalNumberOfItems', which represents the total items (albums) available.
+
+        This performs a minimal API request (limit=1) to fetch metadata about the albums
+        without retrieving all of them. The API response contains 'totalNumberOfItems',
+        which represents the total items (albums) available.
         :return: The number of items available.
         """
         params = {"limit": 1, "offset": 0}
@@ -766,8 +768,9 @@ def playlist_folders(
 
     def get_playlists_count(self) -> int:
         """Get the total number of playlists in the user's root collection.
-        This performs a minimal API request (limit=1) to fetch metadata about
-        the playlists without retrieving all of them. The API response contains
+
+        This performs a minimal API request (limit=1) to fetch metadata about the
+        playlists without retrieving all of them. The API response contains
         'totalNumberOfItems', which represents the total playlists available.
         :return: The number of items available.
         """
@@ -831,9 +834,10 @@ def get_tracks_count(
         self,
     ) -> int:
         """Get the total number of tracks in the user's collection.
-        This performs a minimal API request (limit=1) to fetch metadata about
-        the tracks without retrieving all of them. The API response contains
-        'totalNumberOfItems', which represents the total items (tracks) available.
+
+        This performs a minimal API request (limit=1) to fetch metadata about the tracks
+        without retrieving all of them. The API response contains 'totalNumberOfItems',
+        which represents the total items (tracks) available.
         :return: The number of items available.
         """
         params = {"limit": 1, "offset": 0}