Success! Looks like we've fixed this one. According to our records the fix was applied for EXTJS-8602 in 4.2.0.489.
  1. #1
    Sencha - Community Support Team SamuraiJack1's Avatar
    Join Date
    May 2008
    Posts
    550
    Vote Rating
    3
    SamuraiJack1 will become famous soon enough

      0  

    Default [4.2.0.265] "selectionchange" is not fired

    [4.2.0.265] "selectionchange" is not fired


    The "selectionchange" event is not longer fired when selected records are removed from the store. It happens because the "removed" variable is not initialized with 0 but later is incremented with ++. It becomes NaN and "maybeFireSelectionChange" is never called.

    Code from Ext.selection.Model:

    Code:
     
        // @private
        // Called by subclasses to deselect records upon detection of deletion from the store
        deselectDeletedRecords: function(records) {
            var me = this,
                selected = me.selected,
                i, length = records.length,
                record, removed;
    
            // Deselect records which were removed
            for (i = 0; i < length; i++) {
                record = records[i];
                if (selected.remove(record)) {
                    if (me.lastSelected == record) {
                        me.lastSelected = null;
                    }
                    if (me.getLastFocused() == record) {
                        me.setLastFocused(null);
                    }
                    ++removed;
                }
            }
            if (removed) {
                me.maybeFireSelectionChange(true);
            }
        },

  2. #2
    Sencha - Senior Forum Manager mitchellsimoens's Avatar
    Join Date
    Mar 2007
    Location
    Gainesville, FL
    Posts
    36,754
    Vote Rating
    827
    mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute

      0  

    Default


    Thanks for the report.
    Mitchell Simoens @SenchaMitch
    Sencha Inc, Senior Forum Manager
    ________________
    Check out my GitHub, lots of nice things for Ext JS 4 and Sencha Touch 2
    https://github.com/mitchellsimoens

    Think my support is good? Get more personalized support via a support subscription. https://www.sencha.com/store/

    Need more help with your app? Hire Sencha Services services@sencha.com

    Want to learn Sencha Touch 2? Check out Sencha Touch in Action that is in print!

    When posting code, please use BBCode's CODE tags.

Thread Participants: 1