fixes for ACM and IEEE translators... please open trac tickets
I've fixed two bugs in the ACM and IEEE translators, one major and one minor.
ACM: Wasn't grabbing pdfs at all. Here's the fix:
var linkText = typeLink.textContent.toLowerCase();
linkText = linkText.replace(/(\t|\n| )/g, ""); // add this line
IEEE: Has an error prone way to grab pdf links. Uses hardcoded array indexes so if the url is different at all (for example, when using a proxy) pdf grabbing doesn't work. Here's the fix:
//pdfurlElmt.href = pdfurlElmt.href.substr(0,32) + 'PDF/getPDF' + pdfurlElmt.href.substr(38);
pdfurlElmt.href = pdfurlElmt.href.replace("/stamp.jsp", "PDF/getPDF.jsp"); // replace previous line with this
Please consider integrating these fixes into the translators.
ACM: Wasn't grabbing pdfs at all. Here's the fix:
var linkText = typeLink.textContent.toLowerCase();
linkText = linkText.replace(/(\t|\n| )/g, ""); // add this line
IEEE: Has an error prone way to grab pdf links. Uses hardcoded array indexes so if the url is different at all (for example, when using a proxy) pdf grabbing doesn't work. Here's the fix:
//pdfurlElmt.href = pdfurlElmt.href.substr(0,32) + 'PDF/getPDF' + pdfurlElmt.href.substr(38);
pdfurlElmt.href = pdfurlElmt.href.replace("/stamp.jsp", "PDF/getPDF.jsp"); // replace previous line with this
Please consider integrating these fixes into the translators.
This is an old discussion that has not been active in a long time. Instead of commenting here, you should start a new discussion. If you think the content of this discussion is still relevant, you can link to it from your new discussion.
Note: add the ACM code to the 2.js (inside translator.zip)