#2 fixes modifications of settings.args

lib/index.js

@@ -25,6 +25,9 @@ module.exports = (source, settings, callback) => {
 
     let isBinary = false;
 
+    // copy settings.args to prevent modifications
+    if (settings.args) settings.args = settings.args.slice(0);
+
     if (!settings.args || settings.args.indexOf("-json") == -1) {
         settings.args = settings.args || [];
         settings.args.push("-json");

package.json

@@ -1,6 +1,6 @@
 {
   "name": "simple-exiftool",
-  "version": "1.1.0",
+  "version": "1.1.1",
   "description": "Extracting metadata from media files with exiftool, including but not limited to mp4, mov, 3gp, jpg, png, gif, pdf.",
   "main": "./lib/exif",
   "scripts": {

test/index.js

@@ -173,4 +173,17 @@ describe("giving settings to simple-exiftool", () => {
         });
 
     });
+
+     it("should not modifiy given settings.args", (done) => {
+        var givenArgs=['-FileName'];
+        Exif(Path.join(__dirname, "files/lion.jpg"), {binary: exiftoolPath, args: givenArgs}, (error, metadata) => {
+
+            Expect(error).to.not.exist;
+            // giveArgs should not be modified
+            Expect(givenArgs).to.have.length(1);
+
+            done();
+        });
+
+    });
 });