Success! Looks like we've fixed this one. According to our records the fix was applied for EXTJS-9958 in 4.2.2.1144.
  1. #1
    Touch Premium Member
    Join Date
    Nov 2010
    Location
    Chicago
    Posts
    1,325
    Vote Rating
    114
    LesJ is a glorious beacon of light LesJ is a glorious beacon of light LesJ is a glorious beacon of light LesJ is a glorious beacon of light LesJ is a glorious beacon of light LesJ is a glorious beacon of light

      0  

    Default [4.2] Chart - refreshBuffer used but not documented

    [4.2] Chart - refreshBuffer used but not documented


    I'd like to point out what I believe is a code or documentation issue in the Chart class.

    me.refreshBuffer is not documented or referenced anywhere in the chart package.

    When I search the entire Ext JS code base I get '1 match in 1 file'.

    Code:
    Ext.define('Ext.chart.Chart', {
        ...
        // @private buffered refresh for when we update the store
        delayRefresh: function() {
            var me = this;
            if (!me.refreshTask) {
                me.refreshTask = new Ext.util.DelayedTask(me.refresh, me);
            }
            me.refreshTask.delay(me.refreshBuffer); // <-- refreshBuffer is not documented or referenced
        },
        ....

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

  3. #3
    Touch Premium Member
    Join Date
    Nov 2010
    Location
    Chicago
    Posts
    1,325
    Vote Rating
    114
    LesJ is a glorious beacon of light LesJ is a glorious beacon of light LesJ is a glorious beacon of light LesJ is a glorious beacon of light LesJ is a glorious beacon of light LesJ is a glorious beacon of light

      0  

    Default


    Please check the DelayedTask dependency in the Chart class. Component already uses DelayedTask.

    Code:
    Ext.define('Ext.Component', {
        ...
        uses: [
            'Ext.util.DelayedTask',
            'Ext.Layer',
            'Ext.resizer.Resizer',
            'Ext.util.ComponentDragger'
        ],
        ...
    
    
    
    
    Ext.define('Ext.chart.Chart', {
        ...
        requires: [
            'Ext.util.MixedCollection',
            'Ext.data.StoreManager',
            'Ext.chart.Legend',
            'Ext.chart.theme.Base',
            'Ext.chart.theme.Theme',
            'Ext.util.DelayedTask' // <--- Is this needed?
        ],    
        ...

Thread Participants: 1