xpressengine / xe-core

classes/security/htmlpurifier/library/HTMLPurifier/Printer/ConfigForm.php

Indentation +316 added lines, -316 removed lines

@@ -6,179 +6,179 @@ 
 class HTMLPurifier_Printer_ConfigForm extends HTMLPurifier_Printer
 {
 
-    /**
-     * Printers for specific fields
-     */
-    protected $fields = array();
+	/**
+	 * Printers for specific fields
+	 */
+	protected $fields = array();
 
-    /**
-     * Documentation URL, can have fragment tagged on end
-     */
-    protected $docURL;
+	/**
+	 * Documentation URL, can have fragment tagged on end
+	 */
+	protected $docURL;
 
-    /**
-     * Name of form element to stuff config in
-     */
-    protected $name;
+	/**
+	 * Name of form element to stuff config in
+	 */
+	protected $name;
 
-    /**
-     * Whether or not to compress directive names, clipping them off
-     * after a certain amount of letters. False to disable or integer letters
-     * before clipping.
-     */
-    protected $compress = false;
+	/**
+	 * Whether or not to compress directive names, clipping them off
+	 * after a certain amount of letters. False to disable or integer letters
+	 * before clipping.
+	 */
+	protected $compress = false;
 
-    /**
-     * @param $name Form element name for directives to be stuffed into
-     * @param $doc_url String documentation URL, will have fragment tagged on
-     * @param $compress Integer max length before compressing a directive name, set to false to turn off
-     */
-    public function __construct(
-        $name, $doc_url = null, $compress = false
-    ) {
-        parent::__construct();
-        $this->docURL = $doc_url;
-        $this->name   = $name;
-        $this->compress = $compress;
-        // initialize sub-printers
-        $this->fields[0]    = new HTMLPurifier_Printer_ConfigForm_default();
-        $this->fields[HTMLPurifier_VarParser::BOOL]       = new HTMLPurifier_Printer_ConfigForm_bool();
-    }
+	/**
+	 * @param $name Form element name for directives to be stuffed into
+	 * @param $doc_url String documentation URL, will have fragment tagged on
+	 * @param $compress Integer max length before compressing a directive name, set to false to turn off
+	 */
+	public function __construct(
+		$name, $doc_url = null, $compress = false
+	) {
+		parent::__construct();
+		$this->docURL = $doc_url;
+		$this->name   = $name;
+		$this->compress = $compress;
+		// initialize sub-printers
+		$this->fields[0]    = new HTMLPurifier_Printer_ConfigForm_default();
+		$this->fields[HTMLPurifier_VarParser::BOOL]       = new HTMLPurifier_Printer_ConfigForm_bool();
+	}
 
-    /**
-     * Sets default column and row size for textareas in sub-printers
-     * @param $cols Integer columns of textarea, null to use default
-     * @param $rows Integer rows of textarea, null to use default
-     */
-    public function setTextareaDimensions($cols = null, $rows = null) {
-        if ($cols) $this->fields['default']->cols = $cols;
-        if ($rows) $this->fields['default']->rows = $rows;
-    }
+	/**
+	 * Sets default column and row size for textareas in sub-printers
+	 * @param $cols Integer columns of textarea, null to use default
+	 * @param $rows Integer rows of textarea, null to use default
+	 */
+	public function setTextareaDimensions($cols = null, $rows = null) {
+		if ($cols) $this->fields['default']->cols = $cols;
+		if ($rows) $this->fields['default']->rows = $rows;
+	}
 
-    /**
-     * Retrieves styling, in case it is not accessible by webserver
-     */
-    public static function getCSS() {
-        return file_get_contents(HTMLPURIFIER_PREFIX . '/HTMLPurifier/Printer/ConfigForm.css');
-    }
+	/**
+	 * Retrieves styling, in case it is not accessible by webserver
+	 */
+	public static function getCSS() {
+		return file_get_contents(HTMLPURIFIER_PREFIX . '/HTMLPurifier/Printer/ConfigForm.css');
+	}
 
-    /**
-     * Retrieves JavaScript, in case it is not accessible by webserver
-     */
-    public static function getJavaScript() {
-        return file_get_contents(HTMLPURIFIER_PREFIX . '/HTMLPurifier/Printer/ConfigForm.js');
-    }
+	/**
+	 * Retrieves JavaScript, in case it is not accessible by webserver
+	 */
+	public static function getJavaScript() {
+		return file_get_contents(HTMLPURIFIER_PREFIX . '/HTMLPurifier/Printer/ConfigForm.js');
+	}
 
-    /**
-     * Returns HTML output for a configuration form
-     * @param $config Configuration object of current form state, or an array
-     *        where [0] has an HTML namespace and [1] is being rendered.
-     * @param $allowed Optional namespace(s) and directives to restrict form to.
-     */
-    public function render($config, $allowed = true, $render_controls = true) {
-        if (is_array($config) && isset($config[0])) {
-            $gen_config = $config[0];
-            $config = $config[1];
-        } else {
-            $gen_config = $config;
-        }
+	/**
+	 * Returns HTML output for a configuration form
+	 * @param $config Configuration object of current form state, or an array
+	 *        where [0] has an HTML namespace and [1] is being rendered.
+	 * @param $allowed Optional namespace(s) and directives to restrict form to.
+	 */
+	public function render($config, $allowed = true, $render_controls = true) {
+		if (is_array($config) && isset($config[0])) {
+			$gen_config = $config[0];
+			$config = $config[1];
+		} else {
+			$gen_config = $config;
+		}
 
-        $this->config = $config;
-        $this->genConfig = $gen_config;
-        $this->prepareGenerator($gen_config);
+		$this->config = $config;
+		$this->genConfig = $gen_config;
+		$this->prepareGenerator($gen_config);
 
-        $allowed = HTMLPurifier_Config::getAllowedDirectivesForForm($allowed, $config->def);
-        $all = array();
-        foreach ($allowed as $key) {
-            list($ns, $directive) = $key;
-            $all[$ns][$directive] = $config->get($ns .'.'. $directive);
-        }
+		$allowed = HTMLPurifier_Config::getAllowedDirectivesForForm($allowed, $config->def);
+		$all = array();
+		foreach ($allowed as $key) {
+			list($ns, $directive) = $key;
+			$all[$ns][$directive] = $config->get($ns .'.'. $directive);
+		}
 
-        $ret = '';
-        $ret .= $this->start('table', array('class' => 'hp-config'));
-        $ret .= $this->start('thead');
-        $ret .= $this->start('tr');
-            $ret .= $this->element('th', 'Directive', array('class' => 'hp-directive'));
-            $ret .= $this->element('th', 'Value', array('class' => 'hp-value'));
-        $ret .= $this->end('tr');
-        $ret .= $this->end('thead');
-        foreach ($all as $ns => $directives) {
-            $ret .= $this->renderNamespace($ns, $directives);
-        }
-        if ($render_controls) {
-             $ret .= $this->start('tbody');
-             $ret .= $this->start('tr');
-                 $ret .= $this->start('td', array('colspan' => 2, 'class' => 'controls'));
-                     $ret .= $this->elementEmpty('input', array('type' => 'submit', 'value' => 'Submit'));
-                     $ret .= '[<a href="?">Reset</a>]';
-                 $ret .= $this->end('td');
-             $ret .= $this->end('tr');
-             $ret .= $this->end('tbody');
-        }
-        $ret .= $this->end('table');
-        return $ret;
-    }
+		$ret = '';
+		$ret .= $this->start('table', array('class' => 'hp-config'));
+		$ret .= $this->start('thead');
+		$ret .= $this->start('tr');
+			$ret .= $this->element('th', 'Directive', array('class' => 'hp-directive'));
+			$ret .= $this->element('th', 'Value', array('class' => 'hp-value'));
+		$ret .= $this->end('tr');
+		$ret .= $this->end('thead');
+		foreach ($all as $ns => $directives) {
+			$ret .= $this->renderNamespace($ns, $directives);
+		}
+		if ($render_controls) {
+			 $ret .= $this->start('tbody');
+			 $ret .= $this->start('tr');
+				 $ret .= $this->start('td', array('colspan' => 2, 'class' => 'controls'));
+					 $ret .= $this->elementEmpty('input', array('type' => 'submit', 'value' => 'Submit'));
+					 $ret .= '[<a href="?">Reset</a>]';
+				 $ret .= $this->end('td');
+			 $ret .= $this->end('tr');
+			 $ret .= $this->end('tbody');
+		}
+		$ret .= $this->end('table');
+		return $ret;
+	}
 
-    /**
-     * Renders a single namespace
-     * @param $ns String namespace name
-     * @param $directive Associative array of directives to values
-     */
-    protected function renderNamespace($ns, $directives) {
-        $ret = '';
-        $ret .= $this->start('tbody', array('class' => 'namespace'));
-        $ret .= $this->start('tr');
-            $ret .= $this->element('th', $ns, array('colspan' => 2));
-        $ret .= $this->end('tr');
-        $ret .= $this->end('tbody');
-        $ret .= $this->start('tbody');
-        foreach ($directives as $directive => $value) {
-            $ret .= $this->start('tr');
-            $ret .= $this->start('th');
-            if ($this->docURL) {
-                $url = str_replace('%s', urlencode("$ns.$directive"), $this->docURL);
-                $ret .= $this->start('a', array('href' => $url));
-            }
-                $attr = array('for' => "{$this->name}:$ns.$directive");
+	/**
+	 * Renders a single namespace
+	 * @param $ns String namespace name
+	 * @param $directive Associative array of directives to values
+	 */
+	protected function renderNamespace($ns, $directives) {
+		$ret = '';
+		$ret .= $this->start('tbody', array('class' => 'namespace'));
+		$ret .= $this->start('tr');
+			$ret .= $this->element('th', $ns, array('colspan' => 2));
+		$ret .= $this->end('tr');
+		$ret .= $this->end('tbody');
+		$ret .= $this->start('tbody');
+		foreach ($directives as $directive => $value) {
+			$ret .= $this->start('tr');
+			$ret .= $this->start('th');
+			if ($this->docURL) {
+				$url = str_replace('%s', urlencode("$ns.$directive"), $this->docURL);
+				$ret .= $this->start('a', array('href' => $url));
+			}
+				$attr = array('for' => "{$this->name}:$ns.$directive");
 
-                // crop directive name if it's too long
-                if (!$this->compress || (strlen($directive) < $this->compress)) {
-                    $directive_disp = $directive;
-                } else {
-                    $directive_disp = substr($directive, 0, $this->compress - 2) . '...';
-                    $attr['title'] = $directive;
-                }
+				// crop directive name if it's too long
+				if (!$this->compress || (strlen($directive) < $this->compress)) {
+					$directive_disp = $directive;
+				} else {
+					$directive_disp = substr($directive, 0, $this->compress - 2) . '...';
+					$attr['title'] = $directive;
+				}
 
-                $ret .= $this->element(
-                    'label',
-                    $directive_disp,
-                    // component printers must create an element with this id
-                    $attr
-                );
-            if ($this->docURL) $ret .= $this->end('a');
-            $ret .= $this->end('th');
+				$ret .= $this->element(
+					'label',
+					$directive_disp,
+					// component printers must create an element with this id
+					$attr
+				);
+			if ($this->docURL) $ret .= $this->end('a');
+			$ret .= $this->end('th');
 
-            $ret .= $this->start('td');
-                $def = $this->config->def->info["$ns.$directive"];
-                if (is_int($def)) {
-                    $allow_null = $def < 0;
-                    $type = abs($def);
-                } else {
-                    $type = $def->type;
-                    $allow_null = isset($def->allow_null);
-                }
-                if (!isset($this->fields[$type])) $type = 0; // default
-                $type_obj = $this->fields[$type];
-                if ($allow_null) {
-                    $type_obj = new HTMLPurifier_Printer_ConfigForm_NullDecorator($type_obj);
-                }
-                $ret .= $type_obj->render($ns, $directive, $value, $this->name, array($this->genConfig, $this->config));
-            $ret .= $this->end('td');
-            $ret .= $this->end('tr');
-        }
-        $ret .= $this->end('tbody');
-        return $ret;
-    }
+			$ret .= $this->start('td');
+				$def = $this->config->def->info["$ns.$directive"];
+				if (is_int($def)) {
+					$allow_null = $def < 0;
+					$type = abs($def);
+				} else {
+					$type = $def->type;
+					$allow_null = isset($def->allow_null);
+				}
+				if (!isset($this->fields[$type])) $type = 0; // default
+				$type_obj = $this->fields[$type];
+				if ($allow_null) {
+					$type_obj = new HTMLPurifier_Printer_ConfigForm_NullDecorator($type_obj);
+				}
+				$ret .= $type_obj->render($ns, $directive, $value, $this->name, array($this->genConfig, $this->config));
+			$ret .= $this->end('td');
+			$ret .= $this->end('tr');
+		}
+		$ret .= $this->end('tbody');
+		return $ret;
+	}
 
 }
 
