refactor(ES6): migrating to () => {}

sendilkumarn · sendilkumarn · commit 84d1f660a6e6 · 2017-02-23T15:43:44.000+08:00
diff --git a/lib/DelegatedModuleFactoryPlugin.js b/lib/DelegatedModuleFactoryPlugin.js
@@ -21,30 +21,28 @@ class DelegatedModuleFactoryPlugin {
 	apply(normalModuleFactory) {
 		const scope = this.options.scope;
 		if(scope) {
-			normalModuleFactory.plugin("factory", function(factory) {
-				return function(data, callback) {
-					const dependency = data.dependencies[0];
-					const request = dependency.request;
-					if(request && request.indexOf(scope + "/") === 0) {
-						const innerRequest = "." + request.substr(scope.length);
-						let resolved;
-						if(innerRequest in this.options.content) {
-							resolved = this.options.content[innerRequest];
-							return callback(null, new DelegatedModule(this.options.source, resolved, this.options.type, innerRequest));
-						}
-						for(let i = 0; i < this.options.extensions.length; i++) {
-							const requestPlusExt = innerRequest + this.options.extensions[i];
-							if(requestPlusExt in this.options.content) {
-								resolved = this.options.content[requestPlusExt];
-								return callback(null, new DelegatedModule(this.options.source, resolved, this.options.type, requestPlusExt));
-							}
+			normalModuleFactory.plugin("factory", factory => (data, callback) => {
+				const dependency = data.dependencies[0];
+				const request = dependency.request;
+				if(request && request.indexOf(scope + "/") === 0) {
+					const innerRequest = "." + request.substr(scope.length);
+					let resolved;
+					if(innerRequest in this.options.content) {
+						resolved = this.options.content[innerRequest];
+						return callback(null, new DelegatedModule(this.options.source, resolved, this.options.type, innerRequest));
+					}
+					for(let i = 0; i < this.options.extensions.length; i++) {
+						const requestPlusExt = innerRequest + this.options.extensions[i];
+						if(requestPlusExt in this.options.content) {
+							resolved = this.options.content[requestPlusExt];
+							return callback(null, new DelegatedModule(this.options.source, resolved, this.options.type, requestPlusExt));
 						}
 					}
-					return factory(data, callback);
-				}.bind(this);
-			}.bind(this));
+				}
+				return factory(data, callback);
+			});
 		} else {
-			normalModuleFactory.plugin("module", function(module) {
+			normalModuleFactory.plugin("module", module => {
 				if(module.libIdent) {
 					const request = module.libIdent(this.options);
 					if(request && request in this.options.content) {
@@ -53,7 +51,7 @@ class DelegatedModuleFactoryPlugin {
 					}
 				}
 				return module;
-			}.bind(this));
+			});
 		}
 	}
 }
