Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

属性相关 #98

Open
wuyu8512 opened this issue Nov 13, 2023 · 2 comments
Open

属性相关 #98

wuyu8512 opened this issue Nov 13, 2023 · 2 comments

Comments

@wuyu8512
Copy link

  • 个人觉得通过路径属性设置的Id应该被强制设置,现在代码里会判断是否已经设置ID
  • Season还不支持使用路径属性
  • 在Series层面获取id的时候,如果当前还没有id,可以考虑去检查下面Season有没有被设置id,然后取个最小的

额外的配置文件也同理

@kookxiang
Copy link
Owner

  1. 意思是在配置了路径 id 的时候忽略已有 id 吗?
  2. 看了下 jellyfin 的实现,并不支持这种用法,可以先通过 ini 的方式指定
    https://github.com/jellyfin/jellyfin/blob/master/Emby.Server.Implementations/Library/Resolvers/TV/SeasonResolver.cs
  3. 当 Series 添加的时候应该还没扫到 Season 文件夹,感觉不太行

@wuyu8512
Copy link
Author

wuyu8512 commented Nov 19, 2023

  1. 对的
  2. https://github.com/kookxiang/jellyfin-plugin-bangumi/blob/master/Jellyfin.Plugin.Bangumi/Providers/SeasonProvider.cs 里的 GetMetadata 来一遍是可以的
  3. 确实这会没扫到,但代码里面可以嘛(

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants