Opened 10 years ago

Closed 10 years ago

#411 closed defect (wontfix)

"saving item" message hangs when auto-capturing from amazon.com

Reported by: kkraus Owned by: stakats
Priority: critical Milestone: 1.0 Beta 3
Component: ingester Version: 1.0
Keywords: Cc: simon, dstillman

Description

I've been able to reproduce a bug (on a MAC OS X) reported in the Zotero forums. When clicking on a book icon that shows up in the address bar at Amazon.com, I get a "Saving item" message in the lower right corner of my browser window that doesn't go away. It hovers in front of any new tabs or windows that I open and remains there until Firefox is restarted. Moreover, the auto-captured item doesn't appear in my Zotero Library unless I close the Zotero pane and reopen it in a new *window* (not tab). Here is the relevant error reported in my error console:

Error: uncaught exception: Permission denied to call method Location.toString

Just prior to loading amazon.com, I also got a string of the following error messages:

Error: uncaught exception: Permission denied to get property Window.writeDebug

I've reproduced the "Saving item" bug several times by closing and then restarting my browser.

Attachments (1)

amazon-compressed.sql (5.1 KB) - added by dstillman 10 years ago.
compressed version of Sean's translator

Download all attachments as: .zip

Change History (5)

comment:1 Changed 10 years ago by dstillman

  • Cc simon dstillman added
  • Owner changed from simon to stakats
  • Priority changed from major to critical

The problem reported on the forums (at least by Peter) was, I believe, fixed in r845 a few weeks ago. However, there does seem to be a problem in 1.0.0b2.r2 with the new Amazon translator.

Sean, does the new translator rely on some code that's not present in r2? I don't get this in the bundled 10/02 version, but I get it in your 11/21 version with r2. It works fine with a dev build.

comment:2 Changed 10 years ago by dstillman

Also, the errors in the description are just errors from Amazon, I believe, not from us. (We don't have a writeDebug() call, for example.)

I don't see any errors myself, but the translator is hanging on "running handler 0 for itemDone".

comment:3 Changed 10 years ago by dstillman

Yes, the new Amazon translator seems to be dependent on Simon's modified Ingester.HTTP.doGet(), which isn't available in r2. We'll have to have a discussion about Z-version dependencies for translators, but in the meantime, can the translator be changed to work with the released version?

Changed 10 years ago by dstillman

compressed version of Sean's translator

comment:4 Changed 10 years ago by stakats

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

Dan has reverted to the original r2 translator. We'll roll out the updated Amazon translator in r3.

Note: See TracTickets for help on using tickets.