html-react-parser 4.2.0 → 4.2.2

package/lib/utilities.js

@@ -13,14 +13,12 @@ function invertObject(obj, override) {
     throw new TypeError('First argument must be an object');
   }
 
-  var key;
-  var value;
   var isOverridePresent = typeof override === 'function';
   var overrides = {};
   var result = {};
 
-  for (key in obj) {
-    value = obj[key];
+  for (var key in obj) {
+    var value = obj[key];
 
     if (isOverridePresent) {
       overrides = override(key, value);
@@ -38,6 +36,17 @@ function invertObject(obj, override) {
   return result;
 }
 
+var RESERVED_SVG_MATHML_ELEMENTS = new Set([
+  'annotation-xml',
+  'color-profile',
+  'font-face',
+  'font-face-src',
+  'font-face-uri',
+  'font-face-format',
+  'font-face-name',
+  'missing-glyph'
+]);
+
 /**
  * Check if a given tag is a custom component.
  *
@@ -45,33 +54,24 @@ function invertObject(obj, override) {
  *
  * @param {string} tagName - The name of the html tag.
  * @param {object} props - The props being passed to the element.
- * @returns - Whether tag is custom component.
+ * @returns {boolean} - Whether tag is custom component.
  */
 function isCustomComponent(tagName, props) {
   if (tagName.indexOf('-') === -1) {
     return props && typeof props.is === 'string';
   }
-
-  switch (tagName) {
-    // These are reserved SVG and MathML elements.
-    // We don't mind this whitelist too much because we expect it to never grow.
-    // The alternative is to track the namespace in a few places which is convoluted.
-    // https://w3c.github.io/webcomponents/spec/custom/#custom-elements-core-concepts
-    case 'annotation-xml':
-    case 'color-profile':
-    case 'font-face':
-    case 'font-face-src':
-    case 'font-face-uri':
-    case 'font-face-format':
-    case 'font-face-name':
-    case 'missing-glyph':
-      return false;
-    default:
-      return true;
+  // These are reserved SVG and MathML elements.
+  // We don't mind this whitelist too much because we expect it to never grow.
+  // The alternative is to track the namespace in a few places which is convoluted.
+  // https://w3c.github.io/webcomponents/spec/custom/#custom-elements-core-concepts
+  if (RESERVED_SVG_MATHML_ELEMENTS.has(tagName)) {
+    return false;
   }
+  return true;
 }
 
-var styleToJSOptions = { reactCompat: true };
+// styleToJSOptions
+var STYLE_TO_JS_OPTIONS = { reactCompat: true };
 
 /**
  * Sets style prop.
@@ -84,7 +84,7 @@ function setStyleProp(style, props) {
     return;
   }
   try {
-    props.style = styleToJS(style, styleToJSOptions);
+    props.style = styleToJS(style, STYLE_TO_JS_OPTIONS);
   } catch (err) {
     props.style = {};
   }
@@ -98,7 +98,7 @@ var PRESERVE_CUSTOM_ATTRIBUTES = React.version.split('.')[0] >= 16;
 
 // Taken from
 // https://github.com/facebook/react/blob/cae635054e17a6f107a39d328649137b83f25972/packages/react-dom/src/client/validateDOMNesting.js#L213
-var elementsWithNoTextChildren = new Set([
+var ELEMENTS_WITH_NO_TEXT_CHILDREN = new Set([
   'tr',
   'tbody',
   'thead',
@@ -117,19 +117,25 @@ var elementsWithNoTextChildren = new Set([
  * @returns - Whether node can contain text nodes.
  */
 function canTextBeChildOfNode(node) {
-  return !elementsWithNoTextChildren.has(node.name);
+  return !ELEMENTS_WITH_NO_TEXT_CHILDREN.has(node.name);
 }
 
+/**
+ * Returns the first argument as is.
+ *
+ * @param {any} arg - The argument to be returned.
+ * @returns {any} The input argument `arg`.
+ */
 function returnFirstArg(arg) {
   return arg;
 }
 
 module.exports = {
   PRESERVE_CUSTOM_ATTRIBUTES: PRESERVE_CUSTOM_ATTRIBUTES,
+  ELEMENTS_WITH_NO_TEXT_CHILDREN: ELEMENTS_WITH_NO_TEXT_CHILDREN,
   invertObject: invertObject,
   isCustomComponent: isCustomComponent,
   setStyleProp: setStyleProp,
   canTextBeChildOfNode: canTextBeChildOfNode,
-  elementsWithNoTextChildren: elementsWithNoTextChildren,
   returnFirstArg: returnFirstArg
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "html-react-parser",
-  "version": "4.2.0",
+  "version": "4.2.2",
   "description": "HTML to React parser.",
   "author": "Mark <mark@remarkablemark.org>",
   "main": "index.js",
@@ -52,34 +52,34 @@
     "domhandler": "5.0.3",
     "html-dom-parser": "4.0.0",
     "react-property": "2.0.0",
-    "style-to-js": "1.1.3"
+    "style-to-js": "1.1.4"
   },
   "devDependencies": {
-    "@commitlint/cli": "17.6.7",
-    "@commitlint/config-conventional": "17.6.7",
-    "@rollup/plugin-commonjs": "25.0.3",
-    "@rollup/plugin-node-resolve": "15.1.0",
+    "@commitlint/cli": "17.7.1",
+    "@commitlint/config-conventional": "17.7.0",
+    "@rollup/plugin-commonjs": "25.0.4",
+    "@rollup/plugin-node-resolve": "15.2.1",
     "@size-limit/preset-big-lib": "8.2.6",
-    "@types/react": "18.2.15",
-    "@typescript-eslint/parser": "6.1.0",
+    "@types/react": "18.2.21",
+    "@typescript-eslint/parser": "6.5.0",
     "benchmark": "2.1.4",
     "dtslint": "4.2.1",
-    "eslint": "8.45.0",
+    "eslint": "8.48.0",
     "eslint-plugin-prettier": "5.0.0",
     "husky": "8.0.3",
-    "jest": "29.6.1",
-    "jest-environment-jsdom": "29.6.1",
-    "lint-staged": "13.2.3",
+    "jest": "29.6.4",
+    "jest-environment-jsdom": "29.6.4",
+    "lint-staged": "14.0.1",
     "pinst": "3.0.0",
-    "preact": "10.16.0",
-    "prettier": "3.0.0",
+    "preact": "10.17.1",
+    "prettier": "3.0.3",
     "react": "18.2.0",
     "react-dom": "18.2.0",
     "rimraf": "5.0.1",
     "rollup": "2.79.1",
     "rollup-plugin-terser": "7.0.2",
     "size-limit": "8.2.6",
-    "typescript": "5.1.6"
+    "typescript": "5.2.2"
   },
   "peerDependencies": {
     "react": "0.14 || 15 || 16 || 17 || 18"