mirror of
https://github.com/ezyang/htmlpurifier.git
synced 2024-11-09 23:28:42 +00:00
Refactor auto-paragraph code in preparation for fundamental logic change.
git-svn-id: http://htmlpurifier.org/svnroot/htmlpurifier/trunk@1208 48356398-32a2-884e-a903-53898d9a118a
This commit is contained in:
parent
2166246b7e
commit
53fd096641
@ -21,44 +21,49 @@ class HTMLPurifier_Strategy_MakeWellFormed extends HTMLPurifier_Strategy
|
|||||||
{
|
{
|
||||||
|
|
||||||
function execute($tokens, $config, &$context) {
|
function execute($tokens, $config, &$context) {
|
||||||
|
|
||||||
$definition = $config->getHTMLDefinition();
|
$definition = $config->getHTMLDefinition();
|
||||||
$generator = new HTMLPurifier_Generator();
|
$generator = new HTMLPurifier_Generator();
|
||||||
$result = array();
|
|
||||||
$current_nesting = array();
|
$current_nesting = array();
|
||||||
|
$context->register('CurrentNesting', $current_nesting);
|
||||||
|
|
||||||
|
$tokens_index = null;
|
||||||
|
$context->register('InputIndex', $tokens_index);
|
||||||
|
$context->register('InputTokens', $tokens);
|
||||||
|
|
||||||
|
$result = array();
|
||||||
|
$context->register('OutputTokens', $result);
|
||||||
|
|
||||||
$escape_invalid_tags = $config->get('Core', 'EscapeInvalidTags');
|
$escape_invalid_tags = $config->get('Core', 'EscapeInvalidTags');
|
||||||
$auto_paragraph = $config->get('Core', 'AutoParagraph');
|
$auto_paragraph = $config->get('Core', 'AutoParagraph');
|
||||||
|
|
||||||
for ($k = 0, $tokens_count = count($tokens); $k < $tokens_count; $k++) {
|
for ($tokens_index = 0; isset($tokens[$tokens_index]); $tokens_index++) {
|
||||||
$token = $tokens[$k];
|
|
||||||
|
// if all goes well, this token will be passed through unharmed
|
||||||
|
$token = $tokens[$tokens_index];
|
||||||
|
|
||||||
|
// quick-check: if it's not a tag, no need to process
|
||||||
if (empty( $token->is_tag )) {
|
if (empty( $token->is_tag )) {
|
||||||
if ($auto_paragraph && $token->type === 'text') {
|
if ($auto_paragraph && $token->type === 'text') {
|
||||||
$this->autoParagraphText($result, $current_nesting, $tokens, $k, $token, $context, $config);
|
$this->autoParagraphText($token, $context, $config);
|
||||||
}
|
}
|
||||||
if ($token) $result[] = $token;
|
if ($token) $result[] = $token;
|
||||||
continue;
|
continue;
|
||||||
}
|
}
|
||||||
|
|
||||||
// DEFINITION CALL
|
|
||||||
$info = $definition->info[$token->name]->child;
|
$info = $definition->info[$token->name]->child;
|
||||||
|
|
||||||
// test if it claims to be a start tag but is empty
|
// test if it claims to be a start tag but is empty
|
||||||
if ($info->type == 'empty' &&
|
if ($info->type == 'empty' && $token->type == 'start') {
|
||||||
$token->type == 'start' ) {
|
$result[] = new HTMLPurifier_Token_Empty($token->name, $token->attr);
|
||||||
|
|
||||||
$result[] = new HTMLPurifier_Token_Empty($token->name,
|
|
||||||
$token->attr);
|
|
||||||
continue;
|
continue;
|
||||||
}
|
}
|
||||||
|
|
||||||
// test if it claims to be empty but really is a start tag
|
// test if it claims to be empty but really is a start tag
|
||||||
if ($info->type != 'empty' &&
|
if ($info->type != 'empty' && $token->type == 'empty' ) {
|
||||||
$token->type == 'empty' ) {
|
$result[] = new HTMLPurifier_Token_Start($token->name, $token->attr);
|
||||||
|
|
||||||
$result[] = new HTMLPurifier_Token_Start($token->name,
|
|
||||||
$token->attr);
|
|
||||||
$result[] = new HTMLPurifier_Token_End($token->name);
|
$result[] = new HTMLPurifier_Token_End($token->name);
|
||||||
|
|
||||||
continue;
|
continue;
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -68,21 +73,21 @@ class HTMLPurifier_Strategy_MakeWellFormed extends HTMLPurifier_Strategy
|
|||||||
continue;
|
continue;
|
||||||
}
|
}
|
||||||
|
|
||||||
// we give start tags precedence, so automatically accept unless...
|
// start tags have precedence, so they get passed through...
|
||||||
// it's one of those special cases
|
|
||||||
if ($token->type == 'start') {
|
if ($token->type == 'start') {
|
||||||
|
|
||||||
// if there's a parent, check for special case
|
// ...unless they also have to close their parent
|
||||||
if (!empty($current_nesting)) {
|
if (!empty($current_nesting)) {
|
||||||
|
|
||||||
$parent = array_pop($current_nesting);
|
$parent = array_pop($current_nesting);
|
||||||
$parent_name = $parent->name;
|
$parent_info = $definition->info[$parent->name];
|
||||||
$parent_info = $definition->info[$parent_name];
|
|
||||||
|
|
||||||
// we need to replace this with a more general
|
// this can be replaced with a more general algorithm:
|
||||||
// algorithm
|
// if the token is not allowed by the parent, auto-close
|
||||||
|
// the parent
|
||||||
if (isset($parent_info->auto_close[$token->name])) {
|
if (isset($parent_info->auto_close[$token->name])) {
|
||||||
$result[] = new HTMLPurifier_Token_End($parent_name);
|
// close th e parent, then append the token
|
||||||
|
$result[] = new HTMLPurifier_Token_End($parent->name);
|
||||||
$result[] = $token;
|
$result[] = $token;
|
||||||
$current_nesting[] = $token;
|
$current_nesting[] = $token;
|
||||||
continue;
|
continue;
|
||||||
@ -91,18 +96,16 @@ class HTMLPurifier_Strategy_MakeWellFormed extends HTMLPurifier_Strategy
|
|||||||
$current_nesting[] = $parent; // undo the pop
|
$current_nesting[] = $parent; // undo the pop
|
||||||
}
|
}
|
||||||
|
|
||||||
if ($auto_paragraph) $this->autoParagraphStart($result, $current_nesting, $tokens, $k, $token, $context, $config);
|
if ($auto_paragraph) $this->autoParagraphStart($token, $context, $config);
|
||||||
|
|
||||||
$result[] = $token;
|
if ($token) $result[] = $token;
|
||||||
$current_nesting[] = $token;
|
$current_nesting[] = $token;
|
||||||
continue;
|
continue;
|
||||||
}
|
}
|
||||||
|
|
||||||
// sanity check
|
// sanity check: we should be dealing with a closing tag
|
||||||
if ($token->type != 'end') continue;
|
if ($token->type != 'end') continue;
|
||||||
|
|
||||||
// okay, we're dealing with a closing tag
|
|
||||||
|
|
||||||
// make sure that we have something open
|
// make sure that we have something open
|
||||||
if (empty($current_nesting)) {
|
if (empty($current_nesting)) {
|
||||||
if ($escape_invalid_tags) {
|
if ($escape_invalid_tags) {
|
||||||
@ -115,20 +118,19 @@ class HTMLPurifier_Strategy_MakeWellFormed extends HTMLPurifier_Strategy
|
|||||||
|
|
||||||
// first, check for the simplest case: everything closes neatly
|
// first, check for the simplest case: everything closes neatly
|
||||||
|
|
||||||
// current_nesting is modified
|
|
||||||
$current_parent = array_pop($current_nesting);
|
$current_parent = array_pop($current_nesting);
|
||||||
if ($current_parent->name == $token->name) {
|
if ($current_parent->name == $token->name) {
|
||||||
$result[] = $token;
|
$result[] = $token;
|
||||||
continue;
|
continue;
|
||||||
}
|
}
|
||||||
|
|
||||||
// undo the array_pop
|
|
||||||
$current_nesting[] = $current_parent;
|
|
||||||
|
|
||||||
// okay, so we're trying to close the wrong tag
|
// okay, so we're trying to close the wrong tag
|
||||||
|
|
||||||
// scroll back the entire nest, trying to find our tag
|
// undo the pop previous pop
|
||||||
// feature could be to specify how far you'd like to go
|
$current_nesting[] = $current_parent;
|
||||||
|
|
||||||
|
// scroll back the entire nest, trying to find our tag.
|
||||||
|
// (feature could be to specify how far you'd like to go)
|
||||||
$size = count($current_nesting);
|
$size = count($current_nesting);
|
||||||
// -2 because -1 is the last element, but we already checked that
|
// -2 because -1 is the last element, but we already checked that
|
||||||
$skipped_tags = false;
|
$skipped_tags = false;
|
||||||
@ -140,7 +142,7 @@ class HTMLPurifier_Strategy_MakeWellFormed extends HTMLPurifier_Strategy
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
// we still didn't find the tag, so translate to text
|
// we still didn't find the tag, so remove
|
||||||
if ($skipped_tags === false) {
|
if ($skipped_tags === false) {
|
||||||
if ($escape_invalid_tags) {
|
if ($escape_invalid_tags) {
|
||||||
$result[] = new HTMLPurifier_Token_Text(
|
$result[] = new HTMLPurifier_Token_Text(
|
||||||
@ -157,8 +159,6 @@ class HTMLPurifier_Strategy_MakeWellFormed extends HTMLPurifier_Strategy
|
|||||||
$result[] = new HTMLPurifier_Token_End($skipped_tags[$i]->name);
|
$result[] = new HTMLPurifier_Token_End($skipped_tags[$i]->name);
|
||||||
}
|
}
|
||||||
|
|
||||||
// done!
|
|
||||||
|
|
||||||
}
|
}
|
||||||
|
|
||||||
// we're at the end now, fix all still unclosed tags
|
// we're at the end now, fix all still unclosed tags
|
||||||
@ -171,6 +171,11 @@ class HTMLPurifier_Strategy_MakeWellFormed extends HTMLPurifier_Strategy
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
$context->destroy('CurrentNesting');
|
||||||
|
$context->destroy('InputTokens');
|
||||||
|
$context->destroy('InputIndex');
|
||||||
|
$context->destroy('OutputTokens');
|
||||||
|
|
||||||
return $result;
|
return $result;
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -181,15 +186,17 @@ class HTMLPurifier_Strategy_MakeWellFormed extends HTMLPurifier_Strategy
|
|||||||
* @note This function does not care at all about ending paragraph
|
* @note This function does not care at all about ending paragraph
|
||||||
* tags: the rest of MakeWellFormed handles that!
|
* tags: the rest of MakeWellFormed handles that!
|
||||||
*/
|
*/
|
||||||
function autoParagraphText(&$result, &$current_nesting, $tokens, $k, &$token, &$context, $config) {
|
function autoParagraphText(&$token, &$context, $config) {
|
||||||
$dnl = PHP_EOL . PHP_EOL; // double-newline
|
$dnl = PHP_EOL . PHP_EOL; // double-newline
|
||||||
|
$current_nesting =& $context->get('CurrentNesting');
|
||||||
// paragraphing is on
|
// paragraphing is on
|
||||||
if (empty($current_nesting)) {
|
if (empty($current_nesting)) {
|
||||||
// we're in root node, great time to start a paragraph
|
// we're in root node, great time to start a paragraph
|
||||||
// since we're also dealing with a text node
|
// since we're also dealing with a text node
|
||||||
|
$result =& $context->get('OutputTokens');
|
||||||
$result[] = new HTMLPurifier_Token_Start('p');
|
$result[] = new HTMLPurifier_Token_Start('p');
|
||||||
$current_nesting[] = new HTMLPurifier_Token_Start('p');
|
$current_nesting[] = new HTMLPurifier_Token_Start('p');
|
||||||
$this->autoParagraphSplitText($result, $current_nesting, $tokens, $k, $token, $context, $config);
|
$this->autoParagraphSplitText($token, $context, $config);
|
||||||
} else {
|
} else {
|
||||||
// we're not in root node, so let's see whether or not
|
// we're not in root node, so let's see whether or not
|
||||||
// we're in a paragraph
|
// we're in a paragraph
|
||||||
@ -199,7 +206,7 @@ class HTMLPurifier_Strategy_MakeWellFormed extends HTMLPurifier_Strategy
|
|||||||
$current_nesting[] = $parent;
|
$current_nesting[] = $parent;
|
||||||
|
|
||||||
if ($parent->name === 'p') {
|
if ($parent->name === 'p') {
|
||||||
$this->autoParagraphSplitText($result, $current_nesting, $tokens, $k, $token, $context, $config);
|
$this->autoParagraphSplitText($token, $context, $config);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
@ -209,9 +216,10 @@ class HTMLPurifier_Strategy_MakeWellFormed extends HTMLPurifier_Strategy
|
|||||||
* up into paragraphs unconditionally. Requires that a paragraph was
|
* up into paragraphs unconditionally. Requires that a paragraph was
|
||||||
* already started
|
* already started
|
||||||
*/
|
*/
|
||||||
function autoParagraphSplitText(&$result, &$current_nesting, $tokens, $k, &$token, &$context, $config) {
|
function autoParagraphSplitText(&$token, &$context, $config) {
|
||||||
$dnl = PHP_EOL . PHP_EOL; // double-newline
|
$dnl = PHP_EOL . PHP_EOL; // double-newline
|
||||||
$definition = $config->getHTMLDefinition();
|
$definition = $config->getHTMLDefinition();
|
||||||
|
$current_nesting =& $context->get('CurrentNesting');
|
||||||
|
|
||||||
$raw_paragraphs = explode($dnl, $token->data);
|
$raw_paragraphs = explode($dnl, $token->data);
|
||||||
|
|
||||||
@ -223,6 +231,8 @@ class HTMLPurifier_Strategy_MakeWellFormed extends HTMLPurifier_Strategy
|
|||||||
if (trim($par) !== '') $paragraphs[] = $par;
|
if (trim($par) !== '') $paragraphs[] = $par;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
$result =& $context->get('OutputTokens');
|
||||||
|
|
||||||
if (empty($paragraphs) && count($raw_paragraphs) > 1) {
|
if (empty($paragraphs) && count($raw_paragraphs) > 1) {
|
||||||
$result[] = new HTMLPurifier_Token_End('p');
|
$result[] = new HTMLPurifier_Token_End('p');
|
||||||
array_pop($current_nesting);
|
array_pop($current_nesting);
|
||||||
@ -238,7 +248,11 @@ class HTMLPurifier_Strategy_MakeWellFormed extends HTMLPurifier_Strategy
|
|||||||
|
|
||||||
// check the outside to determine whether or not
|
// check the outside to determine whether or not
|
||||||
// another start tag is needed
|
// another start tag is needed
|
||||||
$end_paragraph = $this->autoParagraphEndParagraph($tokens, $k, $definition);
|
$end_paragraph = $this->autoParagraphEndParagraph(
|
||||||
|
$context->get('InputTokens'),
|
||||||
|
$context->get('InputIndex'),
|
||||||
|
$definition
|
||||||
|
);
|
||||||
if (!$end_paragraph) {
|
if (!$end_paragraph) {
|
||||||
array_pop($result);
|
array_pop($result);
|
||||||
} else {
|
} else {
|
||||||
@ -256,8 +270,11 @@ class HTMLPurifier_Strategy_MakeWellFormed extends HTMLPurifier_Strategy
|
|||||||
$end_paragraph = false;
|
$end_paragraph = false;
|
||||||
for ($j = $k + 1; isset($tokens[$j]); $j++) {
|
for ($j = $k + 1; isset($tokens[$j]); $j++) {
|
||||||
if ($tokens[$j]->type == 'start' || $tokens[$j]->type == 'empty') {
|
if ($tokens[$j]->type == 'start' || $tokens[$j]->type == 'empty') {
|
||||||
if ($tokens[$j]->name == 'p') $end_paragraph = true;
|
if ($tokens[$j]->name == 'p') {
|
||||||
else $end_paragraph = isset($definition->info['p']->auto_close[$tokens[$j]->name]);
|
$end_paragraph = true;
|
||||||
|
} else {
|
||||||
|
$end_paragraph = isset($definition->info['p']->auto_close[$tokens[$j]->name]);
|
||||||
|
}
|
||||||
break;
|
break;
|
||||||
} elseif ($tokens[$j]->type == 'text') {
|
} elseif ($tokens[$j]->type == 'text') {
|
||||||
if (!$tokens[$j]->is_whitespace) {
|
if (!$tokens[$j]->is_whitespace) {
|
||||||
@ -276,12 +293,13 @@ class HTMLPurifier_Strategy_MakeWellFormed extends HTMLPurifier_Strategy
|
|||||||
/**
|
/**
|
||||||
* Sub-function for auto-paragraphing that processes element starts
|
* Sub-function for auto-paragraphing that processes element starts
|
||||||
*/
|
*/
|
||||||
function autoParagraphStart(&$result, &$current_nesting, $tokens, $k, &$token, &$context, $config) {
|
function autoParagraphStart(&$token, &$context, $config) {
|
||||||
|
$current_nesting =& $context->get('CurrentNesting');
|
||||||
if (!empty($current_nesting)) return;
|
if (!empty($current_nesting)) return;
|
||||||
$definition = $config->getHTMLDefinition();
|
$definition = $config->getHTMLDefinition();
|
||||||
// a better check would be to use the projected new algorithm
|
// to be replaced with new auto-auto_close algorithm
|
||||||
// for auto_close
|
|
||||||
if (isset($definition->info['p']->auto_close[$token->name])) return;
|
if (isset($definition->info['p']->auto_close[$token->name])) return;
|
||||||
|
$result =& $context->get('OutputTokens');
|
||||||
$result[] = $current_nesting[] = new HTMLPurifier_Token_Start('p');
|
$result[] = $current_nesting[] = new HTMLPurifier_Token_Start('p');
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user