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

fix(tjupt): 站点搜索范围功能失效 #1999

Merged
merged 2 commits into from
Sep 16, 2024
Merged

fix(tjupt): 站点搜索范围功能失效 #1999

merged 2 commits into from
Sep 16, 2024

Conversation

ixff
Copy link

@ixff ixff commented Sep 15, 2024

本PR修复以下问题:
编辑站点中指定搜索范围失效;
搜索方案中指定站点的搜索范围失效。
两者都会返回全部分类的结果。

@fzlins
Copy link
Collaborator

fzlins commented Sep 15, 2024

现在如果多选好像就不工作了,看网站正确的应该是cat%5B%5D=(cat[]=)

@ixff
Copy link
Author

ixff commented Sep 16, 2024

我测试正常,你说的不工作具体指什么呢?
至于cat[]=和cat=,应该都是可以的。

@fzlins
Copy link
Collaborator

fzlins commented Sep 16, 2024

我测试正常,你说的不工作具体指什么呢? 至于cat[]=和cat=,应该都是可以的。

测试一下同时选电影和剧集

@ixff
Copy link
Author

ixff commented Sep 16, 2024

我测试正常,你说的不工作具体指什么呢? 至于cat[]=和cat=,应该都是可以的。

测试一下同时选电影和剧集

测试没问题。
你测试是什么表现?

@Rhilip
Copy link
Collaborator

Rhilip commented Sep 16, 2024

建议参照:b2d10b4 补充修改 categories.[*].result

&cat401=1 (no other category in filter) changed to &cat=401 (retained from the past) or &cat[]=401
&cat401=1&cat402=1 (more than two category in filter) changed to &cat[]=401&cat[]=402

@ixff
Copy link
Author

ixff commented Sep 16, 2024

建议参照:b2d10b4 补充修改 categories.[*].result

&cat401=1 (no other category in filter) changed to &cat=401 (retained from the past) or &cat[]=401
&cat401=1&cat402=1 (more than two category in filter) changed to &cat[]=401&cat[]=402

按照你的建议修改了。

@Rhilip Rhilip merged commit cd7b050 into pt-plugins:dev Sep 16, 2024
4 checks passed
@ixff ixff deleted the dev branch September 16, 2024 02:03
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