lib/_copy: fixed nasty multiple callback fs.copy() bug. Closes #98

CHANGELOG.md

@@ -3,6 +3,7 @@
 - bugfix `fs.move()` into itself. Closes #104
 - bugfix `fs.move()` moving directory across device. Closes #108
 - added coveralls support
+- bugfix: nasty multiple callback `fs.copy()` bug. Closes #98
 
 0.15.0 / 2015-01-21
 -------------------

lib/_copy.js

@@ -82,15 +82,16 @@ function ncp (source, dest, options, callback) {
     var target = file.name.replace(currentPath, targetPath)
     isWritable(target, function (writable) {
       if (writable) {
-        return copyFile(file, target)
+        copyFile(file, target)
+      } else {
+        if(clobber) {
+          rmFile(target, function () {
+            copyFile(file, target)
+          })
+        } else {
+          cb()
+        }
       }
-      if(clobber) {
-        rmFile(target, function () {
-          copyFile(file, target)
-        })
-      }
-
-      return cb()
     })
   }