From ddff23792d8318024d4fb6ac2981bebc02ee0365 Mon Sep 17 00:00:00 2001 From: Jordan Harband Date: Mon, 20 Sep 2021 12:13:57 -0700 Subject: [PATCH] [Refactor] `no-danger`: use Object.fromEntries and Object.hasOwn instead of reduce --- lib/rules/no-danger.js | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/lib/rules/no-danger.js b/lib/rules/no-danger.js index 9669131037..f373ec4c36 100644 --- a/lib/rules/no-danger.js +++ b/lib/rules/no-danger.js @@ -5,6 +5,9 @@ 'use strict'; +const has = require('object.hasown/polyfill')(); +const fromEntries = require('object.fromentries/polyfill')(); + const docsUrl = require('../util/docsUrl'); const jsxUtil = require('../util/jsx'); const report = require('../util/report'); @@ -17,10 +20,7 @@ const DANGEROUS_PROPERTY_NAMES = [ 'dangerouslySetInnerHTML' ]; -const DANGEROUS_PROPERTIES = DANGEROUS_PROPERTY_NAMES.reduce((props, prop) => { - props[prop] = prop; - return props; -}, Object.create(null)); +const DANGEROUS_PROPERTIES = fromEntries(DANGEROUS_PROPERTY_NAMES.map((prop) => [prop, prop])); // ------------------------------------------------------------------------------ // Helpers @@ -32,7 +32,7 @@ const DANGEROUS_PROPERTIES = DANGEROUS_PROPERTY_NAMES.reduce((props, prop) => { * @returns {boolean} Whether or not the attribute is dnagerous. */ function isDangerous(name) { - return name in DANGEROUS_PROPERTIES; + return has(DANGEROUS_PROPERTIES, name); } // ------------------------------------------------------------------------------