Re: [Coding Standards] Loading all files at once

Travis Swicegood wrote:

Howdy... Joshua Eichorn wrote:

Obviously we can't meet everyones needs, but we do want a solution that is better then the current one. An allfiles.php approach might not be that, but obviously status quo isn't any better either.

Actually, I think you can meet the needs of everyone. From the benchmark tests I ran, it looks like you can easily have the allfiles for the performance geeks and have all of the currently existing require_once declarations at the top of the file. My tests showed that it was more efficient to do the second request through a require_once than through an if(!class_exists()), so I don't see an issue. Remove the text disallowing all require_once statements, and add code to Pyrus to generate an allfiles.php file. I imagine Greg's going to have it know what type of package it's preparing, so it will be extremely easy for it to create the necessary require_once statements, and for that matter inject the dirname(__FILE__) call into all of the require_onces throughout the code when it creates the unzip and go installs. -Travis -Travis Can you show an example of what the file would look like.

If your talking about something like: if (!class_exists('Class_Name')) { require 'filename'; } Then yes you can meet everyones goals, though there is still the matter of a PEAR2_PATH constant and if the flexibility is worth it. -josh