Get popular people refactor

jellyfin / TMDbLib

C#.Net library for TheMovieDB

MIT License

344 stars 128 forks source link

Closed angyanmark closed 1 year ago

angyanmark commented 2 years ago

There is no need for PersonListType.cs and PersonResult.cs. Get popular people method renamed and language parameter added.

Now it matches the get popular movies and get popular tv shows methods, they all return a SearchContainer<SearchBase>.

angyanmark commented 1 year ago

Hi, @LordMike!

Could you take a look at this PR?

I changed the get popular people method's name and return value, now it is consistent with the other get popular methods (GetMoviePopularListAsync, GetTvShowPopularAsync).
I added the language parameter to the method.
I also removed the PersonResult class, it is the same as SearchPerson, and was only used here. Now all get popular methods return a SearchContainer<SearchBase>.

Thanks!