search

zotero / translators

Zotero Translators
http://www.zotero.org/support/dev/translators
1.28k stars 756 forks source link

Embedded Metadata - long author lists don't work? #343

Closed adam3smith closed 12 years ago

adam3smith commented 12 years ago

http://www.pnas.org/content/108/52/20881.full%20Translation%20successful doesn't grab authors in either Highwire2.0 or Embedded Metadata @aurimasv - could you take a look, I can't figure out what's not working.

aurimasv commented 12 years ago

The meta tag for citation_author says "contents" instead of "content". Doesn't look like our RDF translator handles DC.Contributor either. I'm working on the RDF translator though to make it work with CrossRef's rdf+xml output, so I'll try to incorporate DC.Contributor into that.

adam3smith commented 12 years ago

ah thanks - I looked for sth. like that but just didn't see it, thanks for taking a look.

On Sun, May 6, 2012 at 4:13 PM, Aurimas Vinckevicius reply@reply.github.com wrote:

The meta tag for citation_author says "contents" instead of "content". Doesn't look like our RDF translator handles DC.Contributor either. I'm working on the RDF translator though to make it work with CrossRef's rdf+xml output, so I'll try to incorporate DC.Contributor into that.

Reply to this email directly or view it on GitHub: https://github.com/zotero/translators/issues/343#issuecomment-5541331

Sebastian Karcher Ph.D. Candidate Department of Political Science Northwestern University

aurimasv commented 12 years ago

Looks like it's a change (?) in Highwire 2.0 and it affects all Highwire 2.0 sites. We should probably notify them. Or we can conform to their error...

aurimasv commented 12 years ago

Sent them a message, but I don't expect the fix to go out any time soon. Should we just patch the translator to accept both content and contents? At least as a temporary fix.

adam3smith commented 12 years ago

@aurimasv - I don't see a reason not to do that. Could you? You know that one a lot better than me by now.

aurimasv commented 12 years ago

Sure, I'll submit a patch some time today.

simonster commented 12 years ago

Fixed by #350

aurimasv commented 12 years ago

HighWire fixed this today, but we might still want to recognize "contents" in case someone else makes the mistake.

adam3smith commented 12 years ago

I see no downside to allowing both content and contents.