QiuChenlyOpenSource / MusicDownload

歌曲下载
GNU General Public License v3.0
2.48k stars 430 forks source link

文件名 `singer` 问题及命名建议 #135

Open qupig opened 9 months ago

qupig commented 9 months ago

相关代码:

https://github.com/QiuChenlyOpenSource/QQFlacMusicDownloader/blob/b3b84496de59c901ccb6a554ed5d888ef329b576/flaskSystem/src/Common/Tools.py#L157-L159 https://github.com/QiuChenlyOpenSource/QQFlacMusicDownloader/blob/b3b84496de59c901ccb6a554ed5d888ef329b576/flaskSystem/src/Common/Tools.py#L233

目前直接使用了专辑信息的第一个 singer,由此会造成下述问题:

示例专辑:b:003efsms0jxvy5

建议:

可以考虑的文件名格式:[歌手1[,歌手2]...]-[歌曲名][.歌曲ID].Format,例如:

QiuChenly commented 9 months ago

呃 做一下回复: 其他的后面再说,主要回复一下为什么要强制第一个人做专辑艺术家:

  1. emby 显示专辑有bug,举个例子:a b 两首歌都属于xxx专辑,但是两个歌手演唱着singer分别是a b,这就会导致出现两个一摸一样的专辑,并且只有单独的一首歌,其他的非歌手的歌曲全部是另外单独一个专辑,所以我也是经过测试才决定这么写的。
  2. 我为了保证专辑艺术家能够让emby正确分类到一个专辑里,已经对专辑歌手做了查询,会先去qq服务器查询某专辑属于谁,然后就用这个歌手名称作为专辑演唱者,这就是为什么不用歌曲演唱者反而要用专辑演唱者的问题。
qupig commented 9 months ago

@QiuChenly 感谢回复。

抱歉没仔细看,该项目是主要只为 emby 做资源提供吗?

如果是 emby 的 bug,向上游反馈而不是将错就错造成其他错误,可能是更好的选择。

或将其作为一个选项,提供不同的输出方式。

文件名 singer 错误不是太大问题,只是影响查阅文件,但漏下同名歌曲就是硬错误了。

无论如何,感谢创建和维护该项目!