Fix limit bug with trending playlists (#1303)

AudiusProject · Mar 13, 2021 · e7bb435 · e7bb435
1 parent df56278
commit e7bb435
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/discovery-provider/src/api/v1/playlists.py b/discovery-provider/src/api/v1/playlists.py
@@ -362,5 +362,6 @@ def get(self):
         else:
             key = self.get_cache_key()
             playlists = use_redis_cache(key, TRENDING_TTL_SEC, lambda: get_trending_playlists(args))
+            playlists = playlists[offset: limit + offset]
 
         return success_response(playlists)
diff --git a/discovery-provider/src/queries/get_trending_playlists.py b/discovery-provider/src/queries/get_trending_playlists.py
@@ -186,7 +186,7 @@ def get_trending_playlists(args):
 
         # Apply limit + offset early to reduce the amount of
         # population work we have to do
-        if limit and offset:
+        if limit is not None and offset is not None:
             playlists = playlists[offset: limit + offset]
             playlist_ids = playlist_ids[offset: limit + offset]