[4.0.2] groupToggleMenuItem is null

[darkside_ua]

When using 'Ext.grid.feature.Grouping' with 'enableGroupingMenu: false' config parameter there is an error in ext-all-debug.js at line 79141 when toggling grouping feature enable/disable. The error 'groupToggleMenuItem is null' is at line:

groupToggleMenuItem.setChecked(true, true);

So, you just need to add a check for 'groupToggleMenuItem is not null' to resolve this.

[leonardodaza]

When using 'Ext.grid.feature.Grouping' with 'enableGroupingMenu: false' config parameter there is an error in ext-all-debug.js at line 79141 when toggling grouping feature enable/disable. The error 'groupToggleMenuItem is null' is at line:

groupToggleMenuItem.setChecked(true, true);

So, you just need to add a check for 'groupToggleMenuItem is not null' to resolve this.

Thanks for your post. I´ve a question... How add the check?

[mfreund]

Problem exists here too... with 4.1.0

[evant]

This issue is resolved in the latest build, the current code reads:

groupToggleMenuItem = me.view.headerCt.getMenu().down('#groupToggleMenuItem');
if (groupToggleMenuItem) {
    groupToggleMenuItem.setChecked(true, true);
}

[Dumbledore]

this is not fixed in 4.1.3 inside onColumHideShow:

    onColumnHideShow: function() {
        var menu = this.grid.headerCt.getMenu(),
            groupToggleMenuItem = menu.down('#groupMenuItem');
            // "Group by this field" must be disabled if there's only one column left visible.
        if (this.view.headerCt.getVisibleGridColumns(true).length > 1) {
            groupToggleMenuItem.enable(); // here is it not fixed
        } else {
            groupToggleMenuItem.disable();
        }
    },
    showMenuBy: function(t, header) {
        var menu = this.getMenu(),
            groupMenuItem = menu.down('#groupMenuItem'),
            groupMenuMeth = header.groupable === false ? 'disable' : 'enable',
            groupToggleMenuItem = menu.down('#groupToggleMenuItem');
        
        groupMenuItem[groupMenuMeth]();
        if (groupToggleMenuItem) {                  // here is it fixed
            groupToggleMenuItem[this.view.store.isGrouped() ? 'enable' : 'disable']();
        }
        Ext.grid.header.Container.prototype.showMenuBy.apply(this, arguments);
    },