Suggestions [message #15865] |
Wed, 07 January 2004 16:12 ![Go to next message Go to next message](/forum/theme/default/images/down.png) |
Gribnif
![United States United States](/forum/images/flags/us.png) Messages: 82 Registered: December 2003
Karma: 0
|
Member |
|
|
1. Add a "technical" section to the documentation, which describes how the src/*.pm.t files are compiled into the .php files for each theme. It took me quite some time to figure out that I couldn't just make changes to the .php files and have them persist.
2. Add more comments to the PHP source. It's very sparse right now.
3. Change PHP code like this:
if (!($usr_d->users_opt & 131072)) {
to be more readable, by using constants:
define( 'ACCT_CONFIRMED',131072);
...
if (!($usr_d->users_opt & ACCT_CONFIRMED)) {
|
|
|
Re: Suggestions [message #15866 is a reply to message #15865] |
Wed, 07 January 2004 16:25 ![Go to previous message Go to previous message](/forum/theme/default/images/up.png) ![Go to next message Go to next message](/forum/theme/default/images/down.png) |
Ilia
![Canada Canada](/forum/images/flags/ca.png) Messages: 13241 Registered: January 2002
Karma: 0
|
Senior Member Administrator Core Developer |
|
|
It maybe more readable, but WAY slower. Constant look-ups in PHP are very slow.
FUDforum Core Developer
|
|
|
|
Re: Suggestions [message #15870 is a reply to message #15868] |
Wed, 07 January 2004 16:58 ![Go to previous message Go to previous message](/forum/theme/default/images/up.png) ![Go to next message Go to next message](/forum/theme/default/images/down.png) |
Ilia
![Canada Canada](/forum/images/flags/ca.png) Messages: 13241 Registered: January 2002
Karma: 0
|
Senior Member Administrator Core Developer |
|
|
Yeah, but variables are not always in the scope of functions that use them and so on. If you want to see what the various #s mean looks inside GLOBALS_HELP, which documents all global constants. The sql/*.tbl files identify the keys for used in various tables.
FUDforum Core Developer
|
|
|
|
Re: Suggestions [message #15880 is a reply to message #15879] |
Thu, 08 January 2004 01:34 ![Go to previous message Go to previous message](/forum/theme/default/images/up.png) |
Ilia
![Canada Canada](/forum/images/flags/ca.png) Messages: 13241 Registered: January 2002
Karma: 0
|
Senior Member Administrator Core Developer |
|
|
It'll over complicate matters.
FUDforum Core Developer
|
|
|