UI change: show group member role in peer evaluation view
This commit is contained in:
parent
ab8cb36fd8
commit
6b80457da5
|
@ -26,6 +26,7 @@ import com.jaytux.grader.viewmodel.SoloAssignmentState
|
||||||
import com.mohamedrejeb.richeditor.model.rememberRichTextState
|
import com.mohamedrejeb.richeditor.model.rememberRichTextState
|
||||||
import com.mohamedrejeb.richeditor.ui.material3.OutlinedRichTextEditor
|
import com.mohamedrejeb.richeditor.ui.material3.OutlinedRichTextEditor
|
||||||
import kotlinx.datetime.LocalDateTime
|
import kotlinx.datetime.LocalDateTime
|
||||||
|
import org.jetbrains.exposed.sql.transactions.inTopLevelTransaction
|
||||||
|
|
||||||
@Composable
|
@Composable
|
||||||
fun GroupAssignmentView(state: GroupAssignmentState) {
|
fun GroupAssignmentView(state: GroupAssignmentState) {
|
||||||
|
@ -585,9 +586,21 @@ fun PeerEvaluationView(state: PeerEvaluationState) {
|
||||||
}
|
}
|
||||||
MeasuredLazyColumn(key = idx) {
|
MeasuredLazyColumn(key = idx) {
|
||||||
measuredItem { HLine() }
|
measuredItem { HLine() }
|
||||||
items(current.students) { (from, glob, map) ->
|
items(current.students) { (from, role, glob, map) ->
|
||||||
Row(Modifier.height(cellSize)) {
|
Row(Modifier.height(cellSize)) {
|
||||||
Text(from.name, Modifier.width(textLenMeasured.dp).align(Alignment.CenterVertically))
|
Column(Modifier.width(textLenMeasured.dp).align(Alignment.CenterVertically)) {
|
||||||
|
Text(from.name, Modifier.width(textLenMeasured.dp))
|
||||||
|
role?.let { r ->
|
||||||
|
Row {
|
||||||
|
Spacer(Modifier.width(10.dp))
|
||||||
|
Text(
|
||||||
|
r,
|
||||||
|
style = MaterialTheme.typography.bodySmall,
|
||||||
|
fontStyle = FontStyle.Italic
|
||||||
|
)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
LazyRow(state = horScroll) {
|
LazyRow(state = horScroll) {
|
||||||
item { VLine() }
|
item { VLine() }
|
||||||
items(map) { (to, entry) ->
|
items(map) { (to, entry) ->
|
||||||
|
|
|
@ -722,7 +722,7 @@ class SoloAssignmentState(val assignment: SoloAssignment) {
|
||||||
|
|
||||||
class PeerEvaluationState(val evaluation: PeerEvaluation) {
|
class PeerEvaluationState(val evaluation: PeerEvaluation) {
|
||||||
data class Student2StudentEntry(val grade: String, val feedback: String)
|
data class Student2StudentEntry(val grade: String, val feedback: String)
|
||||||
data class StudentEntry(val student: Student, val global: Student2StudentEntry?, val others: List<Pair<Student, Student2StudentEntry?>>)
|
data class StudentEntry(val student: Student, val role: String?, val global: Student2StudentEntry?, val others: List<Pair<Student, Student2StudentEntry?>>)
|
||||||
data class GroupEntry(val group: Group, val content: String, val students: List<StudentEntry>)
|
data class GroupEntry(val group: Group, val content: String, val students: List<StudentEntry>)
|
||||||
val editionCourse = transaction { evaluation.edition.course to evaluation.edition }
|
val editionCourse = transaction { evaluation.edition.course to evaluation.edition }
|
||||||
private val _name = mutableStateOf(evaluation.name); val name = _name.immutable()
|
private val _name = mutableStateOf(evaluation.name); val name = _name.immutable()
|
||||||
|
@ -739,6 +739,9 @@ class PeerEvaluationState(val evaluation: PeerEvaluation) {
|
||||||
}
|
}
|
||||||
|
|
||||||
val students = group.students.map { from ->
|
val students = group.students.map { from ->
|
||||||
|
val role = GroupStudents.selectAll().where { (GroupStudents.studentId eq from.id) and (GroupStudents.groupId eq group.id) }.firstOrNull()?.let {
|
||||||
|
it[GroupStudents.role]
|
||||||
|
}
|
||||||
val s2g = StudentToGroupEvaluation.selectAll().where {
|
val s2g = StudentToGroupEvaluation.selectAll().where {
|
||||||
(StudentToGroupEvaluation.peerEvaluationId eq evaluation.id) and
|
(StudentToGroupEvaluation.peerEvaluationId eq evaluation.id) and
|
||||||
(StudentToGroupEvaluation.studentId eq from.id)
|
(StudentToGroupEvaluation.studentId eq from.id)
|
||||||
|
@ -758,7 +761,7 @@ class PeerEvaluationState(val evaluation: PeerEvaluation) {
|
||||||
other to eval
|
other to eval
|
||||||
}
|
}
|
||||||
|
|
||||||
StudentEntry(from, s2g, others)
|
StudentEntry(from, role, s2g, others)
|
||||||
}
|
}
|
||||||
|
|
||||||
GroupEntry(group, globalNotes ?: "", students)
|
GroupEntry(group, globalNotes ?: "", students)
|
||||||
|
@ -767,7 +770,6 @@ class PeerEvaluationState(val evaluation: PeerEvaluation) {
|
||||||
|
|
||||||
fun upsertGroupFeedback(group: Group, feedback: String) {
|
fun upsertGroupFeedback(group: Group, feedback: String) {
|
||||||
transaction {
|
transaction {
|
||||||
println("Upserting group-level notes for ${group.name} (${evaluation.name})")
|
|
||||||
PeerEvaluationContents.upsert {
|
PeerEvaluationContents.upsert {
|
||||||
it[peerEvaluationId] = evaluation.id
|
it[peerEvaluationId] = evaluation.id
|
||||||
it[groupId] = group.id
|
it[groupId] = group.id
|
||||||
|
@ -780,7 +782,6 @@ class PeerEvaluationState(val evaluation: PeerEvaluation) {
|
||||||
fun upsertIndividualFeedback(from: Student, to: Student?, grade: String, feedback: String) {
|
fun upsertIndividualFeedback(from: Student, to: Student?, grade: String, feedback: String) {
|
||||||
transaction {
|
transaction {
|
||||||
to?.let {
|
to?.let {
|
||||||
println("Upserting individual feedback for ${from.name} -> ${to.name} (${evaluation.name})")
|
|
||||||
StudentToStudentEvaluation.upsert {
|
StudentToStudentEvaluation.upsert {
|
||||||
it[peerEvaluationId] = evaluation.id
|
it[peerEvaluationId] = evaluation.id
|
||||||
it[studentIdFrom] = from.id
|
it[studentIdFrom] = from.id
|
||||||
|
@ -794,10 +795,6 @@ class PeerEvaluationState(val evaluation: PeerEvaluation) {
|
||||||
it[this.grade] = grade
|
it[this.grade] = grade
|
||||||
it[this.note] = feedback
|
it[this.note] = feedback
|
||||||
}
|
}
|
||||||
|
|
||||||
if(to == null) {
|
|
||||||
println("Upserting group-level feedback for ${from.name} ${from.id.value} (${evaluation.name} ${evaluation.id.value})")
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
contents.refresh()
|
contents.refresh()
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue