DEV: Remove HTML parser from Tautologistics. (#7344)

app/assets/javascripts/application.js

@@ -10,7 +10,6 @@
 //= require ./deprecated
 
 // Stuff we need to load first
-//= require ./discourse/helpers/parse-html
 //= require ./discourse/lib/to-markdown
 //= require ./discourse/lib/utilities
 //= require ./discourse/lib/page-visible

app/assets/javascripts/discourse/helpers/parse-html.js.es6

@@ -1,8 +0,0 @@
-/* global Tautologistics */
-export default function parseHTML(rawHtml) {
-  const builder = new Tautologistics.NodeHtmlParser.HtmlBuilder();
-  const parser = new Tautologistics.NodeHtmlParser.Parser(builder);
-
-  parser.parseComplete(rawHtml);
-  return builder.dom;
-}

app/assets/javascripts/discourse/lib/to-markdown.js.es6

@@ -1,5 +1,3 @@
-import parseHTML from "discourse/helpers/parse-html";
-
 const trimLeft = text => text.replace(/^\s+/, "");
 const trimRight = text => text.replace(/\s+$/, "");
 const countPipes = text => (text.replace(/\\\|/, "").match(/\|/g) || []).length;
@@ -495,10 +493,9 @@ function tags() {
 class Element {
   constructor(element, parent, previous, next) {
     this.name = element.name;
-    this.type = element.type;
     this.data = element.data;
     this.children = element.children;
-    this.attributes = element.attributes || {};
+    this.attributes = element.attributes;
 
     if (parent) {
       this.parent = parent;
@@ -554,14 +551,7 @@ class Element {
   }
 
   toMarkdown() {
-    switch (this.type) {
-      case "text":
-        return this.text();
-        break;
-      case "tag":
-        return this.tag().toMarkdown();
-        break;
-    }
+    return this.name === "#text" ? this.text() : this.tag().toMarkdown();
   }
 
   filterParentNames(names) {
@@ -628,7 +618,42 @@ function putPlaceholders(html) {
     match = codeRegEx.exec(origHtml);
   }
 
-  const elements = parseHTML(trimUnwanted(html));
+  const transformNode = node => {
+    if (node.nodeName !== "#text" && node.length !== undefined) {
+      const ret = [];
+      for (let i = 0; i < node.length; ++i) {
+        if (node[i].nodeName !== "#comment") {
+          ret.push(transformNode(node[i]));
+        }
+      }
+      return ret;
+    }
+
+    const ret = {
+      name: node.nodeName.toLowerCase(),
+      data: node.data,
+      children: [],
+      attributes: {}
+    };
+
+    if (node.nodeName === "#text") {
+      return ret;
+    }
+
+    for (let i = 0; i < node.childNodes.length; ++i) {
+      if (node.childNodes[i].nodeName !== "#comment") {
+        ret.children.push(transformNode(node.childNodes[i]));
+      }
+    }
+
+    for (let i = 0; i < node.attributes.length; ++i) {
+      ret.attributes[node.attributes[i].name] = node.attributes[i].value;
+    }
+
+    return ret;
+  };
+
+  const elements = transformNode($.parseHTML(trimUnwanted(html)));
   return { elements, placeholders };
 }
 

app/assets/javascripts/vendor.js

@@ -30,5 +30,4 @@
 //= require virtual-dom
 //= require virtual-dom-amd
 //= require highlight.js
-//= require htmlparser.js
 //= require intersection-observer