Обработчик формы не корректно передает данные из формы. В чем ошибка?

Question

[sergey]

Есть форма с чекбоксами (пример одного):

<input type="checkbox" id="select1" name="inspection" value="ok">
                                    <label for="select1">осмотр коренных вкладышей</label>

есть ajax php обработчик со следующим кодом:

$recepient = "[e-mail]";
$sitename = "........................t";

$name = trim($_POST["name"]);
$phone = trim($_POST["phone"]);
$email = trim($_POST["email"]);


if ( isset($_POST["inspection"]) ) {
    $action = 'Проводить осмотр';
} else {
    $action = 'Не проводить осмотр';
}


$pagetitle = "Новая заявка с сайта \"$sitename\"";
$message = "Имя: $name \nТелефон: $phone \nЕ-майл: $email \nПотребность в ремонте: $action";
mail($recepient, $pagetitle, $message, "Content-type: text/plain; charset=\"utf-8\"\n From: $recepient");

проблема заключается в следующем: на почту приходит сообщение с одним и тем же параметром (отмечен чекбокс или не отмечен). Нужно соответственно при отмеченом чекбоксе - сообщение: "Проводить осмотр;" А при не отмеченном чекбоксе сообщение : "Не проводить осмотр" чтобы приходили на почту.
В pHp не силен, кто может помочь решить проблему - буду благодарен.

Answer 1

[Иван Антонов]

В инпуте не хватает атрибутов name и value
измените
<input type="checkbox" id="select1">
на

<input type="checkbox" id="select1" name="inspection" value="ok">

а потом в php коде так:

if ( isset($_POST["inspection"]) ) {
    echo 'Проводить осмотр';
} else {
    echo 'Не проводить осмотр';
}

Comments

[sergey]

а как быть с этой строкой?
$pagetitle = "Новая заявка с сайта \"$sitename\"";
$message = "Имя: $name \nТелефон: $phone \nЕ-майл: $email \nПотребность в ремонте: $";
mail($recepient, $pagetitle, $message, "Content-type: text/plain; charset=\"utf-8\"\n From: $recepient");

[sergey]

какое значение задать $...

[sergey]

Потребность в ремонте: $ значение которое приходит в письме

[Иван Антонов]

просто вместо "echo" напиши переменную, например так как написал выше Black Jack

[Иван Антонов]

Вот так:

if ( isset($_POST["inspection"]) ) {
    $action = 'Проводить осмотр';
} else {
    $action = 'Не проводить осмотр';
}

$pagetitle = "Новая заявка с сайта \"$sitename\"";
$message = "Имя: $name \nТелефон: $phone \nЕ-майл: $email \nПотребность в ремонте: $action";
mail($recepient, $pagetitle, $message, "Content-type: text/plain; charset=\"utf-8\"\n From: $recepient");

[sergey]

ок, но пока результата нет еще:)

[sergey]

Потребность в ремонте: Не проводить осмотр - вот это сообщение приходит в письме, отмечен чекбокс или же не отмечен

[Иван Антонов]

отмечен чекбок: Потребность в ремонте: Проводить осмотр
не отмечен чекбок: Потребность в ремонте: Не проводить осмотр

[sergey]

Все ок разобрался, первый $action - не проводить осмотр, второй - провести осмотр. так работает:)

[sergey]

Поспешил, выбирает почему-то все время значение второй $action и передает его в письме...

[Иван Антонов]

sergey martynuk: Вы инпут изменили?

[sergey]

да

[Иван Антонов]

sergey martynuk: на

<input type="checkbox" id="select1" name="inspection" value="ok">

[Иван Антонов]

Потому что Вы передаете параметры методом GET. поэтому измените
это
if ( isset($_POST["inspection"]) )
на
if ( isset($_GET["inspection"]) )

[sergey]

я все на POST поменял..., сейчас попробую еще это

[Иван Антонов]

sergey martynuk: у формы не забудьте
<form method="post">

[sergey]

Весь метод GET стоит, при отмеченом чекбоксе - осмотр не проводить, при не отмеченом - осмотр не проводить...

[Иван Антонов]

sergey martynuk: если и в пхп и на форме стоит get. Тогда возможно у вас в форме несколько инпутов с именем inspection

[sergey]

сама форма у меня с методом POST, alax обработчик GET, mail.php также GET, я менял их местами, делал все POST или же GET, но результат один не учитывается значение чекбокса отмеченного, при этом все данные передаются корректно. Имя, телефон, мейл... Инпутов в форме вообще нет, кроме этого...

