Как исправить обратную связь через phpmailer?

Question

[WhiteRussian]

Необходимо получить 4 строки из формы. Приходит все, кроме телефона и типа заявки.

Comments

[Stalker_RED]

Телепаты в отпуске, так что код формы, обработчик и пример post запроса в студию.

[WhiteRussian]

Stalker_RED: С первым легко будет, а как получить post запрос я не в курсе ) А обработчик это файл в атрибуте action ?

[Stalker_RED]

WhiteRussian: да, файл в action. Хотя может оно и через js куда-то в другое место отправляется, но для начала так.
Запрос можно посмотреть нажав F12 -> сеть, включить запись и посмотреть что происходит в момент отправки. Вот к примеру этот мой комментарий:

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

[WhiteRussian]

Stalker_RED: Вот и сам обработчик собственной персоной

<?php

$recipients = '[e-mail]';

try {
    require './phpmailer/PHPMailerAutoload.php';

    preg_match_all("/([\w-]+(?:\.[\w-]+)*)@((?:[\w-]+\.)*\w[\w-]{0,66})\.([a-z]{2,6}(?:\.[a-z]{2})?)/", $recipients, $addresses, PREG_OFFSET_CAPTURE);

    if (!count($addresses[0])) {
        die('MF001');
    }

    if (preg_match('/^(127\.|192\.168\.)/', $_SERVER['REMOTE_ADDR'])) {
        die('MF002');
    }

    $template = file_get_contents('rd-mailform.tpl');

    if (isset($_POST['form-type'])) {
        switch ($_POST['form-type']){
            case 'contact':
                $subject = 'A message from your site visitor';
                break;
            case 'subscribe':
                $subject = 'Subscribe request';
                break;
            case 'order':
                $subject = 'Order request';
                break;
            default:
                $subject = 'A message from your site visitor';
                break;
        }
    }else{
        die('MF004');
    }

    if (isset($_POST['email'])) {
        $template = str_replace(
            array("<!-- #{FromState} -->", "<!-- #{FromEmail} -->"),
            array("Email:", $_POST['email']),
            $template);
    }else{
        die('MF003');
    }

    if (isset($_POST['message'])) {
        $template = str_replace(
            array("<!-- #{MessageState} -->", "<!-- #{MessageDescription} -->"),
            array("Message:", $_POST['message']),
            $template);
    }

    preg_match("/(<!-- #{BeginInfo} -->)(.|\n)+(<!-- #{EndInfo} -->)/", $template, $tmp, PREG_OFFSET_CAPTURE);
    foreach ($_POST as $key => $value) {
        if ($key != "email" && $key != "message" && $key != "form-type" && !empty($value)){
            $info = str_replace(
                array("<!-- #{BeginInfo} -->", "<!-- #{InfoState} -->", "<!-- #{InfoDescription} -->"),
                array("", ucfirst($key) . ':', $value),
                $tmp[0][0]);

            $template = str_replace("<!-- #{EndInfo} -->", $info, $template);
        }
    }

    $template = str_replace(
        array("<!-- #{Subject} -->", "<!-- #{SiteName} -->"),
        array($subject, $_SERVER['SERVER_NAME']),
        $template);

    $mail = new PHPMailer();
    $mail->From = $_POST['email'];

    # Attach file
    if (isset($_FILES['file']) &&
        $_FILES['file']['error'] == UPLOAD_ERR_OK) {
        $mail->AddAttachment($_FILES['file']['tmp_name'],
                             $_FILES['file']['name']);
    }

    if (isset($_POST['name'])){
        $mail->FromName = $_POST['name'];
    }else{
        $mail->FromName = "Site Visitor";
    }

    foreach ($addresses[0] as $key => $value) {
        $mail->addAddress($value[0]);
    }

    $mail->CharSet = 'utf-8';
    $mail->Subject = $subject;
    $mail->MsgHTML($template);
    $mail->send();

    die('MF000');
} catch (phpmailerException $e) {
    die('MF254');
} catch (Exception $e) {
    die('MF255');
}

[WhiteRussian]

Stalker_RED: Вот форма

[WhiteRussian]

Stalker_RED: Post запрос

[WhiteRussian]

Stalker_RED: Вот, он работает и схавал все что я ему дал на вход ) А в письмо приходит только email, имя. Мне помимо этого нужен телефон и тип заявки

Answer 1

[Stalker_RED]

В форме есть поля с именами "name", "email", "phone" и select без атрибута name.
В обработчике проверяются name, email, message и какой-то file.
Похоже, это какой-то левый обработчик, от другой формы вообще.

Нужно добавить select'у атрибут name (name="orderType", например) и добавить обработку этих полей.

Comments

[WhiteRussian]

Мне поменять имена нужно, с phone на message, с select на file я думаю

[Stalker_RED]

WhiteRussian: Вы вообще не понимаете что делаете, да?

[WhiteRussian]

Stalker_RED: Неа, сейчас наступлю на грабли, а потом научусь. Немного людей, которые сразу что-то понимают. :)

[Stalker_RED]

WhiteRussian: У файла совсем не такая обработка как у текстовых полей. попробуйте прочесть код, там же почти все слова понятно что значат.

[WhiteRussian]

Stalker_RED: С php увы не сталкивался в своей жизни много, но аналогия прослеживается. Значит мне придется править php код обработчика на свои имена из формы ?

[WhiteRussian]

Спасибо за помощь чувак! Люто плюсую :D

[WhiteRussian]

Stalker_RED: Все работает, кроме обработчика типа заявки

