Limit set function to the scope where it is used

Author: KillyMXI

lib/helper.js

@@ -118,26 +118,6 @@ function get (obj, path) {
   return obj;
 }
 
-/**
- * Set a nested property of an object.
- *
- * @param { object }   obj   The object to modify.
- * @param { string[] } path  The path to the property.
- * @param { any }      value The value to set.
- */
-function set (obj, path, value) {
-  const valueKey = path.pop();
-  for (const key of path) {
-    let nested = obj[key];
-    if (!nested) {
-      nested = {};
-      obj[key] = nested;
-    }
-    obj = nested;
-  }
-  obj[valueKey] = value;
-}
-
 /**
  * Deduplicate an array by a given key callback.
  * Item properties are merged recursively and with the preference for last defined values.
@@ -171,7 +151,6 @@ module.exports = {
   mergeDuplicatesPreferLast: mergeDuplicatesPreferLast,
   numberToLetterSequence: numberToLetterSequence,
   numberToRoman: numberToRoman,
-  set: set,
   splitClassesAndIds: splitClassesAndIds,
   trimCharacter: trimCharacter
 };

lib/html-to-text.js

@@ -6,7 +6,7 @@ const selderee = require('selderee');
 
 const { BlockTextBuilder } = require('./block-text-builder');
 const defaultFormatters = require('./formatter');
-const { limitedDepthRecursive, mergeDuplicatesPreferLast, set } = require('./helper');
+const { limitedDepthRecursive, mergeDuplicatesPreferLast } = require('./helper');
 
 // eslint-disable-next-line import/no-unassigned-import
 require('./typedefs');
@@ -241,6 +241,19 @@ function handleDeprecatedOptions (options) {
     selectorDefinitions.push(...tagDefinitions);
   }
 
+  function set (obj, path, value) {
+    const valueKey = path.pop();
+    for (const key of path) {
+      let nested = obj[key];
+      if (!nested) {
+        nested = {};
+        obj[key] = nested;
+      }
+      obj = nested;
+    }
+    obj[valueKey] = value;
+  }
+
   function copyFormatterOption (source, format, target) {
     if (options[source] === undefined) { return; }
     for (const definition of selectorDefinitions) {