From 84d8d46a98edce89c5ecd6b8be66c71e8c52ba11 Mon Sep 17 00:00:00 2001 From: austin48 Date: Wed, 10 Jan 2024 15:07:17 -1000 Subject: [PATCH] ISSUE-42 NullPointerException deleting scores --- .../tool/gradebook/ui/AssignmentDetailsBean.java | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/app/ui/src/java/org/sakaiproject/tool/gradebook/ui/AssignmentDetailsBean.java b/app/ui/src/java/org/sakaiproject/tool/gradebook/ui/AssignmentDetailsBean.java index 39f4b766..69a7fdec 100644 --- a/app/ui/src/java/org/sakaiproject/tool/gradebook/ui/AssignmentDetailsBean.java +++ b/app/ui/src/java/org/sakaiproject/tool/gradebook/ui/AssignmentDetailsBean.java @@ -599,9 +599,12 @@ public void processUpdateScores(final ActionEvent event) { String score; try { for (final AssignmentGradeRecord agr : this.updatedGradeRecords) { - score = getGradeEntryByPoints() ? agr.getPointsEarned().toString() : - getGradeEntryByPercent() ? agr.getPercentEarned().toString() : - getGradeEntryByLetter() ? agr.getLetterEarned() : null; + String pointsEarned = agr.getPointsEarned() != null ? agr.getPointsEarned().toString() : null; + String percentEarned = agr.getPercentEarned() != null ? agr.getPercentEarned().toString() : null; + String letterEarned = agr.getLetterEarned() != null ? agr.getLetterEarned() : null; + score = getGradeEntryByPoints() ? pointsEarned : + getGradeEntryByPercent() ? percentEarned : + getGradeEntryByLetter() ? letterEarned : null; getGradebookBean().postEvent("gradebook.updateItemScore", "/gradebook/" + getGradebookUid() + "/" + agr.getAssignment().getName() + "/" + agr.getStudentId() + "/" + score + "/" + getAuthzLevel(), true);