@@ -186,183 +186,183 @@ 
  * Printer decorator for directives that accept null
  */
 class HTMLPurifier_Printer_ConfigForm_NullDecorator extends HTMLPurifier_Printer {
-    /**
-     * Printer being decorated
-     */
-    protected $obj;
-    /**
-     * @param $obj Printer to decorate
-     */
-    public function __construct($obj) {
-        parent::__construct();
-        $this->obj = $obj;
-    }
-    public function render($ns, $directive, $value, $name, $config) {
-        if (is_array($config) && isset($config[0])) {
-            $gen_config = $config[0];
-            $config = $config[1];
-        } else {
-            $gen_config = $config;
-        }
-        $this->prepareGenerator($gen_config);
+	/**
+	 * Printer being decorated
+	 */
+	protected $obj;
+	/**
+	 * @param $obj Printer to decorate
+	 */
+	public function __construct($obj) {
+		parent::__construct();
+		$this->obj = $obj;
+	}
+	public function render($ns, $directive, $value, $name, $config) {
+		if (is_array($config) && isset($config[0])) {
+			$gen_config = $config[0];
+			$config = $config[1];
+		} else {
+			$gen_config = $config;
+		}
+		$this->prepareGenerator($gen_config);
 
-        $ret = '';
-        $ret .= $this->start('label', array('for' => "$name:Null_$ns.$directive"));
-        $ret .= $this->element('span', "$ns.$directive:", array('class' => 'verbose'));
-        $ret .= $this->text(' Null/Disabled');
-        $ret .= $this->end('label');
-        $attr = array(
-            'type' => 'checkbox',
-            'value' => '1',
-            'class' => 'null-toggle',
-            'name' => "$name"."[Null_$ns.$directive]",
-            'id' => "$name:Null_$ns.$directive",
-            'onclick' => "toggleWriteability('$name:$ns.$directive',checked)" // INLINE JAVASCRIPT!!!!
-        );
-        if ($this->obj instanceof HTMLPurifier_Printer_ConfigForm_bool) {
-            // modify inline javascript slightly
-            $attr['onclick'] = "toggleWriteability('$name:Yes_$ns.$directive',checked);toggleWriteability('$name:No_$ns.$directive',checked)";
-        }
-        if ($value === null) $attr['checked'] = 'checked';
-        $ret .= $this->elementEmpty('input', $attr);
-        $ret .= $this->text(' or ');
-        $ret .= $this->elementEmpty('br');
-        $ret .= $this->obj->render($ns, $directive, $value, $name, array($gen_config, $config));
-        return $ret;
-    }
+		$ret = '';
+		$ret .= $this->start('label', array('for' => "$name:Null_$ns.$directive"));
+		$ret .= $this->element('span', "$ns.$directive:", array('class' => 'verbose'));
+		$ret .= $this->text(' Null/Disabled');
+		$ret .= $this->end('label');
+		$attr = array(
+			'type' => 'checkbox',
+			'value' => '1',
+			'class' => 'null-toggle',
+			'name' => "$name"."[Null_$ns.$directive]",
+			'id' => "$name:Null_$ns.$directive",
+			'onclick' => "toggleWriteability('$name:$ns.$directive',checked)" // INLINE JAVASCRIPT!!!!
+		);
+		if ($this->obj instanceof HTMLPurifier_Printer_ConfigForm_bool) {
+			// modify inline javascript slightly
+			$attr['onclick'] = "toggleWriteability('$name:Yes_$ns.$directive',checked);toggleWriteability('$name:No_$ns.$directive',checked)";
+		}
+		if ($value === null) $attr['checked'] = 'checked';
+		$ret .= $this->elementEmpty('input', $attr);
+		$ret .= $this->text(' or ');
+		$ret .= $this->elementEmpty('br');
+		$ret .= $this->obj->render($ns, $directive, $value, $name, array($gen_config, $config));
+		return $ret;
+	}
 }
 
 /**
  * Swiss-army knife configuration form field printer
  */
 class HTMLPurifier_Printer_ConfigForm_default extends HTMLPurifier_Printer {
-    public $cols = 18;
-    public $rows = 5;
-    public function render($ns, $directive, $value, $name, $config) {
-        if (is_array($config) && isset($config[0])) {
-            $gen_config = $config[0];
-            $config = $config[1];
-        } else {
-            $gen_config = $config;
-        }
-        $this->prepareGenerator($gen_config);
-        // this should probably be split up a little
-        $ret = '';
-        $def = $config->def->info["$ns.$directive"];
-        if (is_int($def)) {
-            $type = abs($def);
-        } else {
-            $type = $def->type;
-        }
-        if (is_array($value)) {
-            switch ($type) {
-                case HTMLPurifier_VarParser::LOOKUP:
-                    $array = $value;
-                    $value = array();
-                    foreach ($array as $val => $b) {
-                        $value[] = $val;
-                    }
-                case HTMLPurifier_VarParser::ALIST:
-                    $value = implode(PHP_EOL, $value);
-                    break;
-                case HTMLPurifier_VarParser::HASH:
-                    $nvalue = '';
-                    foreach ($value as $i => $v) {
-                        $nvalue .= "$i:$v" . PHP_EOL;
-                    }
-                    $value = $nvalue;
-                    break;
-                default:
-                    $value = '';
-            }
-        }
-        if ($type === HTMLPurifier_VarParser::MIXED) {
-            return 'Not supported';
-            $value = serialize($value);
-        }
-        $attr = array(
-            'name' => "$name"."[$ns.$directive]",
-            'id' => "$name:$ns.$directive"
-        );
-        if ($value === null) $attr['disabled'] = 'disabled';
-        if (isset($def->allowed)) {
-            $ret .= $this->start('select', $attr);
-            foreach ($def->allowed as $val => $b) {
-                $attr = array();
-                if ($value == $val) $attr['selected'] = 'selected';
-                $ret .= $this->element('option', $val, $attr);
-            }
-            $ret .= $this->end('select');
-        } elseif (
-            $type === HTMLPurifier_VarParser::TEXT ||
-            $type === HTMLPurifier_VarParser::ITEXT ||
-            $type === HTMLPurifier_VarParser::ALIST ||
-            $type === HTMLPurifier_VarParser::HASH ||
-            $type === HTMLPurifier_VarParser::LOOKUP
-        ) {
-            $attr['cols'] = $this->cols;
-            $attr['rows'] = $this->rows;
-            $ret .= $this->start('textarea', $attr);
-            $ret .= $this->text($value);
-            $ret .= $this->end('textarea');
-        } else {
-            $attr['value'] = $value;
-            $attr['type'] = 'text';
-            $ret .= $this->elementEmpty('input', $attr);
-        }
-        return $ret;
-    }
+	public $cols = 18;
+	public $rows = 5;
+	public function render($ns, $directive, $value, $name, $config) {
+		if (is_array($config) && isset($config[0])) {
+			$gen_config = $config[0];
+			$config = $config[1];
+		} else {
+			$gen_config = $config;
+		}
+		$this->prepareGenerator($gen_config);
+		// this should probably be split up a little
+		$ret = '';
+		$def = $config->def->info["$ns.$directive"];
+		if (is_int($def)) {
+			$type = abs($def);
+		} else {
+			$type = $def->type;
+		}
+		if (is_array($value)) {
+			switch ($type) {
+				case HTMLPurifier_VarParser::LOOKUP:
+					$array = $value;
+					$value = array();
+					foreach ($array as $val => $b) {
+						$value[] = $val;
+					}
+				case HTMLPurifier_VarParser::ALIST:
+					$value = implode(PHP_EOL, $value);
+					break;
+				case HTMLPurifier_VarParser::HASH:
+					$nvalue = '';
+					foreach ($value as $i => $v) {
+						$nvalue .= "$i:$v" . PHP_EOL;
+					}
+					$value = $nvalue;
+					break;
+				default:
+					$value = '';
+			}
+		}
+		if ($type === HTMLPurifier_VarParser::MIXED) {
+			return 'Not supported';
+			$value = serialize($value);
+		}
+		$attr = array(
+			'name' => "$name"."[$ns.$directive]",
+			'id' => "$name:$ns.$directive"
+		);
+		if ($value === null) $attr['disabled'] = 'disabled';
+		if (isset($def->allowed)) {
+			$ret .= $this->start('select', $attr);
+			foreach ($def->allowed as $val => $b) {
+				$attr = array();
+				if ($value == $val) $attr['selected'] = 'selected';
+				$ret .= $this->element('option', $val, $attr);
+			}
+			$ret .= $this->end('select');
+		} elseif (
+			$type === HTMLPurifier_VarParser::TEXT ||
+			$type === HTMLPurifier_VarParser::ITEXT ||
+			$type === HTMLPurifier_VarParser::ALIST ||
+			$type === HTMLPurifier_VarParser::HASH ||
+			$type === HTMLPurifier_VarParser::LOOKUP
+		) {
+			$attr['cols'] = $this->cols;
+			$attr['rows'] = $this->rows;
+			$ret .= $this->start('textarea', $attr);
+			$ret .= $this->text($value);
+			$ret .= $this->end('textarea');
+		} else {
+			$attr['value'] = $value;
+			$attr['type'] = 'text';
+			$ret .= $this->elementEmpty('input', $attr);
+		}
+		return $ret;
+	}
 }
 
 /**
  * Bool form field printer
  */
 class HTMLPurifier_Printer_ConfigForm_bool extends HTMLPurifier_Printer {
-    public function render($ns, $directive, $value, $name, $config) {
-        if (is_array($config) && isset($config[0])) {
-            $gen_config = $config[0];
-            $config = $config[1];
-        } else {
-            $gen_config = $config;
-        }
-        $this->prepareGenerator($gen_config);
-        $ret = '';
-        $ret .= $this->start('div', array('id' => "$name:$ns.$directive"));
+	public function render($ns, $directive, $value, $name, $config) {
+		if (is_array($config) && isset($config[0])) {
+			$gen_config = $config[0];
+			$config = $config[1];
+		} else {
+			$gen_config = $config;
+		}
+		$this->prepareGenerator($gen_config);
+		$ret = '';
+		$ret .= $this->start('div', array('id' => "$name:$ns.$directive"));
 
-        $ret .= $this->start('label', array('for' => "$name:Yes_$ns.$directive"));
-        $ret .= $this->element('span', "$ns.$directive:", array('class' => 'verbose'));
-        $ret .= $this->text(' Yes');
-        $ret .= $this->end('label');
+		$ret .= $this->start('label', array('for' => "$name:Yes_$ns.$directive"));
+		$ret .= $this->element('span', "$ns.$directive:", array('class' => 'verbose'));
+		$ret .= $this->text(' Yes');
+		$ret .= $this->end('label');
 
-        $attr = array(
-            'type' => 'radio',
-            'name' => "$name"."[$ns.$directive]",
-            'id' => "$name:Yes_$ns.$directive",
-            'value' => '1'
-        );
-        if ($value === true) $attr['checked'] = 'checked';
-        if ($value === null) $attr['disabled'] = 'disabled';
-        $ret .= $this->elementEmpty('input', $attr);
+		$attr = array(
+			'type' => 'radio',
+			'name' => "$name"."[$ns.$directive]",
+			'id' => "$name:Yes_$ns.$directive",
+			'value' => '1'
+		);
+		if ($value === true) $attr['checked'] = 'checked';
+		if ($value === null) $attr['disabled'] = 'disabled';
+		$ret .= $this->elementEmpty('input', $attr);
 
-        $ret .= $this->start('label', array('for' => "$name:No_$ns.$directive"));
-        $ret .= $this->element('span', "$ns.$directive:", array('class' => 'verbose'));
-        $ret .= $this->text(' No');
-        $ret .= $this->end('label');
+		$ret .= $this->start('label', array('for' => "$name:No_$ns.$directive"));
+		$ret .= $this->element('span', "$ns.$directive:", array('class' => 'verbose'));
+		$ret .= $this->text(' No');
+		$ret .= $this->end('label');
 
-        $attr = array(
-            'type' => 'radio',
-            'name' => "$name"."[$ns.$directive]",
-            'id' => "$name:No_$ns.$directive",
-            'value' => '0'
-        );
-        if ($value === false) $attr['checked'] = 'checked';
-        if ($value === null) $attr['disabled'] = 'disabled';
-        $ret .= $this->elementEmpty('input', $attr);
+		$attr = array(
+			'type' => 'radio',
+			'name' => "$name"."[$ns.$directive]",
+			'id' => "$name:No_$ns.$directive",
+			'value' => '0'
+		);
+		if ($value === false) $attr['checked'] = 'checked';
+		if ($value === null) $attr['disabled'] = 'disabled';
+		$ret .= $this->elementEmpty('input', $attr);
 
-        $ret .= $this->end('div');
+		$ret .= $this->end('div');
 
-        return $ret;
-    }
+		return $ret;
+	}
 }
 
 // vim: et sw=4 sts=4

Braces +34 added lines, -11 removed lines

