On 8/11/09 11:33 AM, dan nessett wrote:
--- On Tue, 8/11/09, Brion Vibberbrion@wikimedia.org wrote:
I'm not sure there's a compelling reason to even have $IP set in LocalSettings.php anymore; the base include path should probably be autodetected in all cases, which is already being done in WebStart.php and commandLine.inc, the web and CLI initialization includes based on their locations in the file tree.
I started this thread because two of the fixes in the patch for bug ticket 20112 (those for Database.t and Global.t) move the require of LocalSettings.php before the require of AutoLoader.php. This is necessary because AutoLoader.php eventually executes: require_once("$IP/js2/mwEmbed/php/jsAutoloadLocalClasses.php").
These scripts should simply be updated to initialize the framework properly instead of trying to half-ass it and load individual classes.
-- brion