javascript Undefined variable/array

Question

I have a small function that takes a dynamic list of textboxes with urls and sends it to the server. I am trying to load the urls into an array then pass them off in a JSON object. The problem is, when I get to that point in the code, I keep getting the error:

TypeError: url_data[this_url] is undefined
http://localhost/2011/admin/public/assets/js/tssol.ui.js
Line 196

Here is the block that returns that error:

$("#dialogEditor").dialog('option', 'buttons', {'Save' : function() {
    /* Define variables */
    var url_data = new Array();
    var this_url = 0;

    /* I've tried this with and without explicitly passing the vars... */
    $("#website_editor input").each(function(url_data, this_url) {
        var id =$(this).attr('name').match(/\d+/);
        var url=$(this).val();

        /* Test the varible each iteration */
        console.log("this_url: " + this_url + " id: " + id + " url: " + url);

    /* Line 196 */ url_data[this_url].id = id;
        url_data[this_url].url = url;

        this_url++;
});


data = JSON.stringify(url_data);
//data = url_data;
//console.log(data);

$.ajax({
    url: 'ajax.websites.php',
    dataType: 'json',
    data: {
        action: "update_resort",
        ResortId: resort,
        data: data
    }
});

$(this).dialog("close");
},
'Cancel': function(){$(this).dialog("close");
}});
},

Sorry about the formatting

What are you trying to obtain? this_url is the input element in .each, so you cannot use that as a key in the array. — pimvdb
– pimvdb, Commented Oct 17, 2012 at 18:00
If url_data[this_url] does not exist yet, you cannot set the id property on it... you have to assign an object to url_data[this_url] first. — Felix Kling
– Felix Kling, Commented Oct 17, 2012 at 18:10

Renato Gama · Accepted Answer · 2012-10-17 18:00:28Z

3

I would do this:

url_data[this_url] = {
    id: id,
    url: url
}

answered Oct 17, 2012 at 18:00

Renato Gama

16.6k12 gold badges64 silver badges98 bronze badges

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

Collectives™ on Stack Overflow

javascript Undefined variable/array

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related