Thank you for reporting this bug. We will make it our priority to review this report.
  1. #1
    Sencha User
    Join Date
    Apr 2010
    Posts
    8
    Vote Rating
    0
    skzr.org is on a distinguished road

      0  

    Post [FIXED-848] GroupingStore sort have a bug when not set the sortInfo of config

    [FIXED-848] GroupingStore sort have a bug when not set the sortInfo of config


    in the Ext.data.GroupingStore.prototype.sort function:
    Code:
      ...
      sorters = [this.sortInfo];//have a bug when not set this.sortInfo
      ...
    So: in the Ext.data.Store.prototype.sortData function
    sorters == [null]
    Ext.data.Store.prototype.sortData:
    Code:
    ....
            //create a sorter function for each sorter field/direction combo
            for (var i=0, j = sorters.length; i < j; i++) {
    //in fact sorters[0] === null will throw a exception
                sortFns.push(this.createSortFunction(sorters[i].field, sorters[i].direction));
            }
    ....
    i fixed :
    Ext.data.GroupingStore.prototype.sort
    Code:
      ...
      sorters = this.sortInfo ? [this.sortInfo] : [];
      //old: sorters = [this.sortInfo];
      ...
    Last edited by skzr.org; 10 Apr 2010 at 11:08 PM. Reason: add where throw exception

  2. #2
    Sencha User
    Join Date
    Apr 2010
    Posts
    8
    Vote Rating
    0
    skzr.org is on a distinguished road

      0  

    Default


    forget:
    ExtJs is version 3.2.0

  3. #3
    Sencha - Community Support Team edspencer's Avatar
    Join Date
    Jan 2009
    Location
    Palo Alto, California
    Posts
    1,939
    Vote Rating
    9
    edspencer is a jewel in the rough edspencer is a jewel in the rough edspencer is a jewel in the rough

      0  

    Default


    This is now fixed in svn, thanks for reporting it. It will be present in the 3.2.1 build which will go out shortly.
    Ext JS Senior Software Architect
    Personal Blog: http://edspencer.net
    Twitter: http://twitter.com/edspencer
    Github: http://github.com/edspencer

Thread Participants: 1

Tags for this Thread