Safari/Chrome Connector on macOS does not work for economist.com

Backer · August 22, 2017

The current Safari/Chrome Connector on macOS does not work for economist.com.

Would anyone please have a look at this and fix it?

bwiernik · August 22, 2017

Can you give a specific URL? And exactly does “doesn’t work” mean here?

Backer · August 22, 2017

https://www.economist.com/news/international/21726276-last-week-newspaper-said-alphabets-boss-should-write-detailed-ringing-rebuttal

leads to the following options (logged in as a subscriber):

Select which items you'd like to add to your library:
# Solar eclipsesOnce portents of terror, eclipses are now an excuse for a partyThis month’s eclipse is a cultural as much as a scientific event
# Least-improvedEfficiency eludes the construction industry
# CharlemagneRussian culture wars
# The Economist explainsWhat are initial coin offerings?

Looks like Zotero can not identify the specific article I opened (the options above are other articles from the homepage).

Zotero Connector 5.0.17
Safari 10.1.2
macOS Sierra 10.12.6

bwiernik · August 22, 2017

Okay, so it's missing the metadata for the main article on the page and is picking up the More Reading links at the bottom. It looks like The Economist changed their page source layout, so the translator will need to be fixed.

In the mean time, if you right-click on the button (Chrome) or long-click it (Safari), you can choose the "Embedded Metadata" option, which will give you more correct data.

Backer · August 22, 2017

Thank you, I tried the "Embedded Metadata" option and it is a valid workaround.
Would appreciate a fix for the dedicated function in the translator very much.

jkr · November 15, 2017

I am also experiencing this problem.

Zotero Connector 5.0.26
Safari 11.0.1
MacOS 10.13.1

I am unable to use the "metadata" option. When I right-click on the Connector icon, I do get option shown, but when I click on it, it doesn't get selected, the options remain visible and neither of them can be selected.

bwiernik · November 15, 2017

In Safari, you need to click-and-hold the Connector icon to second the secondary options, rather than right-click (Safari doesn't allow right-click actions for extensions).

@adamsmith Is this one on your radar?

jkr · November 15, 2017

> In Safari, you need to click-and-hold the Connector icon to second the secondary option

Sorry, that’s of course what I did. Right-clicking does indeed only yield in a regular click. Thanks for pointing that out!

adamsmith · November 15, 2017

ah, no, sorry it wasn't. Ticket here:
https://github.com/zotero/translators/issues/1477
should be pretty quick.

jkr · November 15, 2017

Awesome, thx

adamsmith · November 18, 2017

The Economist issue is now fixed, thanks to @zuphilip

kwilczek · November 27, 2017

I'm still seeing the original issue in both Firefox and Chrome. I have made sure Zotero and the translators are up to date. Is there something else I need to do?

adamsmith · November 27, 2017

That is https://www.economist.com/news/international/21726276-last-week-newspaper-said-alphabets-boss-should-write-detailed-ringing-rebuttal still gives you the option to save multiple items rather than the article itself?

kwilczek · November 27, 2017

Yes, I still see the folder icon and, when clicking on it, none of the articles listed are the one I'm viewing and want to add.

adamsmith · November 27, 2017

You have updated translators from the connector preferences, I assume. Could you go to the zotero data directory https://www.zotero.org/support/zotero_data, in the "translators" folder find "The Economist.js" open it with a text editor and check the "lastUpdated" time, please?

kwilczek · November 27, 2017

I've clicked the Update button 3 times and restarted Zotero and the browsers.

Date modified for The Economist translator is 11/21/2017 at 10:41am.

Thanks for helping with this.

adamsmith · November 27, 2017

Thanks! I didn't mean the date modified of the file -- I mean when you actually open the file (use Notepad or TextEdit) the bit at the top. It'd be something like "lastUpdated": "2017-11-16 21:55:19"

kwilczek · November 27, 2017

lastUpdated": "2017-11-18 15:55:00

adamsmith · November 27, 2017

yeah, that's right. And a little further below that you see this line?
var m = url.match(/economist\.com\/[^\/]*\/[^\/]*\/\d+/);
right?
I don't quite understand why that could detect as a multiple, then.
And just to double-check -- you're looking at
https://www.economist.com/news/international/21726276-last-week-newspaper-said-alphabets-boss-should-write-detailed-ringing-rebuttal
or are you on a different URL? If so, which?

kwilczek · November 27, 2017

Yes, I see that line in the file.

I see the folder on the page you linked to and any other Economist article I click on. My links have Harvard's proxy in them in case that's causing an issue.

https://www-economist-com.ezp-prod1.hul.harvard.edu/news/international/21726276-last-week-newspaper-said-alphabets-boss-should-write-detailed-ringing-rebuttal

I just tried the link without the proxy on a laptop not hard-wired to Harvard's network and the webpage icon does display instead of the folder icon. However the webpage icon isn't active by which I mean clicking on it does nothing. I can successfully add items from the New York Time or CNN on the same computer.

adamsmith · November 27, 2017

interesting -- we'll need to look at that. I'm pretty sure it's the proxy, though it shouldn't be.

kwilczek · November 27, 2017

Thanks! The Economist and Zotero is very heavily used here so it would be great to figure out why they aren't working together for us.

adamsmith · November 28, 2017

OK this should be fixed now once you update your translators. It was indeed the proxy.

@adomasven -- the url passed on to translators (in detectWeb and doWeb) includes the proxy. Is that intended? Do you know if that has changed? I don't recall that.

bwiernik · November 28, 2017

Not sure if relevant--Aurimas once wrote a pull request to de-proxify URLs, but I don't believe that it was ever merged.

kwilczek · November 28, 2017

It works great now. Thank you for your help.

adomasven · November 28, 2017

@adamsmith shouldn't happen, will look into it this week.