feat(skip-empty) added new option to skip downloading empty CSV

README.md

@@ -58,6 +58,7 @@ ngCsv attributes
 * decimal-separator: Defines the decimal separator character (default is .). If set to "locale", it uses the [language sensitive representation of the number](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Number/toLocaleString).
 * text-delimiter: If provided, will use this characters to "escape" fields, otherwise will use double quotes as deafult
 * quote-strings: If provided, will force escaping of every string field.
+* skip-empty: If provided, will not trigger the download of empty data (default is false).
 * lazy-load: If defined and set to true, ngCsv will generate the data string only on demand. See the lazy_load example for more details.
 * add-bom: Add the Byte Order Mark, use this option if you are getting an unexpected char when opening the file on any windows App.
 * charset: Defines the charset of the downloadable Csv file. Default is "utf-8".

build/ng-csv.js

@@ -221,7 +221,8 @@ angular.module('ngCsv.directives').
         addByteOrderMarker: "@addBom",
         ngClick: '&',
         charset: '@charset',
-        label: '&csvLabel'
+        label: '&csvLabel',
+        skipEmpty: '@skipEmpty'
       },
       controller: [
         '$scope',
@@ -284,6 +285,10 @@ angular.module('ngCsv.directives').
       ],
       link: function (scope, element, attrs) {
         function doClick() {
+
+          // drop out for empty data
+          if (!scope.csv.trim() && scope.skipEmpty === "true") {return;}
+
           var charset = scope.charset || "utf-8";
           var blob = new Blob([scope.csv], {
             type: "text/csv;charset="+ charset + ";"

src/ng-csv/directives/ng-csv.js

@@ -21,7 +21,8 @@ angular.module('ngCsv.directives').
         addByteOrderMarker: "@addBom",
         ngClick: '&',
         charset: '@charset',
-        label: '&csvLabel'
+        label: '&csvLabel',
+        skipEmpty: '@skipEmpty'
       },
       controller: [
         '$scope',
@@ -84,6 +85,10 @@ angular.module('ngCsv.directives').
       ],
       link: function (scope, element, attrs) {
         function doClick() {
+
+          // drop out for empty data
+          if (!scope.csv.trim() && scope.skipEmpty === "true") {return;}
+
           var charset = scope.charset || "utf-8";
           var blob = new Blob([scope.csv], {
             type: "text/csv;charset="+ charset + ";"

test/unit/ngCsv/directives/ngCsv.js

@@ -134,14 +134,14 @@ describe('ngCsv directive', function () {
     $rootScope.testDelim = [ {a:1, b:2, c:3}, {a:4, b:5, c:6} ];
     var element = $compile(
       '<div ng-csv="testDelim" csv-label="true" filename="custom.csv"></div>')($rootScope);
-    
+
     $rootScope.$digest();
 
     var scope = element.isolateScope();
 
     // Check that the compiled element contains the templated content
     expect(scope.$eval(scope.data)).toEqual($rootScope.testDelim);
-    
+
 
     scope.buildCSV(scope.data).then(function() {
       expect(scope.csv).toBe('a,b,c\r\n1,2,3\r\n4,5,6\r\n');
@@ -192,6 +192,29 @@ describe('ngCsv directive', function () {
     scope.$apply();
   });
 
+  it('Accepts optional skip-empty attribute', function () {
+
+    $rootScope.noData = [];
+    var element = $compile(
+      '<div ng-csv="noData" skip-empty="true" ' +
+      ' filename="no-download.csv">' +
+      '</div>')($rootScope);
+
+    $rootScope.$digest();
+
+    var scope = element.isolateScope();
+
+    spyOn(scope, 'buildCSV');
+
+    // Check that the compiled element contains the templated content
+    expect(scope.$eval(scope.data)).toEqual($rootScope.noData);
+    expect(scope.skipEmpty).toBe('true');
+
+    scope.$apply();
+
+    expect(scope.buildCSV).not.toHaveBeenCalled();
+  });
+
   it('Accepts optional text-delimiter attribute (input object)', function (done) {
     $rootScope.testDelim = [{a: 1, b: 'a', c: 2}, {a: 'b', b: 'c', c: 3}];
     var element = $compile(