jQuery/javascript object passing

Question

Up until this point, I haven't done much in javascript. I recently started playing with jQuery and came across a tutorial on how to do Edit in Place on a form.

The resulting code works perfectly if you hard-code the id of your container div in the function, but I'm having issues generalizing it for an id passed in to the function.

The code starts like this:

$(document).ready(function() {
    setClickable($('#editInPlace'));
});

function setClickable(target)
{
    $(target).click
    (
        function()
        {
            //Do some dom manipulation

            $('.buttonA').click(function() { saveChanges(this, false); });
            $('.buttonB').click(function() { saveChanges(this, true); });
        }
    )

    .mouseover(function() { $(this).addClass("editable"); })
    .mouseout(function() { $(this).removeClass("editable"); });
}; //end of function setClickable

function saveChanges(obj, cancel)
{
    //Handle the save changes code

    //Reset the div
    $(obj).parent().after('<div id="editInPlace">' + t + '</div>').remove();

    //Call setClickable
    setClickable($('#editInPlace'));
}

The part I'm having issues with is the last call to setClickable inside of the saveChanges function. If I hard-code the value of $('#editInPlace'), it works fine. However, if I change the function signature to

function saveChanges(target, obj, cancel)

and pass in the target object from the setClickable method:

$('.buttonB').click(function() { saveChanges(target, this, true); });

calling

setClickable(target);

doesn't seem to work and I can't figure out why.

You might want to look at the Jeditable plugin, appelsiini.net/projects/jeditable or the Editable plugin, arashkarimzadeh.com/index.php/jquery/…. — tvanfosson
– tvanfosson, Commented Apr 1, 2009 at 13:34

cletus · Accepted Answer · 2009-04-01 13:28:58Z

6

Change:

function setClickable(target)
{
    $(target).click
    (

to:

function setClickable(target)
{
    target.click
    (

You're passing in a wrapped set (also called a jQuery object) and then wrapped it in another wrapped set. The above fixes that. The alternate way to do this is:

$(document).ready(function() {
    setClickable('editInPlace');
});

function setClickable(id)
{
    $("#" + id).click
    (

You see what I'm getting at?

answered Apr 1, 2009 at 13:28

cletus

627k169 gold badges922 silver badges945 bronze badges

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

Comments

Mel · Accepted Answer · 2009-04-01 13:30:23Z

1

You can always try using setClickable($(target)); to make sure you've got a jQuery object.

answered Apr 1, 2009 at 13:30

Mel

2,2991 gold badge18 silver badges18 bronze badges

Collectives™ on Stack Overflow

jQuery/javascript object passing

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related