Change line 47 for correctness

milseg · milseg · commit 0d8908ae7734 · 2016-04-02T01:35:18.000-03:00
MyCustomStorage constructor had an attribution to opts object, which was absolutely wrong, since we would like to access getDestination as a method of MyCustomStorage in line 51. Hence the right statement is assigning the value to 'this' which represents the object itself.
diff --git a/StorageEngine.md b/StorageEngine.md
@@ -44,7 +44,7 @@ function getDestination (req, file, cb) {
 }
 
 function MyCustomStorage (opts) {
-  opts.getDestination = (opts.destination || getDestination)
+  this.getDestination = (opts.destination || getDestination)
 }
 
 MyCustomStorage.prototype._handleFile = function _handleFile (req, file, cb) {

Original file line number	Diff line number	Diff line change
`@@ -44,7 +44,7 @@ function getDestination (req, file, cb) {`
`44`	`44`	`}`
`45`	`45`
`46`	`46`	`function MyCustomStorage (opts) {`
`47`		`- opts.getDestination = (opts.destination \|\| getDestination)`
	`47`	`+ this.getDestination = (opts.destination \|\| getDestination)`
`48`	`48`	`}`
`49`	`49`
`50`	`50`	`MyCustomStorage.prototype._handleFile = function _handleFile (req, file, cb) {`