Thank you for reporting this bug. We will make it our priority to review this report.
  1. #1
    Sencha User leeby's Avatar
    Join Date
    Sep 2008
    Posts
    44
    Vote Rating
    1
    leeby is on a distinguished road

      0  

    Default [FIXED-130] Remove issue of Ext.data.Store

    [FIXED-130] Remove issue of Ext.data.Store


    Hi,

    See below red code, the "this.snapshot" and "this.data" is actually the same object, so the remove function will delete 2 records at the same time. The right record and the next record.

    It can be solved by simply adding a 'IF'.

    =========================
    remove: function(record) {
    if (Ext.isArray(record)) {
    for (var i = 0, length = record.length; i < length; i++) {
    this.remove(record[i]);
    }
    return;
    }

    this.removed.push(record);

    var index = this.data.indexOf(record);

    if (this.snapshot) {
    this.snapshot.remove(record);
    }


    if (index > -1) {
    record.unjoin(this);
    this.data.removeAt(index);

    this.fireEvent('remove', this, record, index);
    this.fireEvent('datachanged', this);
    }
    }
    www.feyasoft.com
    Extjs 3.2 based Calendar Spreadsheet Presentation Word

  2. #2
    Sencha User
    Join Date
    Mar 2007
    Location
    Haarlem, Netherlands
    Posts
    1,243
    Vote Rating
    9
    TommyMaintz will become famous soon enough TommyMaintz will become famous soon enough

      0  

    Default


    This has been fixed and will be part of the next release (0.9.3) due beginning of next week.

Similar Threads

  1. Replies: 3
    Last Post: 8 Jun 2010, 4:19 PM
  2. Replies: 2
    Last Post: 15 May 2009, 2:19 AM
  3. [SOLVED] - Ext.data.Store Issue with retrieving data after page load
    By sneddo in forum Ext 2.x: Help & Discussion
    Replies: 6
    Last Post: 4 Aug 2008, 4:49 AM
  4. Ext.data.Store.remove() ?
    By jdhuang in forum Ext 1.x: Help & Discussion
    Replies: 2
    Last Post: 1 Jul 2007, 5:01 PM
  5. Replies: 1
    Last Post: 29 Jun 2007, 12:40 AM

Thread Participants: 1