[phpBddb-dev1] comments

[Janis P. <jan...@on...>]

Hi!
I thought that I should probably comment the code I wrote because I introduced some new things.
1) global $conf array. I thought this could be better because "global $conf" is much easier than to include every variable.

2) global $msg_codes array. This array has the numeric value for messages that would be passed between files. It would be easier to send plain text but it is not "secure" because someone could put HTML in it.

3) global $lang array. This is very alpha. This is just an idea how languages could be managed. In the very end of all that list there is line where global array $text is set the value of current language. This is because it is shorter to write $text['back'] than $text[$conf['language']]['back'].

4) Single quotes (') vs. double quotes ("). PHP parses all text that is enclosed w double quotes and looks for any variables that could be in there. Of course if it is short text it doesn't matter. I just like to get myself used to them. For SQL it is probably easier to use double quotes.

5) This is for the future.. :) I don't like the default session system built in the PHP. If the site is published on the server farm it would not work. The /tmp folder (if not customized) has to have the right permissions for some users not to hijack the sessions. I would like to rewrite the sessions for MySQL but I thought we need to release the beta by 9th and there may be some more to do. We can't just change php session handler because it requires php.ini editing and that's not good.

Any comments or critics would be great.
Janis


--> http://www.one.lv - your number one mobile email service!