JQuery to PHP with JSON

Question

The JQuery gets the data and send it to the PHP like this:

$(document).ready(function(){

        var email = encodeURIComponent($('#email').val());   
        var act = encodeURIComponent($('#act').val());   
        $('#loadingB').fadeIn(); 
        $.ajax({ 
            type: 'POST', url: 'activate.php',  dataType: "json",  data: { email: email, act: act,  }, 
            success: function(result) {                 
                if (!result.success) { timeout = setTimeout(function(){ $('#loadingB').fadeOut(); }, 1500); $('#fail').fadeIn(); }  
                else { timeout = setTimeout(function(){ $('#loadingB').fadeOut(); }, 1500); $('#success').fadeIn(); } 
            }  
        }); 
        return false; 
    });

The PHP does this:

$email = htmlspecialchars(trim(urldecode($_POST['email']))); 
$act = htmlspecialchars(trim(urldecode($_POST['act'])));
$first = mysql_query("UPDATE members SET active = '1' WHERE active_code = '$act' AND email = '$email' ");
$second = mysql_query("UPDATE member_search SET active = '1' WHERE email = '$email' ");

if(is_bool($first) == true && is_bool($second) == true)
{
$response = array(success => true); 
echo json_encode($response); 
}
else
{
$response = array(success => false); 
echo json_encode($response); 
}

the "loadingB" div fades in but never fades out to return successful or failure. I believe this is a PHP error. I don't think I'm correctly obtaining if the mysql-query returned true or false.

I know the proper data is being collect by the JQuery because I even echoed it just to make sure, the PHP is just not doing anything with it.

Did you try console.log(result.success).... what do you see...? — Sudhir Bastakoti
– Sudhir Bastakoti, Commented Feb 17, 2012 at 4:25
What's the purpose of the timeout. Why not just $('#loadingB').fadeOut(); ? ... which is common to both conditionals, by the way, and can be taken out as a common factor. — Jordan Arsenault
– Jordan Arsenault, Commented Feb 17, 2012 at 4:34
Also, AJAX requests can be analysed in the console if there are PHP errors. Check the Network tab, find the activate.php Path, and click it. You'll be brought to a page as if the request was a basic GET with all PHP errors shown. — Jordan Arsenault
– Jordan Arsenault, Commented Feb 17, 2012 at 4:37
You've also got an extra comma in your ajax config in the data part. — John V.
– John V., Commented Feb 17, 2012 at 4:37
Ah, well make sure to read my comment about is_bool to make sure it will actually check if the queries were successful! — John V.
– John V., Commented Feb 17, 2012 at 4:41

Jaspreet Chahal · Accepted Answer · 2012-02-17 04:38:29Z

2

one thing you can try is to change this

 $response = array(success => true);

to

$response = array("success" => true);

I would also use this

$first = mysql_query("UPDATE members SET active = '1' WHERE active_code = '$act' AND email = '$email' ");
$first_affected_rows = mysql_affected_rows();
$second = mysql_query("UPDATE member_search SET active = '1' WHERE email = '$email' ");
$second_affected_rows = mysql_affected_rows();

if($first_affected_rows > 0 && $second_affected_rows > 0)
{
 $response = array("success" => true); 
 echo json_encode($response); 
}
else 
{
 $response = array("success" => false); 
 echo json_encode($response);  
}

Depending on what you want to achieve above can be changed to suit your logic

edited Feb 17, 2012 at 4:38

answered Feb 17, 2012 at 4:30

Jaspreet Chahal

2,7601 gold badge17 silver badges17 bronze badges

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

4 Comments

user1053263 Over a year ago

Thank you! I like this logic!

John V. Over a year ago

This will work even better than checking if they are both true.

user1053263 Over a year ago

I'm using this instead now. Thanks a bunch guys!

Jaspreet Chahal Over a year ago

Anything for a better world. Cheers!

John V. · Accepted Answer · 2012-02-17 04:33:43Z

1

First of all:

if(is_bool($first) == true && is_bool($second) == true)

is equivlent but longer than:

if(is_bool($first) && is_bool($second))

Second:

$response = array(success => false);
//and
$response = array(success => false);

Should probably be:

$response = array("success" => false);
//and
$response = array("success" => false);

Thirdly (and finally) I think your logic may be off, are you sure you want to send success = true if both are booleans and not both being true?

edited Feb 17, 2012 at 4:33

answered Feb 17, 2012 at 4:30

John V.

4,6704 gold badges28 silver badges26 bronze badges

3 Comments

John V. Over a year ago

@3nigma Thanks, was trying to edit it and noticed you fixed it for me lol

user1053263 Over a year ago

well I just need a way to send to the JQuery that both querys were successfull so it will prompt the user it was so thats why if both querys are true it should tell that to the JQuery via JSON and then the JQuery tells the user so. But if one or both of them return false, it tells the user that it failed. i just thought this was the way to go about it, unless you have a better way, I'm open to try anything. I'll try these updates now. THANK YOU! t

John V. Over a year ago

is_bool just checks if they are a true or false value, you want to check if they are true.

Collectives™ on Stack Overflow

JQuery to PHP with JSON

2 Answers 2

4 Comments

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

4 Comments

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related