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 keyword when searching #742

Merged
merged 1 commit into from
Feb 20, 2025
Merged

Conversation

2894638479
Copy link
Contributor

按我的理解,PopularController.keyword是用来检索视频源的关键词,在每次点进番剧卡片时被赋值为番剧名字。但是原本这儿做了个判断,如果是在搜索结果中就直接使用搜索时的关键词。

我把这个判断去掉了,因为他会造成几个问题:
1.每个搜索结果点开,下面的视频源都是一样的
2.在搜索中点开该番剧,与在追番中点开该番剧看到的视频源不同
3.番剧名与下面的视频不匹配

举个例子,搜索“摇曳百合”,下面也会出现“摇曳露营Δ”,点开“摇曳露营Δ”,下面出现的还是摇曳百合相关的视频源
1.5.8 release:
Screenshot_20250220-113055_Kazumi

Screenshot_20250220-113058_Kazumi

修复后:
Screenshot_20250220-113436_Kazumi

@Predidit
Copy link
Owner

这部分代码在 #53 中被提及

是为了解决译名不同导致的搜索问题

Bangumi API 的搜索很好用,他们的数据库中应该已经预先关联了大量的别名

但是视频来源并不是这样,所以我们需要使用者输入的模糊关键词

@2894638479
Copy link
Contributor Author

这样的话,搜索界面使用Bangumi API的意义就没有了(每个卡片点进去都一样,而且如果Bangumi API搜索没结果,也没法看到视频源的搜索结果),应该直接在各个视频源搜索。但是考虑追番等功能又必须使用bangumi API,但是这样点追番也没意义,因为"使用者输入的模糊关键词“在点了追番之后,在追番列表里打开就不再起作用了,我认为应该用自定义的BangumiItem来解决这个问题,用户在已有BangumiItem的基础上修改name和nameCn,然后可以对这个BangumiItem进行追番等操作

而且从使用者的角度,同样的卡片点开后是不同的结果,用起来也很迷惑,我认为应该统一起来

@Predidit
Copy link
Owner

我在尝试允许在检索没有结果时从 Bangumi 数据库中的别名重新检索

这将解决这个 PR 中的问题和 #53

我们现在应该不能合并这个 PR

@2894638479
Copy link
Contributor Author

好的,确实你这样更方便一些

@Predidit
Copy link
Owner

我们仍然可以合并这个PR

因为新的实现中一样包含这个PR中的修改

@Predidit Predidit merged commit d7d6bf2 into Predidit:main Feb 20, 2025
6 checks passed
ErBWs pushed a commit to ErBWs/Kazumi that referenced this pull request Feb 23, 2025
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