if (isset($_POST['orderType'])) {
        $_template= str_replace(
            array("<!-- #{OrderTypeState} -->", "<!-- #{OrderTypeDescription} -->"),
            array("OrderType:", $_POST['orderType']),
            $template);>

[Stalker_RED]

WhiteRussian: Какие-то ошибки выдает? Может в шаблоне что-то не так?

[WhiteRussian]

Нет не выдает

[WhiteRussian]

Stalker_RED: В форме поменял имя на orderType

[WhiteRussian]

И вот еще раз код всего переделанного обработчика

<?php

$recipients = '[e-mail]';

try {
    require './phpmailer/PHPMailerAutoload.php';

    preg_match_all("/([\w-]+(?:\.[\w-]+)*)@((?:[\w-]+\.)*\w[\w-]{0,66})\.([a-z]{2,6}(?:\.[a-z]{2})?)/", $recipients, $addresses, PREG_OFFSET_CAPTURE);

    if (!count($addresses[0])) {
        die('MF001');
    }

    if (preg_match('/^(127\.|192\.168\.)/', $_SERVER['REMOTE_ADDR'])) {
        die('MF002');
    }

    $template = file_get_contents('rd-mailform.tpl');

    if (isset($_POST['form-type'])) {
        switch ($_POST['form-type']){
            case 'contact':
                $subject = 'Сообщение от посетителя вашего сайта';
                break;
            case 'subscribe':
                $subject = 'Subscribe request';
                break;
            case 'order':
                $subject = 'Order request';
                break;
            default:
                $subject = 'Сообщение от посетителя вашего сайта';
                break;
        }
    }else{
        die('MF004');
    }

    if (isset($_POST['email'])) {
        $template = str_replace(
            array("<!-- #{FromState} -->", "<!-- #{FromEmail} -->"),
            array("Email:", $_POST['email']),
            $template);
    }else{
        die('MF003');
    }

    if (isset($_POST['message'])) {
        $template = str_replace(
            array("<!-- #{MessageState} -->", "<!-- #{MessageDescription} -->"),
            array("Message:", $_POST['message']),
            $template);
    }

    preg_match("/(<!-- #{BeginInfo} -->)(.|\n)+(<!-- #{EndInfo} -->)/", $template, $tmp, PREG_OFFSET_CAPTURE);
    foreach ($_POST as $key => $value) {
        if ($key != "email" && $key != "message" && $key != "form-type" && !empty($value)){
            $info = str_replace(
                array("<!-- #{BeginInfo} -->", "<!-- #{InfoState} -->", "<!-- #{InfoDescription} -->"),
                array("", ucfirst($key) . ':', $value),
                $tmp[0][0]);

            $template = str_replace("<!-- #{EndInfo} -->", $info, $template);
        }
    }

    $template = str_replace(
        array("<!-- #{Subject} -->", "<!-- #{SiteName} -->"),
        array($subject, $_SERVER['SERVER_NAME']),
        $template);

    $mail = new PHPMailer();
    $mail->From = $_POST['email'];

    
    if (isset($_POST['orderType'])) {
        $_template= str_replace(
            array("<!-- {FromState} -->", "<!-- #{FromOrderType} -->"),
            array("OrderType:", $_POST['orderType']),
            $template);
        
    }


    if (isset($_POST['name'])){
        $mail->FromName = $_POST['name'];
    }else{
        $mail->FromName = "Site Visitor";
    }

    foreach ($addresses[0] as $key => $value) {
        $mail->addAddress($value[0]);
    }

    $mail->CharSet = 'utf-8';
    $mail->Subject = $subject;
    $mail->MsgHTML($template);
    $mail->send();

    die('MF000');
} catch (phpmailerException $e) {
    die('MF254');
} catch (Exception $e) {
    die('MF255');
}

[Stalker_RED]

WhiteRussian: А в шаблоне rd-mailform.tpl?

[WhiteRussian]

Stalker_RED: Там тоже нужно поменять эти срочки с orderType? Хорошо, вернусь домой и сразу отвечу, что там да как )

[WhiteRussian]

Присылаю шаблон rd-mailform.tpl Кода очень много поэтому отправляю 4 картинками по порядку

[WhiteRussian]

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

[Stalker_RED]

WhiteRussian: Зачем-же картинками, если есть тег code.

[WhiteRussian]

Stalker_RED Toster Не разрешил отправить столько кода

[Stalker_RED]

WhiteRussian: Можно залить на pastebin, например. Или просто сами добавьте поле в шаблон по образцу какого-то работающего.

[WhiteRussian]

Stalker_RED: Вот, сделал все как надо https://pastebin.com/Gbai5kTW

[WhiteRussian]

Stalker_RED: Что мне сделать дальше, как его добавить туда, чтобы по человечески все заработало. )

[Stalker_RED]

WhiteRussian: Ну вот в php коде вы сделали

str_replace(
            array("<!-- {FromState} -->", "<!-- #{FromOrderType} -->"),
            array("OrderType:", $_POST['orderType']),
            $template);

Этот кусок кода пытается заменить в шаблоне строки <!-- {FromState} --> и <!-- #{FromOrderType} --> на OrderType: и то, что пришло из формы в поле $_POST['orderType']. Но в шаблоне нет таких строк. Значит их надо туда добавить.

[WhiteRussian]

Stalker_RED: Мне бы конкретный пример, как это написать. Чувствую сам не дойду до решения.
Ахаха в глазах двоится, подправил код емае )

[WhiteRussian]

if (isset($_POST['ordertype'])) {
        $_template= str_replace(
            array("<!-- {OrdertypeState} -->", "<!-- #{FromOrdertype} -->"),
            array("Ordertype:", $_POST['ordertype']),
            $template);

Stalker_RED: Вроде как должно работать. Сменил имя в атрибуте на ordertype, c orderType.
Вот мой код https://pastebin.com/mbmiapg2