[cvs] [Wiki] changed: RemoveGlobals

[Jan Schneider]

jan  Tue, 15 Aug 2006 14:44:18 -0700

Modified page: http://wiki.horde.org/RemoveGlobals
New Revision:  1.2
Change log:  My thoughts

@@ -6,4 +6,12 @@
  * //Instantiate// master Horde object (e.g. {{$horde = new Horde($config);}})
  * InstantiateApplicationObjectRefactoring
  * Page handling become method object (e.g. {{$page = new Turba_BrowsePage($app, $request); $page->execute();}})
  
+--Jason
+
+----
+
+* Instantiation of the Horde object doesn't make sense to me because it introduces another global object. And why is using a static Horde class bad?
+* Usage of $_SESSION makes a lot of sense to me, because it is very clear that we are dealing with session state. Encapsulating this in a class or object obfuscates the code.
+
+--Jan