eclipse-sailing-analytics/sailing-analytics
1
0
Fork 0
mirror of https://github.com/eclipse-sailing-analytics/sailing-analytics.git synced 2026-09-27 16:06:39 +00:00
Code Activity

eliminated issue with multiple series per competitor

Browse Source
This commit is contained in:
Axel Uhl
2012-11-16 22:54:40 +01:00
parent c41ee7d795
commit fcdf3ef0dc
1 changed files with 6 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files
java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/ui/shared/charts/LeaderboardRankChart.java
+6 -9
View File
@@ -74,14 +74,7 @@ public class LeaderboardRankChart extends SimplePanel implements RequiresResize
return raceColumnNames.get((int) axisLabelsData.getValueAsLong());
}
}));
chart.getYAxis(0).setAxisTitleText(stringMessages.rank()).setStartOnTick(true).setShowFirstLabel(false)
/* TODO do we need a specific formatter if we don't really format anything?
.setLabels(new YAxisLabels().setFormatter(new AxisLabelsFormatter() {
@Override
public String format(AxisLabelsData axisLabelsData) {
return axisLabelsData.getValueAsString();
}
})) */;
chart.getYAxis().setAxisTitleText(stringMessages.rank()).setStartOnTick(true).setShowFirstLabel(false);
if (compactChart) {
chart.setSpacingBottom(10).setSpacingLeft(10).setSpacingRight(10).setSpacingTop(2)
.setOption("legend/margin", 2)
@@ -101,23 +94,27 @@ public class LeaderboardRankChart extends SimplePanel implements RequiresResize
/* date: null means "now" or "live" */ null, new AsyncCallback<List<Pair<String,List<CompetitorDTO>>>>() {
@Override
public void onSuccess(List<Pair<String, List<CompetitorDTO>>> result) {
List<Series> chartSeries = Arrays.asList(chart.getSeries());
List<Series> chartSeries = new ArrayList<Series>(Arrays.asList(chart.getSeries()));
chart.hideLoading();
raceColumnNames.clear();
int raceNumber = 0;
int maxCompetitorCount = 0;
for (Pair<String, List<CompetitorDTO>> entry : result) {
raceColumnNames.add(entry.getA());
int rank = 1;
maxCompetitorCount = Math.max(maxCompetitorCount, entry.getB().size());
for (CompetitorDTO competitor : entry.getB()) {
Series series = getOrCreateSeries(competitor);
if (!chartSeries.contains(series)) {
chart.addSeries(series);
chartSeries.add(series);
}
series.addPoint(raceNumber, rank);
rank++;
}
raceNumber++;
}
chart.getYAxis().setMax(maxCompetitorCount);
chart.setSizeToMatchContainer();
// it's important here to recall the redraw method, otherwise the bug fix for wrong checkbox positions (nativeAdjustCheckboxPosition)
// in the BaseChart class would not be called