include student points in API CourseSummary and round points to 2 dec…

…imals
mp-access · Dec 14, 2023 · dc9e590 · dc9e590
1 parent c330d7f
commit dc9e590
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/main/kotlin/ch/uzh/ifi/access/projections/CourseSummary.kt b/src/main/kotlin/ch/uzh/ifi/access/projections/CourseSummary.kt
@@ -24,7 +24,7 @@ interface CourseSummary {
     @get:Value("#{@courseService.getTeamMembers(target.assistants)}")
     val assistants: Set<MemberOverview?>?
 
-    @get:Value("#{@courseServiceForCaching.getStudents(target.slug)}")
+    @get:Value("#{@courseServiceForCaching.getStudentsWithPoints(target.slug)}")
     val students: List<StudentDTO?>?
     val assignments: List<AssignmentSummary?>?
 

diff --git a/src/main/kotlin/ch/uzh/ifi/access/service/CourseService.kt b/src/main/kotlin/ch/uzh/ifi/access/service/CourseService.kt
@@ -30,6 +30,7 @@ import org.springframework.lang.Nullable
 import org.springframework.security.core.context.SecurityContextHolder
 import org.springframework.stereotype.Service
 import org.springframework.web.server.ResponseStatusException
+import java.math.RoundingMode
 import java.nio.charset.Charset
 import java.nio.file.Files
 import java.nio.file.NoSuchFileException
@@ -71,7 +72,7 @@ class CourseServiceForCaching(
             val user = roleService.getUserByUsername(it)
             if (user != null) {
                 val coursePoints = courseRepository.getTotalPoints(courseSlug, user.username) ?: 0.0
-                val studentDTO = StudentDTO(user.firstName, user.lastName, user.email, coursePoints, user.username, it)
+                val studentDTO = StudentDTO(user.firstName, user.lastName, user.email, coursePoints.toBigDecimal().setScale(2, RoundingMode.HALF_UP).toDouble(), user.username, it)
                 studentDTO
             } else {
                 StudentDTO(registrationId = it)