Skip to content
Please note that GitHub no longer supports your web browser.

We recommend upgrading to the latest Google Chrome or Firefox.

Learn more
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

Emerald Journals #2036

Closed
adam3smith opened this issue Oct 23, 2019 · 4 comments · Fixed by #2037 or #2046

Comments

@adam3smith
Copy link
Collaborator

@adam3smith adam3smith commented Oct 23, 2019

They changed their site -- it think we used to cover this with one of the default translators, but doesn't look like that's currently working. EM imports nicely but without PDF

https://www.emerald.com/insight/content/doi/10.1108/IJRDM-08-2017-0180/full/html

@adam3smith

This comment has been minimized.

Copy link
Collaborator Author

@adam3smith adam3smith commented Oct 23, 2019

Announcement: https://www.emeraldgrouppublishing.com/about/news/?id=8571 They've switched away from the Atypon platform we did support

adam3smith added a commit to adam3smith/translators that referenced this issue Oct 24, 2019
@zuphilip

This comment has been minimized.

Copy link
Collaborator

@zuphilip zuphilip commented Oct 26, 2019

Okay. I also see the old things in our code:

$ grep -i emerald *.js
Atypon Journals.js:                                     } else if (url.includes('www.emeraldinsight.com')) {
Atypon Journals.js:                                             //for Emerald, improve author's lastname and firstname
Atypon Journals.js:                             //for Emerald, get rid of the "null" that they add at the end of every title:
Atypon Journals.js:                             if (url.includes("www.emeraldinsight.com")){
Atypon Journals.js:             "url": "http://www.emeraldinsight.com/toc/sajgbr/2/2",
Atypon Journals.js:             "url": "http://www.emeraldinsight.com/doi/full/10.1108/SAJGBR-10-2012-0120",
Atypon Journals.js:                             "libraryCatalog": "emeraldinsight.com (Atypon)",
Atypon Journals.js:                             "url": "http://www.emeraldinsight.com/doi/full/10.1108/SAJGBR-10-2012-0120",
Baidu Scholar.js:                               "url": "http://www.emeraldinsight.com/doi/full/10.1108/07419051111154758",

(I already filtered out a false positive result in Rock, Paper, Shotgun.js: from the result above.)

We should clean up this code then also as Emerald is not using Atypon anymore, but first I will continue the review for your new translator.

@zuphilip

This comment has been minimized.

Copy link
Collaborator

@zuphilip zuphilip commented Oct 26, 2019

Note: your translator uses RIS and not EM now, but it looks good.

zuphilip added a commit that referenced this issue Oct 29, 2019
Closes #2036
@zuphilip

This comment has been minimized.

Copy link
Collaborator

@zuphilip zuphilip commented Oct 29, 2019

Reopening, because cleaning up Atypon translator is still a todo.

@zuphilip zuphilip reopened this Oct 29, 2019
zuphilip added a commit to zuphilip/translators that referenced this issue Nov 3, 2019
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.