replace special chars in filename before storing an image. #67

Author: martinsbalodis

extension/scripts/Scraper.js

@@ -60,7 +60,9 @@ Scraper.prototype = {
 
 		var parts = url.split("/");
 		var filename = parts[parts.length-1];
-		filename = filename.split("?")[0];
+		filename = filename.replace(/\?/g, "%3F");
+		filename = filename.replace(/=/g, "%3D");
+		filename = filename.replace(/&/g, "%26");
 		return filename;
 	},
 

tests/spec/ScraperSpec.js

@@ -202,8 +202,8 @@ describe("Scraper", function () {
 
 	it("should extract filename from image url with query string", function() {
 
-		var image = Scraper.prototype.getFileFilename("http://example.com/image.jpg?123");
-		expect(image).toEqual("image.jpg");
+		var image = Scraper.prototype.getFileFilename("http://example.com/image.jpg?a=1&b=2");
+		expect(image).toEqual("image.jpg%3Fa%3D1%26b%3D2");
 	});
 
 	it("should extract filename from image url without http://", function(){