Форма обратной связи - ответное сообщение

frendlix · October 25, 2013

День добрый!

Есть форма обратной связи, она спокойно присылает e-mail на нужный мне адрес, но во первых почему то не валидирует вообще, т.е. нет проверки на заполнение полей, енто раз.

Но главная проблема в том что мне нужно что бы при успешной отправке сообщения пользователю который отправил нам сообщение приходил ответ на его же почту, мол спасибо бла бла бла, сейчас у нас нет:


<?php
  $posttags = get_the_tags();
  if ($posttags) {
    foreach($posttags as $tag) {
	  echo $tag->name . ' ';
    }
  }
?>

, но когда будет мы вас оповестим.

Вот код формы:

php:


<?php
class Mail {    private $_plain = "text/plain";
    public function isTypeHtml($is_html = true) {
	    if ($is_html == true) {
		    $this->_plain = "text/html";
	    } else {
		    $this->_plain = "text/plain";
	    }
    }    public function send($to, $to_name, $from, $from_name, $subject, $message) {
	    $headers = "From: =?utf-8?B?" . base64_encode($from_name) . "?= <$from>\r\n";
	    $headers .= "To: =?utf-8?B?" . base64_encode($to_name) . "?= <$to>\r\n";
	    $headers .= "Subject: =?utf-8?B?" . base64_encode($subject) . "?=\r\n";
	    $headers .= "Content-type: $this->_plain; charset=\"utf-8\"\r\n";
	    $headers .= "X-Mailer: http://sait.ru \r\n";
	    if (mail($to, $subject, $message, $headers)) {
		    return true;
	    }
	    else return false;    }
}if (isset($_POST["send"])) {
    $mail = new Mail();
    $mail->isTypeHtml(false);
    $message .= "E-mail: " . $_POST["email"] . "\n\r";
    $message .= "Ля ля: " . $_POST["message"] . "\n\r";
    if ($mail->send("mail@gmail.com", "Ла ла", "support@gmail.com", "Ля", "Ля", $message)) {
	    echo "<script> alert('Сообшение успешно отправлено!')</script>";
    } else {
	    echo "<script> alert('Возникли ошибки при отправке сообщения!')</script>";
    }
}
?>

html:


<form method="post" action="">
текст <input class="classname" name="message" rows="" cols="0" value="<?php
  $posttags = get_the_tags();
  if ($posttags) {
    foreach($posttags as $tag) {
	  echo $tag->name . ' ';
    }
  }
?>" /> текст:
<input class="inputsubscribe subscribefont" type="text" name="email" size="25" maxlength="255" value="" placeholder="Введите ваш E-mail"/><br>
<button type="submit" class="subscribenews" name="send">Заказать</button>

Помогите ребят, спасибо!

Veseloff · October 25, 2013

Лень код разбирать, да и ни к чему. А почему сейчас не приходит? Функция mail на сервере вообще работает?

frendlix · October 27, 2013

Лень код разбирать, да и ни к чему. А почему сейчас не приходит? Функция mail на сервере вообще работает?

Что не приходит? Сейчас все приходит, не понятно как по результату вместо alert отправлять ответное письмо.

Николя223 · October 27, 2013

$to, $to_name, $from, $from_name, $subject, $message

Вот эти значения и валидируйте ) как хотите

а что вам мешает после

$mail->send("mail@gmail.com", "Ла ла", "support@gmail.com", "Ля", "Ля", $message)

вызвать ещё раз функцию send(), но уже с другими параметрами?

Edited October 27, 2013 by Николя223

frendlix · October 27, 2013

а что вам мешает после
$mail->send("mail@gmail.com", "Ла ла", "support@gmail.com", "Ля", "Ля", $message)
вызвать ещё раз функцию send(), но уже с другими параметрами?

Возможно что мне мешает непонимание того как это сделать (:

Пока что в голову пришло только вот так сделать:

if ($mail->send("mail@gmail.com", "Ла ла", "support@gmail.com", "Ля", "Ля", $message); send("mail@gmail.com", "Ла ла", "support@gmail.com", "Ля", "Ля", $message)) {echo...

Но сдается мне что я написал неправильный синтаксис, вот и вся проблема))

Чукча html занимается php на уровне доначальном))

Николя223 · October 27, 2013

if ($mail->send("mail@gmail.com", "Ла ла", "support@gmail.com", "Ля", "Ля", $message)){

$mail->send("newmail@gmail.com", "Ла ла", "support@gmail.com", "Ля", "Ля", $message);

echo...

}

