Register Info Not Inserting Into Database

Antec

The attached file is my register.php script. What is happening is when I test it but registering the site responds saying that the information has been entered correctly and I get the activation email but the link in the activation email doesn't work, and the reason for that is that nothing is being saved into the database. Now there are no connections errors as a matter of fact there are no errors at all which is confusing me. Any ideas?

Antec

Anybody?

NogDog

At a quick glance, I notice that your code assumes that all queries work, never checking the return value from mysql_query() for a false result, and if so, logging some debug info. A little bit of defensive coding and some error outputs will very likely help you find the problem(s). A typical pattern would be something like:

$sql = "define your SQL string here";
$result = mysql_query($sql);
if(!$result)
{
   user_error(mysql_error() . "<br />\n$sql");
   // or do die() instead of user_error() if such a failure is fatal
}

And if you want to see the debug data on the screen, be sure to turn on display_errors and set the error_reporting level sufficiently high.

Antec

Well im not gettin any errors the issue is that it is not reconnising that there is any info to insert into the database

so would it look something like this?

$sql = "INSERT INTO members (id, status, name, ipaddress, salt, gender, country, password, chatpass, email, joined, act)VALUES ('NULL', '".$status."', '".$member[name]."', '".$ip."', '".$enurl."', '".$member[gender]."', '".$member[country]."', '".$pass2."', '".$member[chatpass]."', '".$member[email]."', 'NULL', '".$anumber."')";
$result = mysql_query($sql);
if(!$result)
{
   user_error(mysql_error() . "<br />\n$sql");
   // or do die() instead of user_error() if such a failure is fatal
}

leatherback

Yes, or if you do it just for debugging purpose:

$sql = "INSERT INTO members (id, status, name, ipaddress, salt, gender, country, password, chatpass, email, joined, act) VALUES ('NULL', '".$status."', '".$member[name]."', '".$ip."', '".$enurl."', '".$member[gender]."', '".$member[country]."', '".$pass2."', '".$member[chatpass]."', '".$member[email]."', 'NULL', '".$anumber."')";
$result = mysql_query($sql) or die(mysql_error());

Note that user_error() is probably a userdefined error handling function which you would need to write (that e.g., logs the error to the database for later verification)

NogDog

leatherback;10880068 wrote:
Note that user_error() is probably a userdefined error handling function which you would need to write (that e.g., logs the error to the database for later verification)

No, it's a built-in function: [man]user_error/man (actually an alias of [man]trigger_error[/man]).

leatherback

coool.. Never used that before. Learned something again!