Skip to content
This repository has been archived by the owner on Nov 19, 2024. It is now read-only.

Fix the Jacoco Coverage Trend portlet #88

Merged
merged 1 commit into from
Jun 7, 2017

Conversation

dcendents
Copy link
Contributor

Since the last update (remove joda-time dependency), the jacoco coverage trend chart portlet is broken. The java.util.Calendar toString method does not return a valid date string and the chart is all messed up:

error

This fix will format the Calendar into a String ("yyyy-MM-dd") and pass that value to JFreeChart:

fixed

Regards

@dcendents
Copy link
Contributor Author

I've changed the commit, turns out to be correct the date needs to be converted into a string earlier to group results correctly in the summaries map.

Here's the correct result:

fixed2

@centic9 centic9 merged commit 770ef97 into jenkinsci:master Jun 7, 2017
@centic9
Copy link
Member

centic9 commented Jun 7, 2017

Good catch! Thanks for the PR!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants