Html & PHP contact us form validation help

Jay1999

Hi all, This is my very first post here 🙂

I am kinda newbie in php.

I have got a html contact us form designed in html and a contactinfo.php page that checks for ur input from the form and checks for any errors. It is working fine but i have got some validation issue.

here is the html form:

<form action="contactinfo.php" method="post" enctype=" ">
<p><b>Please complete the contact us form below:</b></p>
<fieldset class="contact">

<b><legend>Contact Details: </legend><br>

<div> <label for="email">E-Mail Address: </label> <input type='text' name='email' value=""></div>

<div> <label for="subject">Subject: </label> <input type='text' name='subject' value=""></div>

<div> <label for="body">Message: </label> <textarea name="body" rows="5" cols="30"> </textarea></div></fieldset><br>

contactinfo.php:

<html>
<head>
<title>Thanks For Contacting Us</title>
</head>
<body>

<?php
$recipient = ' ';
$email = $POST['email'];
$realName = $POST['realName'];
$subject = $POST['subject'];
$body = $POST['body'];

$messages = array();

if (!preg_match("/^{[\w+-.~]+@[-\w.!]+$/",} $email)) {
$messages[] = "That is not a valid email address.";
}

if (!preg_match("/^[\w\ +-\'\"]+$/", $realName)) {
$messages[] = "The real name field must contain only " .
"alphabetical characters, numbers, spaces, and " .
"reasonable punctuation. We apologize for any inconvenience.";
}

$subject = preg_replace('/\s+/', ' ', $subject);

if (preg_match('/^\s*$/', $subject)) {
$messages[] = "Please specify a subject for your message.";
}

$body = $_POST['body'];

if (preg_match('/^\s*$/', $body)) {
$messages[] = "Your message was blank. Did you mean to say " .
"something?";
}
if (count($messages)) {
foreach ($messages as $message) {
echo("<p>$message</p>\n");
}
echo("<p>Click the back button and correct the problems. " .
"Then click Send Your Message again.</p>");
} else {

mail($recipient,
$subject,
$body,
"From: $realName <$email>\r\n" .
"Reply-To: $realName <$email>\r\n");
echo("<p>Your message has been sent. Thank you!</p>\n");
}
?>
</body>
</html>

isit possible to display the validations next to the forms representative fields instead of them showing on new page?.

Also i am running this on wamp locally, will it send email to my hotmail account or do i need a server for it?

Thanks guys for your help, really appriciate it 🙂

Jay

Jay1999

Sorry guys, here are the color coded version of the displayed php and html codes above. Just checked one of other posts and found out that we have to post using html and php tags.

<form action="contactinfo.php" method="post" enctype=" ">
<p><b>Please complete the contact us form below:</b></p>
<fieldset class="contact">

<b><legend>Contact Details: </legend><br>

<div> <label for="realName">Name: </label> <input type='text' name='realName' value=""></div>

<div> <label for="email">E-Mail Address: </label> <input type='text' name='email' value=""></div>

<div> <label for="subject">Subject: </label> <input type='text' name='subject' value=""></div>

<div> <label for="body">Message: </label> <textarea name="body" rows="5" cols="30"> </textarea></div></fieldset><br>

<input type="submit" name="send" value="Send Message"/>
<input class='button' align='right' type='reset' value='Reset'/>
<input class='button' align='right' type='button' value='Cancel' onclick='window.location.href="index.php"'/></form>

<html>
<head>
<title>Thanks For Contacting Us</title>
</head>
<body>

<?php
$recipient = ' ';
$email = $_POST['email'];
$realName = $_POST['realName'];
$subject = $_POST['subject'];
$body = $_POST['body'];

$messages = array();

if (!preg_match("/^[\w\+\-.~]+\@[\-\w\.\!]+$/", $email)) {
$messages[] = "That is not a valid email address.";
}

if (!preg_match("/^[\w\ \+\-\'\"]+$/", $realName)) {
$messages[] = "The real name field must contain only " .
"alphabetical characters, numbers, spaces, and " .
"reasonable punctuation. We apologize for any inconvenience.";
}

$subject = preg_replace('/\s+/', ' ', $subject);

if (preg_match('/^\s*$/', $subject)) {
$messages[] = "Please specify a subject for your message.";
}

$body = $_POST['body'];

if (preg_match('/^\s*$/', $body)) {
$messages[] = "Your message was blank. Did you mean to say " .
"something?";
}
if (count($messages)) {
foreach ($messages as $message) {
echo("<p>$message</p>\n");
}
echo("<p>Click the back button and correct the problems. " .
"Then click Send Your Message again.</p>");
} else {

mail($recipient,
$subject,
$body,
"From: $realName <$email>\r\n" .
"Reply-To: $realName <$email>\r\n");
echo("<p>Your message has been sent. Thank you!</p>\n");
}
?>
</body>
</html>

Thanks, Jay 🙂

dagon

