I modified the forms to only use $regform and make the form code:
* More readable
* Have no duplicated code
* Allow the servers to remain selected if the validation tests fail
* Allow easy addition of servers, or even automated population of servers from a database
Some suggestions I would make are:
* Why put a maximum length on a password? Especially since you're hashing it.
* If you're using a server that uses magic_quotes_gpc, don't rely on them to make your database queries secure. Always run your own validation on input that ensures that it is safe to use in a query, or use PDO for database functions and PDO->prepare() if you have access to it.
* The rand() function can produce predictable results, consider mt_rand() instead
* When INSERTing data to the database, specify the columns you're inserting before specifying values. That way it's easier to change the database structure without breaking existing code.
sorry for taking so long to reply but talking about the little forum i told you about made me remember i have had a problem with it and thats why i stopped work on it so i tried to work on the problem again and just can not figure out why it is not working this is the link to the code that i think is the problem the-code
i have looked over the code many times but i just can not figure out why it is not working when you click edit reply a drop down is supposed to come but it doesnt do anything it is strange and realy annoying because it messes up my site a lot
As an aside, ensure your dom structure is valid, and try not to use depreciated elements like - the W3C validator and a valid doctype can help with this.
If you look at the example, some of the structure and css has been modified to adhere to XHTML 1.0 Strict.
I'm not sure of the doctype or underlying structure of your live version, but I usually work in strict to ensure future compatibility.