[cvs] [Wiki] changed: BCBreakingHordeIssues

[Chuck Hagenbuch]

chuck  Thu, 03 Aug 2006 09:33:11 -0700

Modified page: https://dev.horde.org/horde/wicked/BCBreakingHordeIssues
New Revision:  2.8
Change log:  notes on autoloading

@@ -27,8 +27,11 @@
  * Remove hack to fix app-specific Notification_Listeners that output <p> instead of <li> (in Notification/Listender/status.php).
  
  ++ To discuss
  * use __autoload() instead of require_once
+ * Using a global __autoload() function is not good practice for a framework because you can only have one __autoload function - I'm not in favor of it (ChuckHagenbuch).
+ * Using it in Horde apps as opposed to the framework is an option, but still leaves us all of the framework requires.
+ * http://php.net/spl-autoload-register is a potential solution. With PHP 5.1.x (or 5.2, or ...) as a minimum version requiring SPL should be okay.
  * Refactor to remove all global variables and other couplings with global state ($_SESSION, $_REQUEST, singleton pattern, etc).  We can do this by:
   * Encapsulting request in a request object
   * //Instantiate// master Horde object (e.g. {{$horde = new Horde($config);}})
   * //Instantiate// application object (e.g. {{$app = new Turba($horde,$config);}})