extract() and execute() should return File instance

extract() and execute() should only execute once
extract() and execute() should work when chaining is not used

CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## v0.9.2
+
+* *extract()* and *execute()* now returns *File* instance. This means you can extract or execute more than once.
+* You can now use *extract()* and *execute()* even when no method chaining is done.
+* *extract()* and *execute()* will only execute *once per download()*.
+
+**Explanation for the last bullet:** In previous versions, if you did: `download(...).extract(...).download(...)`, this would trigger *extract()* twice because we did `.on` bindings to the download stream instead of `.once`. Now if you ever download the file again within the same *File* instance, you have to use *extract()* again to extract it.
+
 ## v0.9.1
 
 * *endl* should now resolve URLs without domains better in *download()* (e.g. If `href` is `/LatestSetup.exe` not `http://example.com/LatestSetup.exe`)

package.json

@@ -1,6 +1,6 @@
 {
   "name": "endl",
-  "version": "0.9.1",
+  "version": "0.9.2",
   "description": "Link extractor, downloader, executer, unzipper",
   "author": {
     "name": "Doğan Çelik",

src/file.coffee

@@ -111,6 +111,7 @@ class File
       options.directory = path.resolve options.directory
 
       thisClass._file = join options.directory, filename
+      thisClass._downloadFinished = false # for extract and execute
 
       callbackData = {
         url: downloadUrl
@@ -120,7 +121,8 @@ class File
       startDownload = ->
         thisClass._stream = createWriteStream thisClass._file
         thisClass._ee.emit 'create'
-        File::_bindCallback(response, callback, callbackData)
+        File::_bindCallback response, callback, callbackData
+        File::_bindCallback response, -> thisClass._downloadFinished = true
         response.pipe thisClass._stream
 
       stat options.directory, (err, stats) ->
@@ -187,7 +189,12 @@ class File
 
   extract: (options, callback) ->
     thisClass = @
-    thisClass._ee.on 'create', -> thisClass._stream.on('finish', thisClass._extractOnFinish.bind(thisClass, options, callback))
+    extractOnFinish = thisClass._extractOnFinish.bind(thisClass, options, callback)
+    if @_downloadFinished == true
+      extractOnFinish()
+    else
+      thisClass._ee.once 'create', -> thisClass._stream.once('finish', extractOnFinish)
+    @
 
   _executeOnFinish: (options) ->
     if options instanceof Array
@@ -202,6 +209,11 @@ class File
 
   execute: (options) ->
     thisClass = @
-    thisClass._ee.on 'create', -> thisClass._stream.on('finish', thisClass._executeOnFinish.bind(thisClass, options))
+    executeOnFinish = thisClass._executeOnFinish.bind(thisClass, options)
+    if @_downloadFinished == true
+      executeOnFinish()
+    else
+      thisClass._ee.once 'create', -> thisClass._stream.once('finish', executeOnFinish)
+    @
 
 module.exports = File