diff --git a/src/main/java/org/olat/course/highscore/ui/HighScoreRunController.java b/src/main/java/org/olat/course/highscore/ui/HighScoreRunController.java index fd855171081a490f2a14797c965fc58d16b45275..b271639c5f618bfed44840bfcd9e69b2a00db181 100644 --- a/src/main/java/org/olat/course/highscore/ui/HighScoreRunController.java +++ b/src/main/java/org/olat/course/highscore/ui/HighScoreRunController.java @@ -22,6 +22,8 @@ package org.olat.course.highscore.ui; * Initial Date: 10.08.2016 <br> * @author fkiefer */ +import java.math.BigDecimal; + import java.util.ArrayList; import java.util.Date; import java.util.List; @@ -140,11 +142,16 @@ public class HighScoreRunController extends FormBasicController{ ownIdentity = userCourseEnv.getIdentityEnvironment().getIdentity(); AssessmentManager assessmentManager = userCourseEnv.getCourseEnvironment().getAssessmentManager(); AssessmentEntry ownEntry = assessmentManager.getAssessmentEntry(courseNode, ownIdentity); - + // check user visibility if (ownEntry != null && ownEntry.getUserVisibility() != null && !ownEntry.getUserVisibility().booleanValue()) { viewHighscore = false; return; } + // ban zero scorer from viewing the highscore on STCourseNode + if(isSTCourseNode && ownEntry != null && ownEntry.getScore().equals(new BigDecimal(0))) { + viewHighscore = false; + return; + } boolean adminORcoach = userCourseEnv.isAdmin() || userCourseEnv.isCoach(); // coaches or admin may see highscore, user only if already scored