decode-html - npm Package Compare versions

Comparing version 1.0.2 to 2.0.0

index.js

@@ -0,14 +1,23 @@
+// Store markers outside of the function scope,
+// not to recreate them on every call
+var entities = {
+  'amp': '&',
+  'apos': '\'',
+  'lt': '<',
+  'gt': '>',
+  'quot': '"',
+  'nbsp': '\xa0'
+};
+var entityPattern = /&([a-z]+);/ig;
+
 module.exports = function decodeHTMLEntities(text) {
-  var replacements = [
-    ['amp', '&'],
-    ['apos', '\''],
-    ['lt', '<'],
-    ['gt', '>']
-  ];
-
-  replacements.forEach(function(replace){
-    text = text.replace(new RegExp('&'+replace[0]+';', 'g'), replace[1]);
+  // A single replace pass with a static RegExp is faster than a loop
+  return text.replace(entityPattern, function(match, entity) {
+    entity = entity.toLowerCase();
+    if (entities.hasOwnProperty(entity)) {
+      return entities[entity];
+    }
+    // return original string if there is no matching entity (no replace)
+    return match;
   });
-
-  return text;
-};
+};

package.json

 {
   "name": "decode-html",
-  "version": "1.0.2",
+  "version": "2.0.0",
   "description": "decode html entities",
   "main": "index.js",
   "dependencies": {},
-  "devDependencies": {},
+  "devDependencies": {
+    "tape": "^4.5.1"
+  },
   "scripts": {
@@ -9,0 +11,0 @@ "test": "node test.js"

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

3349

increased by12.5%

Lines of code

35

increased by45.83%

Worsened metrics

Dev dependency count

1

increased byInfinity%

No dependency changes