the basic process is this - instead of using 2 pages you use 1, the submitted form data is processed before the form, so if there are errors you can then spit out the form with the errors marked - and repopulate the form.

Jay1999

thanks for ur reply

how do i do that and what do i need to put here than?

if using only one page?

dagon

action can be either the current page name, or even bank

Jay1999

thats great thanks mate for the help, really appriciate it 🙂

Jay

Jay1999

guys i have one last question about sql query not functioning properly. I think the syntax I am inputing seems to be wrong.

here's the workin bit in on the php page..

$result = mysql_query("SELECT * FROM ticket where End_Date<'".date("Y-m-d")."'");

but i need it to display overdue tickets where state of if is open.

here's the code i am tryin to put:

$result = mysql_query("SELECT * FROM ticket where End_Date<'".date("Y-m-d"). AND state='open'"'");

here's the error message:

Parse error: parse error in C:\wamp\www\system2\adminhome.php on line 308

Thanks guys, appriciate all the help 🙂

Jay

Derokorian


$result = mysql_query("SELECT * FROM ticket where End_Date<'".date("Y-m-d"). AND state='open'"'");

// change to


$result = mysql_query("SELECT * FROM ticket where End_Date<'".date("Y-m-d")."' AND state='open'");

Also note how easy it is to spot the difference with syntax highlighting. If you are not using an editor that does this I suggest you start =D

bradgrafelman

Also note that there's no reason to use a PHP function to insert today's date into the SQL query string at all - just use MySQL's CURDATE() function instead.

Jay1999

thanks 'Derokorian' that worked like a charm and I will make sure from next time that i use the wrapping tags 🙂 sorry about that. 🙂

bradgrafelman: wot would be the syntax to use MySQL's CURDATE() instead? should it be like this?

$result = mysql_query("SELECT * FROM ticket where End_Date<'".CURDATE()."' AND state='open'");

would that be appropriate?

bradgrafelman

Jay1999 wrote:
should it be like this?

No; in your code above, you're attempting to call the PHP function CURDATE(), which doesn't exist. CURDATE() is a MySQL function - it should just be inside of the query string just like any other text you send to MySQL.

Derokorian

IIRC you also would use it without the single quotes IE: End_Date<CURDATE() AND

Jay1999

thanks guys it worked 🙂

i have one last question:
how can i display a subtotal of numeric data display on the rows.

here's a wrkin sql code that displays number of tickets within thier current state.. but i want it to also display a subtotal of the tickets.