@@ -51,8 +51,12 @@ 
      * @param $rows Integer rows of textarea, null to use default
      */
     public function setTextareaDimensions($cols = null, $rows = null) {
-        if ($cols) $this->fields['default']->cols = $cols;
-        if ($rows) $this->fields['default']->rows = $rows;
+        if ($cols) {
+        	$this->fields['default']->cols = $cols;
+        }
+        if ($rows) {
+        	$this->fields['default']->rows = $rows;
+        }
     }
 
     /**
@@ -155,7 +159,9 @@ 
                     // component printers must create an element with this id
                     $attr
                 );
-            if ($this->docURL) $ret .= $this->end('a');
+            if ($this->docURL) {
+            	$ret .= $this->end('a');
+            }
             $ret .= $this->end('th');
 
             $ret .= $this->start('td');
@@ -167,7 +173,10 @@ 
                     $type = $def->type;
                     $allow_null = isset($def->allow_null);
                 }
-                if (!isset($this->fields[$type])) $type = 0; // default
+                if (!isset($this->fields[$type])) {
+                	$type = 0;
+                }
+                // default
                 $type_obj = $this->fields[$type];
                 if ($allow_null) {
                     $type_obj = new HTMLPurifier_Printer_ConfigForm_NullDecorator($type_obj);
@@ -223,7 +232,9 @@ 
             // modify inline javascript slightly
             $attr['onclick'] = "toggleWriteability('$name:Yes_$ns.$directive',checked);toggleWriteability('$name:No_$ns.$directive',checked)";
         }
-        if ($value === null) $attr['checked'] = 'checked';
+        if ($value === null) {
+        	$attr['checked'] = 'checked';
+        }
         $ret .= $this->elementEmpty('input', $attr);
         $ret .= $this->text(' or ');
         $ret .= $this->elementEmpty('br');
@@ -284,12 +295,16 @@ 
             'name' => "$name"."[$ns.$directive]",
             'id' => "$name:$ns.$directive"
         );
-        if ($value === null) $attr['disabled'] = 'disabled';
+        if ($value === null) {
+        	$attr['disabled'] = 'disabled';
+        }
         if (isset($def->allowed)) {
             $ret .= $this->start('select', $attr);
             foreach ($def->allowed as $val => $b) {
                 $attr = array();
-                if ($value == $val) $attr['selected'] = 'selected';
+                if ($value == $val) {
+                	$attr['selected'] = 'selected';
+                }
                 $ret .= $this->element('option', $val, $attr);
             }
             $ret .= $this->end('select');
@@ -340,8 +355,12 @@ 
             'id' => "$name:Yes_$ns.$directive",
             'value' => '1'
         );
-        if ($value === true) $attr['checked'] = 'checked';
-        if ($value === null) $attr['disabled'] = 'disabled';
+        if ($value === true) {
+        	$attr['checked'] = 'checked';
+        }
+        if ($value === null) {
+        	$attr['disabled'] = 'disabled';
+        }
         $ret .= $this->elementEmpty('input', $attr);
 
         $ret .= $this->start('label', array('for' => "$name:No_$ns.$directive"));
@@ -355,8 +374,12 @@ 
             'id' => "$name:No_$ns.$directive",
             'value' => '0'
         );
-        if ($value === false) $attr['checked'] = 'checked';
-        if ($value === null) $attr['disabled'] = 'disabled';
+        if ($value === false) {
+        	$attr['checked'] = 'checked';
+        }
+        if ($value === null) {
+        	$attr['disabled'] = 'disabled';
+        }
         $ret .= $this->elementEmpty('input', $attr);
 
         $ret .= $this->end('div');

Spacing +7 added lines, -7 removed lines

@@ -41,8 +41,8 @@ 
         $this->name   = $name;
         $this->compress = $compress;
         // initialize sub-printers
-        $this->fields[0]    = new HTMLPurifier_Printer_ConfigForm_default();
-        $this->fields[HTMLPurifier_VarParser::BOOL]       = new HTMLPurifier_Printer_ConfigForm_bool();
+        $this->fields[0] = new HTMLPurifier_Printer_ConfigForm_default();
+        $this->fields[HTMLPurifier_VarParser::BOOL] = new HTMLPurifier_Printer_ConfigForm_bool();
     }
 
     /**
@@ -59,14 +59,14 @@ 
      * Retrieves styling, in case it is not accessible by webserver
      */
     public static function getCSS() {
-        return file_get_contents(HTMLPURIFIER_PREFIX . '/HTMLPurifier/Printer/ConfigForm.css');
+        return file_get_contents(HTMLPURIFIER_PREFIX.'/HTMLPurifier/Printer/ConfigForm.css');
     }
 
     /**
      * Retrieves JavaScript, in case it is not accessible by webserver
      */
     public static function getJavaScript() {
-        return file_get_contents(HTMLPURIFIER_PREFIX . '/HTMLPurifier/Printer/ConfigForm.js');
+        return file_get_contents(HTMLPURIFIER_PREFIX.'/HTMLPurifier/Printer/ConfigForm.js');
     }
 
     /**
@@ -91,7 +91,7 @@ 
         $all = array();
         foreach ($allowed as $key) {
             list($ns, $directive) = $key;
-            $all[$ns][$directive] = $config->get($ns .'.'. $directive);
+            $all[$ns][$directive] = $config->get($ns.'.'.$directive);
         }
 
         $ret = '';
@@ -145,7 +145,7 @@ 
                 if (!$this->compress || (strlen($directive) < $this->compress)) {
                     $directive_disp = $directive;
                 } else {
-                    $directive_disp = substr($directive, 0, $this->compress - 2) . '...';
+                    $directive_disp = substr($directive, 0, $this->compress - 2).'...';
                     $attr['title'] = $directive;
                 }
 
@@ -268,7 +268,7 @@ 
                 case HTMLPurifier_VarParser::HASH:
                     $nvalue = '';
                     foreach ($value as $i => $v) {
-                        $nvalue .= "$i:$v" . PHP_EOL;
+                        $nvalue .= "$i:$v".PHP_EOL;
                     }
                     $value = $nvalue;
                     break;

classes/security/htmlpurifier/library/HTMLPurifier/Printer/HTMLDefinition.php

Indentation +263 added lines, -263 removed lines

@@ -3,269 +3,269 @@
 class HTMLPurifier_Printer_HTMLDefinition extends HTMLPurifier_Printer
 {
 
-    /**
-     * Instance of HTMLPurifier_HTMLDefinition, for easy access
-     */
-    protected $def;
-
-    public function render($config) {
-        $ret = '';
-        $this->config =& $config;
-
-        $this->def = $config->getHTMLDefinition();
-
-        $ret .= $this->start('div', array('class' => 'HTMLPurifier_Printer'));
-
-        $ret .= $this->renderDoctype();
-        $ret .= $this->renderEnvironment();
-        $ret .= $this->renderContentSets();
-        $ret .= $this->renderInfo();
-
-        $ret .= $this->end('div');
-
-        return $ret;
-    }
-
-    /**
-     * Renders the Doctype table
-     */
-    protected function renderDoctype() {
-        $doctype = $this->def->doctype;
-        $ret = '';
-        $ret .= $this->start('table');
-        $ret .= $this->element('caption', 'Doctype');
-        $ret .= $this->row('Name', $doctype->name);
-        $ret .= $this->row('XML', $doctype->xml ? 'Yes' : 'No');
-        $ret .= $this->row('Default Modules', implode($doctype->modules, ', '));
-        $ret .= $this->row('Default Tidy Modules', implode($doctype->tidyModules, ', '));
-        $ret .= $this->end('table');
-        return $ret;
-    }
-
-
-    /**
-     * Renders environment table, which is miscellaneous info
-     */
-    protected function renderEnvironment() {
-        $def = $this->def;
-
-        $ret = '';
-
-        $ret .= $this->start('table');
-        $ret .= $this->element('caption', 'Environment');
-
-        $ret .= $this->row('Parent of fragment', $def->info_parent);
-        $ret .= $this->renderChildren($def->info_parent_def->child);
-        $ret .= $this->row('Block wrap name', $def->info_block_wrapper);
-
-        $ret .= $this->start('tr');
-            $ret .= $this->element('th', 'Global attributes');
-            $ret .= $this->element('td', $this->listifyAttr($def->info_global_attr),0,0);
-        $ret .= $this->end('tr');
-
-        $ret .= $this->start('tr');
-            $ret .= $this->element('th', 'Tag transforms');
-            $list = array();
-            foreach ($def->info_tag_transform as $old => $new) {
-                $new = $this->getClass($new, 'TagTransform_');
-                $list[] = "<$old> with $new";
-            }
-            $ret .= $this->element('td', $this->listify($list));
-        $ret .= $this->end('tr');
-
-        $ret .= $this->start('tr');
-            $ret .= $this->element('th', 'Pre-AttrTransform');
-            $ret .= $this->element('td', $this->listifyObjectList($def->info_attr_transform_pre));
-        $ret .= $this->end('tr');
-
-        $ret .= $this->start('tr');
-            $ret .= $this->element('th', 'Post-AttrTransform');
-            $ret .= $this->element('td', $this->listifyObjectList($def->info_attr_transform_post));
-        $ret .= $this->end('tr');
-
-        $ret .= $this->end('table');
-        return $ret;
-    }
-
-    /**
-     * Renders the Content Sets table
-     */
-    protected function renderContentSets() {
-        $ret = '';
-        $ret .= $this->start('table');
-        $ret .= $this->element('caption', 'Content Sets');
-        foreach ($this->def->info_content_sets as $name => $lookup) {
-            $ret .= $this->heavyHeader($name);
-            $ret .= $this->start('tr');
-            $ret .= $this->element('td', $this->listifyTagLookup($lookup));
-            $ret .= $this->end('tr');
-        }
-        $ret .= $this->end('table');
-        return $ret;
-    }
-
-    /**
-     * Renders the Elements ($info) table
-     */
-    protected function renderInfo() {
-        $ret = '';
-        $ret .= $this->start('table');
-        $ret .= $this->element('caption', 'Elements ($info)');
-        ksort($this->def->info);
-        $ret .= $this->heavyHeader('Allowed tags', 2);
-        $ret .= $this->start('tr');
-        $ret .= $this->element('td', $this->listifyTagLookup($this->def->info), array('colspan' => 2));
-        $ret .= $this->end('tr');
-        foreach ($this->def->info as $name => $def) {
-            $ret .= $this->start('tr');
-                $ret .= $this->element('th', "<$name>", array('class'=>'heavy', 'colspan' => 2));
-            $ret .= $this->end('tr');
-            $ret .= $this->start('tr');
-                $ret .= $this->element('th', 'Inline content');
-                $ret .= $this->element('td', $def->descendants_are_inline ? 'Yes' : 'No');
-            $ret .= $this->end('tr');
-            if (!empty($def->excludes)) {
-                $ret .= $this->start('tr');
-                    $ret .= $this->element('th', 'Excludes');
-                    $ret .= $this->element('td', $this->listifyTagLookup($def->excludes));
-                $ret .= $this->end('tr');
-            }
-            if (!empty($def->attr_transform_pre)) {
-                $ret .= $this->start('tr');
-                    $ret .= $this->element('th', 'Pre-AttrTransform');
-                    $ret .= $this->element('td', $this->listifyObjectList($def->attr_transform_pre));
-                $ret .= $this->end('tr');
-            }
-            if (!empty($def->attr_transform_post)) {
-                $ret .= $this->start('tr');
-                    $ret .= $this->element('th', 'Post-AttrTransform');
-                    $ret .= $this->element('td', $this->listifyObjectList($def->attr_transform_post));
-                $ret .= $this->end('tr');
-            }
-            if (!empty($def->auto_close)) {
-                $ret .= $this->start('tr');
-                    $ret .= $this->element('th', 'Auto closed by');
-                    $ret .= $this->element('td', $this->listifyTagLookup($def->auto_close));
-                $ret .= $this->end('tr');
-            }
-            $ret .= $this->start('tr');
-                $ret .= $this->element('th', 'Allowed attributes');
-                $ret .= $this->element('td',$this->listifyAttr($def->attr), array(), 0);
-            $ret .= $this->end('tr');
-
-            if (!empty($def->required_attr)) {
-                $ret .= $this->row('Required attributes', $this->listify($def->required_attr));
-            }
-
-            $ret .= $this->renderChildren($def->child);
-        }
-        $ret .= $this->end('table');
-        return $ret;
-    }
-
-    /**
-     * Renders a row describing the allowed children of an element
-     * @param $def HTMLPurifier_ChildDef of pertinent element
-     */
-    protected function renderChildren($def) {
-        $context = new HTMLPurifier_Context();
-        $ret = '';
-        $ret .= $this->start('tr');
-            $elements = array();
-            $attr = array();
-            if (isset($def->elements)) {
-                if ($def->type == 'strictblockquote') {
-                    $def->validateChildren(array(), $this->config, $context);
-                }
-                $elements = $def->elements;
-            }
-            if ($def->type == 'chameleon') {
-                $attr['rowspan'] = 2;
-            } elseif ($def->type == 'empty') {
-                $elements = array();
-            } elseif ($def->type == 'table') {
-                $elements = array_flip(array('col', 'caption', 'colgroup', 'thead',
-                    'tfoot', 'tbody', 'tr'));
-            }
-            $ret .= $this->element('th', 'Allowed children', $attr);
-
-            if ($def->type == 'chameleon') {
-
-                $ret .= $this->element('td',
-                    '<em>Block</em>: ' .
-                    $this->escape($this->listifyTagLookup($def->block->elements)),0,0);
-                $ret .= $this->end('tr');
-                $ret .= $this->start('tr');
-                $ret .= $this->element('td',
-                    '<em>Inline</em>: ' .
-                    $this->escape($this->listifyTagLookup($def->inline->elements)),0,0);
-
-            } elseif ($def->type == 'custom') {
-
-                $ret .= $this->element('td', '<em>'.ucfirst($def->type).'</em>: ' .
-                    $def->dtd_regex);
-
-            } else {
-                $ret .= $this->element('td',
-                    '<em>'.ucfirst($def->type).'</em>: ' .
-                    $this->escape($this->listifyTagLookup($elements)),0,0);
-            }
-        $ret .= $this->end('tr');
-        return $ret;
-    }
-
-    /**
-     * Listifies a tag lookup table.
-     * @param $array Tag lookup array in form of array('tagname' => true)
-     */
-    protected function listifyTagLookup($array) {
-        ksort($array);
-        $list = array();
-        foreach ($array as $name => $discard) {
-            if ($name !== '#PCDATA' && !isset($this->def->info[$name])) continue;
-            $list[] = $name;
-        }
-        return $this->listify($list);
-    }
-
-    /**
-     * Listifies a list of objects by retrieving class names and internal state
-     * @param $array List of objects
-     * @todo Also add information about internal state
-     */
-    protected function listifyObjectList($array) {
-        ksort($array);
-        $list = array();
-        foreach ($array as $discard => $obj) {
-            $list[] = $this->getClass($obj, 'AttrTransform_');
-        }
-        return $this->listify($list);
-    }
-
-    /**
-     * Listifies a hash of attributes to AttrDef classes
-     * @param $array Array hash in form of array('attrname' => HTMLPurifier_AttrDef)
-     */
-    protected function listifyAttr($array) {
-        ksort($array);
-        $list = array();
-        foreach ($array as $name => $obj) {
-            if ($obj === false) continue;
-            $list[] = "$name&nbsp;=&nbsp;<i>" . $this->getClass($obj, 'AttrDef_') . '</i>';
-        }
-        return $this->listify($list);
-    }
-
-    /**
-     * Creates a heavy header row
-     */
-    protected function heavyHeader($text, $num = 1) {
-        $ret = '';
-        $ret .= $this->start('tr');
-        $ret .= $this->element('th', $text, array('colspan' => $num, 'class' => 'heavy'));
-        $ret .= $this->end('tr');
-        return $ret;
-    }
+	/**
+	 * Instance of HTMLPurifier_HTMLDefinition, for easy access
+	 */
+	protected $def;
+
+	public function render($config) {
+		$ret = '';
+		$this->config =& $config;
+
+		$this->def = $config->getHTMLDefinition();
+
+		$ret .= $this->start('div', array('class' => 'HTMLPurifier_Printer'));
+
+		$ret .= $this->renderDoctype();
+		$ret .= $this->renderEnvironment();
+		$ret .= $this->renderContentSets();
+		$ret .= $this->renderInfo();
+
+		$ret .= $this->end('div');
+
+		return $ret;
+	}
+
+	/**
+	 * Renders the Doctype table
+	 */
+	protected function renderDoctype() {
+		$doctype = $this->def->doctype;
+		$ret = '';
+		$ret .= $this->start('table');
+		$ret .= $this->element('caption', 'Doctype');
+		$ret .= $this->row('Name', $doctype->name);
+		$ret .= $this->row('XML', $doctype->xml ? 'Yes' : 'No');
+		$ret .= $this->row('Default Modules', implode($doctype->modules, ', '));
+		$ret .= $this->row('Default Tidy Modules', implode($doctype->tidyModules, ', '));
+		$ret .= $this->end('table');
+		return $ret;
+	}
+
+
+	/**
+	 * Renders environment table, which is miscellaneous info
+	 */
+	protected function renderEnvironment() {
+		$def = $this->def;
+
+		$ret = '';
+
+		$ret .= $this->start('table');
+		$ret .= $this->element('caption', 'Environment');
+
+		$ret .= $this->row('Parent of fragment', $def->info_parent);
+		$ret .= $this->renderChildren($def->info_parent_def->child);
+		$ret .= $this->row('Block wrap name', $def->info_block_wrapper);
+
+		$ret .= $this->start('tr');
+			$ret .= $this->element('th', 'Global attributes');
+			$ret .= $this->element('td', $this->listifyAttr($def->info_global_attr),0,0);
+		$ret .= $this->end('tr');
+
+		$ret .= $this->start('tr');
+			$ret .= $this->element('th', 'Tag transforms');
+			$list = array();
+			foreach ($def->info_tag_transform as $old => $new) {
+				$new = $this->getClass($new, 'TagTransform_');
+				$list[] = "<$old> with $new";
+			}
+			$ret .= $this->element('td', $this->listify($list));
+		$ret .= $this->end('tr');
+
+		$ret .= $this->start('tr');
+			$ret .= $this->element('th', 'Pre-AttrTransform');
+			$ret .= $this->element('td', $this->listifyObjectList($def->info_attr_transform_pre));
+		$ret .= $this->end('tr');
+
+		$ret .= $this->start('tr');
+			$ret .= $this->element('th', 'Post-AttrTransform');
+			$ret .= $this->element('td', $this->listifyObjectList($def->info_attr_transform_post));
+		$ret .= $this->end('tr');
+
+		$ret .= $this->end('table');
+		return $ret;
+	}
+
+	/**
+	 * Renders the Content Sets table
+	 */
+	protected function renderContentSets() {
+		$ret = '';
+		$ret .= $this->start('table');
+		$ret .= $this->element('caption', 'Content Sets');
+		foreach ($this->def->info_content_sets as $name => $lookup) {
+			$ret .= $this->heavyHeader($name);
+			$ret .= $this->start('tr');
+			$ret .= $this->element('td', $this->listifyTagLookup($lookup));
+			$ret .= $this->end('tr');
+		}
+		$ret .= $this->end('table');
+		return $ret;
+	}
+
+	/**
+	 * Renders the Elements ($info) table
+	 */
+	protected function renderInfo() {
+		$ret = '';
+		$ret .= $this->start('table');
+		$ret .= $this->element('caption', 'Elements ($info)');
+		ksort($this->def->info);
+		$ret .= $this->heavyHeader('Allowed tags', 2);
+		$ret .= $this->start('tr');
+		$ret .= $this->element('td', $this->listifyTagLookup($this->def->info), array('colspan' => 2));
+		$ret .= $this->end('tr');
+		foreach ($this->def->info as $name => $def) {
+			$ret .= $this->start('tr');
+				$ret .= $this->element('th', "<$name>", array('class'=>'heavy', 'colspan' => 2));
+			$ret .= $this->end('tr');
+			$ret .= $this->start('tr');
+				$ret .= $this->element('th', 'Inline content');
+				$ret .= $this->element('td', $def->descendants_are_inline ? 'Yes' : 'No');
+			$ret .= $this->end('tr');
+			if (!empty($def->excludes)) {
+				$ret .= $this->start('tr');
+					$ret .= $this->element('th', 'Excludes');
+					$ret .= $this->element('td', $this->listifyTagLookup($def->excludes));
+				$ret .= $this->end('tr');
+			}
+			if (!empty($def->attr_transform_pre)) {
+				$ret .= $this->start('tr');
+					$ret .= $this->element('th', 'Pre-AttrTransform');
+					$ret .= $this->element('td', $this->listifyObjectList($def->attr_transform_pre));
+				$ret .= $this->end('tr');
+			}
+			if (!empty($def->attr_transform_post)) {
+				$ret .= $this->start('tr');
+					$ret .= $this->element('th', 'Post-AttrTransform');
+					$ret .= $this->element('td', $this->listifyObjectList($def->attr_transform_post));
+				$ret .= $this->end('tr');
+			}
+			if (!empty($def->auto_close)) {
+				$ret .= $this->start('tr');
+					$ret .= $this->element('th', 'Auto closed by');
+					$ret .= $this->element('td', $this->listifyTagLookup($def->auto_close));
+				$ret .= $this->end('tr');
+			}
+			$ret .= $this->start('tr');
+				$ret .= $this->element('th', 'Allowed attributes');
+				$ret .= $this->element('td',$this->listifyAttr($def->attr), array(), 0);
+			$ret .= $this->end('tr');
+
+			if (!empty($def->required_attr)) {
+				$ret .= $this->row('Required attributes', $this->listify($def->required_attr));
+			}
+
+			$ret .= $this->renderChildren($def->child);
+		}
+		$ret .= $this->end('table');
+		return $ret;
+	}
+
+	/**
+	 * Renders a row describing the allowed children of an element
+	 * @param $def HTMLPurifier_ChildDef of pertinent element
+	 */
+	protected function renderChildren($def) {
+		$context = new HTMLPurifier_Context();
+		$ret = '';
+		$ret .= $this->start('tr');
+			$elements = array();
+			$attr = array();
+			if (isset($def->elements)) {
+				if ($def->type == 'strictblockquote') {
+					$def->validateChildren(array(), $this->config, $context);
+				}
+				$elements = $def->elements;
+			}
+			if ($def->type == 'chameleon') {
+				$attr['rowspan'] = 2;
+			} elseif ($def->type == 'empty') {
+				$elements = array();
+			} elseif ($def->type == 'table') {
+				$elements = array_flip(array('col', 'caption', 'colgroup', 'thead',
+					'tfoot', 'tbody', 'tr'));
+			}
+			$ret .= $this->element('th', 'Allowed children', $attr);
+
+			if ($def->type == 'chameleon') {
+
+				$ret .= $this->element('td',
+					'<em>Block</em>: ' .
+					$this->escape($this->listifyTagLookup($def->block->elements)),0,0);
+				$ret .= $this->end('tr');
+				$ret .= $this->start('tr');
+				$ret .= $this->element('td',
+					'<em>Inline</em>: ' .
+					$this->escape($this->listifyTagLookup($def->inline->elements)),0,0);
+
+			} elseif ($def->type == 'custom') {
+
+				$ret .= $this->element('td', '<em>'.ucfirst($def->type).'</em>: ' .
+					$def->dtd_regex);
+
+			} else {
+				$ret .= $this->element('td',
+					'<em>'.ucfirst($def->type).'</em>: ' .
+					$this->escape($this->listifyTagLookup($elements)),0,0);
+			}
+		$ret .= $this->end('tr');
+		return $ret;
+	}
+
+	/**
+	 * Listifies a tag lookup table.
+	 * @param $array Tag lookup array in form of array('tagname' => true)
+	 */
+	protected function listifyTagLookup($array) {
+		ksort($array);
+		$list = array();
+		foreach ($array as $name => $discard) {
+			if ($name !== '#PCDATA' && !isset($this->def->info[$name])) continue;
+			$list[] = $name;
+		}
+		return $this->listify($list);
+	}
+
+	/**
+	 * Listifies a list of objects by retrieving class names and internal state
+	 * @param $array List of objects
+	 * @todo Also add information about internal state
+	 */
+	protected function listifyObjectList($array) {
+		ksort($array);
+		$list = array();
+		foreach ($array as $discard => $obj) {
+			$list[] = $this->getClass($obj, 'AttrTransform_');
+		}
+		return $this->listify($list);
+	}
+
+	/**
+	 * Listifies a hash of attributes to AttrDef classes
+	 * @param $array Array hash in form of array('attrname' => HTMLPurifier_AttrDef)
+	 */
+	protected function listifyAttr($array) {
+		ksort($array);
+		$list = array();
+		foreach ($array as $name => $obj) {
+			if ($obj === false) continue;
+			$list[] = "$name&nbsp;=&nbsp;<i>" . $this->getClass($obj, 'AttrDef_') . '</i>';
+		}
+		return $this->listify($list);
+	}
+
+	/**
+	 * Creates a heavy header row
+	 */
+	protected function heavyHeader($text, $num = 1) {
+		$ret = '';
+		$ret .= $this->start('tr');
+		$ret .= $this->element('th', $text, array('colspan' => $num, 'class' => 'heavy'));
+		$ret .= $this->end('tr');
+		return $ret;
+	}
 
 }
 

