Cannot save from SAE International website

vihar · August 8, 2008

I am having trouble saving citations from the SAE website (http://www.sae.org/). Hovering the mouse pointer over the Firefox addressbar icon shows the correct translator being used (SAE International). But when clicked, it gives the generic 'Could not save item' error.

I have tried all the seven steps described to troubleshoot translator issues form the relevant resource on the Zotero website at http://www.zotero.org/documentation/troubleshooting_translator_issues

I am using Firefox 3.0.1 with Zotero 1.0.7, which are both currently the latest respective versions.

Please help.

Regards,
ViHAR.

vihar · August 8, 2008

... to add to the above information, normal SAE books are saved just fine. I am having trouble specifically with papers.

Acrymble · August 9, 2008

Which article in particular did you have trouble with? I had troubles too, but with a book. So I can't be sure I've fixed your problem.

Please post a link.

briantipton · August 11, 2008

I, too, am having the same problem. Here is a sample paper link that would not save: http://www.sae.org/technical/papers/952053

vihar · February 9, 2009

Submitted an error report via Zotero. Report ID: 1875180056.

vihar · November 4, 2009

http://www.sae.org/technical/papers/2000-01-0488

ryan.gehmlich · December 8, 2009

I am curious if anybody has done anything further on this? I am having the same problem with SAE International papers and have to enter them all in by hand (it's one of my major sources so it's annoying!) I wouldn't know how to start editing the js file to fix this... here is where the error log says it fails:

(2)(+0000000): Translate: Translation using SAE International failed:
message => doc.evaluate("//p[strong[contains(text(), \"ISBN\")]]", doc, null, XPathResult.ANY_TYPE, null).iterateNext() is null
fileName => chrome://zotero/content/xpcom/translate.js
lineNumber => 854
stack => ([object XPCNativeWrapper])@chrome://zotero/content/xpcom/translate.js:854

name => TypeError
url => http://www.sae.org/technical/papers/970866
downloadAssociatedFiles => false
automaticSnapshots => true

russelldub · December 30, 2009

Hi,

I've found the problem and have a solution. The following line of SAE International.js:

if (type == "paper") {

should read:

if (type == "conferencePaper") {

It's unclear from the docs what the proper way to submit this fix is, so I'm posting it here in the hopes that devs will find and fix, and in the meantime the above posters can fix their local copies themselves. (SAE International.js is under your FIrefox profile directory in zotero\translators\)

Cheers.

dstillman · December 30, 2009

Fixed. Thanks.

For everyone else, your copy of Zotero should auto-update within 24 hours, or you can update manually by clicking Update Now in the General pane of the Zotero prefs.

EndingPop · March 18, 2010

After upgrading to Zotero 2.0.2, I'm now unable to read at least one paper on SAE's site. Using Firefox 3.5.8.

http://www.sae.org/technical/papers/2010-01-0907

I don't know if it's relevant that this paper is not published yet.

EndingPop · March 23, 2010

I'm also seeing this same problem on another computer using Zotero 2.0.2 and Firefox 3.6.2. This hasn't made it onto the known translator issues page. Should I be contacting someone about this?

migit · October 26, 2010

I have the same issue with the new version. ALL SAE paper don't work.

(3)(+0000195): Translate: Could not find a result using CrossRef:
message => refXML is undefined
fileName => chrome://zotero/content/xpcom/translate.js
lineNumber => 948
stack => processCrossRef("\n<doi_records xmlns:xsi=\"http://www.w3.org/2001/XMLSchema-insta
..............
nixref")@chrome://zotero/content/xpcom/translate.js:1024

name => TypeError
url => undefined
downloadAssociatedFiles => false
automaticSnapshots => true

Can anybody help, or give me a tip where in the code I could have a look to fix that?

ajlyon · October 26, 2010

The SAE translator itself is broken, but that's not a major issue since they all have DOIs assigned. Unfortunately, Zotero doesn't yet load bibliographic data for documents of type report-paper correctly -- this will have to be fixed in the translator.

The DOI route will not let us save the report abstracts, so if that is important, please let me know. It would also be possible to create a hybrid translator that pulled most of its data from the DOI lookup service (CrossRef) and added addition data like abstracts. It also won't do automatic PDF saving, but since the PDFs at SAE are paid downloads, that might not be possible anyway.

ajlyon · October 26, 2010

The key place in the code is CrossRef.js in the translators directory of your Zotero data directory; we need to add support for a new type. The CrossRef output for an SAE paper looks like this: http://www.crossref.org/openurl/?pid=zter:zter321&url_ver=Z39.88-2004&ctx_ver=Z39.88-2004&rft_id=info:doi/10.4271/2010-01-0907&rft_val_fmt=info:ofi/fmt:kev:mtx:journal&rft.genre=article&noredirect=true&format=unixref

The currently broken code for the SAE translator is in "SAE International.js" in the same directory.

migit · October 27, 2010

Thanks for the quick answer.

For my self the CrossRef output is already all I need. The abstract would be just a bonus.

I will have a look on it later on, but I'm not really involved in Zotero Dev. Is there anybody around who could do that quickly?

migit · November 1, 2010

OK I had a go and realised, that I'm to stupid for that.

Why does CrossRef.js not appear in Scaffold?

I started of in CrossRef.js with...
} else if(xml.doi_record[0].crossref.report-paper.length()) {
var item = new Zotero.Item("journalArticle");
var itemXML = xml.doi_record.crossref.report-paper;
var refXML = itemXML.report-paper_article;
var metadataXML = itemXML.report-paper_metadata;

//item.DOI = refXML.doi_data.doi.toString();
item.DOI = itemXML.report-paper_metadata.DOI.toString();
item.publicationTitle = itemXML.report-paper_metadata.full_title.toString();
item.volume = itemXML.journal_issue.journal_volume.volume.toString();
item.issue = itemXML.journal_issue.issue.toString();
}

But I even can't see if it is actually executed.

HELP :)

ajlyon · November 1, 2010

Replace your CrossRef.js with this: http://github.com/ajlyon/zotero-bits/raw/master/CrossRef.js
Then try again. You may have to restart Firefox for it to take effect, I don't remember.

migit · November 2, 2010

It works. Thank you, you are my hero.

I still had some issues, for those that have the same problem:

Issue #1: E4X dowsn't seem to like the "-" in "report-paper", but I fixed that by adding this in line 30:
xmlOutput = xmlOutput.replace(/report[^>]*paper/g, "reportpaper");

Issue #2:
in the added code "item" should be "Item" (capital i) and rNum made problems so the added code looks like this now:

} else if(xml.doi_record[0].crossref.reportpaper.length()) {
// Report Paper
// Example: doi: 10.4271/2010-01-0907
// http://www.crossref.org/openurl/?pid=zter:zter321&url_ver=Z39.88-2004&ctx_ver=Z39.88-2004&rft_id=info%3Adoi/10.4271/2010-01-0907&rft_val_fmt=info%3Aofi%2Ffmt%3Akev%3Amtx%3Ajournal&rft.genre=article&noredirect=true&format=unixref
var itemXML = xml.doi_record[0].crossref.reportpaper;
var refXML = itemXML.reportpaper_metadata;
var metadataXML = itemXML.reportpaper_metadata;
var item = new Zotero.Item("report");
if (metadataXML.publisher_item.item_number.toString() )
item.reportNumber = metadataXML.publisher_item.item_number.toString();
item.institution = metadataXML.publisher.publisher_name.toString();
item.place = metadataXML.publisher.publisher_place.toString();

ajlyon · November 2, 2010

Glad it works for you. As you can probably tell, I didn't actually test the code. :)

I'll look into adding this to the CrossRef.js that ships with Zotero. Do you know of any other publishers that assign DOIs to items of type report-paper? I'd like to test it with non-SAE DOIs if possible.

migit · November 2, 2010

I actually found this button at the top only because of that problem :)

So far I haven't seen any report-paper types. But it looks like I am gonna use Zotero for the next 3 years, so I can let you know if I find something.

I would say its still worth to ship it with Zotero. SAE is quite a big database for automotive and aerospace. Thing is I've changed my CrossRef.js already to have the the items as Journal Article. I don't know how the mass would like it best.

ajlyon · November 2, 2010

I've updated the translator at the address above. It should now work without any modifications from you-- I opted for a different approach to handling the hyphenated element names (thanks to http://wso2.org/project/mashup/0.2/docs/e4xquickstart.html).

Since I'm confident that this will have no negative impact on DOIs for items not of type report-paper, this can certainly go into the Zotero trunk even without non-SAE examples. I was mainly interested in further refining processing of report-paper from the start, if possible.

russelldub · March 7, 2011

This is broken for me again.

I get the following in my error console:

DOI Translator: could not find DOI

I poked around a bit, but could not find a solution. Any help is appreciated.

russelldub · March 7, 2011

Alright. After way too much wasted time on this, I have the following solution:

CrossRef from here:
https://github.com/ajlyon/zotero-bits/raw/85c2196e2e4f66bb8bececb4392311f4c726a899/CrossRef.js

Edited replacing
} else if(xml.doi_record[0].crossref["report-paper"]length()) {
with
} else if(xml.doi_record[0].crossref["report-paper"].length()) {

(Note the dot.) Can this fix (including the rest of the report-paper type block) be incorporated into the official CrossRef.js? Also, It'd be nice if the SAE specific translator were fixed.

ajlyon · March 8, 2011

Sorry for the typo, and for accidentally dropping report support in a subsequent edit of the translator.

This is now fixed on Github and in the Zotero trunk.

russelldub · March 8, 2011

Thanks for making the changes.

Maybe some day down the line I'll get the motivation/time to fix the SAE translator myself. Or the SAE site admins will get smart and allow exporting of references...