-
Notifications
You must be signed in to change notification settings - Fork 525
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
Update WePu.js #350
Update WePu.js #350
Conversation
@yfdyh000 新版已经上传,新旧两版都需要继续完善附件下载的代码。 |
@jiaojiaodubai |
是的,有些 DOI 还指向万方或万方医学,因此现在把中文 DOI 抓取写到 CNKI.js 是不够的,应该独立出来,#343 正是为解决这一问题而提出的。
最适合提交到 DOI Content Negotiation.js,但这里存在一点阻碍。 能搜索 DOI 的前提是写出 DOI 目标地址的 translator。如你所知,web translator 使用元数据中的 不过目前,在大多数中文用户都安装茉莉花且更新了本仓库转换器的前提下,他们可以正常使用魔法棒来搜索知网的某些 DOI。 |
发现在 #59 中提出个这个问题。 |
知网的应该没有问题,但是万方网页的数据是通过 protobuff 异步加载的,而 protobuff 需要依赖第三方库进行解析……直到现在仍是一个难点,我们将来可能需要将最少必需的有关代码迁移到它的 translator 中。 |
在本地运行 npm run lint -- "WeiPu.js" 没有发现问题,可能存在延迟。 |
@jiaojiaodubai |
原先获取 PDF 链接的请求中缺少一个参数,缺少该参数能否正确获取到有效的 PDF 下载链接仍有待验证 |
@doubanchan
@yfdyh000