так пробуй

Zverushka · October 27, 2013

а что вам мешает после
$mail->send("mail@gmail.com", "Ла ла", "support@gmail.com", "Ля", "Ля", $message)
вызвать ещё раз функцию send(), но уже с другими параметрами?
Возможно что мне мешает непонимание того как это сделать (:
Пока что в голову пришло только вот так сделать:
if ($mail->send("mail@gmail.com", "Ла ла", "support@gmail.com", "Ля", "Ля", $message); send("mail@gmail.com", "Ла ла", "support@gmail.com", "Ля", "Ля", $message)) {echo...
Но сдается мне что я написал неправильный синтаксис, вот и вся проблема))
Чукча html занимается php на уровне доначальном))

Конечно, чувак, неправильно. Какой send сработает просто так? Если бы он работал, то вместо конструкции $mail->send, так бы и было написано - send;

Просто скопируй функцию отправки еще раз и все.


if ($mail->send("mail@gmail.com", "Ла ла", "support@gmail.com", "Ля", "Ля", $message)) {
			 echo "<script> alert('Сообшение успешно отправлено!')</script>";
	 } else {
			 echo "<script> alert('Возникли ошибки при отправке сообщения!')</script>";
	 }

if ($mail->send("SECONDgmail.com", "Ла ла", "support@gmail.com", "Ля", "Ля", $message)) {
			 echo "<script> alert('Сообшение на другой ящик успешно отправлено!')</script>";
	 } else {
			 echo "<script> alert('Возникли ошибки при отправке сообщения!')</script>";
	 }

Edited October 27, 2013 by Zverushka

frendlix · October 28, 2013

if ($mail->send("mail@gmail.com", "Ла ла", "support@gmail.com", "Ля", "Ля", $message)){
$mail->send("newmail@gmail.com", "Ла ла", "support@gmail.com", "Ля", "Ля", $message);
echo...
}
так пробуй

Все гениальное просто, спасибо (:

Чуть еще допилил, стало вообще отлично!

frendlix · October 30, 2013

Хочется еще добавить кое какой вопрос (:

Форма при отправке данных выводит jquery fancybox скрипт, который собственно выводит всплывающее окно типа pop-up с сообщением о успешной отправке.

Вопрос в следующем при обновлении страницы форма еще раз отправляет данные.

Что пробовал - о чем думал:

1. Редирект на страницу с сообщением о результате действия (успешном или нет) - тут все ясно, кроме того, что тогда у меня пропадает красивое фенсибокс окошко.

2. Очистка данных из формы aka reset но он не помог

3. Почитал что можно использовать: unset($_POST); и что бы отрабатывался он сразу после отправки данных, но что то корректно всунуть его у меня не вышло.

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

Спасибо!

rus · October 30, 2013

Редирект на страницу с сообщением о результате действия (успешном или нет) - тут все ясно, кроме того, что тогда у меня пропадает красивое фенсибокс окошко.

не совсем правильная логика, редиректить нужно только после УСПЕШНЫХ действий, а если введенные данные не корректны - выдавать соотв. предупреждение.

ну и если заголовки как-то будут пересекаться во время редиректа, то применять буферизацию вывода:

ob_start();

ob_clean();

есть правда еще один способ, который я взял себе на заметку у нашего уважаемого гуру Int-а:


if (isset($_SESSION['flag']) && (time()-$_SESSION['flag'] < 15)) {
echo '<h1>Защита от повторного обновления! Нужно подождать 15 сек...</h1>';
} else {
// тут ваш код...
echo '<h1>Информация отправлена!</h1>';
$_SESSION['flag']=time();
}

frendlix · October 30, 2013

не совсем правильная логика, редиректить нужно только после УСПЕШНЫХ действий, а если введенные данные не корректны - выдавать соотв. предупреждение. ну и если заголовки как-то будут пересекаться во время редиректа, то применять буферизацию вывода: ob_start(); ob_clean();

Ну да, я просто не так написал)) В случае неправильных данных он alert выдает через jquery опять таки.

А если sucsess то всплывает поп ап.

Поэтому редирект меня не очень устраивает.

А тот вариант что вы предложили возьму на вооружение, может очень даже пригодится, если ничего более элегантного не найду.

Zverushka · October 30, 2013

