Флажок проверки PHP - PullRequest
0 голосов
/ 15 мая 2018

Я искал весь интернет, пытаясь найти решение того, что мне здесь не хватает (или делаю не так).Моя форма не проверяется, даже если флажок установлен.Все остальное работает отлично.

Это флажок, который я не могу заставить работать правильно.

Я испробовал много разных идей, но он не подтвердится даже при проверенных «условиях» (таких как этот пример)ниже).

Вот мой HTML:

<div class="form-check">
      <input type="checkbox" class="form-check-input" name="terms" id="terms"  />
     <label class="form-check-label" for="terms"><p>I agree to terms of service.</p></label>
</div>

Вот вся моя проверка PHP (обновленная с комментариями / ответами ниже):

<?php

if(!$_POST) exit;

// Email address verification.
function isEmail($email) { // lots of email validation stuff in here // }

if (!defined("PHP_EOL")) define("PHP_EOL", "\r\n");

$name     = $_POST['name'];
$email    = $_POST['email'];
$website   = $_POST['website'];
$subject  = $_POST['subject'];
$comments = $_POST['comments'];
$terms = $_POST['terms'];


//set an error counter to trigger the `exit`
$error_counter = 0;
if(trim($name) == '') {
echo '<div class="error_message">Attention! You must enter your name</div>';
$error_counter++;
} 
if(trim($email) == '') {
echo '<div class="error_message">Attention! Please enter a valid email 
address.</div>';
$error_counter++;
}
if(!isEmail($email)) {
echo '<div class="error_message">Attention! You have entered an invalid e- 
mail address, try again.</div>';
$error_counter++;
} 
if(trim($subject) == '') {
echo '<div class="error_message">Attention! Please enter a subject.</div>';
$error_counter++;
}
if(trim($comments) == '') {
echo '<div class="error_message">Attention! Please enter your message. 
</div>';
$error_counter++;
}
if(empty($terms)) {
echo '<div class="error_message">Attention! Please agree to our terms of 
service.</div>';
$error_counter++;
}
//if `$error_counter > 0 > 0` it will trigger the `exit()` to stop the script and display the errors.
if($error_counter > 0){
exit();
}

if(get_magic_quotes_gpc()) {
$comments = stripslashes($comments);
}


// Configuration option.
// Enter the email address that you want to emails to be sent to.
// Example $address = "joe.doe@yourdomain.com";

$address = "cousinjack@mydomain.com";


// Configuration option.
// i.e. The standard subject will appear as, "You've been contacted by John Doe."

// Example, $e_subject = '$name . ' has contacted you via Your Website.';

$e_subject = 'You have been contacted by ' . $name . '.';


// Configuration option.
// You can change this if you feel that you need to.
// Developers, you may wish to add more fields to the form, in which case you must be sure to add them here.

$e_body = "You have been contacted by $name with regards to $subject, their additional message is as follows." . PHP_EOL . PHP_EOL;
$e_content = "\"$comments\"" . PHP_EOL . PHP_EOL;
$e_reply = "You can contact $name via email, $email ";

$msg = wordwrap( $e_body . $e_content . $e_reply, 70 );

$headers = "From: $email" . PHP_EOL;
$headers .= "Reply-To: $email" . PHP_EOL;
$headers .= "MIME-Version: 1.0" . PHP_EOL;
$headers .= "Content-type: text/plain; charset=utf-8" . PHP_EOL;
$headers .= "Content-Transfer-Encoding: quoted-printable" . PHP_EOL;

if(mail($address, $e_subject, $msg, $headers)) {

// Email has sent successfully, echo a success page.

echo "<fieldset>";
echo "<div id='success_page'>";
echo "<h1>Email Sent Successfully.</h1>";
echo "<p>Thank you <strong>$name</strong>, your message has been submitted to us.</p>";
echo "</div>";
echo "</fieldset>";

} else {

echo 'ERROR!';

}

?>

Ответы [ 3 ]

0 голосов
/ 15 мая 2018

Я вижу, что ваш флажок не имеет значения, поэтому часть PHP получает мусор / пустое значение.

Короче, просто добавьте атрибут required в вашу сторону HTML. Нет необходимости проверки на конце сервера

<div class="form-check">
      <input type="checkbox" class="form-check-input" name="terms" id="terms" required/>
     <label class="form-check-label" for="terms"><p>I agree to terms of service.</p></label>
</div>
0 голосов
/ 15 мая 2018

Проблема с вашей текущей проверкой заключается в том, что вы используете if-else неправильно и что вы используете exit для каждого оператора, который преждевременно завершит ваш скрипт.

Вы можете сделать это вместо:

For Terms:
<input type="checkbox" class="form-check-input" name="terms" id="terms" value="1"/>    

//set an error counter to trigger the `exit`
$error_counter = 0;
if(trim($name) == '') {
    echo '<div class="error_message">Attention! You must enter your name</div>';
    $error_counter++;
} 
if(trim($email) == '') {
    echo '<div class="error_message">Attention! Please enter a valid email 
address.</div>';
    $error_counter++;
}
if(!isEmail($email)) {
    echo '<div class="error_message">Attention! You have entered an invalid e- 
mail address, try again.</div>';
    $error_counter++;
} 
if(trim($subject) == '') {
    echo '<div class="error_message">Attention! Please enter a subject.</div>';
    $error_counter++;
}
if(trim($comments) == '') {
    echo '<div class="error_message">Attention! Please enter your message.</div>';
    $error_counter++;
}
if(empty($terms)) {
    echo '<div class="error_message">Attention! Please agree to our terms of service.</div>';
    $error_counter++;
}
//if `$error_counter > 0 > 0` it will trigger the `exit()` to stop the script and display the errors.
if($error_counter > 0){
    exit();
}

Для этого есть более красивый подход. Не стесняйтесь добавлять предложения для ОП.

0 голосов
/ 15 мая 2018

Вы не установили никакого атрибута значения для своего флажка:

<input type="checkbox" class="form-check-input" name="terms" id="terms" value="yes" />

Дополнительная информация: https://developer.mozilla.org/fr/docs/Web/HTML/Element/Input/checkbox

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...