Success! Looks like we've fixed this one. According to our records the fix was applied for EXTJS-7186 in 4.1.2.
  1. #1
    Sencha Premium Member
    Join Date
    Apr 2012
    Posts
    9
    Vote Rating
    2
    sidoron is on a distinguished road

      0  

    Default Wrong calculation of step in Ext.draw.Draw

    Wrong calculation of step in Ext.draw.Draw


    REQUIRED INFORMATION

    Ext version tested:
    • Ext 4.1
    Browser versions tested against:
    • Chrome 21.0.1180.79 m (Windows)
    • IE8
    Description:
    • Wrong calculation of Case of day in snapEndsByDateAndStep in Ext.draw.Draw
    Steps to reproduce the problem:
    • Create a chart with step Day and a value greater than 1
    The result that was expected:
    • An error occured and the chart does not load correctly with step greater than 1

    Test Case:
    The situation now

    Code:
    Currently is: parentheses missing:
    Code:
    new Date(fromStat[0], fromStat[1],
                                Math.floor(fromStat[2] - 1 / step[1]) * step[1] + 1, 0, 0, 0, 0);
    
    Should be:
    
    new Date(fromStat[0], fromStat[1],
    Math.floor((fromStat[2] - 1) / step[1]) * step[1] + 1, 0, 0, 0, 0);
    



    HELPFUL INFORMATION


    Debugging already done:
    • This function should be corrected
    Possible fix:
    • override this method?

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


    This has been fixed in 4.1.2
    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