Todo List

patTemplate

patTemplate_Compiler

• check for existing compiled template
• implement all template types
• implement getParsedTemplate
• implement variable modifiers

patTemplate_Dump_Html

• move this into patTemplate_Dump_Dhtml and keep it free from javascript

patTemplate_Dump_XUL

• move this into patTemplate_Dump_Dhtml and keep it free from javascript

patTemplate_Compiler::_compileModuloTemplate()

• check special conditions (__first, __last, __default)

patTemplate::_getConditionValue()

• split this method into smaller check methods that will be called according to a priority list

patTemplate_Compiler::_getVar()

• check for 'unusedvars' attribute

core

GeSHi::$enable_important_blocks

• REMOVE THIS FUNCTIONALITY!

GeSHi::$error

• check err reporting works

GeSHi::$important_styles

• As above - rethink the whole idea of important blocks as it is buggy and will be hard to implement in 1.2

GeSHi::$line_numbers_start

• Warning documentation about XHTML compliance

GeSHi::add_url_to_keyword()

• Get rid of ender

GeSHi::disable_highlighting()

• Rewrite with an array traversal

GeSHi::enable_highlighting()

• Rewrite with array traversal

GeSHi::enable_important_blocks()

• REMOVE THIS SHIZ FROM GESHI!

GeSHi::get_attributes()

• Document behaviour change - class is outputted regardless of whether we're using classes or not. Same with style

GeSHi::get_language_name_from_extension()

• Re-think about how this method works (maybe make it private and/or make it a extension->lang lookup?)
• static?

GeSHi::highlight_lines_extra()

• Some data replication here that could be cut down on

GeSHi::load_from_file()

• Complete rethink of this and above method

GeSHi::load_language()

• Needs to load keys for lexic permissions for keywords, regexps etc

GeSHi::parse_non_string_part()

• BUGGY! Why? Why not build string and return?

GeSHi::set_case_keywords()

• Error check the passed value

patError

patErrorManager

• implement expectError() to ignore an error with a certain code only once.
• implement ignoreError() to ignore errrors with a certain code

patErrorManager::raise()

• either remove HTML tags and entities from output or test for enviroment!!! in shell is ugly!
• implement 'simple' mode that returns just false (BC for patConfiguration)

patError::__construct()

• all calls to patErrorManager::raise* should not be included in backtrace