[INTERNAL] Use Object.create(null) for dynamic maps

Author: matz3

lib/lbt/resources/LibraryFileAnalyzer.js

@@ -49,7 +49,7 @@ function createRawInfo(rawModule) {
 }
 
 export function getDependencyInfos( name, content ) {
-	const infos = {};
+	const infos = Object.create(null);
 	parser.parseString(content, (err, result) => {
 		if ( result &&
 				result.library &&

lib/lbt/resources/ModuleInfo.js

@@ -39,7 +39,7 @@ class ModuleInfo {
 	constructor(name) {
 		this._name = name;
 		this.subModules = [];
-		this._dependencies = {};
+		this._dependencies = Object.create(null);
 		this.dynamicDependencies = false;
 
 		/**

lib/processors/manifestCreator.js

@@ -349,7 +349,7 @@ async function createManifest(
 		}
 
 		function collectThemes() {
-			const themes = {};
+			const themes = Object.create(null);
 
 			// find theme resources and determine theme names from their paths
 			libBundle.getResources(/(?:[^/]+\/)*themes\//).forEach((res) => {

lib/processors/nonAsciiEscaper.js

@@ -14,7 +14,7 @@ import escapeUnicode from "escape-unicode";
 const CHAR_CODE_OF_LAST_ASCII_CHARACTER = 127;
 
 // use memoization for escapeUnicode function for performance
-const memoizeEscapeUnicodeMap = {};
+const memoizeEscapeUnicodeMap = Object.create(null);
 const memoizeEscapeUnicode = function(sChar) {
 	if (memoizeEscapeUnicodeMap[sChar]) {
 		return memoizeEscapeUnicodeMap[sChar];

lib/processors/versionInfoGenerator.js

@@ -60,15 +60,15 @@ function getTimestamp() {
 const processManifest = async (manifestResource) => {
 	const manifestContent = await manifestResource.getString();
 	const manifestObject = JSON.parse(manifestContent);
-	const manifestInfo = {};
+	const manifestInfo = Object.create(null);
 
 	// sap.ui5/dependencies is used for the "manifestHints/libs"
 	if (manifestObject["sap.ui5"]) {
 		const manifestDependencies = manifestObject["sap.ui5"]["dependencies"];
 		if (manifestDependencies && manifestDependencies.libs) {
-			const libs = {};
+			const libs = Object.create(null);
 			for (const [libKey, libValue] of Object.entries(manifestDependencies.libs)) {
-				libs[libKey] = {};
+				libs[libKey] = Object.create(null);
 				if (libValue.lazy) {
 					libs[libKey].lazy = true;
 				}
@@ -230,7 +230,7 @@ class DependencyInfo {
  * @returns {object} the object with sorted keys
  */
 const sortObjectKeys = (obj) => {
-	const sortedObject = {};
+	const sortedObject = Object.create(null);
 	const keys = Object.keys(obj);
 	keys.sort();
 	keys.forEach((key) => {
@@ -433,7 +433,7 @@ export default async function({options}) {
 	let components;
 	artifactInfos.forEach((artifactInfo) => {
 		artifactInfo.embeds.forEach((embeddedArtifactInfo) => {
-			const componentObject = {};
+			const componentObject = Object.create(null);
 			const bundledComponents = artifactInfo.bundledComponents;
 			const componentName = embeddedArtifactInfo.componentName;
 			if (!bundledComponents.has(componentName)) {
@@ -446,7 +446,7 @@ export default async function({options}) {
 				componentObject.manifestHints = manifestHints;
 			}
 
-			components = components || {};
+			components = components || Object.create(null);
 			components[componentName] = componentObject;
 		});
 	});

lib/tasks/bundlers/utils/createModuleNameMapping.js

@@ -15,7 +15,7 @@ import {getNonDebugName} from "../../../lbt/utils/ModuleName.js";
  * @returns {object} Module name mapping
  */
 export default function({resources, taskUtil}) {
-	const moduleNameMapping = {};
+	const moduleNameMapping = Object.create(null);
 	for (let i = resources.length - 1; i >= 0; i--) {
 		const resource = resources[i];
 		if (taskUtil.getTag(resource, taskUtil.STANDARD_TAGS.IsDebugVariant)) {

lib/tasks/generateCachebusterInfo.js

@@ -57,7 +57,7 @@ export default function({workspace, options}) {
 	const basePath = `/resources/${namespace}/`;
 	return workspace.byGlob(`/resources/${namespace}/**/*`)
 		.then(async (resources) => {
-			const cachebusterInfo = {};
+			const cachebusterInfo = Object.create(null);
 			const signer = getSigner(signatureType);
 
 			await Promise.all(resources.map(async (resource) => {