From 5b747d9bb148f8185d4f900d5dedabdb365488c6 Mon Sep 17 00:00:00 2001 From: refaelos Date: Wed, 12 Jun 2013 18:56:12 +0300 Subject: [PATCH] fixed an issue with error handling in 'rmdirRecursive' --- lib/wrench.js | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/lib/wrench.js b/lib/wrench.js index 1066ee7..3092012 100644 --- a/lib/wrench.js +++ b/lib/wrench.js @@ -293,11 +293,16 @@ exports.chownSyncRecursive = function(sourceDir, uid, gid) { */ exports.rmdirRecursive = function rmdirRecursive(dir, failSilent, clbk){ fs.readdir(dir, function(err, files){ - if(err && typeof failSilent === 'boolean' && !failSilent) - return clbk(err); + if(err) { + if (typeof failSilent === 'boolean' && failSilent) { + return clbk(null); + } else { + return clbk(err); + } + } - if(typeof failSilent === 'function') - clbk = failSilent; + if(typeof failSilent === 'function') + clbk = failSilent; (function rmFile(err){ if (err) return clbk(err);