Refactor tag stack

This commit is contained in:
Liam Newman 2018-08-23 11:44:10 -07:00
parent 4a1940e9c7
commit 4278f10b7f

View File

@ -171,59 +171,61 @@ function in_array(what, arr) {
return arr.indexOf(what) !== -1;
}
function TagFrame(parent, tag, parser_token, indent_level) {
function TagFrame(parent, parser_token, indent_level) {
this.parent = parent || null;
this.tag = tag || '';
this.tag = parser_token ? parser_token.tag_name : '';
this.indent_level = indent_level || 0;
this.parser_token = parser_token || null;
}
TagFrame.prototype.record_tag = function(tag, parser_token, indent_level) { //function to record a tag and its parent in this.tags Object
var new_parent = new TagFrame(this.parent, this.tag, this.parser_token, this.indent_level);
this.parent = new_parent;
this.tag = tag;
this.indent_level = indent_level;
this.parser_token = parser_token;
function TagStack(printer) {
this._printer = printer;
this._current_frame = null;
}
TagStack.prototype.get_parser_token = function() {
return this._current_frame ? this._current_frame.parser_token : null;
};
TagFrame.prototype.try_pop = function(tag, printer) { //function to retrieve the opening tag to the corresponding closer
TagStack.prototype.record_tag = function(parser_token) { //function to record a tag and its parent in this.tags Object
var new_frame = new TagFrame(this._current_frame, parser_token, this._printer.indent_level);
this._current_frame = new_frame;
};
TagStack.prototype._try_pop_frame = function(frame) { //function to retrieve the opening tag to the corresponding closer
var parser_token = null;
var temp_parent = this;
while (temp_parent) { //till we reach '' (the initial value);
if (temp_parent.tag === tag) { //if this is it use it
break;
}
temp_parent = temp_parent.parent;
}
if (temp_parent) {
parser_token = temp_parent.parser_token;
printer.indent_level = temp_parent.indent_level;
temp_parent = temp_parent.parent ? temp_parent.parent : temp_parent;
this.parent = temp_parent.parent;
this.tag = temp_parent.tag;
this.indent_level = temp_parent.indent_level;
this.parser_token = temp_parent.parser_token;
if (frame) {
parser_token = frame.parser_token;
this._printer.indent_level = frame.indent_level;
this._current_frame = frame.parent;
}
return parser_token;
};
TagFrame.prototype.indent_to_tag = function(tag_list, printer) {
var temp_parent = this;
TagStack.prototype._get_frame = function(tag_list) { //function to retrieve the opening tag to the corresponding closer
var frame = this._current_frame;
while (temp_parent) { //till we reach '' (the initial value);
if (tag_list.indexOf(temp_parent.tag) !== -1) { //if this is it use it
while (frame) { //till we reach '' (the initial value);
if (tag_list.indexOf(frame.tag) !== -1) { //if this is it use it
break;
}
temp_parent = temp_parent.parent;
frame = frame.parent;
}
if (temp_parent) {
printer.indent_level = temp_parent.indent_level;
return frame;
};
TagStack.prototype.try_pop = function(tag) { //function to retrieve the opening tag to the corresponding closer
var frame = this._get_frame([tag]);
return this._try_pop_frame(frame);
};
TagStack.prototype.indent_to_tag = function(tag_list) {
var frame = this._get_frame(tag_list);
if (frame) {
this._printer.indent_level = frame.indent_level;
}
};
@ -344,8 +346,6 @@ Beautifier.prototype.beautify = function() {
// HACK: newline parsing inconsistent. This brute force normalizes the input.
source_text = source_text.replace(allLineBreaks, '\n');
this._tag_stack = new TagFrame();
var last_token = {
text: '',
type: ''
@ -357,6 +357,8 @@ Beautifier.prototype.beautify = function() {
this._options.wrap_line_length, this._options.max_preserve_newlines, this._options.preserve_newlines);
var tokens = new Tokenizer(source_text, this._options).tokenize();
this._tag_stack = new TagStack(printer);
var parser_token = null;
var raw_token = tokens.next();
while (raw_token.type !== TOKEN.EOF) {
@ -616,7 +618,7 @@ var TagOpenParserToken = function(parent, raw_token) {
};
Beautifier.prototype._get_tag_open_token = function(raw_token) { //function to get a full tag and parse its type
var parser_token = new TagOpenParserToken(this._tag_stack.parser_token, raw_token);
var parser_token = new TagOpenParserToken(this._tag_stack.get_parser_token(), raw_token);
parser_token.alignment_size = this._options.wrap_attributes_indent_size;
@ -637,13 +639,13 @@ Beautifier.prototype._set_tag_position = function(printer, raw_token, parser_tok
if (!parser_token.is_empty_element) {
if (parser_token.is_end_tag) { //this tag is a double tag so check for tag-ending
parser_token.start_tag_token = this._tag_stack.try_pop(parser_token.tag_name, printer); //remove it and all ancestors
parser_token.start_tag_token = this._tag_stack.try_pop(parser_token.tag_name); //remove it and all ancestors
} else { // it's a start-tag
// check if this tag is starting an element that has optional end element
// and do an ending needed
this._do_optional_end_element(parser_token, printer);
this._do_optional_end_element(parser_token);
this._tag_stack.record_tag(parser_token.tag_name, parser_token, printer.indent_level); //push it on the tag stack
this._tag_stack.record_tag(parser_token); //push it on the tag stack
if ((parser_token.tag_name === 'script' || parser_token.tag_name === 'style') &&
!(parser_token.is_unformatted || parser_token.is_content_unformatted)) {
@ -662,7 +664,7 @@ Beautifier.prototype._set_tag_position = function(printer, raw_token, parser_tok
if (parser_token.is_empty_element) { //if this tag name is a single tag type (either in the list or has a closing /)
if (parser_token.tag_start_char === '{' && parser_token.tag_check === 'else') {
this._tag_stack.indent_to_tag(['if', 'unless'], printer);
this._tag_stack.indent_to_tag(['if', 'unless']);
parser_token.indent_content = true;
// Don't add a newline if opening {{#if}} tag is on the current line
var foundIfOnCurrentLine = printer.current_line_has_match(/{{#if/);
@ -706,7 +708,7 @@ Beautifier.prototype._set_tag_position = function(printer, raw_token, parser_tok
};
Beautifier.prototype._do_optional_end_element = function(parser_token, printer) {
Beautifier.prototype._do_optional_end_element = function(parser_token) {
// NOTE: cases of "if there is no more content in the parent element"
// are handled automatically by the beautifier.
// It assumes parent or ancestor close tag closes all children.
@ -716,36 +718,36 @@ Beautifier.prototype._do_optional_end_element = function(parser_token, printer)
} else if (parser_token.tag_name === 'body') {
// A head elements end tag may be omitted if the head element is not immediately followed by a space character or a comment.
this._tag_stack.try_pop('head', printer);
this._tag_stack.try_pop('head');
} else if (parser_token.parent.tag_name === 'body') {
// TODO: A body elements end tag may be omitted if the body element is not immediately followed by a comment.
} else if (parser_token.tag_name === 'li') {
// An li elements end tag may be omitted if the li element is immediately followed by another li element or if there is no more content in the parent element.
this._tag_stack.try_pop('li', printer);
this._tag_stack.try_pop('li');
} else if (parser_token.tag_name === 'dd' || parser_token.tag_name === 'dt') {
// A dd elements end tag may be omitted if the dd element is immediately followed by another dd element or a dt element, or if there is no more content in the parent element.
// A dt elements end tag may be omitted if the dt element is immediately followed by another dt element or a dd element.
this._tag_stack.try_pop('dt', printer);
this._tag_stack.try_pop('dd', printer);
this._tag_stack.try_pop('dt');
this._tag_stack.try_pop('dd');
} else if (parser_token.parent.tag_name === 'rp' || parser_token.parent.tag_name === 'rt') {
// An rt elements end tag may be omitted if the rt element is immediately followed by an rt or rp element, or if there is no more content in the parent element.
// An rp elements end tag may be omitted if the rp element is immediately followed by an rt or rp element, or if there is no more content in the parent element.
this._tag_stack.try_pop('rt', printer);
this._tag_stack.try_pop('rp', printer);
this._tag_stack.try_pop('rt');
this._tag_stack.try_pop('rp');
} else if (parser_token.tag_name === 'optgroup') {
// An optgroup elements end tag may be omitted if the optgroup element is immediately followed by another optgroup element, or if there is no more content in the parent element.
// An option elements end tag may be omitted if the option element is immediately followed by another option element, or if it is immediately followed by an optgroup element, or if there is no more content in the parent element.
this._tag_stack.try_pop('optgroup', printer);
this._tag_stack.try_pop('option', printer);
this._tag_stack.try_pop('optgroup');
this._tag_stack.try_pop('option');
} else if (parser_token.tag_name === 'option') {
// An option elements end tag may be omitted if the option element is immediately followed by another option element, or if it is immediately followed by an optgroup element, or if there is no more content in the parent element.
this._tag_stack.try_pop('option', printer);
this._tag_stack.try_pop('option');
} else if (parser_token.parent.tag_name === 'colgroup') {
// TODO: A colgroup elements end tag may be omitted if the colgroup element is not immediately followed by a space character or a comment.
@ -756,21 +758,21 @@ Beautifier.prototype._do_optional_end_element = function(parser_token, printer)
} else if (parser_token.tag_name === 'tbody' || parser_token.tag_name === 'tfoot') {
// A thead elements end tag may be omitted if the thead element is immediately followed by a tbody or tfoot element.
// A tbody elements end tag may be omitted if the tbody element is immediately followed by a tbody or tfoot element, or if there is no more content in the parent element.
this._tag_stack.try_pop('thead', printer);
this._tag_stack.try_pop('tbody', printer);
this._tag_stack.try_pop('thead');
this._tag_stack.try_pop('tbody');
} else if (parser_token.tag_name === 'tfoot') {
// TODO: A tfoot elements end tag may be omitted if there is no more content in the parent element.
} else if (parser_token.tag_name === 'tr') {
// A tr elements end tag may be omitted if the tr element is immediately followed by another tr element, or if there is no more content in the parent element.
this._tag_stack.try_pop('tr', printer);
this._tag_stack.try_pop('tr');
} else if (parser_token.tag_name === 'th' || parser_token.tag_name === 'td') {
// A td elements end tag may be omitted if the td element is immediately followed by a td or th element, or if there is no more content in the parent element.
// A th elements end tag may be omitted if the th element is immediately followed by a td or th element, or if there is no more content in the parent element.
this._tag_stack.try_pop('td', printer);
this._tag_stack.try_pop('th', printer);
this._tag_stack.try_pop('td');
this._tag_stack.try_pop('th');
}
// Start element omission not handled currently
@ -779,7 +781,7 @@ Beautifier.prototype._do_optional_end_element = function(parser_token, printer)
// A colgroup elements start tag may be omitted if the first thing inside the colgroup element is a col element, and if the element is not immediately preceded by another colgroup element whose end tag has been omitted. (It cant be omitted if the element is empty.)
// Fix up the parent of the parser token
parser_token.parent = this._tag_stack.parser_token;
parser_token.parent = this._tag_stack.get_parser_token();
};