Make Spotify interfaces return actual paging objects

[jzheng2017]

Is your feature request related to a problem? Please describe.
Currently some api calls of the SpotifyApi interface return an object wrapping a Paging<T> object. This is done because of how the Spotify Web API return their json.

For instance PlaylistSimplifiedPaging is just wrapping around Paging<PlaylistSimplified>. The end user would have to make an additional getter call to get the actual contents of the paging object. This is unnecessary because it is an internal detail of how the json is mapped.

The following functions need to be changed:

• BrowseApi - getCategoryPlaylists
• BrowseApi - getCategories
• BrowseApi - getNewReleases

If there are anymore I've missed, please comment it on this issue.

Describe the solution you'd like
It would be better if the wrapped object contents got returned instead of the wrapper object itself.

So instead of returning the wrapped object, for instance PlaylistSimplifiedPaging, it would be better to returned the actual Paging object.

Describe alternatives you've considered
None

Additional context
The feature is actually already done once in FollowApiRetrofit class for the function getFollowedArtists. Take this function as reference when implementing the feature.

[Pratik-Nataraj516]

Hi, is this something I can work on?

[jzheng2017]

Hi, is this something I can work on?

@Pratik-Nataraj516 Yes, of course. I'll assign this issue to you. Implement the feature in the feature/unwrap-paging-objects branch. When creating the pull request to the upstream repository, make sure to point it to the same branch.

Also do not forget to read the contribution guidelines before starting.

Good luck!

EDIT: also do not change the function directly, but make new function and deprecate the old function with @Deprecated annotation. The annotation must have the following information: deprecated since <version>, an explanation why it is deprecated and a link to the new function that has to be used. This is done so the change is backwards compatible.

[jzheng2017]

@Pratik-Nataraj516 please confirm by 27-12-2020 that you're going to implement this feature. If there is no response by then, I'll have to remove you as assignee.

[SumanohariR]

is this issue Still open? if yes, can I work on it?
I work as a backend dev and I think I can work on this.

[jzheng2017]

is this issue Still open? if yes, can I work on it? I work as a backend dev and I think I can work on this.

Yeah, it's still open and you can work on this.

[SumanohariR]

Okay, Thank you for the confirmation.
I will start working on this

[jzheng2017]

Okay, Thank you for the confirmation. I will start working on this

Any update on the progress of this issue?