Site testing


#1

Hi everyone

I know this could seems a weird post but I would like, if possible, that few of you (2 or 3) can test the php version of my site at this adress
http://membres.lycos.fr/luciferchrist/index_new.php/
'cause I don’t have much time to test it ('cause of my job and my other web projects) and I also have problem with my web browser.

If it’s not possible, excuse me for that post and I won’t post another one like this one.

Friendly, Lucifer Christ, php beginner


#2

Few things. If you don’t have time to test it for yourself, I find it really hard to make time to test it for you. Your link is broken:
http://membres.lycos.fr/luciferchrist/index_new.php
Last but not least, je ne parle pas le francais.


#3

Sorry about the french speaking inconvenient. It’s the next big step i’ll work on, to make it bilingual depending on the browser’s language preference.

I thank you for taking part of your time to do it. I’ve done a litttle testing of it this night and i’ve found just a problem. I don’t knowif it’s my coding or my host (multimania) 'cause it worked fine on my computer. The confirmation page doesn’t loads the table it contains when the registration is done. Here’s the code

[code]<?php
include “include/connexion.inc.php”;
$loginame = $_POST[‘loginame’];
$loginpword = $_POST[‘loginpword’];
$usermail = $_POST[‘usermail’];
$userlocation = $_POST[‘userlocation’];
// check if the username entered already exist in the table
$sql_1 = “select * from reg_member where loginame=’$loginame’”;
$reg_result_1 = mysql_db_query($dbname,$sql_1,$id_link);
$reg_num_1 = @mysql_num_rows($reg_result_1);
if(!$reg_num_1 || $reg_num_1>1) {
// check if the email entered already exist in the table
$sql_2 = “select * from reg_member where usermail=’$usermail’”;
$reg_result_2 = mysql_db_query($dbname,$sql_2,$id_link);
$reg_num_2 = @mysql_num_rows($reg_result_2);
if(!$reg_num_2 || $reg_num_2>1) {
// record the data’s in the table if everything is ok
$sql=“INSERT INTO reg_member (loginame,loginpword,usermail,userlocation) VALUES (’$loginame’,’$loginpword’,’$usermail’,’$userlocation’)”;
@mysql_db_query($dbname,$sql,$id_link);
?>

Welcome <?php echo($loginame) ?>
<?php } else { // if the email already exist header("Location: pages.php?page=register&msg=umail"); } } else { // if the username already exist header("Location: pages.php?page=register&msg=uname"); } // ?>[/code]

When i go to the PhpMyAdmin, the data’s are recorded in the appropriate table.

Thanks again and in advance for any help.

Friendly, Lucifer Christ, php beginner


#4
The confirmation page doesn't loads the table it contains when the registration is done.

???


#5

Lig,

I was talking about the html table, almost at the end of the coding i’ve posted.

[code]?>

Welcome <?php echo($loginame) ?>
<?php [/code]

#6

Dude you need to make your code more readable… Anyway…
One question… why multiple queries to find out if there is a matching user?

I don’t understand your logic if this is a login page…

Psuedocode:

//  check if the username entered already exist in the table
if (you do not get an invalid return or the user name is in the db at least once)
{
    //  check if the email entered already exist in the table 
    if (you do not get an invalid return or the email address is in the db at least once)
    {
        //  record the data's in the table if everything is ok
        Insert the member into the db with their info
        show welcome member
    }
    else 
    {  
        //  if the email already exist 
        submit the new header
    }
}
else 
{  
    //  if the username already exist
    submit new header
} 

You allow there to be 1 user with that name already in the DB… same with the email address. but you never check to make sure the name and the email address go together. I don’t know if that was done intentionally of not.