Merge pull request #30 from pconerly/dont-parse-javascript-script-content-as-tag

Dont parse javascript script content "<" as tag

Author: pconerly

.circleci/config.yml

@@ -0,0 +1,34 @@
+# Javascript Node CircleCI 2.0 configuration file
+#
+# Check https://circleci.com/docs/2.0/language-javascript/ for more details
+#
+version: 2.1
+jobs:
+  build:
+    docker:
+      # - image: circleci/node:8.9.4
+      - image: circleci/node:8.16.0
+
+    working_directory: ~/project
+
+    steps:
+      - checkout
+
+      # Download and cache dependencies
+      - restore_cache:
+          keys:
+            - deps-{{ checksum "package-lock.json" }}
+            # fallback to using the latest cache if no exact match is found
+            - deps-
+
+      - run: npm install
+
+      - save_cache:
+          paths:
+            - frontend/node_modules
+          key: deps-{{ checksum "package-lock.json" }}
+
+      # run tests
+      - run: npm run test-node
+      # run lint
+      - run: npm run jshint

.jshintrc

@@ -1,7 +1,6 @@
 {
   "curly": true,
   "latedef": true,
-  "quotmark": true,
   "undef": true,
   "unused": true,
   "trailing": true,

lib/parse-tag.js

@@ -1,4 +1,4 @@
-var attrRE = /\s([^'"/\s><]+?)[\s/>]|([^\s=]+)=\s?(".*?"|'.*?')/g
+var attrRE = /\s([^'"/\s><]+?)[\s/>]|([^\s=]+)=\s?(".*?"|'.*?')/g;
 
 // create optimized lookup object for
 // void elements as listed here:
@@ -30,40 +30,45 @@ module.exports = function (tag) {
         children: []
     };
 
-    let tagMatch = tag.match(/<\/?([^\s]+?)[/\s>]/)
+    var tagMatch = tag.match(/<\/?([^\s]+?)[/\s>]/);
     if(tagMatch)
     {
         res.name = tagMatch[1];
-        if (lookup[tagMatch[1].toLowerCase()] || tag.charAt(tag.length - 2) === '/')
+        if (lookup[tagMatch[1].toLowerCase()] || tag.charAt(tag.length - 2) === '/') {
             res.voidElement = true;
+        }
 
     }
 
-    let reg = new RegExp(attrRE)
-    let result = null;
+    var reg = new RegExp(attrRE);
+    var result = null;
     for (; ;)
     {
         result = reg.exec(tag);
 
-        if (result === null)
+        if (result === null) {
             break;
+        }
 
-        if (!result[0].trim())
+        if (!result[0].trim()) {
             continue;
+        }
 
         if (result[1])
         {
-            let attr = result[1].trim();
-            let arr = [attr, ""];
+            var attr = result[1].trim();
+            var arr = [attr, ''];
 
-            if(attr.indexOf("=") > -1)
-                arr = attr.split("=");
+            if(attr.indexOf('=') > -1) {
+                arr = attr.split('=');
+            }
 
             res.attrs[arr[0]] = arr[1];
             reg.lastIndex--;
         }
-        else if (result[2])
+        else if (result[2]) {
             res.attrs[result[2]] = result[3].trim().substring(1,result[3].length - 1);
+        }
     }
 
     return res;

lib/parse.js

@@ -1,5 +1,6 @@
 /*jshint -W030 */
-var tagRE = /<(?:"[^"]*"['"]*|'[^']*'['"]*|[^'">])+>/g;
+var tagRE = /<[a-zA-Z\-\!\/](?:"[^"]*"['"]*|'[^']*'['"]*|[^'">])*>/g;
+
 var parseTag = require('./parse-tag');
 // re-used obj for quick lookups of components
 var empty = Object.create ? Object.create(null) : {};