[FIXED-1184] Store.insert() bug - reverse order

[radubrehar]

Hi,

I have noticed an issue in the store insert() method. Here is the code.

insert : function(index, records){
        records = [].concat(records);
        for(var i = 0, len = records.length; i < len; i++){
            this.data.insert(index, records[i]);
            records[i].join(this);
        }
        if(this.snapshot){
            this.snapshot.addAll(records);
        }
        this.fireEvent('add', this, records, index);
    },

What I get is that I insert an array of records at a certain index, and after insertion, they are in the reverse order. The reason is clear, as in the method above records[i] is inserted always at the "index" index, while the insertion index should increment by 1 at every iteration.

So if I have records [r1,r2,r3] and insert them in an empty store, at index 0, I get the records in the store in the following order: [r3,r2,r1], which is annoying.

[radubrehar]

Well, after a search, I have found the same bug already reported at http://www.sencha.com/forum/showthre...-reverse-order. Hope it will be fixed in the future version of Ext, with the fix that Animal pointed a=out.

[Jamie Avins]

Thank you for the report. If the post isn't in the bug forum with a number assigned to it in the title, there's a good chance we don't know about it.

[edspencer]

Thanks, this has been fixed in svn and will appear in 3.3 beta 2