Merge remote-tracking branch 'server/master'

This commit is contained in:
Axel Uhl committed 2021-03-12 14:24:29 +01:00
commit 406da2b832
1 file changed
+5 -2
@@ -632,8 +632,11 @@ public class EventsResource extends AbstractSailingServerResource {
/* leaderboardDiscardThresholdsParam */ leaderboardDiscardThresholdsParam, numberOfRacesParam, canBoatsOfCompetitorsChangePerRace,
competitorRegistrationType, localCompetitorRegistrationSecret);
if (leaderboardGroup != null) {
// no security checks for any overall leaderboard required here as we don't manipulate it through this service
getService().apply(new UpdateLeaderboardGroup(leaderboardGroup.getId(), leaderboardGroup.getName(),
// no security checks for any overall leaderboard required here as we don't manipulate it
// through this service. leaderboardGroup.getOverallLeaderboard() == null will be true, due to
// overallLeaderboardScoringSchemeTypeParam being null in validateAndAddLeaderboardGroup call.
getService().apply(new UpdateLeaderboardGroup(leaderboardGroup.getId(),
leaderboardGroup.getName(),
leaderboardGroup.getDescription(), leaderboardGroup.getDisplayName(),
Collections.singletonList(leaderboard.getName()),
leaderboardGroup.getOverallLeaderboard() == null ? null