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

Juris-M · Jan 14, 2017 · f2440eb · f2440eb
1 parent f44264c
commit f2440eb
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/chrome/content/zotero/xpcom/file.js b/chrome/content/zotero/xpcom/file.js
@@ -273,7 +273,7 @@ Zotero.File = new function(){
 		if (source instanceof Components.interfaces.nsIFile) {
 			source = source.path;
 		}
-		else if (source.startsWith('^file:')) {
+		else if (source.startsWith('file:')) {
 			source = OS.Path.fromFileURI(source);
 		}
 		var options = {

diff --git a/test/tests/fileTest.js b/test/tests/fileTest.js
@@ -48,6 +48,13 @@ describe("Zotero.File", function () {
 			assert.lengthOf(contents, 6);
 			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 () {