Braces +6 added lines, -2 removed lines

@@ -222,7 +222,9 @@ 
         ksort($array);
         $list = array();
         foreach ($array as $name => $discard) {
-            if ($name !== '#PCDATA' && !isset($this->def->info[$name])) continue;
+            if ($name !== '#PCDATA' && !isset($this->def->info[$name])) {
+            	continue;
+            }
             $list[] = $name;
         }
         return $this->listify($list);
@@ -250,7 +252,9 @@ 
         ksort($array);
         $list = array();
         foreach ($array as $name => $obj) {
-            if ($obj === false) continue;
+            if ($obj === false) {
+            	continue;
+            }
             $list[] = "$name&nbsp;=&nbsp;<i>" . $this->getClass($obj, 'AttrDef_') . '</i>';
         }
         return $this->listify($list);

Spacing +11 added lines, -11 removed lines

@@ -10,7 +10,7 @@ 
 
     public function render($config) {
         $ret = '';
-        $this->config =& $config;
+        $this->config = & $config;
 
         $this->def = $config->getHTMLDefinition();
 
@@ -60,7 +60,7 @@ 
 
         $ret .= $this->start('tr');
             $ret .= $this->element('th', 'Global attributes');
-            $ret .= $this->element('td', $this->listifyAttr($def->info_global_attr),0,0);
+            $ret .= $this->element('td', $this->listifyAttr($def->info_global_attr), 0, 0);
         $ret .= $this->end('tr');
 
         $ret .= $this->start('tr');
@@ -150,7 +150,7 @@ 
             }
             $ret .= $this->start('tr');
                 $ret .= $this->element('th', 'Allowed attributes');
-                $ret .= $this->element('td',$this->listifyAttr($def->attr), array(), 0);
+                $ret .= $this->element('td', $this->listifyAttr($def->attr), array(), 0);
             $ret .= $this->end('tr');
 
             if (!empty($def->required_attr)) {
@@ -192,23 +192,23 @@ 
             if ($def->type == 'chameleon') {
 
                 $ret .= $this->element('td',
-                    '<em>Block</em>: ' .
-                    $this->escape($this->listifyTagLookup($def->block->elements)),0,0);
+                    '<em>Block</em>: '.
+                    $this->escape($this->listifyTagLookup($def->block->elements)), 0, 0);
                 $ret .= $this->end('tr');
                 $ret .= $this->start('tr');
                 $ret .= $this->element('td',
-                    '<em>Inline</em>: ' .
-                    $this->escape($this->listifyTagLookup($def->inline->elements)),0,0);
+                    '<em>Inline</em>: '.
+                    $this->escape($this->listifyTagLookup($def->inline->elements)), 0, 0);
 
             } elseif ($def->type == 'custom') {
 
-                $ret .= $this->element('td', '<em>'.ucfirst($def->type).'</em>: ' .
+                $ret .= $this->element('td', '<em>'.ucfirst($def->type).'</em>: '.
                     $def->dtd_regex);
 
             } else {
                 $ret .= $this->element('td',
-                    '<em>'.ucfirst($def->type).'</em>: ' .
-                    $this->escape($this->listifyTagLookup($elements)),0,0);
+                    '<em>'.ucfirst($def->type).'</em>: '.
+                    $this->escape($this->listifyTagLookup($elements)), 0, 0);
             }
         $ret .= $this->end('tr');
         return $ret;
@@ -251,7 +251,7 @@ 
         $list = array();
         foreach ($array as $name => $obj) {
             if ($obj === false) continue;
-            $list[] = "$name&nbsp;=&nbsp;<i>" . $this->getClass($obj, 'AttrDef_') . '</i>';
+            $list[] = "$name&nbsp;=&nbsp;<i>".$this->getClass($obj, 'AttrDef_').'</i>';
         }
         return $this->listify($list);
     }

classes/security/htmlpurifier/library/HTMLPurifier/PropertyList.php

Indentation +66 added lines, -66 removed lines

@@ -5,82 +5,82 @@
  */
 class HTMLPurifier_PropertyList
 {
-    /**
-     * Internal data-structure for properties
-     */
-    protected $data = array();
+	/**
+	 * Internal data-structure for properties
+	 */
+	protected $data = array();
 
-    /**
-     * Parent plist
-     */
-    protected $parent;
+	/**
+	 * Parent plist
+	 */
+	protected $parent;
 
-    protected $cache;
+	protected $cache;
 
-    public function __construct($parent = null) {
-        $this->parent = $parent;
-    }
+	public function __construct($parent = null) {
+		$this->parent = $parent;
+	}
 
-    /**
-     * Recursively retrieves the value for a key
-     */
-    public function get($name) {
-        if ($this->has($name)) return $this->data[$name];
-        // possible performance bottleneck, convert to iterative if necessary
-        if ($this->parent) return $this->parent->get($name);
-        throw new HTMLPurifier_Exception("Key '$name' not found");
-    }
+	/**
+	 * Recursively retrieves the value for a key
+	 */
+	public function get($name) {
+		if ($this->has($name)) return $this->data[$name];
+		// possible performance bottleneck, convert to iterative if necessary
+		if ($this->parent) return $this->parent->get($name);
+		throw new HTMLPurifier_Exception("Key '$name' not found");
+	}
 
-    /**
-     * Sets the value of a key, for this plist
-     */
-    public function set($name, $value) {
-        $this->data[$name] = $value;
-    }
+	/**
+	 * Sets the value of a key, for this plist
+	 */
+	public function set($name, $value) {
+		$this->data[$name] = $value;
+	}
 
-    /**
-     * Returns true if a given key exists
-     */
-    public function has($name) {
-        return array_key_exists($name, $this->data);
-    }
+	/**
+	 * Returns true if a given key exists
+	 */
+	public function has($name) {
+		return array_key_exists($name, $this->data);
+	}
 
-    /**
-     * Resets a value to the value of it's parent, usually the default. If
-     * no value is specified, the entire plist is reset.
-     */
-    public function reset($name = null) {
-        if ($name == null) $this->data = array();
-        else unset($this->data[$name]);
-    }
+	/**
+	 * Resets a value to the value of it's parent, usually the default. If
+	 * no value is specified, the entire plist is reset.
+	 */
+	public function reset($name = null) {
+		if ($name == null) $this->data = array();
+		else unset($this->data[$name]);
+	}
 
-    /**
-     * Squashes this property list and all of its property lists into a single
-     * array, and returns the array. This value is cached by default.
-     * @param $force If true, ignores the cache and regenerates the array.
-     */
-    public function squash($force = false) {
-        if ($this->cache !== null && !$force) return $this->cache;
-        if ($this->parent) {
-            return $this->cache = array_merge($this->parent->squash($force), $this->data);
-        } else {
-            return $this->cache = $this->data;
-        }
-    }
+	/**
+	 * Squashes this property list and all of its property lists into a single
+	 * array, and returns the array. This value is cached by default.
+	 * @param $force If true, ignores the cache and regenerates the array.
+	 */
+	public function squash($force = false) {
+		if ($this->cache !== null && !$force) return $this->cache;
+		if ($this->parent) {
+			return $this->cache = array_merge($this->parent->squash($force), $this->data);
+		} else {
+			return $this->cache = $this->data;
+		}
+	}
 
-    /**
-     * Returns the parent plist.
-     */
-    public function getParent() {
-        return $this->parent;
-    }
+	/**
+	 * Returns the parent plist.
+	 */
+	public function getParent() {
+		return $this->parent;
+	}
 
-    /**
-     * Sets the parent plist.
-     */
-    public function setParent($plist) {
-        $this->parent = $plist;
-    }
+	/**
+	 * Sets the parent plist.
+	 */
+	public function setParent($plist) {
+		$this->parent = $plist;
+	}
 }
 
 // vim: et sw=4 sts=4

Braces +14 added lines, -5 removed lines

@@ -25,9 +25,13 @@ 
      * Recursively retrieves the value for a key
      */
     public function get($name) {
-        if ($this->has($name)) return $this->data[$name];
+        if ($this->has($name)) {
+        	return $this->data[$name];
+        }
         // possible performance bottleneck, convert to iterative if necessary
-        if ($this->parent) return $this->parent->get($name);
+        if ($this->parent) {
+        	return $this->parent->get($name);
+        }
         throw new HTMLPurifier_Exception("Key '$name' not found");
     }
 
