Contact Form Error in PHP

Question

Trying to get this contact form to work, not sure what's going on. Here's the HTML:

   <form id="contact-form" action="contact.php">
      <input type="text" class="form-control" placeholder="Your Name" name="name" />
      <input type="text" class="form-control" placeholder="Email" name="email" />
      <input type="text" class="form-control" placeholder="Phone Number" name="phone" />
      <input type="submit" class="btn btn-jackalope btn-centered">Request Quote</input>
    </form>

Here's the contact.php I'm using:

<?php
/*
* Contact Form Class
*/


header('Cache-Control: no-cache, must-revalidate');
header('Expires: Mon, 26 Jul 1997 05:00:00 GMT');
header('Content-type: application/json');

$admin_email = 'mcollinsblog@gmail.com'; // Your Email
//$message_min_length = 0; // Min Message Length


class Contact_Form{
function __construct($details, $email_admin, $message_min_length){

    $this->name = stripslashes($details['name']);
    $this->email = trim($details['email']);
    $this->phone = trim($details['phone']);
    $this->subject = 'Contact from Your Website'; // Subject 
    $this->message = stripslashes($details['message']);

    $this->email_admin = $email_admin;
    $this->message_min_length = $message_min_length;

    $this->response_status = 1;
    $this->response_html = '';
}


private function validateEmail(){
    $regex = '/^([\w-]+(?:\.[\w-]+)*)@((?:[\w-]+\.)*\w[\w-]{0,66})\.([a-z]{2,6}    (?:\.[a-z]{2})?)$/i';

    if($this->email == '') { 
        return false;
    } else {
        $string = preg_replace($regex, '', $this->email);
    }

    return empty($string) ? true : false;
}


private function validateFields(){
    // Check name
    if(!$this->name)
    {
        $this->response_html .= '<p>Please enter your name</p>';
        $this->response_status = 0;
    }

    // Check email
    if(!$this->email)
    {
        $this->response_html .= '<p>Please enter an e-mail address</p>';
        $this->response_status = 0;
    }

    // Check valid email
    if($this->email && !$this->validateEmail())
    {
        $this->response_html .= '<p>Please enter a valid e-mail address</p>';
        $this->response_status = 0;
    }

    // Check message length
    if(!$this->message || strlen($this->message) < $this->message_min_length)
    {
        $this->response_html .= '<p>Please enter your message. It should have at least '.$this->message_min_length.' characters</p>';
        $this->response_status = 0;
    }
}


private function sendEmail(){
    $mail = mail($this->email_admin, $this->subject, $this->message,
         "From: ".$this->name." <".$this->email.">\r\n"
        ."Reply-To: ".$this->email."\r\n"
    ."X-Mailer: PHP/" . phpversion());

    if($mail)
    {
        $this->response_status = 1;
        $this->response_html = '<p>Thank You!</p>';
    }
}


function sendRequest(){
    $this->validateFields();
    if($this->response_status)
    {
        $this->sendEmail();
    }

    $response = array();
    $response['status'] = $this->response_status;   
    $response['html'] = $this->response_html;

    echo json_encode($response);
}
}


$contact_form = new Contact_Form($_POST, $admin_email, $message_min_length);
$contact_form->sendRequest();

?>

Here's the URL: http://jackalopemedia.com/benefits.html

REALLY appreciate any help here! Thanks.

Answer 1

Without writing a method attribute in your <form> tag, by default it takes it as a GET request.

What you need to do is this:

<form id="contact-form" action="contact.php" method="post">
  <input type="text" class="form-control" placeholder="Your Name" name="name" />
  <input type="text" class="form-control" placeholder="Email" name="email" />
  <input type="text" class="form-control" placeholder="Phone Number" name="phone" />
  <input type="submit" class="btn btn-jackalope btn-centered">Request Quote</input>
</form>

Note how I included method="post" .

Check if that works.

UPDATE:

Instead of:

$response = array();
$response['status'] = $this->response_status;   
$response['html'] = $this->response_html;

echo json_encode($response);

try:

$response = array();
$response['status'] = $this->response_status;
$response['html'] = $this->response_html;

if ($response['status'] == '0') { // If error
  echo json_encode($response);
} else {
  echo $response['html'];
}

UPDATE: Change:

$this->name = stripslashes($details['name']);
$this->email = trim($details['email']);
$this->phone = trim($details['phone']);
$this->subject = 'Contact from Your Website'; // Subject 
$this->message = stripslashes($details['message']);

To this:

$this->name = stripslashes($details['name']);
$this->email = trim($details['email']);
$this->phone = trim($details['phone']);
$this->subject = 'Contact from Your Website'; // Subject 
$this->message = $this->name . " " . $this->email . " " . $this->phone;

Should do the trick.