mirror of
https://github.com/eclipse-sailing-analytics/sailing-analytics.git
synced 2026-09-29 08:56:43 +00:00
fix for bug 1222 by extending the LeaderboardDescriptor constructor
This commit is contained in:
+8
-8
@@ -256,7 +256,7 @@ public class LeaderboardConfigPanel extends FormPanel implements RegattaDisplaye
|
||||
} else {
|
||||
if (leaderboardDTO.isRegattaLeaderboard) {
|
||||
LeaderboardDescriptor descriptor = new LeaderboardDescriptor(leaderboardDTO.name,
|
||||
null, leaderboardDTO.discardThresholds, leaderboardDTO.regattaName);
|
||||
leaderboardDTO.displayName, null, leaderboardDTO.discardThresholds, leaderboardDTO.regattaName);
|
||||
AbstractLeaderboardDialog dialog = new RegattaLeaderboardEditDialog(Collections
|
||||
.unmodifiableCollection(otherExistingLeaderboard), Collections.unmodifiableCollection(allRegattas),
|
||||
descriptor, stringMessages, errorReporter,
|
||||
@@ -272,7 +272,7 @@ public class LeaderboardConfigPanel extends FormPanel implements RegattaDisplaye
|
||||
});
|
||||
dialog.show();
|
||||
} else {
|
||||
LeaderboardDescriptor descriptor = new LeaderboardDescriptor(leaderboardDTO.name, leaderboardDTO.scoringScheme, leaderboardDTO.discardThresholds);
|
||||
LeaderboardDescriptor descriptor = new LeaderboardDescriptor(leaderboardDTO.name, leaderboardDTO.displayName, leaderboardDTO.scoringScheme, leaderboardDTO.discardThresholds);
|
||||
FlexibleLeaderboardEditDialog dialog = new FlexibleLeaderboardEditDialog(Collections
|
||||
.unmodifiableCollection(otherExistingLeaderboard),
|
||||
descriptor, stringMessages, errorReporter,
|
||||
@@ -954,9 +954,9 @@ public class LeaderboardConfigPanel extends FormPanel implements RegattaDisplaye
|
||||
leaderboardSelectionChanged();
|
||||
}
|
||||
|
||||
private void updateLeaderboard(final String oldLeaderboardName, final LeaderboardDescriptor leaderboardToUdate) {
|
||||
sailingService.updateLeaderboard(oldLeaderboardName, leaderboardToUdate.getName(), leaderboardToUdate.getDisplayName(),
|
||||
leaderboardToUdate.getDiscardThresholds(), new AsyncCallback<Void>() {
|
||||
private void updateLeaderboard(final String oldLeaderboardName, final LeaderboardDescriptor leaderboardToUpdate) {
|
||||
sailingService.updateLeaderboard(oldLeaderboardName, leaderboardToUpdate.getName(), leaderboardToUpdate.getDisplayName(),
|
||||
leaderboardToUpdate.getDiscardThresholds(), new AsyncCallback<Void>() {
|
||||
@Override
|
||||
public void onFailure(Throwable t) {
|
||||
errorReporter.reportError("Error trying to update leaderboard " + oldLeaderboardName + ": "
|
||||
@@ -968,9 +968,9 @@ public class LeaderboardConfigPanel extends FormPanel implements RegattaDisplaye
|
||||
for (int i = 0; i < leaderboardList.getList().size(); i++) {
|
||||
StrippedLeaderboardDTO dao = leaderboardList.getList().get(i);
|
||||
if (dao.name.equals(oldLeaderboardName)) {
|
||||
dao.name = leaderboardToUdate.getName();
|
||||
dao.displayName = leaderboardToUdate.getDisplayName();
|
||||
dao.discardThresholds = leaderboardToUdate.getDiscardThresholds();
|
||||
dao.name = leaderboardToUpdate.getName();
|
||||
dao.displayName = leaderboardToUpdate.getDisplayName();
|
||||
dao.discardThresholds = leaderboardToUpdate.getDiscardThresholds();
|
||||
break;
|
||||
}
|
||||
}
|
||||
|
||||
+4
-2
@@ -25,15 +25,17 @@ public class LeaderboardDescriptor {
|
||||
public LeaderboardDescriptor() {
|
||||
}
|
||||
|
||||
public LeaderboardDescriptor(String name, ScoringSchemeType scoringScheme, int[] discardThresholds, String regattaName) {
|
||||
public LeaderboardDescriptor(String name, String displayName, ScoringSchemeType scoringScheme, int[] discardThresholds, String regattaName) {
|
||||
this.name = name;
|
||||
this.displayName = displayName;
|
||||
this.scoringScheme = scoringScheme;
|
||||
this.discardThresholds = discardThresholds;
|
||||
this.regattaName = regattaName;
|
||||
}
|
||||
|
||||
public LeaderboardDescriptor(String name, ScoringSchemeType scoringScheme, int[] discardThresholds) {
|
||||
public LeaderboardDescriptor(String name, String displayName, ScoringSchemeType scoringScheme, int[] discardThresholds) {
|
||||
this.name = name;
|
||||
this.displayName = displayName;
|
||||
this.scoringScheme = scoringScheme;
|
||||
this.discardThresholds = discardThresholds;
|
||||
}
|
||||
|
||||
+1
-1
@@ -107,9 +107,9 @@ public class LeaderboardGroupDialog extends DataEntryDialog<LeaderboardGroupDial
|
||||
final List<StrippedLeaderboardDTO> emptyStrippedLeaderboardDTOList = Collections.emptyList();
|
||||
errorMessage = new FlexibleLeaderboardDialog.LeaderboardParameterValidator(stringMessages,
|
||||
emptyStrippedLeaderboardDTOList).getErrorMessage(new LeaderboardDescriptor("Overall",
|
||||
/* displayName */ null,
|
||||
groupToValidate.getOverallLeaderboardScoringSchemeType(),
|
||||
groupToValidate.getOverallLeaderboardDiscardThresholds()));
|
||||
;
|
||||
}
|
||||
return errorMessage;
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user