Success! Looks like we've fixed this one. According to our records the fix was applied for EXTJS-5076 in 4.2.0.265.
  1. #1
    Ext JS Premium Member
    Join Date
    May 2009
    Posts
    95
    Vote Rating
    40
    extjs@peacetech.com has a spectacular aura about extjs@peacetech.com has a spectacular aura about extjs@peacetech.com has a spectacular aura about

      0  

    Default Store.getGroupString ignores Grouper.getGroupString

    Store.getGroupString ignores Grouper.getGroupString


    Store.getGroupString ignores Grouper.getGroupString so if you provided a Grouper with your own getGroupString it never used

    Snippet from Store.js:

    Code:
        getGroupString: function(instance) {
            var group = this.groupers.first();
            if (group) {
                return instance.get(group.property);
            }
            return '';
        },

  2. #2
    Sencha - Senior Forum Manager mitchellsimoens's Avatar
    Join Date
    Mar 2007
    Location
    Gainesville, FL
    Posts
    37,547
    Vote Rating
    872
    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


    Thank you 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.

  3. #3
    Sencha Premium Member
    Join Date
    Feb 2012
    Posts
    14
    Vote Rating
    0
    Jan Pogadl is on a distinguished road

      0  

    Default


    It would be nice if this could get fixed.

    This code can be used for an overwrite in Ext.data.Store.

    Code:
        getGroupString: function(instance) {
            var group = this.groupers.first();
            if (group) {
                return group.getGroupString(instance);
            }
            return '';
        }

  4. #4
    Sencha User nukboon's Avatar
    Join Date
    Oct 2007
    Location
    BKK, Thailand
    Posts
    73
    Vote Rating
    1
    nukboon is on a distinguished road

      0  

    Default EXTJSIV-5076 STILL NOT FIXED

    EXTJSIV-5076 STILL NOT FIXED


    http://www.sencha.com/forum/showthre...getGroupString

    Problem not solved.

  5. #5
    Sencha Premium Member
    Join Date
    Mar 2010
    Posts
    111
    Vote Rating
    -1
    predator has a little shameless behaviour in the past

      0  

    Default


    I can see that in 4.2.2.1144, the suggested change is included
    However, this code is causing infinite recursion to my grouped grids. No fancy settings to grouping feature.
    PHP Code:
    getGroupString: function(instance) {
            var 
    group this.groupers.first();
            if (
    group) {
                return 
    group.getGroupString(instance); //Infinite recursion
            
    }
            return 
    '';
        },