Получается у тебя происходит якобы нажатие кнопки отправить (то есть кнопки submit) на обычной перезагрузке страницы?И страница не просит подтвердить отправку данных при обновлении? Конечно без кода страницы тут не посоветуешь ничего...

frendlix · October 30, 2013

Как бы да получается, страница по крайней мере в опере (новой) и хроме не просит.

Код страницы:


<?php
class Mail {
    private $_plain = "text/plain";
    public function isTypeHtml($is_html = true) {
	    if ($is_html == true) {
		    $this->_plain = "text/html";
	    } else {
		    $this->_plain = "text/plain";
	    }
    }
    public function send($to, $to_name, $from, $from_name, $subject, $message) {
	    $headers = "From: =?utf-8?B?" . base64_encode($from_name) . "?= <$from>\r\n";
	    $headers .= "To: =?utf-8?B?" . base64_encode($to_name) . "?= <$to>\r\n";
	    $headers .= "Subject: =?utf-8?B?" . base64_encode($subject) . "?=\r\n";
	    $headers .= "Content-type: $this->_plain; charset=\"utf-8\"\r\n";
	    $headers .= "X-Mailer: sitename \r\n";
	    if (mail($to, $subject, $message, $headers)) {
   return true;
	    }
	    else return false;
    }
}
if (isset($_POST["send"])) {
    $mail = new Mail();
    $mail->isTypeHtml(false);
$message2 =$_POST["fio"];
    $message .= $_POST["email"];
    $message1 .= $_POST["message1"];
    if ($mail->send("mail@gmail.com", "Бла 1", "mail@gmail.com", "Sitename", "Заказ на  $message1", "E-mail: $message \r\nИмя: $message2 \r\nНужен: $message1"))
{$mail->send("$message", "Бла 1", "mail@gmail.com", "Sitename", "Подтверждение Вашего запроса  на сайте Sitename!", "Здравствуйте $message2! \r\nВаше сообщение успешно отправлено! \r\nКак только $message1 появится на нашем сайте, мы сразу сообщим Вам об этом! \r\n--\r\n
Спасибо за Ваше обращение! \r\nСлужба поддержки сайта: Sitename  \r\nE-mail для общих вопросов: mail@gmail.com \r\nE-mail для вопросов по заказам или оплате: mail@gmail.com");
	    echo "
<a class='fancybox'  href='../sucsess.html'></a>
  <script>jQuery(document).ready(function($){
  $('.fancybox')
    .attr('rel', 'gallery')
    .fancybox({
	    'padding' : '0',
  'scrolling'   : 'no',
    });
$('.fancybox').eq(0).trigger('click');
}); </script>";
    } else {
	    echo "<script> alert('Возникли ошибки при отправке сообщения!')</script>";
    }
}
?>
<form method="post" action="">
<div id="marginyes">
<div id="backsubnews" style="height: 235px;">
<div id="backbuttonyes" style="height: 215px;">
<div style="border-right: 3px solid #AA1212;text-align:center;width:290px;text-transform: uppercase;font-size: 15px!important;text-shadow: 1px 0px 1px #3f3f3f;color:#ffffff;margin-top: 5px;
float: left;">
Сейчас у нас нет нужного товара<br/>
<input style="font-weight: bold;font-size: 18px;background: none;border: none;color: #fff;text-transform: uppercase;padding: 0px;margin: 0 15%;
text-align: center;" name="message1" rows="" cols="0" value="<?php
  $posttags = get_the_tags();
  if ($posttags) {
    foreach($posttags as $tag) {
	  echo $tag->name . ' ';
    }
  }
?>" />Но вы можете предварительно его заказать:
<div class="buyautograph">
<p class="marginpyes"><input style="margin-top: -13px;padding: 8px!important;" class="inputsubscribe subscribefont" type="text" name="email" id="email" size="25" maxlength="255" value="" placeholder="Введите ваш E-mail"/><br>
<input style="margin-top: -25px; margin-bottom: 0px;padding: 8px!important;" class="inputsubscribe subscribefont" type="text" value="" name="fio" id="name"size="25" maxlength="255" placeholder="Введите ваше Имя"/><br>
<button id="img_button" type="submit" class="subscribenews" name="send">Заказать</button></p>
</div>
</div>
<div class="godirectory" style="margin-top: 15px; width:285px;">
<a style="color:#ffffff;" href="../cat/">
<div class="subscribenews" style="width: 255px;margin: 140px 15px;font-size: 18px!important;line-height: 30px;text-shadow: 1px 1px 0px #333;">
Посмотреть весь<br/>каталог
</div>
</a>
</div>
</div>
</div>
</div>
</form>

Там еще проверка на заполнение полей есть но она jquery и к делу особого отношения не имеет думаю.

Zverushka · October 30, 2013

Может для начала отделить форму от содержимого? В любом случае это будет удобней. Вынеси отсылку сообщений в contanct.php и соотвественно action="contact.php"; А, хотя у тебя странице магазин имитирует.. Я просто делаю это на ajax чтобы не было перезагрузки и перехода на другую страницу.

Хотя я помню делала на новой странице) и тут клиент - а сделайте мне в том же окошке без перезагрузки - ну блин думаю, надо было сразу не халявить и ajax рисовать).

