Skip to content

fix(drivers/189pc,drivers/189tv): fix extra requests when viewing files#2791

Open
wzdc wants to merge 3 commits into
OpenListTeam:mainfrom
wzdc:main
Open

fix(drivers/189pc,drivers/189tv): fix extra requests when viewing files#2791
wzdc wants to merge 3 commits into
OpenListTeam:mainfrom
wzdc:main

Conversation

@wzdc

@wzdc wzdc commented Jul 14, 2026

Copy link
Copy Markdown

Summary / 摘要

原有代码中,翻页的终止条件只判断了 Count == 0(总数为0),但并未判断当前返回的文件数量是否未达到数量,导致额外请求获取下一页的文件。这里添加了判断返回的文件数量是否达到所设置的pageSize才触发请求下一页。

fileListAO.count获取的是当前文件夹下总共的文件数量而不是当前返回的文件数量,但实际测试发现:当你请求的页码超出实际范围时(比如总共只有 3 页,你却请求第 4 页),接口返回的 Count 变成了 0。

  • This PR has breaking changes.
    / 此 PR 包含破坏性变更。
  • This PR changes public API, config, storage format, or migration behavior.
    / 此 PR 修改了公开 API、配置、存储格式或迁移行为。
  • This PR requires corresponding changes in related repositories.
    / 此 PR 需要关联仓库同步修改。

Related repository PRs / 关联仓库 PR:

  • OpenList-Frontend:
  • OpenList-Docs:

Related Issues / 关联 Issue

Testing / 测试

  • go test ./...
  • Manual test / 手动测试: 编译运行后使用网络调试工具检查网络请求行为

Checklist / 检查清单

  • I have read CONTRIBUTING.
    / 我已阅读 CONTRIBUTING
  • I confirm this contribution follows the repository license, contribution policy, and code of conduct.
    / 我确认此贡献符合仓库许可证、贡献规范和行为准则。
  • I have formatted the changed code with gofmt, go fmt, or prettier where applicable.
    / 我已按适用情况使用 gofmtgo fmtprettier 格式化变更代码。
  • I have requested review from relevant maintainers or code owners where applicable.
    / 我已在适用情况下请求相关维护者或代码所有者审查。

AI Disclosure / AI 使用声明

  • This PR includes AI-assisted content.
    / 此 PR 包含 AI 辅助内容。

Tools used / 使用工具:

  • ChatGPT
  • Codex
  • GitHub Copilot
  • Claude
  • Gemini
  • Other (please specify) / 其他(请注明):

Usage scope / 使用范围:

  • Code generation / 代码生成

  • Refactoring / 重构

  • Documentation / 文档

  • Tests / 测试

  • Translation / 翻译

  • Review assistance / 审查辅助

  • I have reviewed and validated all AI-assisted content included in this PR.
    / 我已审核并验证此 PR 中的所有 AI 辅助内容。

  • I have ensured that all AI-assisted commits include Co-Authored-By attribution.
    / 我已确保所有 AI 辅助提交都包含 Co-Authored-By 归属信息。

  • I can reproduce all AI-assisted content included in this PR without any AI tools.
    / 我可以在没有任何 AI 工具的情况下重现此 PR 中包含的所有 AI 辅助内容。

@jyxjjj jyxjjj left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

建议复用请求时使用的 pageSize,避免再次硬编码 130 和 1000,否则以后修改分页大小时容易遗漏这里。

@wzdc
wzdc requested a review from jyxjjj July 15, 2026 13:02
@xrgzs xrgzs added Module: Driver Driver-Related Issue/PR bug labels Jul 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Module: Driver Driver-Related Issue/PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants