Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: qb 支持自动添加分类 #1588

Merged
merged 1 commit into from
Sep 17, 2023
Merged

feat: qb 支持自动添加分类 #1588

merged 1 commit into from
Sep 17, 2023

Conversation

IITII
Copy link
Member

@IITII IITII commented Sep 17, 2023

  • 这是一个测试中的功能

#77 #1030 #1518 #1564

使用步骤

  • 如果升级后发现未生效, 可以试试先重启下浏览器
  1. 下载服务器 -> 编辑 -> 启用分类 -> 添加分类名称和路径 -> 保存
  2. 下载目录设置 -> 选择下载服务器 -> 所有站点 -> 添加一样的路径 -> 保存
  3. 刷新 PT 站网页, 选择下载到配置过的路径. 一键下载也可以.
  4. 查看 qb, 相关信息应该已经推过去了.

反馈要求

  • qb 配置截图
  • 下载路径截图
  • 下载操作截图
  • 对应站点
  • PTPP 背景页日志截图

注意

autoTMM: 自动种子管理

  • qb 4.5.2 实测 autoTMM 为 true 的情况下, 如果指定分类, 存储路径以分类为准.
  • 分类不存在时, 会自动创建分类, 路径为 默认 下载路径/分类名称
  • PTPP 里面配置的路径不匹配的情况下, 会自动禁用 autoTMM
  • PTPP 分类路径和名称需要和 QB 一直

截图

image image image image

参考文档

其他

  • 目前这个路径处理比较麻烦, 考虑了很久还是采取直接判断路径的方式, 而不是选择下载到分类这种操作.
  • 标签功能, 能再区分的也就只有站点名称了, 那个按 tracker 排序即可, 实际意义不大.
  • 而且, 从个人使用体验上来说, 标签和自动分类没有可比性.

@IITII IITII merged commit f2eaa32 into pt-plugins:dev Sep 17, 2023
@IITII IITII requested review from Rhilip and fzlins September 17, 2023 12:30
@fzlins
Copy link
Collaborator

fzlins commented Sep 17, 2023

这个会和现有的分类逻辑冲突吗? ffc4681

@fzlins
Copy link
Collaborator

fzlins commented Sep 17, 2023

现在qb分类是在下载目录里配置的。
image

@IITII
Copy link
Member Author

IITII commented Sep 17, 2023 via email

Comment on lines +174 to +186
if (clientOptions && clientOptions.enableCategory) {
qbCategories = clientOptions.qbCategories
if (qbCategories && qbCategories.length > 0) {
let qbCategory = qbCategories.find(c => c.path === savePath)
// qb 4.5.2 实测 autoTMM 为 true 的情况下, 如果指定分类, 存储路径以分类为准.
// 分类不存在时, 会自动创建分类, 路径为 `默认下载路径/分类名称`.
// nastool 会自动维护分类, 这里不做那么细致的处理.
// 所以路径不匹配的情况下, 需要禁用 autoTMM
if (qbCategory) {
// 以用户手动设置为准
category = qbCategory.name
autoTMM = true
} else {
Copy link
Member Author

@IITII IITII Sep 17, 2023

Choose a reason for hiding this comment

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

也就说现在有两处可以配置分类. 一是下载路径,二是下载器配置. 如果分类有重叠的话, 从实现上来说会以我这个为准. @fzlins

@IITII IITII mentioned this pull request Sep 24, 2023
4 tasks
@sjtuross
Copy link

sjtuross commented Oct 2, 2023

个人想法供参考:路径和分类只能二选一配置,用了分类autoTMM就开启否则关闭。会用分类的用户自然会在QB里配置好路径等等。这样逻辑和配置都会简化

@mitlearn
Copy link

mitlearn commented Oct 2, 2023

个人想法供参考:路径和分类只能二选一配置,用了分类autoTMM就开启否则关闭。会用分类的用户自然会在QB里配置好路径等等。这样逻辑和配置都会简化

这么设计也是为了避免出错 假设路径与分类没有匹配就会下到默认路径下
而且从这里开始已经是这么设计#1588 (comment)

@IITII
Copy link
Member Author

IITII commented Oct 3, 2023

个人想法供参考:路径和分类只能二选一配置,用了分类autoTMM就开启否则关闭。会用分类的用户自然会在QB里配置好路径等等。这样逻辑和配置都会简化

可以忽略 fzlins 提出的这种配置方法.

  1. 没文档没说明, 没见过的人都不会配
  2. 按之前的逻辑不会启用 autoTMM, 我给加上了

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.

4 participants