If checkbox changes, use jquery to call php

Question

I have a simple checkbox on a page that allows a user to say if they'd like to receive email notifications. I am using jquery for this to call some php code when the checkbox changes. However, I am not having much luck even calling the jquery function (clicking the checkbox does nothing) let alone test the backend functionality.

Any help in pointing out the error would be great. Thanks.

The checkbox HTML:

<input id="notify_checkbox" type="checkbox" value="y" name="notify">

The jquery:

$('#notify_checkbox').change(function(){

    if($('#notify_checkbox').attr('checked'))
    {
        $.post("/update_notify", { checked: "y", email: "<?php echo $this->session->userdata('email');?>" });
        $( "#notifyresult" ).html( "<p>Awesome, we'll send you an email!</p>" );
    }
    else
    {
        $.post("/update_notify", { checked: "n", email: "<?php echo $this->session->userdata('email');?>" });
        $( "#notifyresult" ).html( "<p>Okay, we won't email you.</p>" );
    }
});

And finally the PHP:

function update_notify()
{
    // Passed through AJAX
    $notify = $_POST[checked];
    $email = $_POST[email];

    $this->load->model('musers');
    $query = $this->musers->update_user_notify($email, $notify);
}

RESOLUTION: The comments below were helpful but not the ultimate solution. The solution was to add the following around my code.

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

In your PHP, you should quote your array keys: $_POST['checked'] and $_POST['email']. — gen_Eric
– gen_Eric, Commented May 19, 2011 at 19:38

Community · Accepted Answer · 2017-05-23 11:47:53Z

3

Why not use .click() instead?

JSFIDDLE

Also, as you can see in my JSFiddle example, use .is(':checked') instead of attr('checked').

edit after @Rocket commented on your post:

You should indeed quote your $_POST values in your php! Didn't notice it myself, credits to rocket

edited May 23, 2017 at 11:47

CommunityBot

11 silver badge

answered May 19, 2011 at 19:34

user657496

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

2 Comments

zee Over a year ago

Thanks. I see this works in your jsFiddle code, but for the life of me I can't figure out why clikcing on my checkbox (with the .is(':checked') changes) nothing happens! I even put an alert like you jsFiddle, but still clicking the checkbox does nothing. Grrr.

zee Over a year ago

Might be a silly question, but does it matter if the jquery function is not in between the <head></head tags? I have it in the body of the view, not the header.

gen_Eric · Accepted Answer · 2011-05-19 19:39:53Z

0

What's the name of your controller? You need to put that in the URL.

 $.post("/controller/update_notify", ...

answered May 19, 2011 at 19:39

gen_Eric

228k42 gold badges304 silver badges343 bronze badges

1 Comment

zee Over a year ago

Correct, I actually had this in my code but removed it from the sample paster here for brevity.

lonesomeday · Accepted Answer · 2011-05-19 19:42:06Z

0

The problem is with the redefinition of the attr function in jQuery 1.6, and with the difference between attributes and properties.

With attributes (retrieved with attr), the value of checked="checked" or its absence stays the same, regardless of whether the element is actually checked or not.

With properties (retrieved with prop as of jQuery 1.6), the actual state of the element is found. This is equivalent to checking the checked property of the element (which is preferable because you don't need to do a new jQuery selection). The best soltion would be as follows:

if (this.checked) {

See jsFiddles showing this:

answered May 19, 2011 at 19:42

lonesomeday

239k54 gold badges330 silver badges329 bronze badges

Collectives™ on Stack Overflow

If checkbox changes, use jquery to call php

3 Answers 3

2 Comments

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

2 Comments

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related