suzanne.soy/zotero
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fixes #795, Importing certain pages via favicon drag results in multiple snapshots

Browse Source
This commit is contained in:
Dan Stillman 2007-10-21 07:26:05 +00:00
parent 5b003ff641
commit 33e5b867eb
1 changed files with 8 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
chrome/content/zotero/xpcom/attachments.js
View File

@ -195,14 +195,21 @@ Zotero.Attachments = new function(){
// get the charset and title and index the document)
if (Zotero.MIME.hasNativeHandler(mimeType, ext)){
var browser = Zotero.Browser.createHiddenBrowser();
var imported = false;
var onpageshow = function() {
// pageshow can be triggered multiple times on some pages,
// so make sure we only import once
// (https://www.zotero.org/trac/ticket/795)
if (imported) {
return;
}
var callback = function () {
browser.removeEventListener("pageshow", onpageshow, false);
Zotero.Browser.deleteHiddenBrowser(browser);
};
Zotero.Attachments.importFromDocument(browser.contentDocument,
sourceItemID, forceTitle, parentCollectionIDs, callback);
imported = true;
};
browser.addEventListener("pageshow", onpageshow, false);
browser.loadURI(url);