refactor: limit should be passed to itunes.search()

danjac · Aug 29, 2024 · 5a834d9 · 5a834d9
1 parent 8db9151
commit 5a834d9
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 7 deletions.
diff --git a/radiofeed/podcasts/itunes.py b/radiofeed/podcasts/itunes.py
@@ -3,7 +3,6 @@
 from collections.abc import Iterator
 
 import httpx
-from django.conf import settings
 
 from radiofeed.http_client import Client
 from radiofeed.podcasts.models import Podcast
@@ -28,11 +27,7 @@ class Feed:
     podcast: Podcast | None = None
 
 
-def search(
-    client: Client,
-    search_term: str,
-    limit: int = settings.PAGE_SIZE,
-) -> Iterator[Feed]:
+def search(client: Client, search_term: str, limit: int) -> Iterator[Feed]:
     """Search iTunes podcast API."""
     return _insert_podcasts(
         _parse_feeds_from_json(

diff --git a/radiofeed/podcasts/views.py b/radiofeed/podcasts/views.py
@@ -111,7 +111,11 @@ def search_itunes(request: HttpRequest) -> HttpResponseRedirect | TemplateRespon
     discover_url = reverse("podcasts:discover")
 
     if request.search:
-        feeds = itunes.search(get_client(), request.search.value)
+        feeds = itunes.search(
+            get_client(),
+            request.search.value,
+            settings.PAGE_SIZE,
+        )
 
         return TemplateResponse(
             request,