Passing Variables Through Callback

[Gregg Bolinger]

I'm trying to go through some of my code and get rid of some duplicate boilerplate. Through out my app, I need to confirm certain things, like deleting. So I have something like this...so far...See the comments in the live click event for what I actually need to do.

$('.remove').live('click', function() {

    // I need to get these 2 var's into confirmRemoveMember()
    var clicker = $(this);
    var data = clicker.metadata();    
    // ###############################

    showConfirmDialog("Really Delete?", confirmRemoveMember);
});

function confirmRemoveMember() {
     var commData = selectedCommittee.metadata()
     $.post(contextPath + "/committee/removeMember/" + commData.id, {'userId':data.id}, function(data) {
       if (data.success) {
         $(clicker).closest('li').remove();
       }
     }, 'json');
}

function showConfirmDialog(message, callback) {
    $("#message-dialog").html("<p>" + message + "</p>");
    $("#message-dialog").dialog({
            resizable: false,
            height:140,
            modal: true,
            buttons: {
                'Yes, delete': function() {
                    $("#message-dialog").html("");
                    $(this).dialog('close');
                    callback.call(this)
                },
                Cancel: function() {
                    $("#message-dialog").html("");
                    $(this).dialog('close');
                }
            }
        });
}

[Ankit Garg]

What if you add the two values as parameter to showConfirmDialog method and pass it to the callback (I'm no javascript guru so this might be a wrong solution)

# $('.remove').live('click', function() {    
    // I need to get these 2 var's into confirmRemoveMember()  
    var clicker = $(this);  
    var data = clicker.metadata();
 
    showConfirmDialog("Really Delete?", confirmRemoveMember, clicker, data);  
});

function confirmRemoveMember(clicker, data) {
:::
}

function showConfirmDialog(message, callback, clicker, data) {
:::
        callback.call(this,clicker,data)
}

Edit: I'm completely sure now that this post is useless, changing the parameters of showConfirmDialog/confirmRemoveMember must not be a choice...

[Gregg Bolinger]

Right, because not everything I use a confirm for uses those parameters.