使用特殊字符时,POST联系人表单缺少数据

时间:2018-06-25 09:09:48

标签: php jquery character contact-form

我正在使用一个简单的联系表:http://luiszuno.com/previews/formy/ 问题是,每当在表格上键入字符“&”时,电子邮件中都不会收到“&”之后的其余消息。编辑:添加了代码,窗体正常工作,而不是缺少数据问题。是什么原因导致此问题?

    jQuery(document).ready(function($) {

    $("#formy").on( "submit", function( event ) {
        $(this).serialize();
    });
    // Hide messages 
    $("#formy-success").hide();
    $("#formy-error").hide();
    $("input,textarea").blur(function(){
            $(this).css("border-color","#596a87");
        });

    // on submit...
    $("#formy #submit").click(function() {

        $(this).serialize();

        // Required fields:

        //name
        var name = $("#name").val();
        if(name == "" || name == "Name *"){
            $("#name").focus();
            $("#formy-error").fadeIn().text("Name required");
            $("#name").css("border-color","#a22528");
            return false;   
        }
        else {$("#name").css("border-color","#596a87");}


        // email

        var email = $("#email").val();
        if(email == "" || email == "Email *"){
            $("#email").focus();
            $("#formy-error").fadeIn().text("Email required");
            $("#email").css("border-color","#a22528");
            return false;
        }
        else {$("#email").css("border-color","#596a87");}

        // email validation
        function validateEmail(email) {
            var filter = /^([a-zA-Z0-9_.+-])+\@(([a-zA-Z0-9-])+\.)+([a-zA-Z0-9]{2,4})+$/;
            return filter.test(email);
        }
        if (!validateEmail(email)) {
            $("#formy-error").fadeIn().text("Invalid email address");
            $("#email").css("border-color","#a22528");
            return false;
        }

        //budget
        var budget = $("#budget").val();
        if(budget == "" || budget == "Budget"){
            return false;
        }


        // comments
        var comments = $("#comments").val();
        if(comments == "" || comments == "Message *"){
            $("#comments").focus();
            $("#formy-error").fadeIn().text("Message required");
            $("#comments").css("border-color","#a22528");
            return false;
        }
        else {$("#comments").css("border-color","#596a87");}

        // send mail php
        var sendMailUrl = $("#sendMailUrl").val();

        // Retrieve values for to, from & subject at the form
        var to = $("#to").val();
        var from = $("#from").val();
        var subject = $("#subject").val();

        // Create the data string
        var dataString = 'name=' + name
                        + '&email=' + email        
                        + '&comments=' + comments
                        + '&to=' + to
                        + '&from=' + from
                        + '&budget=' + budget   
                        + '&subject=' + subject;                             
        // ajax 
        $.ajax({
            type:"POST",
            url: sendMailUrl,
            data: dataString,
            success: success()
        });
    });  


    // On success...
     function success(){
        $("#formy-success").fadeIn(250).text("Thanks, I will contact you soon!");
        $("#formy-error").hide();
        $("#formy fieldset").slideUp(250);
     }

    return false;
});

send-mail.php

    <?php header("Content-Type: text/html; charset=utf-8"); 


//vars
$subject = $_POST['subject'];
$to = explode(',', $_POST['to'] );

$from = $_POST['email'];

//data
$msg = "NAME: "  .$_POST['name']    ."<br>\n";
$msg .= "EMAIL: "  .$_POST['email']    ."<br>\n";
$msg .= "BUDGET: "  .$_POST['budget']    ."<br>\n";
$msg .= "COMMENTS: "  .$_POST['comments']    ."<br>\n";

//Headers
$headers  = "MIME-Version: 1.0\r\n";
$headers .= "Content-type: text/html; charset=UTF-8\r\n";
$headers .= "From: <".$from. ">" ;


//send for each mail
foreach($to as $mail){
   mail($mail, $subject, $msg, $headers);
}

?>

2 个答案:

答案 0 :(得分:1)

您的问题是您正在手动构建数据字符串,而不是对插入到字符串中的数据值进行URI编码。因此,将假定文本中的任何原始&字符都是URI的一部分,而不是数据的一部分,这意味着它将被解释为指示下一个参数的开始,而不仅仅是文本中的字符。对该数据进行URI编码会将其和其他特殊字符转换为它们的编码等效项,因此不会被误解。

现在,您当然可以通过用encodeURIComponent()包装每个字段变量来解决此问题,但这很冗长乏味:

var dataString = 'name=' + encodeURIComponent(name)
                        + '&email=' + encodeURIComponent(email)        
                        + '&comments=' + encodeURIComponent(comments)
                        + '&to=' + encodeURIComponent(to)
                        + '&from=' + encodeURIComponent(from)
                        + '&budget=' + encodeURIComponent(budget)   
                        + '&subject=' + encodeURIComponent(subject); 

有更好的方法:

当前,您的$(this).serialize();无法正常工作,因为this代表所单击的按钮,而不是表单。但是,如果将其指向表单,则可以使用它轻松地自动序列化表单中的字段,并且jQuery将代表您处理任何编码问题。这样可以节省手动编码和单独获取每个字段值的代码(尽管我注意到您当前需要使用它进行验证,尽管还有其他实现验证的方法可以消除这种需求,但这完全是另一个主题)。

$.ajax({
    type:"POST",
    url: sendMailUrl,
    data: $("#formy").serialize(),
    success: success //Unrelated: I also removed the brackets here, so it becomes a _reference_ to the "success" function - writing success() as you did means the function is immediately executed, and what gets passed to jQuery is the _result_ of the function, which isn't what you want in this case
});

如您所见,这省去了很多麻烦。而且,如果将来您在表单中添加更多字段,则完全不必更改此代码。


另一点-使用JavaScript验证表单输入会遇到很多麻烦。这是很好并且用户友好的,但是它没有提供任何安全性。在服务器端,您似乎很高兴将浏览器直接发送的任何值插入到您的电子邮件中。具有少量知识的任何用户都可以使用其开发人员工具来修改您的JavaScript,或者关闭JS,或者完全使用其他工具(例如PostMan或自定义应用程序)在您的服务器上触发HTTP请求,而无需接触您的表单。他们可能会发送有问题的值,这可能会破坏您的电子邮件-例如一个不同的“ from”值,或一些讨厌的HTML,或者如果您正在与应用程序中任何地方的数据库进行交互,请执行SQL Injection攻击以将其弄乱。在将其用于其他任何用途之前,您应该始终验证PHP代码中的所有传入数据是否存在安全问题,并确保它们符合您的业务规则。

答案 1 :(得分:0)

在使用联系表单时,建议将encodeURIComponent()用作@ADyson,以修复收到的电子邮件中丢失的数据/ URL。

var comments = $("#comments").val();
var e_comments = encodeURIComponent(comments);