fixed issue #14, when closureBuilder run from a grunt watch task, the…

… root param got lost
thanpolas · Dec 12, 2012 · 24eb4d7 · 24eb4d7
1 parent 8f9ea4a
commit 24eb4d7
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/tasks/closureBuilder.js b/tasks/closureBuilder.js
@@ -145,7 +145,7 @@ function validate(grunt, data)
   }
 
   var root = data.root;
-  if (!root) {
+  if (!root || 0 === root.length) {
     grunt.log.error('ERROR'.red + ' :: ' + 'root'.red + ' property is required');
     return false;
   }

diff --git a/tasks/helpers.js b/tasks/helpers.js
@@ -46,10 +46,8 @@ module.exports = function(grunt) {
     if (Array.isArray(param)) {
       if (opt_parsePath){
         paramValue = [];
-        var item = param.shift();
-        while(item) {
-          paramValue.push(grunt.file.expand(item));
-          item = param.shift();
+        for(var i = 0, len = param.length; i < len; i++) {
+          paramValue.push(grunt.file.expand(param[i]));
         }
       }
       return ' ' + directive + sp + paramValue.join(' ' + directive + sp);