Results 1 to 2 of 2

Thread: Wrong calculation of step in Ext.draw.Draw

    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
      0  

    Default 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 - Sr Software Engineer mitchellsimoens's Avatar
    Join Date
    Mar 2007
    Location
    Gainesville, FL
    Posts
    39,551
    Vote Rating
    1271
      0  

    Default

    This has been fixed in 4.1.2
    Mitchell Simoens @LikelyMitch
    Sencha Inc, Senior Software Engineer
    ________________
    Learn BBCode and use it! Checkout the CODE tag!

    Check out my GitHub, lots of nice things for Ext JS and Sencha Touch
    https://github.com/mitchellsimoens

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •