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

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