Skip to content

fix: 修正 VIP 判断逻辑#92

Merged
imsyy merged 1 commit into
SPlayer-Dev:devfrom
MoYingJi:pr/fix/vip
Jul 21, 2026
Merged

fix: 修正 VIP 判断逻辑#92
imsyy merged 1 commit into
SPlayer-Dev:devfrom
MoYingJi:pr/fix/vip

Conversation

@MoYingJi

Copy link
Copy Markdown
Collaborator

将「会员高音质」也视作免费。这符合逻辑(可以免费听低音质),也符合官方客户端的显示逻辑

改动类型

  • 新功能(feat)
  • 缺陷修复(fix)
  • 重构 / 优化(不改变对外行为)
  • 文档(docs)
  • 其他(请在「改动说明」中注明)

是否包含破坏性变更

  • 是(请在「改动说明」中详细描述)

改动说明

将「会员高音质」也视作免费。这符合逻辑(可以免费听低音质),也符合官方客户端的显示逻辑

测试情况

对比了我的歌单中修复后 SPlayer Next 和官方客户端(Android)显示 VIP 徽标的情况,均符合

自查清单

  • 本 PR 只包含一个主要功能 / 修复,没有夹带无关改动
  • 已在本地完整测试通过;AI 生成的代码同样自行测试并审阅过,未做未经验证的提交
  • 已运行 pnpm format,并确认 pnpm typecheckpnpm lint 通过
  • 改动涉及原生模块时已 pnpm build:native 验证;未手写 native/*/index.d.ts
  • 已向 dev 分支提交

将「会员高音质」也视作免费。这符合逻辑(可以免费听低音质),也符合官方客户端的显示逻辑
Copilot AI review requested due to automatic review settings July 19, 2026 17:29

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

该 PR 修复网易云歌曲 fee 字段到应用层 TrackFee 的映射逻辑:将「会员高音质(fee=8)」从“视作 VIP”调整为“视作免费”,从而使 UI 的 VIP 徽标展示与官方客户端逻辑保持一致(可免费听低音质时不应显示 VIP)。

Changes:

  • 调整 toTrackFee 映射:fee===8 归类为免费(TrackFee=0),不再归类为 VIP(TrackFee=1
  • 同步更新对应注释说明,避免文档与实现不一致

@imsyy
imsyy merged commit 77f2ddb into SPlayer-Dev:dev Jul 21, 2026
1 check passed
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.

3 participants