search

jellyfin / jellyfin-plugin-anilist

GNU General Public License v2.0
37 stars 22 forks source link

Studio data includes both Studio and Producers #73

Open sjorge opened 4 months ago

sjorge commented 4 months ago
Screenshot 2024-07-12 at 12 10 49 Screenshot 2024-07-12 at 12 11 27

Ref: https://anilist.co/anime/116589/86-Eighty-Six/

As you can see the studio tags contains both the Studio data and Producers data from anilist. I having the producers be added there seems wrong to me IMHO. It would be nice to have that as a configuration option to pick if only Studio or Studio + Producers get added.

Edit: I'm not sure but I think they are differentiated in GraphQL response by isMainStudio where it is false for the producers.

Edit2: I also noticed we currently have dups because of this, https://anilist.co/anime/150672/Oshi-no-Ko/ seems to have Doga Kobo twice. Because it's listed twice on anilist (ones under studio and ones under producers)

jack-mil commented 4 months ago

This has been a pet peeve of mine for a while. I usually have to go and manually remove duplicate or unnecessary 'Studios'

dr-dialup commented 4 months ago

I'm not able to make a PR at the moment but I believe it's as trivial as changing the following line https://github.com/jellyfin/jellyfin-plugin-anilist/blob/01c6986e1fe4a68c27122b591310672367179af4/Jellyfin.Plugin.AniList/Providers/AniList/AniListApi.cs#L98 to this to only retrieve an entry's Studio(s) rather than Studio(s) + Producers. 

studios(isMain: true) {

This appears to work even for shows that have multiple animation studios.