@@ -50,8 +54,11 @@ 
      * no value is specified, the entire plist is reset.
      */
     public function reset($name = null) {
-        if ($name == null) $this->data = array();
-        else unset($this->data[$name]);
+        if ($name == null) {
+        	$this->data = array();
+        } else {
+        	unset($this->data[$name]);
+        }
     }
 
     /**
@@ -60,7 +67,9 @@ 
      * @param $force If true, ignores the cache and regenerates the array.
      */
     public function squash($force = false) {
-        if ($this->cache !== null && !$force) return $this->cache;
+        if ($this->cache !== null && !$force) {
+        	return $this->cache;
+        }
         if ($this->parent) {
             return $this->cache = array_merge($this->parent->squash($force), $this->data);
         } else {

classes/security/htmlpurifier/library/HTMLPurifier/PropertyListIterator.php

Indentation +18 added lines, -18 removed lines

@@ -6,26 +6,26 @@
 class HTMLPurifier_PropertyListIterator extends FilterIterator
 {
 
-    protected $l;
-    protected $filter;
+	protected $l;
+	protected $filter;
 
-    /**
-     * @param $data Array of data to iterate over
-     * @param $filter Optional prefix to only allow values of
-     */
-    public function __construct(Iterator $iterator, $filter = null) {
-        parent::__construct($iterator);
-        $this->l = strlen($filter);
-        $this->filter = $filter;
-    }
+	/**
+	 * @param $data Array of data to iterate over
+	 * @param $filter Optional prefix to only allow values of
+	 */
+	public function __construct(Iterator $iterator, $filter = null) {
+		parent::__construct($iterator);
+		$this->l = strlen($filter);
+		$this->filter = $filter;
+	}
 
-    public function accept() {
-        $key = $this->getInnerIterator()->key();
-        if( strncmp($key, $this->filter, $this->l) !== 0 ) {
-            return false;
-        }
-        return true;
-    }
+	public function accept() {
+		$key = $this->getInnerIterator()->key();
+		if( strncmp($key, $this->filter, $this->l) !== 0 ) {
+			return false;
+		}
+		return true;
+	}
 
 }
 

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
 
     public function accept() {
         $key = $this->getInnerIterator()->key();
-        if( strncmp($key, $this->filter, $this->l) !== 0 ) {
+        if (strncmp($key, $this->filter, $this->l) !== 0) {
             return false;
         }
         return true;

classes/security/htmlpurifier/library/HTMLPurifier/Strategy.php

Indentation +8 added lines, -8 removed lines

@@ -12,14 +12,14 @@
 abstract class HTMLPurifier_Strategy
 {
 
-    /**
-     * Executes the strategy on the tokens.
-     *
-     * @param $tokens Array of HTMLPurifier_Token objects to be operated on.
-     * @param $config Configuration options
-     * @returns Processed array of token objects.
-     */
-    abstract public function execute($tokens, $config, $context);
+	/**
+	 * Executes the strategy on the tokens.
+	 *
+	 * @param $tokens Array of HTMLPurifier_Token objects to be operated on.
+	 * @param $config Configuration options
+	 * @returns Processed array of token objects.
+	 */
+	abstract public function execute($tokens, $config, $context);
 
 }
 

classes/security/htmlpurifier/library/HTMLPurifier/Strategy/Composite.php

Indentation +10 added lines, -10 removed lines

@@ -6,17 +6,17 @@
 abstract class HTMLPurifier_Strategy_Composite extends HTMLPurifier_Strategy
 {
 
-    /**
-     * List of strategies to run tokens through.
-     */
-    protected $strategies = array();
+	/**
+	 * List of strategies to run tokens through.
+	 */
+	protected $strategies = array();
 
-    public function execute($tokens, $config, $context) {
-        foreach ($this->strategies as $strategy) {
-            $tokens = $strategy->execute($tokens, $config, $context);
-        }
-        return $tokens;
-    }
+	public function execute($tokens, $config, $context) {
+		foreach ($this->strategies as $strategy) {
+			$tokens = $strategy->execute($tokens, $config, $context);
+		}
+		return $tokens;
+	}
 
 }
 

classes/security/htmlpurifier/library/HTMLPurifier/Strategy/Core.php

Indentation +6 added lines, -6 removed lines

@@ -6,12 +6,12 @@
 class HTMLPurifier_Strategy_Core extends HTMLPurifier_Strategy_Composite
 {
 
-    public function __construct() {
-        $this->strategies[] = new HTMLPurifier_Strategy_RemoveForeignElements();
-        $this->strategies[] = new HTMLPurifier_Strategy_MakeWellFormed();
-        $this->strategies[] = new HTMLPurifier_Strategy_FixNesting();
-        $this->strategies[] = new HTMLPurifier_Strategy_ValidateAttributes();
-    }
+	public function __construct() {
+		$this->strategies[] = new HTMLPurifier_Strategy_RemoveForeignElements();
+		$this->strategies[] = new HTMLPurifier_Strategy_MakeWellFormed();
+		$this->strategies[] = new HTMLPurifier_Strategy_FixNesting();
+		$this->strategies[] = new HTMLPurifier_Strategy_ValidateAttributes();
+	}
 
 }
 

classes/security/htmlpurifier/library/HTMLPurifier/Strategy/FixNesting.php

Indentation +291 added lines, -291 removed lines

@@ -31,297 +31,297 @@
 class HTMLPurifier_Strategy_FixNesting extends HTMLPurifier_Strategy
 {
 
-    public function execute($tokens, $config, $context) {
-        //####################################################################//
-        // Pre-processing
-
-        // get a copy of the HTML definition
-        $definition = $config->getHTMLDefinition();
-
-        // insert implicit "parent" node, will be removed at end.
-        // DEFINITION CALL
-        $parent_name = $definition->info_parent;
-        array_unshift($tokens, new HTMLPurifier_Token_Start($parent_name));
-        $tokens[] = new HTMLPurifier_Token_End($parent_name);
-
-        // setup the context variable 'IsInline', for chameleon processing
-        // is 'false' when we are not inline, 'true' when it must always
-        // be inline, and an integer when it is inline for a certain
-        // branch of the document tree
-        $is_inline = $definition->info_parent_def->descendants_are_inline;
-        $context->register('IsInline', $is_inline);
-
-        // setup error collector
-        $e =& $context->get('ErrorCollector', true);
-
-        //####################################################################//
-        // Loop initialization
-
-        // stack that contains the indexes of all parents,
-        // $stack[count($stack)-1] being the current parent
-        $stack = array();
-
-        // stack that contains all elements that are excluded
-        // it is organized by parent elements, similar to $stack,
-        // but it is only populated when an element with exclusions is
-        // processed, i.e. there won't be empty exclusions.
-        $exclude_stack = array();
-
-        // variable that contains the start token while we are processing
-        // nodes. This enables error reporting to do its job
-        $start_token = false;
-        $context->register('CurrentToken', $start_token);
-
-        //####################################################################//
-        // Loop
-
-        // iterate through all start nodes. Determining the start node
-        // is complicated so it has been omitted from the loop construct
-        for ($i = 0, $size = count($tokens) ; $i < $size; ) {
-
-            //################################################################//
-            // Gather information on children
-
-            // child token accumulator
-            $child_tokens = array();
-
-            // scroll to the end of this node, report number, and collect
-            // all children
-            for ($j = $i, $depth = 0; ; $j++) {
-                if ($tokens[$j] instanceof HTMLPurifier_Token_Start) {
-                    $depth++;
-                    // skip token assignment on first iteration, this is the
-                    // token we currently are on
-                    if ($depth == 1) continue;
-                } elseif ($tokens[$j] instanceof HTMLPurifier_Token_End) {
-                    $depth--;
-                    // skip token assignment on last iteration, this is the
-                    // end token of the token we're currently on
-                    if ($depth == 0) break;
-                }
-                $child_tokens[] = $tokens[$j];
-            }
-
-            // $i is index of start token
-            // $j is index of end token
-
-            $start_token = $tokens[$i]; // to make token available via CurrentToken
-
-            //################################################################//
-            // Gather information on parent
-
-            // calculate parent information
-            if ($count = count($stack)) {
-                $parent_index = $stack[$count-1];
-                $parent_name  = $tokens[$parent_index]->name;
-                if ($parent_index == 0) {
-                    $parent_def   = $definition->info_parent_def;
-                } else {
-                    $parent_def   = $definition->info[$parent_name];
-                }
-            } else {
-                // processing as if the parent were the "root" node
-                // unknown info, it won't be used anyway, in the future,
-                // we may want to enforce one element only (this is
-                // necessary for HTML Purifier to clean entire documents
-                $parent_index = $parent_name = $parent_def = null;
-            }
-
-            // calculate context
-            if ($is_inline === false) {
-                // check if conditions make it inline
-                if (!empty($parent_def) && $parent_def->descendants_are_inline) {
-                    $is_inline = $count - 1;
-                }
-            } else {
-                // check if we're out of inline
-                if ($count === $is_inline) {
-                    $is_inline = false;
-                }
-            }
-
-            //################################################################//
-            // Determine whether element is explicitly excluded SGML-style
-
-            // determine whether or not element is excluded by checking all
-            // parent exclusions. The array should not be very large, two
-            // elements at most.
-            $excluded = false;
-            if (!empty($exclude_stack)) {
-                foreach ($exclude_stack as $lookup) {
-                    if (isset($lookup[$tokens[$i]->name])) {
-                        $excluded = true;
-                        // no need to continue processing
-                        break;
-                    }
-                }
-            }
-
-            //################################################################//
-            // Perform child validation
-
-            if ($excluded) {
-                // there is an exclusion, remove the entire node
-                $result = false;
-                $excludes = array(); // not used, but good to initialize anyway
-            } else {
-                // DEFINITION CALL
-                if ($i === 0) {
-                    // special processing for the first node
-                    $def = $definition->info_parent_def;
-                } else {
-                    $def = $definition->info[$tokens[$i]->name];
-
-                }
-
-                if (!empty($def->child)) {
-                    // have DTD child def validate children
-                    $result = $def->child->validateChildren(
-                        $child_tokens, $config, $context);
-                } else {
-                    // weird, no child definition, get rid of everything
-                    $result = false;
-                }
-
-                // determine whether or not this element has any exclusions
-                $excludes = $def->excludes;
-            }
-
-            // $result is now a bool or array
-
-            //################################################################//
-            // Process result by interpreting $result
-
-            if ($result === true || $child_tokens === $result) {
-                // leave the node as is
-
-                // register start token as a parental node start
-                $stack[] = $i;
-
-                // register exclusions if there are any
-                if (!empty($excludes)) $exclude_stack[] = $excludes;
-
-                // move cursor to next possible start node
-                $i++;
-
-            } elseif($result === false) {
-                // remove entire node
-
-                if ($e) {
-                    if ($excluded) {
-                        $e->send(E_ERROR, 'Strategy_FixNesting: Node excluded');
-                    } else {
-                        $e->send(E_ERROR, 'Strategy_FixNesting: Node removed');
-                    }
-                }
-
-                // calculate length of inner tokens and current tokens
-                $length = $j - $i + 1;
-
-                // perform removal
-                array_splice($tokens, $i, $length);
-
-                // update size
-                $size -= $length;
-
-                // there is no start token to register,
-                // current node is now the next possible start node
-                // unless it turns out that we need to do a double-check
-
-                // this is a rought heuristic that covers 100% of HTML's
-                // cases and 99% of all other cases. A child definition
-                // that would be tricked by this would be something like:
-                // ( | a b c) where it's all or nothing. Fortunately,
-                // our current implementation claims that that case would
-                // not allow empty, even if it did
-                if (!$parent_def->child->allow_empty) {
-                    // we need to do a double-check
-                    $i = $parent_index;
-                    array_pop($stack);
-                }
-
-                // PROJECTED OPTIMIZATION: Process all children elements before
-                // reprocessing parent node.
-
-            } else {
-                // replace node with $result
-
-                // calculate length of inner tokens
-                $length = $j - $i - 1;
-
-                if ($e) {
-                    if (empty($result) && $length) {
-                        $e->send(E_ERROR, 'Strategy_FixNesting: Node contents removed');
-                    } else {
-                        $e->send(E_WARNING, 'Strategy_FixNesting: Node reorganized');
-                    }
-                }
-
-                // perform replacement
-                array_splice($tokens, $i + 1, $length, $result);
-
-                // update size
-                $size -= $length;
-                $size += count($result);
-
-                // register start token as a parental node start
-                $stack[] = $i;
-
-                // register exclusions if there are any
-                if (!empty($excludes)) $exclude_stack[] = $excludes;
-
-                // move cursor to next possible start node
-                $i++;
-
-            }
-
-            //################################################################//
-            // Scroll to next start node
-
-            // We assume, at this point, that $i is the index of the token
-            // that is the first possible new start point for a node.
-
-            // Test if the token indeed is a start tag, if not, move forward
-            // and test again.
-            $size = count($tokens);
-            while ($i < $size and !$tokens[$i] instanceof HTMLPurifier_Token_Start) {
-                if ($tokens[$i] instanceof HTMLPurifier_Token_End) {
-                    // pop a token index off the stack if we ended a node
-                    array_pop($stack);
-                    // pop an exclusion lookup off exclusion stack if
-                    // we ended node and that node had exclusions
-                    if ($i == 0 || $i == $size - 1) {
-                        // use specialized var if it's the super-parent
-                        $s_excludes = $definition->info_parent_def->excludes;
-                    } else {
-                        $s_excludes = $definition->info[$tokens[$i]->name]->excludes;
-                    }
-                    if ($s_excludes) {
-                        array_pop($exclude_stack);
-                    }
-                }
-                $i++;
-            }
-
-        }
-
-        //####################################################################//
-        // Post-processing
-
-        // remove implicit parent tokens at the beginning and end
-        array_shift($tokens);
-        array_pop($tokens);
-
-        // remove context variables
-        $context->destroy('IsInline');
-        $context->destroy('CurrentToken');
-
-        //####################################################################//
-        // Return
-
-        return $tokens;
-
-    }
+	public function execute($tokens, $config, $context) {
+		//####################################################################//
+		// Pre-processing
+
+		// get a copy of the HTML definition
+		$definition = $config->getHTMLDefinition();
+
+		// insert implicit "parent" node, will be removed at end.
+		// DEFINITION CALL
+		$parent_name = $definition->info_parent;
+		array_unshift($tokens, new HTMLPurifier_Token_Start($parent_name));
+		$tokens[] = new HTMLPurifier_Token_End($parent_name);
+
+		// setup the context variable 'IsInline', for chameleon processing
+		// is 'false' when we are not inline, 'true' when it must always
+		// be inline, and an integer when it is inline for a certain
+		// branch of the document tree
+		$is_inline = $definition->info_parent_def->descendants_are_inline;
+		$context->register('IsInline', $is_inline);
+
+		// setup error collector
+		$e =& $context->get('ErrorCollector', true);
+
+		//####################################################################//
+		// Loop initialization
+
+		// stack that contains the indexes of all parents,
+		// $stack[count($stack)-1] being the current parent
+		$stack = array();
+
+		// stack that contains all elements that are excluded
+		// it is organized by parent elements, similar to $stack,
+		// but it is only populated when an element with exclusions is
+		// processed, i.e. there won't be empty exclusions.
+		$exclude_stack = array();
+
+		// variable that contains the start token while we are processing
+		// nodes. This enables error reporting to do its job
+		$start_token = false;
+		$context->register('CurrentToken', $start_token);
+
+		//####################################################################//
+		// Loop
+
+		// iterate through all start nodes. Determining the start node
+		// is complicated so it has been omitted from the loop construct
+		for ($i = 0, $size = count($tokens) ; $i < $size; ) {
+
+			//################################################################//
+			// Gather information on children
+
+			// child token accumulator
+			$child_tokens = array();
+
+			// scroll to the end of this node, report number, and collect
+			// all children
+			for ($j = $i, $depth = 0; ; $j++) {
+				if ($tokens[$j] instanceof HTMLPurifier_Token_Start) {
+					$depth++;
+					// skip token assignment on first iteration, this is the
+					// token we currently are on
+					if ($depth == 1) continue;
+				} elseif ($tokens[$j] instanceof HTMLPurifier_Token_End) {
+					$depth--;
+					// skip token assignment on last iteration, this is the
+					// end token of the token we're currently on
+					if ($depth == 0) break;
+				}
+				$child_tokens[] = $tokens[$j];
+			}
+
+			// $i is index of start token
+			// $j is index of end token
+
+			$start_token = $tokens[$i]; // to make token available via CurrentToken
+
+			//################################################################//
+			// Gather information on parent
+
+			// calculate parent information
+			if ($count = count($stack)) {
+				$parent_index = $stack[$count-1];
+				$parent_name  = $tokens[$parent_index]->name;
+				if ($parent_index == 0) {
+					$parent_def   = $definition->info_parent_def;
+				} else {
+					$parent_def   = $definition->info[$parent_name];
+				}
+			} else {
+				// processing as if the parent were the "root" node
+				// unknown info, it won't be used anyway, in the future,
+				// we may want to enforce one element only (this is
+				// necessary for HTML Purifier to clean entire documents
+				$parent_index = $parent_name = $parent_def = null;
+			}
+
+			// calculate context
+			if ($is_inline === false) {
+				// check if conditions make it inline
+				if (!empty($parent_def) && $parent_def->descendants_are_inline) {
+					$is_inline = $count - 1;
+				}
+			} else {
+				// check if we're out of inline
+				if ($count === $is_inline) {
+					$is_inline = false;
+				}
+			}
+
+			//################################################################//
+			// Determine whether element is explicitly excluded SGML-style
+
+			// determine whether or not element is excluded by checking all
+			// parent exclusions. The array should not be very large, two
+			// elements at most.
+			$excluded = false;
+			if (!empty($exclude_stack)) {
+				foreach ($exclude_stack as $lookup) {
+					if (isset($lookup[$tokens[$i]->name])) {
+						$excluded = true;
+						// no need to continue processing
+						break;
+					}
+				}
+			}
+
+			//################################################################//
+			// Perform child validation
+
+			if ($excluded) {
+				// there is an exclusion, remove the entire node
+				$result = false;
+				$excludes = array(); // not used, but good to initialize anyway
+			} else {
+				// DEFINITION CALL
+				if ($i === 0) {
+					// special processing for the first node
+					$def = $definition->info_parent_def;
+				} else {
+					$def = $definition->info[$tokens[$i]->name];
+
+				}
+
+				if (!empty($def->child)) {
+					// have DTD child def validate children
+					$result = $def->child->validateChildren(
+						$child_tokens, $config, $context);
+				} else {
+					// weird, no child definition, get rid of everything
+					$result = false;
+				}
+
+				// determine whether or not this element has any exclusions
+				$excludes = $def->excludes;
+			}
+
+			// $result is now a bool or array
+
+			//################################################################//
+			// Process result by interpreting $result
+
+			if ($result === true || $child_tokens === $result) {
+				// leave the node as is
+
+				// register start token as a parental node start
+				$stack[] = $i;
+
+				// register exclusions if there are any
+				if (!empty($excludes)) $exclude_stack[] = $excludes;
+
+				// move cursor to next possible start node
+				$i++;
+
+			} elseif($result === false) {
+				// remove entire node
+
+				if ($e) {
+					if ($excluded) {
+						$e->send(E_ERROR, 'Strategy_FixNesting: Node excluded');
+					} else {
+						$e->send(E_ERROR, 'Strategy_FixNesting: Node removed');
+					}
+				}
+
+				// calculate length of inner tokens and current tokens
+				$length = $j - $i + 1;
+
+				// perform removal
+				array_splice($tokens, $i, $length);
+
+				// update size
+				$size -= $length;
+
+				// there is no start token to register,
+				// current node is now the next possible start node
+				// unless it turns out that we need to do a double-check
+
+				// this is a rought heuristic that covers 100% of HTML's
+				// cases and 99% of all other cases. A child definition
+				// that would be tricked by this would be something like:
+				// ( | a b c) where it's all or nothing. Fortunately,
+				// our current implementation claims that that case would
+				// not allow empty, even if it did
+				if (!$parent_def->child->allow_empty) {
+					// we need to do a double-check
+					$i = $parent_index;
+					array_pop($stack);
+				}
+
+				// PROJECTED OPTIMIZATION: Process all children elements before
+				// reprocessing parent node.
+
+			} else {
+				// replace node with $result
+
+				// calculate length of inner tokens
+				$length = $j - $i - 1;
+
+				if ($e) {
+					if (empty($result) && $length) {
+						$e->send(E_ERROR, 'Strategy_FixNesting: Node contents removed');
+					} else {
+						$e->send(E_WARNING, 'Strategy_FixNesting: Node reorganized');
+					}
+				}
+
+				// perform replacement
+				array_splice($tokens, $i + 1, $length, $result);
+
+				// update size
+				$size -= $length;
+				$size += count($result);
+
+				// register start token as a parental node start
+				$stack[] = $i;
+
+				// register exclusions if there are any
+				if (!empty($excludes)) $exclude_stack[] = $excludes;
+
+				// move cursor to next possible start node
+				$i++;
+
+			}
+
+			//################################################################//
+			// Scroll to next start node
+
+			// We assume, at this point, that $i is the index of the token
+			// that is the first possible new start point for a node.
+
+			// Test if the token indeed is a start tag, if not, move forward
+			// and test again.
+			$size = count($tokens);
+			while ($i < $size and !$tokens[$i] instanceof HTMLPurifier_Token_Start) {
+				if ($tokens[$i] instanceof HTMLPurifier_Token_End) {
+					// pop a token index off the stack if we ended a node
+					array_pop($stack);
+					// pop an exclusion lookup off exclusion stack if
+					// we ended node and that node had exclusions
+					if ($i == 0 || $i == $size - 1) {
+						// use specialized var if it's the super-parent
+						$s_excludes = $definition->info_parent_def->excludes;
+					} else {
+						$s_excludes = $definition->info[$tokens[$i]->name]->excludes;
+					}
+					if ($s_excludes) {
+						array_pop($exclude_stack);
+					}
+				}
+				$i++;
+			}
+
+		}
+
+		//####################################################################//
+		// Post-processing
+
+		// remove implicit parent tokens at the beginning and end
+		array_shift($tokens);
+		array_pop($tokens);
+
+		// remove context variables
+		$context->destroy('IsInline');
+		$context->destroy('CurrentToken');
+
+		//####################################################################//
+		// Return
+
+		return $tokens;
+
+	}
 
 }
 

Spacing +4 added lines, -4 removed lines

@@ -52,7 +52,7 @@ 
         $context->register('IsInline', $is_inline);
 
         // setup error collector
-        $e =& $context->get('ErrorCollector', true);
+        $e = & $context->get('ErrorCollector', true);
 
         //####################################################################//
         // Loop initialization
@@ -77,7 +77,7 @@ 
 
         // iterate through all start nodes. Determining the start node
         // is complicated so it has been omitted from the loop construct
-        for ($i = 0, $size = count($tokens) ; $i < $size; ) {
+        for ($i = 0, $size = count($tokens); $i < $size;) {
 
             //################################################################//
             // Gather information on children
@@ -112,7 +112,7 @@ 
 
             // calculate parent information
             if ($count = count($stack)) {
-                $parent_index = $stack[$count-1];
+                $parent_index = $stack[$count - 1];
                 $parent_name  = $tokens[$parent_index]->name;
                 if ($parent_index == 0) {
                     $parent_def   = $definition->info_parent_def;
@@ -204,7 +204,7 @@ 
                 // move cursor to next possible start node
                 $i++;
 
-            } elseif($result === false) {
+            } elseif ($result === false) {
                 // remove entire node
 
                 if ($e) {

Braces +12 added lines, -4 removed lines

@@ -92,12 +92,16 @@ 
                     $depth++;
                     // skip token assignment on first iteration, this is the
                     // token we currently are on
-                    if ($depth == 1) continue;
+                    if ($depth == 1) {
+                    	continue;
+                    }
                 } elseif ($tokens[$j] instanceof HTMLPurifier_Token_End) {
                     $depth--;
                     // skip token assignment on last iteration, this is the
                     // end token of the token we're currently on
-                    if ($depth == 0) break;
+                    if ($depth == 0) {
+                    	break;
+                    }
                 }
                 $child_tokens[] = $tokens[$j];
             }
@@ -199,7 +203,9 @@ 
                 $stack[] = $i;
 
                 // register exclusions if there are any
-                if (!empty($excludes)) $exclude_stack[] = $excludes;
+                if (!empty($excludes)) {
+                	$exclude_stack[] = $excludes;
+                }
 
                 // move cursor to next possible start node
                 $i++;
@@ -268,7 +274,9 @@ 
                 $stack[] = $i;
 
                 // register exclusions if there are any
-                if (!empty($excludes)) $exclude_stack[] = $excludes;
+                if (!empty($excludes)) {
+                	$exclude_stack[] = $excludes;
+                }
 
                 // move cursor to next possible start node
                 $i++;

classes/security/htmlpurifier/library/HTMLPurifier/Strategy/MakeWellFormed.php

Indentation +512 added lines, -512 removed lines

@@ -14,518 +14,518 @@
 class HTMLPurifier_Strategy_MakeWellFormed extends HTMLPurifier_Strategy
 {
 
-    /**
-     * Array stream of tokens being processed.
-     */
-    protected $tokens;
-
-    /**
-     * Current index in $tokens.
-     */
-    protected $t;
-
-    /**
-     * Current nesting of elements.
-     */
-    protected $stack;
-
-    /**
-     * Injectors active in this stream processing.
-     */
-    protected $injectors;
-
-    /**
-     * Current instance of HTMLPurifier_Config.
-     */
-    protected $config;
-
-    /**
-     * Current instance of HTMLPurifier_Context.
-     */
-    protected $context;
-
-    public function execute($tokens, $config, $context) {
-
-        $definition = $config->getHTMLDefinition();
-
-        // local variables
-        $generator = new HTMLPurifier_Generator($config, $context);
-        $escape_invalid_tags = $config->get('Core.EscapeInvalidTags');
-        // used for autoclose early abortion
-        $global_parent_allowed_elements = array();
-        if (isset($definition->info[$definition->info_parent])) {
-            // may be unset under testing circumstances
-            $global_parent_allowed_elements = $definition->info[$definition->info_parent]->child->getAllowedElements($config);
-        }
-        $e = $context->get('ErrorCollector', true);
-        $t = false; // token index
-        $i = false; // injector index
-        $token      = false; // the current token
-        $reprocess  = false; // whether or not to reprocess the same token
-        $stack = array();
-
-        // member variables
-        $this->stack   =& $stack;
-        $this->t       =& $t;
-        $this->tokens  =& $tokens;
-        $this->config  = $config;
-        $this->context = $context;
-
-        // context variables
-        $context->register('CurrentNesting', $stack);
-        $context->register('InputIndex',     $t);
-        $context->register('InputTokens',    $tokens);
-        $context->register('CurrentToken',   $token);
-
-        // -- begin INJECTOR --
-
-        $this->injectors = array();
-
-        $injectors = $config->getBatch('AutoFormat');
-        $def_injectors = $definition->info_injector;
-        $custom_injectors = $injectors['Custom'];
-        unset($injectors['Custom']); // special case
-        foreach ($injectors as $injector => $b) {
-            // XXX: Fix with a legitimate lookup table of enabled filters
-            if (strpos($injector, '.') !== false) continue;
-            $injector = "HTMLPurifier_Injector_$injector";
-            if (!$b) continue;
-            $this->injectors[] = new $injector;
-        }
-        foreach ($def_injectors as $injector) {
-            // assumed to be objects
-            $this->injectors[] = $injector;
-        }
-        foreach ($custom_injectors as $injector) {
-            if (!$injector) continue;
-            if (is_string($injector)) {
-                $injector = "HTMLPurifier_Injector_$injector";
-                $injector = new $injector;
-            }
-            $this->injectors[] = $injector;
-        }
-
-        // give the injectors references to the definition and context
-        // variables for performance reasons
-        foreach ($this->injectors as $ix => $injector) {
-            $error = $injector->prepare($config, $context);
-            if (!$error) continue;
-            array_splice($this->injectors, $ix, 1); // rm the injector
-            trigger_error("Cannot enable {$injector->name} injector because $error is not allowed", E_USER_WARNING);
-        }
-
-        // -- end INJECTOR --
-
-        // a note on reprocessing:
-        //      In order to reduce code duplication, whenever some code needs
-        //      to make HTML changes in order to make things "correct", the
-        //      new HTML gets sent through the purifier, regardless of its
-        //      status. This means that if we add a start token, because it
-        //      was totally necessary, we don't have to update nesting; we just
-        //      punt ($reprocess = true; continue;) and it does that for us.
-
-        // isset is in loop because $tokens size changes during loop exec
-        for (
-            $t = 0;
-            $t == 0 || isset($tokens[$t - 1]);
-            // only increment if we don't need to reprocess
-            $reprocess ? $reprocess = false : $t++
-        ) {
-
-            // check for a rewind
-            if (is_int($i) && $i >= 0) {
-                // possibility: disable rewinding if the current token has a
-                // rewind set on it already. This would offer protection from
-                // infinite loop, but might hinder some advanced rewinding.
-                $rewind_to = $this->injectors[$i]->getRewind();
-                if (is_int($rewind_to) && $rewind_to < $t) {
-                    if ($rewind_to < 0) $rewind_to = 0;
-                    while ($t > $rewind_to) {
-                        $t--;
-                        $prev = $tokens[$t];
-                        // indicate that other injectors should not process this token,
-                        // but we need to reprocess it
-                        unset($prev->skip[$i]);
-                        $prev->rewind = $i;
-                        if ($prev instanceof HTMLPurifier_Token_Start) array_pop($this->stack);
-                        elseif ($prev instanceof HTMLPurifier_Token_End) $this->stack[] = $prev->start;
-                    }
-                }
-                $i = false;
-            }
-
-            // handle case of document end
-            if (!isset($tokens[$t])) {
-                // kill processing if stack is empty
-                if (empty($this->stack)) break;
-
-                // peek
-                $top_nesting = array_pop($this->stack);
-                $this->stack[] = $top_nesting;
-
-                // send error [TagClosedSuppress]
-                if ($e && !isset($top_nesting->armor['MakeWellFormed_TagClosedError'])) {
-                    $e->send(E_NOTICE, 'Strategy_MakeWellFormed: Tag closed by document end', $top_nesting);
-                }
-
-                // append, don't splice, since this is the end
-                $tokens[] = new HTMLPurifier_Token_End($top_nesting->name);
-
-                // punt!
-                $reprocess = true;
-                continue;
-            }
-
-            $token = $tokens[$t];
-
-            //echo '<br>'; printTokens($tokens, $t); printTokens($this->stack);
-            //flush();
-
-            // quick-check: if it's not a tag, no need to process
-            if (empty($token->is_tag)) {
-                if ($token instanceof HTMLPurifier_Token_Text) {
-                    foreach ($this->injectors as $i => $injector) {
-                        if (isset($token->skip[$i])) continue;
-                        if ($token->rewind !== null && $token->rewind !== $i) continue;
-                        $injector->handleText($token);
-                        $this->processToken($token, $i);
-                        $reprocess = true;
-                        break;
-                    }
-                }
-                // another possibility is a comment
-                continue;
-            }
-
-            if (isset($definition->info[$token->name])) {
-                $type = $definition->info[$token->name]->child->type;
-            } else {
-                $type = false; // Type is unknown, treat accordingly
-            }
-
-            // quick tag checks: anything that's *not* an end tag
-            $ok = false;
-            if ($type === 'empty' && $token instanceof HTMLPurifier_Token_Start) {
-                // claims to be a start tag but is empty
-                $token = new HTMLPurifier_Token_Empty($token->name, $token->attr, $token->line, $token->col, $token->armor);
-                $ok = true;
-            } elseif ($type && $type !== 'empty' && $token instanceof HTMLPurifier_Token_Empty) {
-                // claims to be empty but really is a start tag
-                $this->swap(new HTMLPurifier_Token_End($token->name));
-                $this->insertBefore(new HTMLPurifier_Token_Start($token->name, $token->attr, $token->line, $token->col, $token->armor));
-                // punt (since we had to modify the input stream in a non-trivial way)
-                $reprocess = true;
-                continue;
-            } elseif ($token instanceof HTMLPurifier_Token_Empty) {
-                // real empty token
-                $ok = true;
-            } elseif ($token instanceof HTMLPurifier_Token_Start) {
-                // start tag
-
-                // ...unless they also have to close their parent
-                if (!empty($this->stack)) {
-
-                    // Performance note: you might think that it's rather
-                    // inefficient, recalculating the autoclose information
-                    // for every tag that a token closes (since when we
-                    // do an autoclose, we push a new token into the
-                    // stream and then /process/ that, before
-                    // re-processing this token.)  But this is
-                    // necessary, because an injector can make an
-                    // arbitrary transformations to the autoclosing
-                    // tokens we introduce, so things may have changed
-                    // in the meantime.  Also, doing the inefficient thing is
-                    // "easy" to reason about (for certain perverse definitions
-                    // of "easy")
-
-                    $parent = array_pop($this->stack);
-                    $this->stack[] = $parent;
-
-                    if (isset($definition->info[$parent->name])) {
-                        $elements = $definition->info[$parent->name]->child->getAllowedElements($config);
-                        $autoclose = !isset($elements[$token->name]);
-                    } else {
-                        $autoclose = false;
-                    }
-
-                    if ($autoclose && $definition->info[$token->name]->wrap) {
-                        // Check if an element can be wrapped by another 
-                        // element to make it valid in a context (for 
-                        // example, <ul><ul> needs a <li> in between)
-                        $wrapname = $definition->info[$token->name]->wrap;
-                        $wrapdef = $definition->info[$wrapname];
-                        $elements = $wrapdef->child->getAllowedElements($config);
-                        $parent_elements = $definition->info[$parent->name]->child->getAllowedElements($config);
-                        if (isset($elements[$token->name]) && isset($parent_elements[$wrapname])) {
-                            $newtoken = new HTMLPurifier_Token_Start($wrapname);
-                            $this->insertBefore($newtoken);
-                            $reprocess = true;
-                            continue;
-                        }
-                    }
-
-                    $carryover = false;
-                    if ($autoclose && $definition->info[$parent->name]->formatting) {
-                        $carryover = true;
-                    }
-
-                    if ($autoclose) {
-                        // check if this autoclose is doomed to fail
-                        // (this rechecks $parent, which his harmless)
-                        $autoclose_ok = isset($global_parent_allowed_elements[$token->name]);
-                        if (!$autoclose_ok) {
-                            foreach ($this->stack as $ancestor) {
-                                $elements = $definition->info[$ancestor->name]->child->getAllowedElements($config);
-                                if (isset($elements[$token->name])) {
-                                    $autoclose_ok = true;
-                                    break;
-                                }
-                                if ($definition->info[$token->name]->wrap) {
-                                    $wrapname = $definition->info[$token->name]->wrap;
-                                    $wrapdef = $definition->info[$wrapname];
-                                    $wrap_elements = $wrapdef->child->getAllowedElements($config);
-                                    if (isset($wrap_elements[$token->name]) && isset($elements[$wrapname])) {
-                                        $autoclose_ok = true;
-                                        break;
-                                    }
-                                }
-                            }
-                        }
-                        if ($autoclose_ok) {
-                            // errors need to be updated
-                            $new_token = new HTMLPurifier_Token_End($parent->name);
-                            $new_token->start = $parent;
-                            if ($carryover) {
-                                $element = clone $parent;
-                                // [TagClosedAuto]
-                                $element->armor['MakeWellFormed_TagClosedError'] = true;
-                                $element->carryover = true;
-                                $this->processToken(array($new_token, $token, $element));
-                            } else {
-                                $this->insertBefore($new_token);
-                            }
-                            // [TagClosedSuppress]
-                            if ($e && !isset($parent->armor['MakeWellFormed_TagClosedError'])) {
-                                if (!$carryover) {
-                                    $e->send(E_NOTICE, 'Strategy_MakeWellFormed: Tag auto closed', $parent);
-                                } else {
-                                    $e->send(E_NOTICE, 'Strategy_MakeWellFormed: Tag carryover', $parent);
-                                }
-                            }
-                        } else {
-                            $this->remove();
-                        }
-                        $reprocess = true;
-                        continue;
-                    }
-
-                }
-                $ok = true;
-            }
-
-            if ($ok) {
-                foreach ($this->injectors as $i => $injector) {
-                    if (isset($token->skip[$i])) continue;
-                    if ($token->rewind !== null && $token->rewind !== $i) continue;
-                    $injector->handleElement($token);
-                    $this->processToken($token, $i);
-                    $reprocess = true;
-                    break;
-                }
-                if (!$reprocess) {
-                    // ah, nothing interesting happened; do normal processing
-                    $this->swap($token);
-                    if ($token instanceof HTMLPurifier_Token_Start) {
-                        $this->stack[] = $token;
-                    } elseif ($token instanceof HTMLPurifier_Token_End) {
-                        throw new HTMLPurifier_Exception('Improper handling of end tag in start code; possible error in MakeWellFormed');
-                    }
-                }
-                continue;
-            }
-
-            // sanity check: we should be dealing with a closing tag
-            if (!$token instanceof HTMLPurifier_Token_End) {
-                throw new HTMLPurifier_Exception('Unaccounted for tag token in input stream, bug in HTML Purifier');
-            }
-
-            // make sure that we have something open
-            if (empty($this->stack)) {
-                if ($escape_invalid_tags) {
-                    if ($e) $e->send(E_WARNING, 'Strategy_MakeWellFormed: Unnecessary end tag to text');
-                    $this->swap(new HTMLPurifier_Token_Text(
-                        $generator->generateFromToken($token)
-                    ));
-                } else {
-                    $this->remove();
-                    if ($e) $e->send(E_WARNING, 'Strategy_MakeWellFormed: Unnecessary end tag removed');
-                }
-                $reprocess = true;
-                continue;
-            }
-
-            // first, check for the simplest case: everything closes neatly.
-            // Eventually, everything passes through here; if there are problems
-            // we modify the input stream accordingly and then punt, so that
-            // the tokens get processed again.
-            $current_parent = array_pop($this->stack);
-            if ($current_parent->name == $token->name) {
-                $token->start = $current_parent;
-                foreach ($this->injectors as $i => $injector) {
-                    if (isset($token->skip[$i])) continue;
-                    if ($token->rewind !== null && $token->rewind !== $i) continue;
-                    $injector->handleEnd($token);
-                    $this->processToken($token, $i);
-                    $this->stack[] = $current_parent;
-                    $reprocess = true;
-                    break;
-                }
-                continue;
-            }
-
-            // okay, so we're trying to close the wrong tag
-
-            // undo the pop previous pop
-            $this->stack[] = $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($this->stack);
-            // -2 because -1 is the last element, but we already checked that
-            $skipped_tags = false;
-            for ($j = $size - 2; $j >= 0; $j--) {
-                if ($this->stack[$j]->name == $token->name) {
-                    $skipped_tags = array_slice($this->stack, $j);
-                    break;
-                }
-            }
-
-            // we didn't find the tag, so remove
-            if ($skipped_tags === false) {
-                if ($escape_invalid_tags) {
-                    $this->swap(new HTMLPurifier_Token_Text(
-                        $generator->generateFromToken($token)
-                    ));
-                    if ($e) $e->send(E_WARNING, 'Strategy_MakeWellFormed: Stray end tag to text');
-                } else {
-                    $this->remove();
-                    if ($e) $e->send(E_WARNING, 'Strategy_MakeWellFormed: Stray end tag removed');
-                }
-                $reprocess = true;
-                continue;
-            }
-
-            // do errors, in REVERSE $j order: a,b,c with </a></b></c>
-            $c = count($skipped_tags);
-            if ($e) {
-                for ($j = $c - 1; $j > 0; $j--) {
-                    // notice we exclude $j == 0, i.e. the current ending tag, from
-                    // the errors... [TagClosedSuppress]
-                    if (!isset($skipped_tags[$j]->armor['MakeWellFormed_TagClosedError'])) {
-                        $e->send(E_NOTICE, 'Strategy_MakeWellFormed: Tag closed by element end', $skipped_tags[$j]);
-                    }
-                }
-            }
-
-            // insert tags, in FORWARD $j order: c,b,a with </a></b></c>
-            $replace = array($token);
-            for ($j = 1; $j < $c; $j++) {
-                // ...as well as from the insertions
-                $new_token = new HTMLPurifier_Token_End($skipped_tags[$j]->name);
-                $new_token->start = $skipped_tags[$j];
-                array_unshift($replace, $new_token);
-                if (isset($definition->info[$new_token->name]) && $definition->info[$new_token->name]->formatting) {
-                    // [TagClosedAuto]
-                    $element = clone $skipped_tags[$j];
-                    $element->carryover = true;
-                    $element->armor['MakeWellFormed_TagClosedError'] = true;
-                    $replace[] = $element;
-                }
-            }
-            $this->processToken($replace);
-            $reprocess = true;
-            continue;
-        }
-
-        $context->destroy('CurrentNesting');
-        $context->destroy('InputTokens');
-        $context->destroy('InputIndex');
-        $context->destroy('CurrentToken');
-
-        unset($this->injectors, $this->stack, $this->tokens, $this->t);
-        return $tokens;
-    }
-
-    /**
-     * Processes arbitrary token values for complicated substitution patterns.
-     * In general:
-     *
-     * If $token is an array, it is a list of tokens to substitute for the
-     * current token. These tokens then get individually processed. If there
-     * is a leading integer in the list, that integer determines how many
-     * tokens from the stream should be removed.
-     *
-     * If $token is a regular token, it is swapped with the current token.
-     *
-     * If $token is false, the current token is deleted.
-     *
-     * If $token is an integer, that number of tokens (with the first token
-     * being the current one) will be deleted.
-     *
-     * @param $token Token substitution value
-     * @param $injector Injector that performed the substitution; default is if
-     *        this is not an injector related operation.
-     */
-    protected function processToken($token, $injector = -1) {
-
-        // normalize forms of token
-        if (is_object($token)) $token = array(1, $token);
-        if (is_int($token))    $token = array($token);
-        if ($token === false)  $token = array(1);
-        if (!is_array($token)) throw new HTMLPurifier_Exception('Invalid token type from injector');
-        if (!is_int($token[0])) array_unshift($token, 1);
-        if ($token[0] === 0) throw new HTMLPurifier_Exception('Deleting zero tokens is not valid');
-
-        // $token is now an array with the following form:
-        // array(number nodes to delete, new node 1, new node 2, ...)
-
-        $delete = array_shift($token);
-        $old = array_splice($this->tokens, $this->t, $delete, $token);
-
-        if ($injector > -1) {
-            // determine appropriate skips
-            $oldskip = isset($old[0]) ? $old[0]->skip : array();
-            foreach ($token as $object) {
-                $object->skip = $oldskip;
-                $object->skip[$injector] = true;
-            }
-        }
-
-    }
-
-    /**
-     * Inserts a token before the current token. Cursor now points to
-     * this token.  You must reprocess after this.
-     */
-    private function insertBefore($token) {
-        array_splice($this->tokens, $this->t, 0, array($token));
-    }
-
-    /**
-     * Removes current token. Cursor now points to new token occupying previously
-     * occupied space.  You must reprocess after this.
-     */
-    private function remove() {
-        array_splice($this->tokens, $this->t, 1);
-    }
-
-    /**
-     * Swap current token with new token. Cursor points to new token (no
-     * change).  You must reprocess after this.
-     */
-    private function swap($token) {
-        $this->tokens[$this->t] = $token;
-    }
+	/**
+	 * Array stream of tokens being processed.
+	 */
+	protected $tokens;
+
+	/**
+	 * Current index in $tokens.
+	 */
+	protected $t;
+
+	/**
+	 * Current nesting of elements.
+	 */
+	protected $stack;
+
+	/**
+	 * Injectors active in this stream processing.
+	 */
+	protected $injectors;
+
+	/**
+	 * Current instance of HTMLPurifier_Config.
+	 */
+	protected $config;
+
+	/**
+	 * Current instance of HTMLPurifier_Context.
+	 */
+	protected $context;
+
+	public function execute($tokens, $config, $context) {
+
+		$definition = $config->getHTMLDefinition();
+
+		// local variables
+		$generator = new HTMLPurifier_Generator($config, $context);
+		$escape_invalid_tags = $config->get('Core.EscapeInvalidTags');
+		// used for autoclose early abortion
+		$global_parent_allowed_elements = array();
+		if (isset($definition->info[$definition->info_parent])) {
+			// may be unset under testing circumstances
+			$global_parent_allowed_elements = $definition->info[$definition->info_parent]->child->getAllowedElements($config);
+		}
+		$e = $context->get('ErrorCollector', true);
+		$t = false; // token index
+		$i = false; // injector index
+		$token      = false; // the current token
+		$reprocess  = false; // whether or not to reprocess the same token
+		$stack = array();
+
+		// member variables
+		$this->stack   =& $stack;
+		$this->t       =& $t;
+		$this->tokens  =& $tokens;
+		$this->config  = $config;
+		$this->context = $context;
+
+		// context variables
+		$context->register('CurrentNesting', $stack);
+		$context->register('InputIndex',     $t);
+		$context->register('InputTokens',    $tokens);
+		$context->register('CurrentToken',   $token);
+
+		// -- begin INJECTOR --
+
+		$this->injectors = array();
+
+		$injectors = $config->getBatch('AutoFormat');
+		$def_injectors = $definition->info_injector;
+		$custom_injectors = $injectors['Custom'];
+		unset($injectors['Custom']); // special case
+		foreach ($injectors as $injector => $b) {
+			// XXX: Fix with a legitimate lookup table of enabled filters
+			if (strpos($injector, '.') !== false) continue;
+			$injector = "HTMLPurifier_Injector_$injector";
+			if (!$b) continue;
+			$this->injectors[] = new $injector;
+		}
+		foreach ($def_injectors as $injector) {
+			// assumed to be objects
+			$this->injectors[] = $injector;
+		}
+		foreach ($custom_injectors as $injector) {
+			if (!$injector) continue;
+			if (is_string($injector)) {
+				$injector = "HTMLPurifier_Injector_$injector";
+				$injector = new $injector;
+			}
+			$this->injectors[] = $injector;
+		}
+
+		// give the injectors references to the definition and context
+		// variables for performance reasons
+		foreach ($this->injectors as $ix => $injector) {
+			$error = $injector->prepare($config, $context);
+			if (!$error) continue;
+			array_splice($this->injectors, $ix, 1); // rm the injector
+			trigger_error("Cannot enable {$injector->name} injector because $error is not allowed", E_USER_WARNING);
+		}
+
+		// -- end INJECTOR --
+
+		// a note on reprocessing:
+		//      In order to reduce code duplication, whenever some code needs
+		//      to make HTML changes in order to make things "correct", the
+		//      new HTML gets sent through the purifier, regardless of its
+		//      status. This means that if we add a start token, because it
+		//      was totally necessary, we don't have to update nesting; we just
+		//      punt ($reprocess = true; continue;) and it does that for us.
+
+		// isset is in loop because $tokens size changes during loop exec
+		for (
+			$t = 0;
+			$t == 0 || isset($tokens[$t - 1]);
+			// only increment if we don't need to reprocess
+			$reprocess ? $reprocess = false : $t++
+		) {
+
+			// check for a rewind
+			if (is_int($i) && $i >= 0) {
+				// possibility: disable rewinding if the current token has a
+				// rewind set on it already. This would offer protection from
+				// infinite loop, but might hinder some advanced rewinding.
+				$rewind_to = $this->injectors[$i]->getRewind();
+				if (is_int($rewind_to) && $rewind_to < $t) {
+					if ($rewind_to < 0) $rewind_to = 0;
+					while ($t > $rewind_to) {
+						$t--;
+						$prev = $tokens[$t];
+						// indicate that other injectors should not process this token,
+						// but we need to reprocess it
+						unset($prev->skip[$i]);
+						$prev->rewind = $i;
+						if ($prev instanceof HTMLPurifier_Token_Start) array_pop($this->stack);
+						elseif ($prev instanceof HTMLPurifier_Token_End) $this->stack[] = $prev->start;
+					}
+				}
+				$i = false;
+			}
+
+			// handle case of document end
+			if (!isset($tokens[$t])) {
+				// kill processing if stack is empty
+				if (empty($this->stack)) break;
+
+				// peek
+				$top_nesting = array_pop($this->stack);
+				$this->stack[] = $top_nesting;
+
+				// send error [TagClosedSuppress]
+				if ($e && !isset($top_nesting->armor['MakeWellFormed_TagClosedError'])) {
+					$e->send(E_NOTICE, 'Strategy_MakeWellFormed: Tag closed by document end', $top_nesting);
+				}
+
+				// append, don't splice, since this is the end
+				$tokens[] = new HTMLPurifier_Token_End($top_nesting->name);
+
+				// punt!
+				$reprocess = true;
+				continue;
+			}
+
+			$token = $tokens[$t];
+
+			//echo '<br>'; printTokens($tokens, $t); printTokens($this->stack);
+			//flush();
+
+			// quick-check: if it's not a tag, no need to process
+			if (empty($token->is_tag)) {
+				if ($token instanceof HTMLPurifier_Token_Text) {
+					foreach ($this->injectors as $i => $injector) {
+						if (isset($token->skip[$i])) continue;
+						if ($token->rewind !== null && $token->rewind !== $i) continue;
+						$injector->handleText($token);
+						$this->processToken($token, $i);
+						$reprocess = true;
+						break;
+					}
+				}
+				// another possibility is a comment
+				continue;
+			}
+
+			if (isset($definition->info[$token->name])) {
+				$type = $definition->info[$token->name]->child->type;
+			} else {
+				$type = false; // Type is unknown, treat accordingly
+			}
+
+			// quick tag checks: anything that's *not* an end tag
+			$ok = false;
+			if ($type === 'empty' && $token instanceof HTMLPurifier_Token_Start) {
+				// claims to be a start tag but is empty
+				$token = new HTMLPurifier_Token_Empty($token->name, $token->attr, $token->line, $token->col, $token->armor);
+				$ok = true;
+			} elseif ($type && $type !== 'empty' && $token instanceof HTMLPurifier_Token_Empty) {
+				// claims to be empty but really is a start tag
+				$this->swap(new HTMLPurifier_Token_End($token->name));
+				$this->insertBefore(new HTMLPurifier_Token_Start($token->name, $token->attr, $token->line, $token->col, $token->armor));
+				// punt (since we had to modify the input stream in a non-trivial way)
+				$reprocess = true;
+				continue;
+			} elseif ($token instanceof HTMLPurifier_Token_Empty) {
+				// real empty token
+				$ok = true;
+			} elseif ($token instanceof HTMLPurifier_Token_Start) {
+				// start tag
+
+				// ...unless they also have to close their parent
+				if (!empty($this->stack)) {
+
+					// Performance note: you might think that it's rather
+					// inefficient, recalculating the autoclose information
+					// for every tag that a token closes (since when we
+					// do an autoclose, we push a new token into the
+					// stream and then /process/ that, before
+					// re-processing this token.)  But this is
+					// necessary, because an injector can make an
+					// arbitrary transformations to the autoclosing
+					// tokens we introduce, so things may have changed
+					// in the meantime.  Also, doing the inefficient thing is
+					// "easy" to reason about (for certain perverse definitions
+					// of "easy")
+
+					$parent = array_pop($this->stack);
+					$this->stack[] = $parent;
+
+					if (isset($definition->info[$parent->name])) {
+						$elements = $definition->info[$parent->name]->child->getAllowedElements($config);
+						$autoclose = !isset($elements[$token->name]);
+					} else {
+						$autoclose = false;
+					}
+
+					if ($autoclose && $definition->info[$token->name]->wrap) {
+						// Check if an element can be wrapped by another 
+						// element to make it valid in a context (for 
+						// example, <ul><ul> needs a <li> in between)
+						$wrapname = $definition->info[$token->name]->wrap;
+						$wrapdef = $definition->info[$wrapname];
+						$elements = $wrapdef->child->getAllowedElements($config);
+						$parent_elements = $definition->info[$parent->name]->child->getAllowedElements($config);
+						if (isset($elements[$token->name]) && isset($parent_elements[$wrapname])) {
+							$newtoken = new HTMLPurifier_Token_Start($wrapname);
+							$this->insertBefore($newtoken);
+							$reprocess = true;
+							continue;
+						}
+					}
+
+					$carryover = false;
+					if ($autoclose && $definition->info[$parent->name]->formatting) {
+						$carryover = true;
+					}
+
+					if ($autoclose) {
+						// check if this autoclose is doomed to fail
+						// (this rechecks $parent, which his harmless)
+						$autoclose_ok = isset($global_parent_allowed_elements[$token->name]);
+						if (!$autoclose_ok) {
+							foreach ($this->stack as $ancestor) {
+								$elements = $definition->info[$ancestor->name]->child->getAllowedElements($config);
+								if (isset($elements[$token->name])) {
+									$autoclose_ok = true;
+									break;
+								}
+								if ($definition->info[$token->name]->wrap) {
+									$wrapname = $definition->info[$token->name]->wrap;
+									$wrapdef = $definition->info[$wrapname];
+									$wrap_elements = $wrapdef->child->getAllowedElements($config);
+									if (isset($wrap_elements[$token->name]) && isset($elements[$wrapname])) {
+										$autoclose_ok = true;
+										break;
+									}
+								}
+							}
+						}
+						if ($autoclose_ok) {
+							// errors need to be updated
+							$new_token = new HTMLPurifier_Token_End($parent->name);
+							$new_token->start = $parent;
+							if ($carryover) {
+								$element = clone $parent;
+								// [TagClosedAuto]
+								$element->armor['MakeWellFormed_TagClosedError'] = true;
+								$element->carryover = true;
+								$this->processToken(array($new_token, $token, $element));
+							} else {
+								$this->insertBefore($new_token);
+							}
+							// [TagClosedSuppress]
+							if ($e && !isset($parent->armor['MakeWellFormed_TagClosedError'])) {
+								if (!$carryover) {
+									$e->send(E_NOTICE, 'Strategy_MakeWellFormed: Tag auto closed', $parent);
+								} else {
+									$e->send(E_NOTICE, 'Strategy_MakeWellFormed: Tag carryover', $parent);
+								}
+							}
+						} else {
+							$this->remove();
+						}
+						$reprocess = true;
+						continue;
+					}
+
+				}
+				$ok = true;
+			}
+
+			if ($ok) {
+				foreach ($this->injectors as $i => $injector) {
+					if (isset($token->skip[$i])) continue;
+					if ($token->rewind !== null && $token->rewind !== $i) continue;
+					$injector->handleElement($token);
+					$this->processToken($token, $i);
+					$reprocess = true;
+					break;
+				}
+				if (!$reprocess) {
+					// ah, nothing interesting happened; do normal processing
+					$this->swap($token);
+					if ($token instanceof HTMLPurifier_Token_Start) {
+						$this->stack[] = $token;
+					} elseif ($token instanceof HTMLPurifier_Token_End) {
+						throw new HTMLPurifier_Exception('Improper handling of end tag in start code; possible error in MakeWellFormed');
+					}
+				}
+				continue;
+			}
+
+			// sanity check: we should be dealing with a closing tag
+			if (!$token instanceof HTMLPurifier_Token_End) {
+				throw new HTMLPurifier_Exception('Unaccounted for tag token in input stream, bug in HTML Purifier');
+			}
+
+			// make sure that we have something open
+			if (empty($this->stack)) {
+				if ($escape_invalid_tags) {
+					if ($e) $e->send(E_WARNING, 'Strategy_MakeWellFormed: Unnecessary end tag to text');
+					$this->swap(new HTMLPurifier_Token_Text(
+						$generator->generateFromToken($token)
+					));
+				} else {
+					$this->remove();
+					if ($e) $e->send(E_WARNING, 'Strategy_MakeWellFormed: Unnecessary end tag removed');
+				}
+				$reprocess = true;
+				continue;
+			}
+
+			// first, check for the simplest case: everything closes neatly.
+			// Eventually, everything passes through here; if there are problems
+			// we modify the input stream accordingly and then punt, so that
+			// the tokens get processed again.
+			$current_parent = array_pop($this->stack);
+			if ($current_parent->name == $token->name) {
+				$token->start = $current_parent;
+				foreach ($this->injectors as $i => $injector) {
+					if (isset($token->skip[$i])) continue;
+					if ($token->rewind !== null && $token->rewind !== $i) continue;
+					$injector->handleEnd($token);
+					$this->processToken($token, $i);
+					$this->stack[] = $current_parent;
+					$reprocess = true;
+					break;
+				}
+				continue;
+			}
+
+			// okay, so we're trying to close the wrong tag
+
+			// undo the pop previous pop
+			$this->stack[] = $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($this->stack);
+			// -2 because -1 is the last element, but we already checked that
+			$skipped_tags = false;
+			for ($j = $size - 2; $j >= 0; $j--) {
+				if ($this->stack[$j]->name == $token->name) {
+					$skipped_tags = array_slice($this->stack, $j);
+					break;
+				}
+			}
+
+			// we didn't find the tag, so remove
+			if ($skipped_tags === false) {
+				if ($escape_invalid_tags) {
+					$this->swap(new HTMLPurifier_Token_Text(
+						$generator->generateFromToken($token)
+					));
+					if ($e) $e->send(E_WARNING, 'Strategy_MakeWellFormed: Stray end tag to text');
+				} else {
+					$this->remove();
+					if ($e) $e->send(E_WARNING, 'Strategy_MakeWellFormed: Stray end tag removed');
+				}
+				$reprocess = true;
+				continue;
+			}
+
+			// do errors, in REVERSE $j order: a,b,c with </a></b></c>
+			$c = count($skipped_tags);
+			if ($e) {
+				for ($j = $c - 1; $j > 0; $j--) {
+					// notice we exclude $j == 0, i.e. the current ending tag, from
+					// the errors... [TagClosedSuppress]
+					if (!isset($skipped_tags[$j]->armor['MakeWellFormed_TagClosedError'])) {
+						$e->send(E_NOTICE, 'Strategy_MakeWellFormed: Tag closed by element end', $skipped_tags[$j]);
+					}
+				}
+			}
+
+			// insert tags, in FORWARD $j order: c,b,a with </a></b></c>
+			$replace = array($token);
+			for ($j = 1; $j < $c; $j++) {
+				// ...as well as from the insertions
+				$new_token = new HTMLPurifier_Token_End($skipped_tags[$j]->name);
+				$new_token->start = $skipped_tags[$j];
+				array_unshift($replace, $new_token);
+				if (isset($definition->info[$new_token->name]) && $definition->info[$new_token->name]->formatting) {
+					// [TagClosedAuto]
+					$element = clone $skipped_tags[$j];
+					$element->carryover = true;
+					$element->armor['MakeWellFormed_TagClosedError'] = true;
+					$replace[] = $element;
+				}
+			}
+			$this->processToken($replace);
+			$reprocess = true;
+			continue;
+		}
+
+		$context->destroy('CurrentNesting');
+		$context->destroy('InputTokens');
+		$context->destroy('InputIndex');
+		$context->destroy('CurrentToken');
+
+		unset($this->injectors, $this->stack, $this->tokens, $this->t);
+		return $tokens;
+	}
+
+	/**
+	 * Processes arbitrary token values for complicated substitution patterns.
+	 * In general:
+	 *
+	 * If $token is an array, it is a list of tokens to substitute for the
+	 * current token. These tokens then get individually processed. If there
+	 * is a leading integer in the list, that integer determines how many
+	 * tokens from the stream should be removed.
+	 *
+	 * If $token is a regular token, it is swapped with the current token.
+	 *
+	 * If $token is false, the current token is deleted.
+	 *
+	 * If $token is an integer, that number of tokens (with the first token
+	 * being the current one) will be deleted.
+	 *
+	 * @param $token Token substitution value
+	 * @param $injector Injector that performed the substitution; default is if
+	 *        this is not an injector related operation.
+	 */
+	protected function processToken($token, $injector = -1) {
+
+		// normalize forms of token
+		if (is_object($token)) $token = array(1, $token);
+		if (is_int($token))    $token = array($token);
+		if ($token === false)  $token = array(1);
+		if (!is_array($token)) throw new HTMLPurifier_Exception('Invalid token type from injector');
+		if (!is_int($token[0])) array_unshift($token, 1);
+		if ($token[0] === 0) throw new HTMLPurifier_Exception('Deleting zero tokens is not valid');
+
+		// $token is now an array with the following form:
+		// array(number nodes to delete, new node 1, new node 2, ...)
+
+		$delete = array_shift($token);
+		$old = array_splice($this->tokens, $this->t, $delete, $token);
+
+		if ($injector > -1) {
+			// determine appropriate skips
+			$oldskip = isset($old[0]) ? $old[0]->skip : array();
+			foreach ($token as $object) {
+				$object->skip = $oldskip;
+				$object->skip[$injector] = true;
+			}
+		}
+
+	}
+
+	/**
+	 * Inserts a token before the current token. Cursor now points to
+	 * this token.  You must reprocess after this.
+	 */
+	private function insertBefore($token) {
+		array_splice($this->tokens, $this->t, 0, array($token));
+	}
+
+	/**
+	 * Removes current token. Cursor now points to new token occupying previously
+	 * occupied space.  You must reprocess after this.
+	 */
+	private function remove() {
+		array_splice($this->tokens, $this->t, 1);
+	}
+
+	/**
+	 * Swap current token with new token. Cursor points to new token (no
+	 * change).  You must reprocess after this.
+	 */
+	private function swap($token) {
+		$this->tokens[$this->t] = $token;
+	}
 
 }
 

Spacing +6 added lines, -6 removed lines

@@ -65,17 +65,17 @@
         $stack = array();
 
         // member variables
-        $this->stack   =& $stack;
-        $this->t       =& $t;
-        $this->tokens  =& $tokens;
+        $this->stack   = & $stack;
+        $this->t       = & $t;
+        $this->tokens  = & $tokens;
         $this->config  = $config;
         $this->context = $context;
 
         // context variables
         $context->register('CurrentNesting', $stack);
-        $context->register('InputIndex',     $t);
-        $context->register('InputTokens',    $tokens);
-        $context->register('CurrentToken',   $token);
+        $context->register('InputIndex', $t);
+        $context->register('InputTokens', $tokens);
+        $context->register('CurrentToken', $token);
 
         // -- begin INJECTOR --
 

Braces +71 added lines, -24 removed lines

@@ -87,9 +87,13 @@ 
         unset($injectors['Custom']); // special case
         foreach ($injectors as $injector => $b) {
             // XXX: Fix with a legitimate lookup table of enabled filters
-            if (strpos($injector, '.') !== false) continue;
+            if (strpos($injector, '.') !== false) {
+            	continue;
+            }
             $injector = "HTMLPurifier_Injector_$injector";
-            if (!$b) continue;
+            if (!$b) {
+            	continue;
+            }
             $this->injectors[] = new $injector;
         }
         foreach ($def_injectors as $injector) {
@@ -97,7 +101,9 @@ 
             $this->injectors[] = $injector;
         }
         foreach ($custom_injectors as $injector) {
-            if (!$injector) continue;
+            if (!$injector) {
+            	continue;
+            }
             if (is_string($injector)) {
                 $injector = "HTMLPurifier_Injector_$injector";
                 $injector = new $injector;
@@ -109,7 +115,9 @@ 
         // variables for performance reasons
         foreach ($this->injectors as $ix => $injector) {
             $error = $injector->prepare($config, $context);
-            if (!$error) continue;
+            if (!$error) {
+            	continue;
+            }
             array_splice($this->injectors, $ix, 1); // rm the injector
             trigger_error("Cannot enable {$injector->name} injector because $error is not allowed", E_USER_WARNING);
         }
@@ -139,7 +147,9 @@ 
                 // infinite loop, but might hinder some advanced rewinding.
                 $rewind_to = $this->injectors[$i]->getRewind();
                 if (is_int($rewind_to) && $rewind_to < $t) {
-                    if ($rewind_to < 0) $rewind_to = 0;
+                    if ($rewind_to < 0) {
+                    	$rewind_to = 0;
+                    }
                     while ($t > $rewind_to) {
                         $t--;
                         $prev = $tokens[$t];
@@ -147,8 +157,11 @@ 
                         // but we need to reprocess it
                         unset($prev->skip[$i]);
                         $prev->rewind = $i;
-                        if ($prev instanceof HTMLPurifier_Token_Start) array_pop($this->stack);
-                        elseif ($prev instanceof HTMLPurifier_Token_End) $this->stack[] = $prev->start;
+                        if ($prev instanceof HTMLPurifier_Token_Start) {
+                        	array_pop($this->stack);
+                        } elseif ($prev instanceof HTMLPurifier_Token_End) {
+                        	$this->stack[] = $prev->start;
+                        }
                     }
                 }
                 $i = false;
@@ -157,7 +170,9 @@ 
             // handle case of document end
             if (!isset($tokens[$t])) {
                 // kill processing if stack is empty
-                if (empty($this->stack)) break;
+                if (empty($this->stack)) {
+                	break;
+                }
 
                 // peek
                 $top_nesting = array_pop($this->stack);
@@ -185,8 +200,12 @@ 
             if (empty($token->is_tag)) {
                 if ($token instanceof HTMLPurifier_Token_Text) {
                     foreach ($this->injectors as $i => $injector) {
-                        if (isset($token->skip[$i])) continue;
-                        if ($token->rewind !== null && $token->rewind !== $i) continue;
+                        if (isset($token->skip[$i])) {
+                        	continue;
+                        }
+                        if ($token->rewind !== null && $token->rewind !== $i) {
+                        	continue;
+                        }
                         $injector->handleText($token);
                         $this->processToken($token, $i);
                         $reprocess = true;
@@ -325,8 +344,12 @@ 
 
             if ($ok) {
                 foreach ($this->injectors as $i => $injector) {
-                    if (isset($token->skip[$i])) continue;
-                    if ($token->rewind !== null && $token->rewind !== $i) continue;
+                    if (isset($token->skip[$i])) {
+                    	continue;
+                    }
+                    if ($token->rewind !== null && $token->rewind !== $i) {
+                    	continue;
+                    }
                     $injector->handleElement($token);
                     $this->processToken($token, $i);
                     $reprocess = true;
@@ -352,13 +375,17 @@ 
             // make sure that we have something open
             if (empty($this->stack)) {
                 if ($escape_invalid_tags) {
-                    if ($e) $e->send(E_WARNING, 'Strategy_MakeWellFormed: Unnecessary end tag to text');
+                    if ($e) {
+                    	$e->send(E_WARNING, 'Strategy_MakeWellFormed: Unnecessary end tag to text');
+                    }
                     $this->swap(new HTMLPurifier_Token_Text(
                         $generator->generateFromToken($token)
                     ));
                 } else {
                     $this->remove();
-                    if ($e) $e->send(E_WARNING, 'Strategy_MakeWellFormed: Unnecessary end tag removed');
+                    if ($e) {
+                    	$e->send(E_WARNING, 'Strategy_MakeWellFormed: Unnecessary end tag removed');
+                    }
                 }
                 $reprocess = true;
                 continue;
@@ -372,8 +399,12 @@ 
             if ($current_parent->name == $token->name) {
                 $token->start = $current_parent;
                 foreach ($this->injectors as $i => $injector) {
-                    if (isset($token->skip[$i])) continue;
-                    if ($token->rewind !== null && $token->rewind !== $i) continue;
+                    if (isset($token->skip[$i])) {
+                    	continue;
+                    }
+                    if ($token->rewind !== null && $token->rewind !== $i) {
+                    	continue;
+                    }
                     $injector->handleEnd($token);
                     $this->processToken($token, $i);
                     $this->stack[] = $current_parent;
@@ -406,10 +437,14 @@ 
                     $this->swap(new HTMLPurifier_Token_Text(
                         $generator->generateFromToken($token)
                     ));
-                    if ($e) $e->send(E_WARNING, 'Strategy_MakeWellFormed: Stray end tag to text');
+                    if ($e) {
+                    	$e->send(E_WARNING, 'Strategy_MakeWellFormed: Stray end tag to text');
+                    }
                 } else {
                     $this->remove();
-                    if ($e) $e->send(E_WARNING, 'Strategy_MakeWellFormed: Stray end tag removed');
+                    if ($e) {
+                    	$e->send(E_WARNING, 'Strategy_MakeWellFormed: Stray end tag removed');
+                    }
                 }
                 $reprocess = true;
                 continue;
@@ -479,12 +514,24 @@ 
     protected function processToken($token, $injector = -1) {
 
         // normalize forms of token
-        if (is_object($token)) $token = array(1, $token);
-        if (is_int($token))    $token = array($token);
-        if ($token === false)  $token = array(1);
-        if (!is_array($token)) throw new HTMLPurifier_Exception('Invalid token type from injector');
-        if (!is_int($token[0])) array_unshift($token, 1);
-        if ($token[0] === 0) throw new HTMLPurifier_Exception('Deleting zero tokens is not valid');
+        if (is_object($token)) {
+        	$token = array(1, $token);
+        }
+        if (is_int($token)) {
+        	$token = array($token);
+        }
+        if ($token === false) {
+        	$token = array(1);
+        }
+        if (!is_array($token)) {
+        	throw new HTMLPurifier_Exception('Invalid token type from injector');
+        }
+        if (!is_int($token[0])) {
+        	array_unshift($token, 1);
+        }
+        if ($token[0] === 0) {
+        	throw new HTMLPurifier_Exception('Deleting zero tokens is not valid');
+        }
 
         // $token is now an array with the following form:
         // array(number nodes to delete, new node 1, new node 2, ...)