[Solved] Science Direct Translator broken on Journal Page

pedrom34

Hi,

Science Direct translator is broken for multiple entries when on a journal issue page.
For example:
https://www.sciencedirect.com/journal/le-pharmacien-hospitalier-et-clinicien/vol/52/issue/4
Zotero Connector sees a web page.

I'm on last version of Firefox (57.0.4) , Zotero Connector (5.0.31) and Zotero (5.0.33). Translator for science direct is up to date as well, and is working on other scenarii (search results, article page, book, chapter, etc.).

Third part cookies are allowed.

adamsmith

ticket here: https://github.com/zotero/translators/issues/1510
we've just not seen that page design before (and Elsevier says it's new). Thanks for reporting.

pedrom34

Thanks to you!

pedrom34

hey @adamsmith

I saw on github that the issue is closed. Thanks for the fix, but it doesn't work when not using https.
For instance:
https://www.sciencedirect.com/journal/le-pharmacien-hospitalier-et-clinicien/vol/52/issue/4 > OK
http://www.sciencedirect.com/journal/le-pharmacien-hospitalier-et-clinicien/vol/52/issue/4 > KO

Doesn't really bother me, just to let you know.

PS: I didn't know if I should comment here or on GitHub?

adamsmith

works for me and I don't see any reason it shouldn't. Do you not get a folder icon or does importing produce an error?

pedrom34

No folder icon when http.
Folder icon when https.

adamsmith

Have you tried updating the translator from the connector preferences? Reload the page? And are you going through a proxy or are those the URLs as you see them?

I can't replicate this on Firefox 57/ Zotero connector 5.0.31 on two different computers.

pedrom34

Fudge!
I did update the translator from both Zotero and Zotero Connector, I did reload the page, I wasn't using a proxy, and still it didn't work.
Now it does. Sorry that I made you test...

Have a nice day.

adamsmith

no, thanks for testing. Especially with this type of thing it's very easy to miss a specific scenario in testing, so good to have a 2nd set of eyes.