refactor: minor refactor of basic-loader

Author: zfeher

lib/basic-loader-amd.js

@@ -6,7 +6,7 @@ define(["exports"], function (_exports) {
   });
   _exports.default = void 0;
 
-  function _load(tag) {
+  var _load = function _load(tag) {
     // attributes example: { 'data-test': 'new-attribute-here' }
     return function (url) {
       var attributes = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : {};
@@ -36,17 +36,17 @@ define(["exports"], function (_exports) {
             element.rel = 'stylesheet';
             attr = 'href';
             parent = 'head';
-        } // Inject into document to kick off loading
-
+        }
 
         element[attr] = url;
-        Object.keys(attributes).forEach(function (key) {
-          element.setAttribute(key, attributes[key]);
-        });
+        Object.keys(attributes).forEach(function (name) {
+          return element.setAttribute(name, attributes[name]);
+        }); // Inject into document to kick off loading
+
         document[parent].appendChild(element);
       });
     };
-  } // exporting a "default" would render the amd package to work differently
+  }; // exporting a "default" would render the amd package to work differently
 
 
   var _default = {

lib/basic-loader.js

@@ -5,7 +5,7 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.default = void 0;
 
-function _load(tag) {
+var _load = function _load(tag) {
   // attributes example: { 'data-test': 'new-attribute-here' }
   return function (url) {
     var attributes = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : {};
@@ -35,17 +35,17 @@ function _load(tag) {
           element.rel = 'stylesheet';
           attr = 'href';
           parent = 'head';
-      } // Inject into document to kick off loading
-
+      }
 
       element[attr] = url;
-      Object.keys(attributes).forEach(function (key) {
-        element.setAttribute(key, attributes[key]);
-      });
+      Object.keys(attributes).forEach(function (name) {
+        return element.setAttribute(name, attributes[name]);
+      }); // Inject into document to kick off loading
+
       document[parent].appendChild(element);
     });
   };
-} // exporting a "default" would render the amd package to work differently
+}; // exporting a "default" would render the amd package to work differently
 
 
 var _default = {

src/basic-loader.js

@@ -1,6 +1,6 @@
-function _load (tag) {
+const _load = (tag) => {
   // attributes example: { 'data-test': 'new-attribute-here' }
-  return function (url, attributes = {}) {
+  return (url, attributes = {}) => {
     // This promise will be used by Promise.all to determine success or failure
     return new Promise((resolve, reject) => {
       const element = document.createElement(tag);
@@ -23,18 +23,17 @@ function _load (tag) {
           parent = 'head';
       }
 
-      // Inject into document to kick off loading
       element[attr] = url;
-      Object.keys(attributes).forEach(key => {
-        element.setAttribute(key, attributes[key]);
-      });
+      Object.keys(attributes).forEach((name) => element.setAttribute(name, attributes[name]));
+
+      // Inject into document to kick off loading
       document[parent].appendChild(element);
     });
   };
-}
+};
 
 export default {
   css: _load('link'),
   js: _load('script'),
-  img: _load('img')
+  img: _load('img'),
 };