[cvs] [Wiki] changed: BCBreakingHordeIssues

[Chuck Hagenbuch]

chuck  Mon, 18 Dec 2006 13:03:24 -0800

Modified page: https://dev.horde.org/horde/wicked/BCBreakingHordeIssues
New Revision:  2.33
Change log:  group template issues

@@ -10,9 +10,9 @@
  * Encrypt deprecations in Horde_Crypt_pgp::.
  * Use one place for clear text sendmail error messages.
  * rename files in imp/templates/prefs to more sensible defaults (e.g. draftsfolderselect.inc, not folderselect.inc)
  * Remove {{$foo = &new Foo();}} calls and replace with {{$foo = new Foo();}} - The former is deprecated as of PHP 5 and causes errors when strict code checking is on.
-* Remove IMP_Template::.
+* Similarly, don't pass objects by reference ( function foo(&$bar) )
  * Move share edit form into framework package to make it extendable.
  * Convert all configuration files to XML format.
  * Remove IMP_IMAP_Search:: and Ingo_IMAP_Search::.
  * Remove imap_reopen() call in IMAP_Search::.
@@ -20,12 +20,11 @@
  * Move date/time parsing methods from iCalendar:: to Horde_Date::.
  * Checkboxes in Horde_Form that are marked as required should be required to be checked.
  * Remove IMP_SpellChecker::.
  * Add a single function for changing the language (handle the bindTextDomain() call also)
-* Remove hack to fix app-specific Notification_Listeners that output <p> instead of <li> (in Notification/Listender/status.php).
+* Remove hack to fix app-specific Notification_Listeners that output <p> instead of <li> (in Notification/Listener/status.php).
  * Prefix all classes with Horde_ that don't have that prefix yet (also see http://www.php.net/manual/en/userlandnaming.php).
  * Filter all input through filter extension or library.
-* Escape all template tags by default.
  * Use phone form fields, i.e. in Turba.
  * Use exceptions instead of PEAR_Errors.
  * Clean up Horde::loadConfiguration() mess.
  * Consistent return types (array vs. timestamp) in Horde_iCalendar::_parse*() methods.
@@ -48,8 +47,15 @@
  * Remove app-specific 'open_calendar.js' files and extra code in Horde's open_calendar.js.
  * Replace horde's popup.js with IMP's version.
  * Basically consolidate _all_ js present in multiple apps
  * Put core javascript into a package so it can be versioned/upgraded?
+
+++ Templates
+* Remove IMP_Template, Ingo_Template, and Gollem_Template.
+* Replace all templates with a Horde_View class, with a default of simple PHP templates
+* Introduce templates for ALL HTML generated in libraries
+* Figure out how to handle templates for framework packages
+* Default HTML Horde_View should escape all output (htmlspecialchars)
  
  ++ Bugs
  * Bug [http://bugs.horde.org/ticket/?id=4221 4221].
  * Bug [http://bugs.horde.org/ticket/?id=3271 3271] - more specifically, how to deal with addition of configspecial tags. Of course, this depends on what changes are made to Horde's configuration design.