[Suggestion] Add support for reserialisation about node-htmlparser

[tautologistics]

That is actually something I had wanted to do a while back but then life got in the way. Going to tag this as "Fix in 1.8". I guess one could call it a bug because it is missing =)

[jaap3]

Something like this does what you want. You probably want to special case some things in order to get the output you wish...

function toHTML(nodeSet, asArray) {
  var html = [];
  nodeSet.forEach(function (node) {
      switch (node.type) {
          case 'text':
          html.push(node.data);
          break;
          case 'directive':
          html.push('<!' + node.data + '>');
          break;
          case 'comment':
          html.push('<!--' + node.data + '>');
          break;
          default:
          html.push('<' + node.data + '>');
          if (node.children)
              html = html.concat(toHTML(node.children, true));
          html.push('</' + node.name + '>');
          break;
      }
  });
  if (asArray)
      return html;
  return html.join('');
} 

[tlrobinson]

This would be nice to have. In the meantime I'm using @jaap3's snippet, except I had to change the "directive" case to '<' + node.data + '>', the "comment" case to '<!--' + node.data + '-->', and strip self-closing "/" in for default tags using node.data.replace(/\s*\/$/,''), otherwise it works great.

[resistdesign]

Here is a version I fleshed out in case it helps anyone:
(NOTE: It only returns a string.)

var VOID_HTML_ELEMENT_MAP = {
  area: true,
  base: true,
  br: true,
  col: true,
  command: true,
  embed: true,
  hr: true,
  img: true,
  input: true,
  keygen: true,
  link: true,
  meta: true,
  param: true,
  source: true,
  track: true,
  wbr: true
};

function toHTML(nodeSet) {
    var html = [];

    if (nodeSet) {
      nodeSet.forEach(function(node) {
        switch (node.type) {
          case 'text':
            html.push(node.data);
            break;
          case 'directive':
            html.push('<' + node.data + '>');
            break;
          case 'comment':
            html.push('<!-- ' + node.data + ' -->');
            break;
          default:
            html.push('<' + node.data.replace(/\/$/, ''));
            if (!/\/$/.test(node.data) && !VOID_HTML_ELEMENT_MAP[node.name]) {
              html.push('>');
              if (node.children) {
                html.push(toHTML(node.children));
              }
              html.push('</' + node.name + '>');
            } else if (!/\/$/.test(node.data)) {
              html.push('>');
            } else {
              html.push('/>');
            }
            break;
        }
      });
    }

    return html.join('');
  }

I'm sure it could be optimized but it's a start :)

[resistdesign]

Just a note, the point of the VOID_HTML_ELEMENT_MAP is so that it can do something like <br> or <br/> depending on how it was in the html before it was parsed.