$('.fancybox').eq(0).trigger('click'); - что это интересно?

Edited October 30, 2013 by Zverushka

frendlix · October 30, 2013

Может для начала отделить форму от содержимого? В любом случае это будет удобней. Вынеси отсылку сообщений в contanct.php и соотвественно action="contact.php"; А, хотя у тебя странице магазин имитирует.. Я просто делаю это на ajax чтобы не было перезагрузки и перехода на другую страницу.
Хотя я помню делала на новой странице) и тут клиент - а сделайте мне в том же окошке без перезагрузки - ну блин думаю, надо было сразу не халявить и ajax рисовать).
$('.fancybox').eq(0).trigger('click'); - что это интересно?

Это собственно как раз подключение красивого fancybox окошечка))

Ajax я думал, но что то сразу затупил, а теперь уже хочется этот вариант добить если выйдет))

Zverushka · October 30, 2013

Ну по меньше мере стоит обрабатывать данные формы не на той же не странице, а на другой - раз все равно пользователя в итоге перенаправляет на другую страницу. Мне кажется в этом случае проблема отпадет сама собой. Просто через javascript проверить данные формы на корректность.

Николя223 · October 30, 2013

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

... Делай редирект(можно даже на ту же страницу), тогда проблема решиться. ридирект делается через header.

Могу ещё внимание обратить , что у форм есть важный атрибут - enctype

ещё и его стоило бы прописать

Edited October 30, 2013 by Николя223

Zverushka · October 30, 2013

А почту с заполненными полями второй раз отсылает? Или пустые?

frendlix · October 30, 2013

А почту с заполненными полями второй раз отсылает? Или пустые?

Пустые

аз все равно пользователя в итоге перенаправляет на другую страницу.

Не, он на той же остается просто alert типа выскакивает если все окей.

Делай редирект(можно даже на ту же страницу)

Ну да, пока что только такой вариант оптимальный, блин жаль что никак по другому...

А как бы тогда сделать так что бы редиректилось на ту же страницу, но при этом всплывал поп ап уже на обновленной странице.

Николя223 · October 30, 2013

к окончанию страницы добавляй: ?send=true

и в коде:


if(isset($_GET['send'])){
if($_GET['send']=="true"){
echo " ";
}
}

либо примерно то же самое только в сессиях, или кукки

Edited October 30, 2013 by Николя223

frendlix · October 30, 2013

к окончанию страницы добавляй: ?send=true
и в коде:
if(isset($_GET['send'])){
if($_GET['send']=="true"){
echo " ";
}
}
либо примерно то же самое только в сессиях, или кукки

Спасибо! Буду ваять (:

Zverushka · October 30, 2013

Ну да, пока что только такой вариант оптимальный, блин жаль что никак по другому...

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

И вот так тоже не пиши


$message .= $_POST["email"];
	 $message1 .= $_POST["message1"];

Код критически уязвим к взлому.

Пиши вот так

$message1 = addslashes(trim($_POST['message1 ']))

Одна удаляет пробелы (но это фигня), другая преобразует спец-символы в безопасные коды.

И если не уверен в безопасности своего кода, никогда не пиши вот так:


Как только $message1 появится

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

Николя223 · October 30, 2013

Ну да, пока что только такой вариант оптимальный, блин жаль что никак по другому...

а unset ($_POST["send"] ) b т.д. ты пробовал? в конце работы скрипта?

Sign In

Форма обратной связи - ответное сообщение

Question

Link to comment

Share on other sites

22 answers to this question

Recommended Posts

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Join the conversation

Similar Content

Обсуждения

Important Information