l0o0 / translators_CN

Zotero translator中文网页抓取翻译器🎉This is Zotero translators for Chinese Sites(beta), not the official Zotero repo
GNU Affero General Public License v3.0
4.03k stars 523 forks source link

豆瓣电影标题为多语言失败 #300

Closed aGolduck closed 7 months ago

aGolduck commented 7 months ago

你遇到了什么问题? [必填]

发生问题的链接 [必填] https://movie.douban.com/subject/1308843/ https://movie.douban.com/subject/1450031/ https://movie.douban.com/subject/3183628/

问题描述 [必填] 如题

你的预期结果 正确剪辑

浏览器

自查清单

jiaojiaodubai commented 7 months ago

修复于 52c442c97cbeaf5bc982b24c1a3ade9cbace8086。

aGolduck commented 7 months ago

建议是把这些 bad case 加到测试里,感觉快速迭代很容易挂一漏万。如果原网页不频烦更新,这些 test cases 还是能在一段时间内有效。

jiaojiaodubai commented 7 months ago

感谢你的建议。test cases主要展示的是对各种条目的支持,因为这些条目的抓取方法和对字段的要求不太一样。而bad case我会以注释形式放到相关代码处,我认为这样更有指向性、解释性。