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

Fix passing file: URI to Zotero.File.getContentsAsync()

Browse Source
This commit is contained in:
Dan Stillman 2017-01-13 06:48:02 -05:00
parent f44264cd4d
commit f2440eb826
2 changed files with 8 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
chrome/content/zotero/xpcom/file.js
View File

@ -273,7 +273,7 @@ Zotero.File = new function(){
if (source instanceof Components.interfaces.nsIFile) { if (source instanceof Components.interfaces.nsIFile) {
source = source.path; source = source.path;
} }
else if (source.startsWith('^file:')) { else if (source.startsWith('file:')) {
source = OS.Path.fromFileURI(source); source = OS.Path.fromFileURI(source);
} }
var options = { var options = {

7
test/tests/fileTest.js
View File

@ -48,6 +48,13 @@ describe("Zotero.File", function () {
assert.lengthOf(contents, 6); assert.lengthOf(contents, 6);
assert.equal(contents, "Zotero"); assert.equal(contents, "Zotero");
}); });
it("should get a file from a file: URI", function* () {
var contents = yield Zotero.File.getContentsAsync(
OS.Path.toFileURI(OS.Path.join(getTestDataDirectory().path, "test.txt"))
);
assert.isTrue(contents.startsWith('Zotero'));
});
}) })
describe("#getBinaryContentsAsync()", function () { describe("#getBinaryContentsAsync()", function () {