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

Use passed URL rather than doc.location in translator processDocuments

Browse Source 
Outside of page contexts (e.g., from an XHR with responseType =
'document'), document.location is null.
This commit is contained in:
Dan Stillman 2017-08-16 17:58:59 +02:00
parent d1de8b751a
commit da2dedf7af
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
chrome/content/zotero/xpcom/utilities_translate.js
View File

@ -279,7 +279,7 @@ Zotero.Utilities.Translate.prototype.processDocuments = function(urls, processor
} }
translate.incrementAsyncProcesses("Zotero.Utilities.Translate#processDocuments"); translate.incrementAsyncProcesses("Zotero.Utilities.Translate#processDocuments");
var hiddenBrowser = Zotero.HTTP.processDocuments(urls, function(doc) { var hiddenBrowser = Zotero.HTTP.processDocuments(urls, function (doc, url) {
if(!processor) return; if(!processor) return;
var newLoc = doc.location; var newLoc = doc.location;
@ -289,10 +289,10 @@ Zotero.Utilities.Translate.prototype.processDocuments = function(urls, processor
// DEBUG: Is there a better fix for this? // DEBUG: Is there a better fix for this?
|| Zotero.isServer) { || Zotero.isServer) {
// Cross-site; need to wrap // Cross-site; need to wrap
processor(translate._sandboxManager.wrap(doc), newLoc.toString()); processor(translate._sandboxManager.wrap(doc), url);
} else { } else {
// Not cross-site; no need to wrap // Not cross-site; no need to wrap
processor(doc, newLoc.toString()); processor(doc, url);
} }
}, },
function() { function() {