Merge pull request #77 from SamyPesse/patch-1

Fix copyDirRecursive error callback
ryanmcgrath · Feb 10, 2014 · e5f08b8 · e5f08b8
2 parents 666e0c7 + b655e26
commit e5f08b8
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/lib/wrench.js b/lib/wrench.js
@@ -437,14 +437,17 @@ exports.copyDirRecursive = function copyDirRecursive(srcDir, newDir, opts, clbk)
  newFile = newDir+'/'+filename;
 
  fs.stat(file, function(err, fileStat){
+ if (err) return clbk(err);
  if (fileStat.isDirectory())
- copyDirRecursive(file, newFile, copyFiles);
+ copyDirRecursive(file, newFile, copyFiles, clbk);
  else if (fileStat.isSymbolicLink())
  fs.readlink(file, function(err, link){
+ if (err) return clbk(err);
  fs.symlink(link, newFile, copyFiles);
  });
  else
  fs.readFile(file, function(err, data){
+ if (err) return clbk(err);
  fs.writeFile(newFile, data, copyFiles);
  });
  });