Re: cvs: pear /LiveUser/Perm Simple.php /LiveUser/Perm/Storage
| From: | Lukas Smith | Date: | Wed, 15 Dec 2004 20:17:43 +0000 |
| Subject: | Re: cvs: pear /LiveUser/Perm Simple.php /LiveUser/Perm/Storage | ||
| References: | 1 2 3 | Groups: | php.pear.dev |
| Request: | Send a blank email to pear-dev+get-35087@lists.php.net to get a copy of this message | ||
Lukas Smith wrote:
Lukas Smith wrote:ok appearently nobody cares so i guess i will go with the super global variant. *sigh* *grumble* so much for the community experience regards, LukasLukas Smith wrote:in an effort to entice a few more replies: Arnaud told me in IRC he prefers the super global. hmm maybe i need to make the content of the thread more flameable .. it seems such content gets much more attention .. ;-)In LiveUser we have several rather large configuration arrays, that essentially cover the entire db schema. Since we need those arrays in the client as well as the admin classes I have placed them inside static methods of the client. See the code in CVS or the latest commits for details. Now the question is shouldnt I better place them inside a separate script and just place them inside the super global ..? $GLOBALS[_LiveUser]['tables'] = array( .. ); That way I dont need to call a static method to get the array both inside the client and inside the admin. Inside the admin I would also get around including a few lines of code (however in practice its likely that these lines of code will be included already anyways).lsmith Sun Dec 12 16:25:56 2004 EDTModified files: /pear/LiveUser/Perm Simple.php /pear/LiveUser/Perm/Storage SQL.php Log: - added new default for config arrays inside static methods so we can fetch them from the admin classes)