Bug 62283 - Report Dashboard - Date is not correctly displayed on chart when granularity is >= 1 day
Summary: Report Dashboard - Date is not correctly displayed on chart when granularity ...
Status: RESOLVED FIXED
Alias: None
Product: JMeter
Classification: Unclassified
Component: HTTP (show other bugs)
Version: 3.3
Hardware: All All
: P2 normal (vote)
Target Milestone: JMETER_5.0
Assignee: JMeter issues mailing list
URL:
Keywords: FixedInTrunk
Depends on:
Blocks:
 
Reported: 2018-04-11 14:33 UTC by pdutertre
Modified: 2018-05-27 19:54 UTC (History)
2 users (show)


Attachments
Date is not correctly displayed (125.08 KB, image/png)
2018-04-11 14:33 UTC, pdutertre 		Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description pdutertre 2018-04-11 14:33:31 UTC 
Created attachment 35857 [details]
Date is not correctly displayed

The date is not displayed correctly when the granularity is equal or greater than 1 day (see screen capture)

jmeter.reportgenerator.overall_granularity=86400000

A hack to correct the bug:
 replace the %H:%M:%S by %Y.%m.%d inside the content\js\graph.js file after JMETER report generation
Comment 1 Philippe Mouawad 2018-05-27 13:19:07 UTC 
Regarding this bug:

    should we automatically compute flot timeformat for overtime graphs based on granularity ?:
        if granularity >= day => %y%m%d/
        if granularity >= hour => %m%d/%H
        if granularity >= minute => %H:%M
        if granularity >= second => %H:%M:%S
    Or make user set it ? knowing that format is Javascript not java ?
Comment 2 Philippe Mouawad 2018-05-27 19:53:44 UTC 
Author: pmouawad
Date: Sun May 27 19:53:20 2018
New Revision: 1832345

URL: http://svn.apache.org/viewvc?rev=1832345&view=rev
Log:
Bug 62283 - Report Dashboard - Date is not correctly displayed on chart when granularity is >= 1 day
Bugzilla Id: 62283

Modified:
    jmeter/trunk/bin/report-template/content/js/dashboard-commons.js
    jmeter/trunk/bin/report-template/content/js/graph.js.fmkr
    jmeter/trunk/xdocs/changes.xml
Comment 3 Philippe Mouawad 2018-05-27 19:54:20 UTC 
I chose to make it dynamic without user specifying anything.
Please test and confirm if it's ok for you.

Regards