mirror of
https://github.com/spotipy-dev/spotipy.git
synced 2026-04-27 00:25:54 +03:00
[GH-ISSUE #534] total parameter should refer to all types in search_markets method #319
Labels
No labels
api-bug
bug
dependencies
documentation
duplicate
enhancement
external-ide
headless-mode
implicit-grant-flow
invalid
missing-endpoint
pr-welcome
private-api
pull-request
question
spotipy3
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/spotipy#319
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @stephanebruckert on GitHub (Jul 7, 2020).
Original GitHub issue: https://github.com/spotipy-dev/spotipy/issues/534
Currently we can stop searching all markets when an
totalresults are found. However this only applies to the first search type when multiple types are given:github.com/plamere/spotipy@f7fb8757e9/spotipy/client.py (L541)@quanuw commented on GitHub (Apr 11, 2021):
Hello, I'm new to this project and would like to contribute! Can I be assigned to this issue @stephanebruckert?
@rngolam commented on GitHub (Nov 27, 2022):
Hi @stephanebruckert, I'm currently looking into this issue and just want to clarify the desired behavior.
The
limitparameter specifies the number of items to be returned for each search type. The code will currently truncate the limit as the item count approachestotal. But what should happen when multiple types are specified?For example, if the user enters this query:
The code will search the first market for 10 artist items followed by 10 track items, bringing the count to 20. At this point, we want to truncate the limit to 5 prior to searching the next market. But should we still get 5 items for both artists and tracks, resulting in a count of 30 items? Or do we just want to get the 5 artists items and immediately return the 25 results at this point (which would result in the second market having no
tracksproperty)?Thanks!