Fix this context in callbacks using arrow funtions.

evanvosberg · evanvosberg · commit eb61233396b7 · 2020-02-10T19:19:52.000+01:00
diff --git a/grunt/tasks/modularize.js b/grunt/tasks/modularize.js
@@ -2,7 +2,7 @@
 
 var _ = require("lodash"),
 
-	fmd = require("fmd");
+    fmd = require("fmd");
 
 module.exports = function (grunt) {
 
@@ -14,78 +14,76 @@ module.exports = function (grunt) {
             modules = {},
 
             config = {
-    			target: this.target + '/',
-    			factories: ["commonjs", "amd", "global"],
-    			trim_whitespace: true,
-    			new_line: "unix",
-    			indent: "\t"
-    		};
+                target: this.target + '/',
+                factories: ["commonjs", "amd", "global"],
+                trim_whitespace: true,
+                new_line: "unix",
+                indent: "\t"
+            };
 
         // Prepare Factory-Module-Definition settings
-        _.each(options, function (conf, name) {
+        _.each(options, (conf, name) => {
             var sources = [],
 
-				opts = {
-					depends: {}
-				},
+                opts = {
+                    depends: {}
+                },
 
-				deps = [];
+                deps = [];
 
-			if (conf.exports) {
-				opts.exports = conf.exports;
-			}
+            if (conf.exports) {
+                opts.exports = conf.exports;
+            }
 
-			if (conf.global) {
-				opts.global = conf.global;
-			}
+            if (conf.global) {
+                opts.global = conf.global;
+            }
 
             // Find and add self as source
-            _.each(this.filesSrc, function (source) {
-    			if (grunt.file.exists(source + name + ".js")) {
-    				sources.push(source + name + ".js");
-    			}
-            }, this);
+            _.each(this.filesSrc, (source) => {
+                if (grunt.file.exists(source + name + ".js")) {
+                    sources.push(source + name + ".js");
+                }
+            });
 
             if (conf.pack) {
-    			// Collect all components
-    			deps = _.chain(conf.components)
-    			    .map(function (depName) {
-        			    return options[depName].components;
-    			    })
-    			    .flatten()
-    			    .uniq()
-    			    .without(name)
-    			    .sort(function (a, b) {
-            			return options[a].components.indexOf(b) === -1 ? -1 : 1;
-        			})
-        			.value();
+                // Collect all components
+                deps = _.chain(conf.components)
+                    .map(depName => options[depName].components)
+                    .flatten()
+                    .uniq()
+                    .without(name)
+                    .sort((a, b) => {
+                        return options[a].components.indexOf(b) === -1 ? -1 : 1;
+                    })
+                    .value();
 
                 // Add components as source files -> results a single file
-                _.each(this.filesSrc, function (source) {
-                    _.each(deps, function (depName) {
-            			if (grunt.file.exists(source + depName + ".js")) {
-            				sources.push(source + depName + ".js");
-            			}
-        			});
-                }, this);
+                _.each(this.filesSrc, (source) => {
+                    _.each(deps, (depName) => {
+                        if (grunt.file.exists(source + depName + ".js")) {
+                            sources.push(source + depName + ".js");
+                        }
+                    });
+                });
             } else {
-    			// Read components and add them as dependecies
-    			_.each(_.without(conf.components, name), function (value, i) {
-    				opts.depends['./' + value] = value === "core" ? "CryptoJS" : null;
-    			});
-			}
+                // Read components and add them as dependecies
+                _.each(_.without(conf.components, name), (value, i) => {
+                    opts.depends['./' + value] = value === "core" ? "CryptoJS" : null;
+                });
+            }
 
-			// Remove duplicates
-			sources = _.uniq(sources);
+            // Remove duplicates
+            sources = _.uniq(sources);
 
             // Add module settings to fmd definition
-			modules[name] = [sources, opts];
-		}, this);
+            modules[name] = [sources, opts];
+        });
 
-		// Build packege modules
-		fmd(config)
-			.define(modules)
-			.build(function (createdFiles) {
+        // Build packege modules
+        fmd(config)
+            .define(modules)
+            .build(() => {
 
                 done();
             });
