Skip to content

修复“搜索歌单”功能在获取收藏夹视频列表时缺失最后一页的问题 - #60

Open
Xlucidator wants to merge 3 commits into
kenmingwang:masterfrom
Xlucidator:master
Open

修复“搜索歌单”功能在获取收藏夹视频列表时缺失最后一页的问题#60
Xlucidator wants to merge 3 commits into
kenmingwang:masterfrom
Xlucidator:master

Conversation

@Xlucidator

Copy link
Copy Markdown

问题描述在 #59

问题段落修复为

    for (const response of v) {
        const js = await response.json()
        js.data.medias.forEach(m => BVidPromises.push(fetchVideoInfo(m.bvid)))
    }

修复后测试正常。media_id=3349906669 的收藏夹共142个视频,修复后缺失的最后2个视频正确加入列表,如图所示
image

p.s 顺手修正了以下几处'Delete'的错误拼写

@lovegaoshi

Copy link
Copy Markdown
Contributor

建议一步到位 直接用不分页的API 参考

@Xlucidator

Copy link
Copy Markdown
Author

@lovegaoshi 已改用不分页的API并进行简单测试。还有一些附加说明和疑问:

  1. 似乎还可使用Promise.allSettled应对中途网络错误,不过看其他地方都是all就不改了
  2. 返回信息里bvid和bv_id是什么关系呢?b站的屎山吗(doge
  3. 合集搜索相关功能似乎已经旧了用不了了,而且本来用起来也很麻烦,相关函数看上去也冗杂,可能(?)再开个issue修下
  4. 这个项目好冷啊,是要弃置了的感觉吗..

@lovegaoshi

Copy link
Copy Markdown
Contributor

死挺久了
https://github.com/lovegaoshi/NoxPlayer

@Xlucidator

Copy link
Copy Markdown
Author

乐了

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

Successfully merging this pull request may close these issues.

2 participants