Skip to content
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

Fixes S #1343

Merged
merged 26 commits into from Jun 26, 2017

Conversation

@zuphilip
Copy link
Collaborator

commented Jun 25, 2017

No description provided.

zuphilip and others added some commits Jun 24, 2017

Extend BMC translator to cover also Springer Open
Moreover, now the Springer Science+Business Media.js translator
is not anymore needed because all journals moved to either
BioMed Central or Springer Open.
Update SAGE Knowledge.js
The RIS data they provide is very problematic in
various aspects. Therefore, we have to correct this
as good as possible in the translator.
@adam3smith
Copy link
Collaborator

left a comment

A couple of small nits or questions

@@ -2,14 +2,14 @@
"translatorID": "d21dcd90-c997-4e14-8fe0-353b8e19a47a",
"label": "SAGE Knowledge",
"creator": "ProQuest",

This comment has been minimized.

Copy link
@adam3smith

adam3smith Jun 25, 2017

Collaborator

add your name here and in copyright

}
//Z.debug(citeLink)

ZU.doGet(citeLink, function(text) {
var re = /<textarea name="records".*?>([\s\S]*?)<\/textarea>/;

This comment has been minimized.

Copy link
@adam3smith

adam3smith Jun 25, 2017

Collaborator

I don't see that used anymore. Delete or comment out

.replace(/NV\s+-\s+1\n/, "")
.replace(/^(AU\s+-\s+.+?),? Ph\.? ?D\.?\n/mg, '$1\n');
.replace(/^AU\s+-\s+,\s+$/m, '')

This comment has been minimized.

Copy link
@adam3smith

adam3smith Jun 25, 2017

Collaborator

This seems like you're deleting the entire Author field or what am I missing?

This comment has been minimized.

Copy link
@zuphilip

zuphilip Jun 25, 2017

Author Collaborator

Yes, but this is an empty author field, i.e. something like

AU  -  , 

(Note the beginning ^ and end of line $ characters in the regexp.)

This comment has been minimized.

Copy link
@adam3smith

adam3smith Jun 25, 2017

Collaborator

ah didn't catch that these were just spaces and commas. Makes sense then.

SSRN.js Outdated
"reportNumber": "ID 1450387",
"reportType": "SSRN Scholarly Paper",
"shortTitle": "Who Doesn't Support the Genocide Convention?",
"url": "https://papers.ssrn.com/abstract=1450387",
"attachments": [
{
"title": "Full Text PDF",

This comment has been minimized.

Copy link
@adam3smith

adam3smith Jun 25, 2017

Collaborator

curious: did you test if we are now able to download the PDF? (that didn't use to work)

This comment has been minimized.

Copy link
@zuphilip

zuphilip Jun 25, 2017

Author Collaborator

I tested this now, but PDFs are still not working. There are various problems why this does not work (and might not work easily):

This comment has been minimized.

Copy link
@adam3smith

adam3smith Jun 25, 2017

Collaborator

we should remove this, then -- otherwise it looks like a bug or something people should expect Zotero to do.

@zuphilip

This comment has been minimized.

Copy link
Collaborator Author

commented Jun 25, 2017

Please have a look at the new version.

adam3smith and others added some commits Jun 26, 2017

@zuphilip

This comment has been minimized.

Copy link
Collaborator Author

commented Jun 26, 2017

I removed the non-working PDF attachment in SSRN as you suggested. Please have a look.

@adam3smith adam3smith merged commit c250250 into zotero:master Jun 26, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@adam3smith

This comment has been minimized.

Copy link
Collaborator

commented Jun 26, 2017

Thanks!

@zuphilip zuphilip deleted the zuphilip:fixes-ST branch Jun 26, 2017

zuphilip added a commit to zuphilip/translators that referenced this pull request Mar 28, 2018

Fixes S (zotero#1343)
* Update SAE Papers.js
Closes zotero#1342.
* Update Safari Books Online.js
* Update tests in SAGE Journals.js
* Update sbn.it.js
* Update Scholars Portal Journals.js
* Update SciELO.js
* Delete Science Links Japan.js
Closing at 2014-03-31 see https://web.archive.org/web/20140314103855/http://www.sciencelinks.jp/
* Update tests in Silverchair.js
* JACC is now covered by Highwire translator, e.g. http://www.onlinejacc.org/content/55/25/2858#Abstract
* JAPhA is now covered by Elsevier Health Journals translator, e.g. http://www.japha.org/article/S1544-3191(15)30515-X/fulltext
* Neuropsychiatry is now covered by Atypon translator, e.g. http://neuro.psychiatryonline.org/toc/jnp/24/2
TODO http://annals.org/aim/article/1358680/flexible-sigmoidoscopy-screening-reduced-colorectal-cancer-incidence-mortality-older-adults
* Update tests in SlideShare.js
* Update SORA.js
* Update Spiegel Online.js
* Update Springer Link.js
* Extend BMC translator to cover also Springer Open
The Springer Science+Business Media.js translator
is not longer needed because all journals moved to either
BioMed Central or Springer Open.
* Update tests in SSOAR.js
* Update SSRN.js
Delete non-working PDF attachment
* Update tests in Stanford Encyclopedia of Philosophy.js
* Update tests in Store norske leksikon.js
* Update Sueddeutsche.de.js
* Update Summon 2.js
* Update Schweizer Radio und Fernsehen SRF.js
* Update SAGE Knowledge.js
The RIS data they provide is very problematic in
various aspects. Therefore, we have to correct this
as good as possible in the translator.

zuphilip added a commit to zuphilip/translators that referenced this pull request Mar 28, 2018

Fixes S (zotero#1343)
* Update SAE Papers.js
Closes zotero#1342.
* Update Safari Books Online.js
* Update tests in SAGE Journals.js
* Update sbn.it.js
* Update Scholars Portal Journals.js
* Update SciELO.js
* Delete Science Links Japan.js
Closing at 2014-03-31 see https://web.archive.org/web/20140314103855/http://www.sciencelinks.jp/
* Update tests in Silverchair.js
* JACC is now covered by Highwire translator, e.g. http://www.onlinejacc.org/content/55/25/2858#Abstract
* JAPhA is now covered by Elsevier Health Journals translator, e.g. http://www.japha.org/article/S1544-3191(15)30515-X/fulltext
* Neuropsychiatry is now covered by Atypon translator, e.g. http://neuro.psychiatryonline.org/toc/jnp/24/2
TODO http://annals.org/aim/article/1358680/flexible-sigmoidoscopy-screening-reduced-colorectal-cancer-incidence-mortality-older-adults
* Update tests in SlideShare.js
* Update SORA.js
* Update Spiegel Online.js
* Update Springer Link.js
* Extend BMC translator to cover also Springer Open
The Springer Science+Business Media.js translator
is not longer needed because all journals moved to either
BioMed Central or Springer Open.
* Update tests in SSOAR.js
* Update SSRN.js
Delete non-working PDF attachment
* Update tests in Stanford Encyclopedia of Philosophy.js
* Update tests in Store norske leksikon.js
* Update Sueddeutsche.de.js
* Update Summon 2.js
* Update Schweizer Radio und Fernsehen SRF.js
* Update SAGE Knowledge.js
The RIS data they provide is very problematic in
various aspects. Therefore, we have to correct this
as good as possible in the translator.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.