code/wrench-js
Archived
1
Fork 0
Code Issues 22 Pull requests 7 Projects Releases Packages Wiki Activity Actions

Remove redundant check

Browse source 
We default opts to {} at the top of the function.

Also fix indentation.
This commit is contained in:
Jo Liss 2014-04-03 23:22:20 +00:00
parent 13f486d867
commit 8fef99a450
1 changed files with 5 additions and 11 deletions
Download patch file Download diff file Expand all files Collapse all files

16
lib/wrench.js
View file

@ -231,9 +231,9 @@ exports.copyDirSyncRecursive = function(sourceDir, newDirLocation, opts) {
opts = opts || {};
try {
if(fs.statSync(newDirLocation).isDirectory()) {
if(fs.statSync(newDirLocation).isDirectory()) {
if(opts.forceDelete) {
exports.rmdirSyncRecursive(newDirLocation);
exports.rmdirSyncRecursive(newDirLocation);
} else {
return new Error('You are trying to delete a directory that already exists. Specify forceDelete in the opts argument to override this. Bailing~');
}
@ -256,15 +256,9 @@ exports.copyDirSyncRecursive = function(sourceDir, newDirLocation, opts) {
for(var i = 0; i < files.length; i++) {
// ignores all files or directories which match the RegExp in opts.filter
if(typeof opts !== 'undefined') {
if (hasFilter) {
if (!isFileIncluded(opts, sourceDir, files[i])) {
continue;
}
}
if (opts.excludeHiddenUnix && /^\./.test(files[i])) continue;
}
if (hasFilter && !isFileIncluded(opts, sourceDir, files[i])) continue;
if (opts.excludeHiddenUnix && /^\./.test(files[i])) continue;
var currFile = fs.lstatSync(_path.join(sourceDir, files[i]));