Change API

Author: Philipp Alferov

src/angular-file-saver.service.js

@@ -19,32 +19,24 @@ module.exports = function FileSaver(Blob, SaveAs, FileSaverUtils) {
     /**
     * saveAs - Immediately starts saving a file, returns undefined.
     *
-    * @param  {object} config Set of options such as data, filename
-    * and Blob constructor options. Options - optional parameter if data
-    * is represented by blob instance
+    * @param  {object} config Set of options such as filename and Blob options.
     * @return {undefined}
     */
 
     saveAs: function(config) {
-      config = config || {};
+      config = angular.extend({}, config);
       var data = config.data;
       var filename = config.filename;
-      var options = config.options;
 
-      if (!FileSaverUtils.isArray(data) && !isBlobInstance(data)) {
-        FileSaverUtils.handleErrors('Data argument should be represented as an array or Blob instance');
+      if (!isBlobInstance(data)) {
+        FileSaverUtils.handleErrors('Data argument should be a blob instance');
       }
 
       if (!FileSaverUtils.isString(filename)) {
         FileSaverUtils.handleErrors('Filename argument should be a string');
       }
 
-      if (isBlobInstance(data)) {
-        return save(data, filename);
-      }
-
-      var blob = new Blob(data, options);
-      return save(blob, filename);
+      return save(data, filename);
     }
   };
 };

test/angular-file-saver-bundle.spec.js

@@ -2,50 +2,38 @@
 
 describe('angular-file-saver', function() {
 
-  var FileSaver, config;
+  var FileSaver, Blob;
 
   beforeEach(function() {
 
     angular.mock.module('ngFileSaver');
 
-    angular.mock.inject(function(_FileSaver_) {
+    angular.mock.inject(function(_FileSaver_, _Blob_) {
       FileSaver = _FileSaver_;
+      Blob = _Blob_;
     });
   });
 
   describe('passed arguments are correct', function() {
 
-    beforeEach(function() {
-      config = {
-        filename: 'xfiles.txt',
-        options: {
-          type: 'text/plain;charset=utf-8'
-        }
+    it('should throw an error if provided data is an array', function() {
+      var config = {
+        data: ['text'],
+        filename: 'xfiles.txt'
       };
-    });
-
-    it('should save a file if provided data is an array', function() {
-      config.data = ['text'];
 
-      expect(function() {
-        FileSaver.saveAs(config);
-      }).not.toThrow();
+      expect(function() { FileSaver.saveAs(config); }).toThrow();
     });
 
     it('should save a file if provided data is a blob', function() {
-      config.data = new Blob(['text'], config.options);
+      var data = new Blob(['text'], { type: 'text/plain;charset=utf-8' });
 
-      expect(function() {
-        FileSaver.saveAs(config);
-      }).not.toThrow();
-    });
-
-    it('should save a file if options object is not passed', function() {
-      config.data = ['text'];
+      var config = {
+        data: data,
+        filename: 'xfiles.txt'
+      };
 
-      expect(function() {
-        FileSaver.saveAs(config);
-      }).not.toThrow();
+      expect(function() { FileSaver.saveAs(config); }).not.toThrow();
     });
 
   });