javascript array empty using .push .text

Question

I am having an issue with the code that I was given as an example from a previous SO question - it is working about 95% but the order array is appearing empty why?

HTML:

<input class="order" value="<?php echo $order; ?>" type="text"  />

JS:

$('body').on("click", "#brands_by_category_submit_btn", function (e) {
         e.preventDefault();               
        var self       = $(this);
        var order      =  []; 
        var id         = $("#manID").data("id");
        var brand_name = $("#brand_name").data("id");
        var data       = grabData(true);

        $(".order").each(function(){
            order.push($(this).text());
        })

        if(data.length)
        {
            var data_array = { 
                id : id,
                brand_name : brand_name, 
                cat_id     : data,
                order      : order, 
                state      : 1
            };

Psst: var order = $(".order").map(function() { return $(this).val(); }).get(); — Ry-
– Ry- ♦, Commented Sep 22, 2013 at 20:15

pawel · Accepted Answer · 2013-09-22 20:15:05Z

2

.text() returns the text content of a node, like <p>this text here</p>. An <input /> element doesn't have text content, so $('input').text() will just return an empty string. Your order array should be an array of empty strings then. Maybe you want to extract the values?

$(".order").each(function(){
            order.push($(this).val());
});

answered Sep 22, 2013 at 20:15

pawel

37.1k7 gold badges59 silver badges54 bronze badges

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

Comments

Andy · Accepted Answer · 2013-09-22 20:14:00Z

1

How about:

order.push($(this).val());

answered Sep 22, 2013 at 20:14

Andy

63.6k13 gold badges72 silver badges99 bronze badges

Collectives™ on Stack Overflow

javascript array empty using .push .text

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