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.
Note: add the ACM code to the 2.js (inside translator.zip)