Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#622 closed defect (fixed)

Zotero Overides Google Scholar OpenURL Link Resolver Preferences

Reported by: tjowens Owned by: simon
Priority: major Milestone: 1.0.0
Component: ingester Version: 1.0
Keywords: Cc: cartesian, stakats

Description

When you ingest information from Google scholar Zotero changes the preferences in Google scholar. Zotero sets Google Scholar's OpenURL settings to OhioLink, OCLC and GMU. This bumps out whatever local resolver the user had specified in Google Scholar was using.

Attachments (4)

google scholar translator code.txt (4.2 KB) - added by cartesian 9 years ago.
Edits to translator by cartesian (SR)
Google scholar translator code v2.txt (4.3 KB) - added by cartesian 9 years ago.
This code replaces my earlier fix and fixes a bug that arises when Google interface language wasn't set to English
Google scholar translator code v3.txt (4.2 KB) - added by cartesian 9 years ago.
Fixed typo in "itemGrabLink"; this fixes my random bonus bug #1
google scholar translator v4.sql (4.4 KB) - added by cartesian 9 years ago.
Places Zotero.wait()s and done()s better. Also handles additional language prefs in the url. Does NOT fix #750. This will be my last contribution for a while!

Download all attachments as: .zip

Change History (13)

comment:1 Changed 9 years ago by stakats

  • Priority changed from minor to major

I suspect this problem arises from our GS scholar translator setting the GS cookie to include the Endnote (RIS) citation links. We're probably also overwriting the OpenURL parameter.

Changed 9 years ago by cartesian

Edits to translator by cartesian (SR)

comment:2 Changed 9 years ago by cartesian

  • Cc cartesian added

I've edited the translator to solve a related problem with EZProxy (couldn't set preferences with cookie). This version sets preferences to show EndNote links via URL, instead of cookie.

Bonus random bugs for this translator:
1) Do a google scholar search for "url.replace". This page crashes both the rewritten and existing translators. Don't know why.
2) Also there seem to be timing issues: scaffold often shows "translation successful" a while before it's done. This bug is also carried over from existing translator.

comment:3 Changed 9 years ago by dstillman

  • Cc stakats added

Changed 9 years ago by cartesian

This code replaces my earlier fix and fixes a bug that arises when Google interface language wasn't set to English

comment:4 Changed 9 years ago by cartesian

Bonus random bug 3) I get a busy hourglass on my Firefox when I mouse over various parts of the browser at any time after this translator runs the Zotero.Utilities.loadDocument() code, even after items are successfully added to my library. The hourglass goes away if I hit refresh for the tab that was scraped. I didn't alter that code except to change the URL it uses; this bug is also a holdover from the current translator.

comment:5 follow-up: Changed 9 years ago by dstillman

I think that may be related to the hidden browser, either in translate code or my attachments code. (Is the translator saving a PDF? Another type of attachment? Does it happen if the PDF-saving pref is disabled?)

Changed 9 years ago by cartesian

Fixed typo in "itemGrabLink"; this fixes my random bonus bug #1

comment:6 in reply to: ↑ 5 ; follow-up: Changed 9 years ago by cartesian

Replying to dstillman:

I think that may be related to the hidden browser, either in translate code or my attachments code. (Is the translator saving a PDF? Another type of attachment? Does it happen if the PDF-saving pref is disabled?)

I don't think it's related to attachments - it's the same even if the pdf save pref is off in Options. I think it might be because the translator doesn't wait for the call to loadDocument() to finish properly. But I don't understand enough about the Zotero API to fix it myself.

comment:7 in reply to: ↑ 6 Changed 9 years ago by dstillman

Replying to cartesian:

Replying to dstillman:

I think that may be related to the hidden browser, either in translate code or my attachments code. (Is the translator saving a PDF? Another type of attachment? Does it happen if the PDF-saving pref is disabled?)

I don't think it's related to attachments - it's the same even if the pdf save pref is off in Options. I think it might be because the translator doesn't wait for the call to loadDocument() to finish properly. But I don't understand enough about the Zotero API to fix it myself.

A link item is still an attachment (and Zotero still loads the page in a hidden browser to index the page content). If I just return at the beginning of Zotero.Attachments.importFromURL(), it doesn't happen.

Created #750 for this problem. I'll see what I can do.

Changed 9 years ago by cartesian

Places Zotero.wait()s and done()s better. Also handles additional language prefs in the url. Does NOT fix #750. This will be my last contribution for a while!

comment:8 Changed 9 years ago by stakats

  • Resolution set to fixed
  • Status changed from new to closed

(In [1741]) Closes #622 by updating Google Scholar translator with cartesian's code.

comment:9 Changed 9 years ago by dstillman

Pushed to repo.

Note: See TracTickets for help on using tickets.