theguy0000

For some reason, either the form input is not working, or it is not emailing correctly.

here is the html:


and register.php:

[php]<?php

/*************************

* *

* Created with RapidPHP *

* version 6.1.0.54 *

* *

* *

*************************/



unset($user, $pass, $pass_conf, $fname, $lname, $email, $address1, $address2);

$user = $_POST['user'];

$pass = $_POST['pass'];

$pass_conf = $_POST['pass_conf'];

$fname = $_POST['fname'];

$lname = $_POST['lname'];

$email = $_POST['email'];

$address1 = $_POST['address1'];

$address2 = $_POST['address2'];

$address = $_POST['address1'] . "\n" . $_POST['address2'];

$email_from = "noreply@xxxxxxxxxxxxxxxxx.com";

$email_to = "xxxxxxxxxx@xxxxx.com";

$email_msg = "noreply@xxxxxxxxxxxxxxxxx.com\n----------\nDo not respond to this email. Any response sent will not be recieved.\n\nYour message is as follows\n-------------------------\n\n";

$email_subj = "Undaground Hosting: user registration";



If ( $user = "" ||$pass = "" || $pass_conf = "" || $fname = "" || $lname = "" || $email = "" || $address1 = "" || $address2 = "" )

{

Die ("One or more fields were not filled out. Please click <script language=\"javascript\" type=\"text/javascript\">document.write('<a href=\"javascript:history.go(-1)\">here</a>');</script><noscript>your browser's back button</noscript> to fill out the empty fields.");

}



If ( $pass != $pass_conf )

{

Die ("Your password and your confirmation password did not match. Please click <script language=\"javascript\" type=\"text/javascript\">document.write('<a href=\"javascript:history.go(-1)\">here</a>');</script><noscript>your browser's back button</noscript> to correct these passwords, then try again.");

}



$email_msg .= "A new user account has been requested. The user sent the following information about the account:\n";

$email_msg .= "\nUsername:\n".$user."\n";

$email_msg .= "\nPassword:\n".$pass."\n";

$email_msg .= "\nFull Name:\n".$lname.", ".$fname."\n";

$email_msg .= "\nEmail address:\n".$email."\n";

$email_msg .= "\nPostal Address:\n".$address."\n";



mail ($email_to, $email_subj, $email_msg, "From: ".$email_from);



?>
[/php]

and here is the email I recieved:

why is it emailing me blank fields?

thanks!

DaWei

This is assigning "" to each variable. I b'leve you prolly meant to use "=="

__________________
Abstraction doesn't make it impossible to write bad code; it makes it possible to write superior code.
Contributor's Corner: Grumpy on C++ Exceptions DaWei on Pointers

theguy0000

oops that was a bit stupid...

sorry, my php has gotten a little rusty...

thanks man!

theguy0000

wait...now it's telling me that some fileds were not filled out, but I am positive that I filled out everything...

DaWei

Effective debugging requires information. If you don't have a debugger, sprinkle 'echo' or 'print_r' statements through the code to make sure variables are being stuffed with the values you expect. A print_r ($_POST) will tell you if the stuff is arriving. Echoing the values inside the string of "" tests (where the DIE is) will also tell you something. Sometimes you will need to view the source as well as the rendered page.

__________________
Abstraction doesn't make it impossible to write bad code; it makes it possible to write superior code.
Contributor's Corner: Grumpy on C++ Exceptions DaWei on Pointers