Problems using return value from PHP script in jQuery.ajax success handler

Question

I'm getting a weird result from a jQuery ajax request sending form details to a PHP script. The same scripts are used elsewhere problem free. Basically the form is submitted using jQuery.ajax like this:

//if submit button is clicked
$('#form1').submit(function () {    

    //Get the data from all the fields
    var name = $('input[name=name]');
    var email = $('input[name=email]');
    var con_email = $('input[name=con_email]');
    var comments = $('textarea[name=comments]');

    //organize the data properly
    var data = 'name=' + name.val() + '&email=' + email.val() + '&con_email=' + con_email.val() + '&comments='  + encodeURIComponent(comments.val());


    //show the loading sign
    $('.loading').show();

    //start the ajax
    $.ajax({
        //this is the php file that processes the data and send mail
        url: "process-email/process.php",   

        //GET method is used
        type: "GET",

        //pass the data         
        data: data,     

        //Do not cache the page
        cache: false,

        //success
        success: function () {              
            //if process.php returned 1/true (send mail success)
            if (html==1) {                  
                //hide the form
                $('.form').fadeOut('slow');                 

                $('.done').delay(1000).fadeIn('slow');

            }               
        }       
    });



    //cancel the submit button default behaviours
    return false;
});

The PHP script works fine, the email is sent and 1 is returned (email sent) but the script stops at: if(html==1). I get this error

html is not defined

As said above exactly the same script works fine somewhere else, but here I get that error and the script is stopped. Can someone please help to understand where there might be the problem?

"organize the data properly" — you aren't, the data needs escaping. Just construct an object and use that instead of a string and jQuery will do it for you. I'd show you how … but "Get the data from all the fields" means you should just be using jQuery('form#myForm').serialize(). — Quentin
– Quentin, Commented Jan 3, 2012 at 13:16

user447356 · Accepted Answer · 2012-01-03 13:08:11Z

1

You have to add the parameter reference:

success: function (html) {
    //if process.php returned 1/true (send mail success)
    //.....
}

Then you can use this parameter, which will be the response from server.

answered Jan 3, 2012 at 13:08

user447356

Sign up to request clarification or add additional context in comments.

1 Comment

firefiter Over a year ago

sorry got you. I will try now

Sudhir Bastakoti · Accepted Answer · 2012-01-03 13:08:57Z

1

Check your success funcion, should be like:

success: function (response) {              
            //if process.php returned 1/true (send mail success)
            if (response == "1") {                  
                //hide the form
                $('.form').fadeOut('slow');                 

                $('.done').delay(1000).fadeIn('slow');

            }               
        }

answered Jan 3, 2012 at 13:08

Sudhir Bastakoti

100k15 gold badges163 silver badges169 bronze badges

Comments

White Elephant · Accepted Answer · 2012-01-03 13:09:17Z

1

It looks like you are not returning the response from the PHP script to the JavaScript function. If you do something like the following for your success function it should get you on the right track:

success: function( html )
{
    if(html=='1')
    {
        [...]
    }
}

answered Jan 3, 2012 at 13:09

White Elephant

1,3813 gold badges13 silver badges28 bronze badges

Collectives™ on Stack Overflow

Problems using return value from PHP script in jQuery.ajax success handler

3 Answers 3

1 Comment

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related