here's the code:


 <div id="headerInfo-1">
     <?php
     if(isset($_GET['token']))
     {
$to=($_GET['dateto']);
    $from=($_GET['datefrom']);
     ?>
     <b><h2>Report: <span style='color:#0b77b7'><u><?php echo $from ; ?></u></span></b> - <b><span style='color:#0b77b7'><u><?php echo $to ;?></u></span></b></h2><h5><a href="reportprint.php?datefrom=<?php echo $from ; ?>&dateto=<?php echo $to ; ?>" target="_blank">Print Report</a></h5>

<table width="100%" cellpadding="5" style="text-align:justify;border-collapse:collapse;padding:10px;">
<tr style="border-bottom:1px solid #000000;">
<th>Dates</th>
<th>Ticket Status</th>
<th>Total Tickets</th>

</tr>

<?php
 $con = mysql_connect("localhost","root","root") or die ("Couldn't connect");
mysql_select_db("system", $con);

$result = $result = mysql_query("SELECT Create_Date AS 'CreateDate', state AS 'State', count( State ) AS 'Number'
FROM ticket
WHERE Create_Date > '".$from."'
AND Create_Date < '".$to."'
GROUP BY Create_Date, State");

while($row = mysql_fetch_array($result))
  {      

    ?>

<tr style="background:#EEEEE8;">
<td><?php echo $dmydate = date('d-m-Y',strtotime($row['CreateDate'])); ?></td>
<td><?php echo $row['State']; ?></td>
<td><?php echo $row['Number']; ?></td>
</tr>
    <?php
  }
  ?>
</table>

<table width="100%" cellpadding="5" style="text-align:justify;border-collapse:collapse;padding:10px;">
<tr style="border-bottom:1px solid #000000;">
<th></th>
<th></th>
<th>Subtotal: </th></tr>

</table></div>

plz check the attached file i just don't get subtotal.

thanks guys

jay 🙂

Jay1999

anyone? 😕

Derokorian

<?php
// ... More code above here, but I didn't copy paste it all :P
$subtotal = 0;
    while($row = mysql_fetch_array($result)) 
  {       

      $subtotal = $subtotal + $row['Number'];
    ?> 

<tr style="background:#EEEEE8;"> 
<td><?php echo $dmydate = date('d-m-Y',strtotime($row['CreateDate'])); ?></td> 
<td><?php echo $row['State']; ?></td> 
<td><?php echo $row['Number']; ?></td> 
</tr> 
    <?php 
  } 
  ?> 
</table> 

<table width="100%" cellpadding="5" style="text-align:justify;border-collapse:collapse;padding:10px;"> 
<tr style="border-bottom:1px solid #000000;"> 
<th></th> 
<th></th> 
<th>Subtotal: <?php echo $subtotal; ?> </th></tr>

See if that works =D

Jay1999

Derokorian;10979152 wrote:

<?php
// ... More code above here, but I didn't copy paste it all :P
$subtotal = 0;
    while($row = mysql_fetch_array($result)) 
  {       

      $subtotal = $subtotal + $row['Number'];
    ?> 

<tr style="background:#EEEEE8;"> 
<td><?php echo $dmydate = date('d-m-Y',strtotime($row['CreateDate'])); ?></td> 
<td><?php echo $row['State']; ?></td> 
<td><?php echo $row['Number']; ?></td> 
</tr> 
    <?php 
  } 
  ?> 
</table> 

<table width="100%" cellpadding="5" style="text-align:justify;border-collapse:collapse;padding:10px;"> 
<tr style="border-bottom:1px solid #000000;"> 
<th></th> 
<th></th> 
<th>Subtotal: <?php echo $subtotal; ?> </th></tr>

See if that works =D

bro thanks for the help but unfortunetly it did't work 😕

    <?php
     $con = mysql_connect("localhost","root","root") or die ("Couldn't connect");
    mysql_select_db("system", $con);

$result = $result = mysql_query("SELECT Create_Date AS 'CreateDate', state AS 'State', count( State ) AS 'Number'
FROM ticket
WHERE Create_Date > '".$from."'
AND Create_Date < '".$to."'
GROUP BY Create_Date, State");
$subtotal = 0;
    while($row = mysql_fetch_array($result))
  {       

      $subtotal = $subtotal + $row['Number'];
    ?>

<th></th>
<th></th>
<th>Subtotal: <?php echo $subtotal; ?> </th></tr>  <?php
  }
  ?>

This is wot it displayed:

Subtotal: 1
Subtotal: 3
Subtotal: 5
Subtotal: 7
Subtotal: 8
Subtotal: 9

thanks bro really appriciate it 🙂

Jay

bradgrafelman

That's because you're echo'ing the subtotal inside of the loop as it is being updated. That is not what Derokorian was suggesting you do.

Besides, if you just want the subtotal without using the individual rows, then why loop through them all in the first place? Just use MySQL's SUM() function and you're done.

Jay1999

arrrrrggghhhhhhhh may bad i ran that script two times, u were right 'bradgrafelman' and thanks Derokorian 🙂 awesome guys 🙂

Jay1999

hey guys i am in trouble again.. not creating a new thread for this as i gues it relates to the title:

my login is workin but i can't get the validations to work (empty) well, isset funtion is working when invalid details are entered but and lets user/admin to login but no form feilds validations.. anythin i'm doin wrong here?.. if possible if u could correct the code 😕 i tryin over google but maybe i didn't understood it.

<?php

session_start();
$username = $_POST['username'];
  if(empty($username))
  {  echo("Please Input UserName"); }
$password = $_POST['password'];
  if(empty($password))
  { echo("Please Input Password"); }

$con = mysql_connect("localhost","root","alpine") or die ("Couldn't connect");
if($type=="admin")
{
mysql_select_db("database_acc", $con);
$result = mysql_query("SELECT * FROM admin");
while($row = mysql_fetch_array($result))
  {  if($row['user_id']==$username && $row['password']==$password)  {
        $_SESSION['user']=$username;
        $_SESSION['type']="admin";
        $acc=$acc+1;
     header( 'Location: admin.php');
    }
  }
}
else
{
    mysql_select_db("database_acc", $con);
    $result = mysql_query("SELECT * FROM members");
    while($row = mysql_fetch_array($result))
  {
    if($row['email']==$username && $row['password']==$password)
    {
        $_SESSION['user']=$username;
        $_SESSION['type']="normal";
        $acc=$acc+1;
     header( 'Location: members.php');
    }
  }
}
if($flag==0)
{   header( 'Location: index.php?invalidlogindetails=1');
    $_SESSION['details']="1"; } 
?>

here's the form workin code with isset function next to field 🙂

<?php
session_start();
if(!isset($_SESSION['user']) ) {   

?>
<div id="Login">
<form action='login.php'  method='POST'> 
<div> <label for="username"><b>E-Mail Address:</b></label> <input type='text' name='username'/><br><br></div>
<div> <label for="password"><b>Password:</b> </label>      <input type='password' name='password'/><br><br></div>
<span style="color:red">
<?php } else echo " ";
     if(isset($_GET['details'])) 
     { echo "<b>Incorrect Login! </b>   "; } ?>
<input type='submit' value='Login'><br></span></div></span>

if possible i gues with isset function would be better bcoz i want the message to display on the same page and next to field where with empty i don't think thats possilbe as the one i called for 'details' is workin like a charm, but i could be wrong.

thanks guys any help appriciated 🙂

Jay 🙂

Jay1999

anyone for help ? 🙁