[Иван Антонов]

sergey martynuk: Какая-то ерунда. Покажите код все вместе.

[sergey]

ок, сейчас выложу на jsfiddle.net

[sergey]

здесь кидаю

[sergey]

<form class="form" method="get"><input type="hidden" name="type" value="catalog">
                                <div class="form-title">Записаться на ремонт</div>
                                <div>
                                    <input class="<?php if ($ini['name_required']) print 'required'; ?> name" placeholder="Ваше имя" type="text" name="name"></div>
                                <div>
                                    <input class="<?php if ($ini['phone_required']) print 'required'; ?> phone" placeholder="Ваш телефон" type="text" name="phone"></div>
                                <div>
                                    <input data-valid="email" class="<?php if ($ini['email_required']) print 'required'; ?> email" placeholder="Ваш e-mail" type="text" name="email"></div>

                                <div>
                                    <input class="red-btn" type="submit" name="sub" value="ОТПРАВИТЬ"></div>
                                    <?php if ($ini['catalog_file']) { ?><p class="bot" contenteditable="true" id="id-block8">Мы Вам перезвоним в течении 15 минут</p><?php } ?>
                            </form>

[sergey]

<input type="checkbox" id="select1" name="inspection" value="ok">
                                    <label for="select1">осмотр коренных вкладышей</label>

[sergey]

ajax обработчик mail.php

$recepient = "....";
$sitename = "....";

$name = trim($_GET["name"]);
$phone = trim($_GET["phone"]);
$email = trim($_GET["email"]);


if ( isset($_GET["inspection"]) ) {
    $action = 'Проводить осмотр корреных вкладышей';
} else {
    $action = 'Не проводить осмотр';
}


$pagetitle = "Новая заявка с сайта \"$sitename\"";
$message = "Имя: $name \nТелефон: $phone \nЕ-майл: $email \nПотребность в ремонте: $action";
mail($recepient, $pagetitle, $message, "Content-type: text/plain; charset=\"utf-8\"\n From: $recepient");

[sergey]

и сообственно код обработчика

$("form").submit(function() {
		$.ajax({
			type: "GET",
			url: "mail.php",
			data: $("form").serialize()
		}).done(function() {
			alert("Спасибо за заявку!");
			setTimeout(function() {
				$.fancybox.close();
			}, 1000);
		});
		return false;
	});

[sergey]

Ну и собственно все, больше ничего примечательного

[Иван Антонов]

sergey martynuk: так чекбокс должен быть в форме, либо он игнориться будет

[sergey]

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

[sergey]

Обернул его в форму и о чудо все ок. Огромное спасибо!!!

Answer 2

[IceJOKER]

if(isset($_POST['some_field']))
  echo 'Do something =)';
else 
  echo "Don't do it!";

Comments

[sergey]

стесняюсь спросить? some_field это параметр name?

[Евгений Одинец]

sergey martynuk: да

Answer 3

[Евгений Одинец]

Как вы отправляете форму?

$_POST["select1"] - приходит от елемента с name = select1

Comments

[sergey]

в смысле как отправляю? есть форма в ней поля, чекбоксы и отправить:)))

[sergey]

а как должно приходить?

[Евгений Одинец]

sergey martynuk: у вас файл називается ajax.php, значит логичто что вы отправляете форму ajax запросом

[sergey]

да ajax запросом

[Евгений Одинец]

sergey martynuk: var data = $("#myform").serialize(); ? Если да то нужно указивайть name для полей, а может вы отправляете например так: var input $('#input').val(); data: "input="+input тогда name не нужен. Вы не описали полностью метод отправки, не предоставили код и хотите что-бы вам помогли. Как минимум нужно было показать ваш ajax запрос, может вообще вы его через get отправляете а в файле пытаетесь получить post

[sergey]

выложил php файл

Answer 4

[Виталий Хоменко]

Как правило для удобства работы используется такая связка для чекбокса:

<input type="hidden" name="checkbox1" value="0">
<input type="checkbox" name="checkbox1" value="1">

- Если чекбокс выбран получаете значение 1
- Если чекбокс не выбран получаете значение 0

Выбранный чекбокс затрет значение скрытого поля при сабмите формы.

Comments

[WarGot]

Странный какой-то способ. Это чтобы не на isset проверять, а снимать значение в любом случае ?

[Виталий Хоменко]

WarGot: Верно. Надежный способ