mebtte / cicada

A multi-user music service for self-hosting.
GNU General Public License v3.0
1.03k stars 80 forks source link

fix: 歌手列表查询不出来的问题 #37

Closed CodeByZack closed 1 year ago

CodeByZack commented 1 year ago

https://stackoverflow.com/questions/27985863/mysql-left-join-and-count-and-group-by

CodeByZack commented 1 year ago

写导入歌手脚本的时候发现的,添加了多个歌手,点我的歌手查询出来的数据不对。

mebtte commented 1 year ago

@CodeByZack 应该是某个表的外键不正确导致出现 NULL 吧 如果外键正确工作的话 LEFT JOIN 不会出现 NULL 的情况

CodeByZack commented 1 year ago

@mebtte 大佬SQL 我不太熟悉哈, 我只是试了下换成s.id能查出来数据

场景是 先不添加歌曲 正常添加多个歌手, /api/self_singer_list 这个接口查出来的数据是不对的

mebtte commented 1 year ago

我复现一下

CodeByZack commented 1 year ago

@mebtte 大佬需要一起开发么,感觉很多功能都还可以加一些

mebtte commented 1 year ago

@CodeByZack 有新功能或者建议的话可以提 issue 讨论一下, 谈论下来没问题的话就可以实现和提 pull request 了

mebtte commented 1 year ago

@CodeByZack 我复现出来了, 也理解 https://stackoverflow.com/questions/27985863/mysql-left-join-and-count-and-group-by 这里说的意思了, 确实是存在问题

mebtte commented 1 year ago

估计其他 SQL 也有这种问题, 我统一 review 一下

mebtte commented 1 year ago

@CodeByZack 已发版 0.67.1