diff --git a/TODO b/TODO index b8b2caa1..71ffb20c 100644 --- a/TODO +++ b/TODO @@ -9,6 +9,7 @@ Ongoing - Additional support for poorly written HTML - Implement all non-essential attribute transforms - Microsoft Word HTML cleaning (i.e. MsoNormal) + - Error logging for filtering and cleanup procedures 1.3 release - Formatters for plaintext @@ -41,6 +42,8 @@ Unknown release (on a scratch-an-itch basis) - Pretty-printing HTML (adds dependency of Generator to HTMLDefinition) - Non-lossy dumb alternate character encoding transformations, achieved by numerically encoding all non-ASCII characters + - Preservation of indentation in tables (tricky since the contents can be + shuffled around) Wontfix - Non-lossy smart alternate character encoding transformations diff --git a/library/HTMLPurifier/AttrContext.php b/library/HTMLPurifier/AttrContext.php index c3316737..5f0c1679 100644 --- a/library/HTMLPurifier/AttrContext.php +++ b/library/HTMLPurifier/AttrContext.php @@ -3,15 +3,15 @@ /** * Internal data-structure used in attribute validation to accumulate state. * - * All it is is a data-structure that holds objects that accumulate state, like - * HTMLPurifier_IDAccumulator. + * This is a data-structure that holds objects that accumulate state, like + * HTMLPurifier_IDAccumulator. It's better than using globals! * - * @param Many functions that accept this object have it as a mandatory - * parameter, even when there is no use for it. Though this is - * for the same reasons as why HTMLPurifier_Config is a mandatory - * parameter, it is also because you cannot assign a default value - * to a parameter passed by reference (passing by reference is essential - * for context to work in PHP 4). + * @note Many functions that accept this object have it as a mandatory + * parameter, even when there is no use for it. Though this is + * for the same reasons as why HTMLPurifier_Config is a mandatory + * parameter, it is also because you cannot assign a default value + * to a parameter passed by reference (passing by reference is essential + * for context to work in PHP 4). */ class HTMLPurifier_AttrContext diff --git a/library/HTMLPurifier/ConfigSchema.php b/library/HTMLPurifier/ConfigSchema.php index 270fd8fd..99806bd9 100644 --- a/library/HTMLPurifier/ConfigSchema.php +++ b/library/HTMLPurifier/ConfigSchema.php @@ -2,7 +2,6 @@ /** * Configuration definition, defines directives and their defaults. - * @todo Build documentation generation capabilities. * @todo The ability to define things multiple times is confusing and should * be factored out to its own function named registerDependency() or * addNote(), where only the namespace.name and an extra descriptions @@ -39,7 +38,6 @@ class HTMLPurifier_ConfigSchema { /** * Lookup table of allowed types. - * @todo Add descriptions */ var $types = array( 'string' => 'String', @@ -82,9 +80,6 @@ class HTMLPurifier_ConfigSchema { /** * Defines a directive for configuration * @warning Will fail of directive's namespace is defined - * @todo Collect information on description and allow redefinition - * so that multiple files can register a dependency on a - * configuration directive. * @param $namespace Namespace the directive is in * @param $name Key of directive * @param $default Default value of directive diff --git a/library/HTMLPurifier/EntityParser.php b/library/HTMLPurifier/EntityParser.php index 83593f7a..478b6ba2 100644 --- a/library/HTMLPurifier/EntityParser.php +++ b/library/HTMLPurifier/EntityParser.php @@ -88,7 +88,6 @@ class HTMLPurifier_EntityParser * either index 1, 2 or 3 set with a hex value, dec value, * or string (respectively). * @returns Replacement string. - * @todo Implement string translations */ // +----------+----------+----------+----------+ diff --git a/library/HTMLPurifier/Lexer/DOMLex.php b/library/HTMLPurifier/Lexer/DOMLex.php index 2df01f52..a24d1014 100644 --- a/library/HTMLPurifier/Lexer/DOMLex.php +++ b/library/HTMLPurifier/Lexer/DOMLex.php @@ -12,15 +12,12 @@ require_once 'HTMLPurifier/TokenFactory.php'; * documents, it performs twenty times faster than * HTMLPurifier_Lexer_DirectLex,and is the default choice for PHP 5. * - * @notice - * Any empty elements will have empty tokens associated with them, even if + * @note Any empty elements will have empty tokens associated with them, even if * this is prohibited by the spec. This is cannot be fixed until the spec * comes into play. * - * @todo Determine DOM's entity parsing behavior, point to local entity files - * if necessary. - * @todo Make div access less fragile, and refrain from preprocessing when - * HTML tag and friends are already present. + * @note PHP's DOM extension does not actually parse any entities, we use + * our own function to do that. */ class HTMLPurifier_Lexer_DOMLex extends HTMLPurifier_Lexer