inflatablefriends / lastfm

Portable .Net library for Last.fm
Other
100 stars 62 forks source link

Fixed GetTop commands from the Tag Api #176

Open eivanmariscal opened 1 year ago

eivanmariscal commented 1 year ago

Lastfm's documentation is off for these endpoints:

Documentation says that it should return a "topalbums" and "toptracks" object respectively but it's actually returning "albums" and "tracks".

This is related to issue #172 and this PR fixes that.

There was another bug in the tag.getTopTags command. For this endpoint Lastfm doesn't return an URL object and this was throwig a NullReferenceException in the LastTag constructor. I added an overloaded constructor to fix the problem.