-
Type:
Bug
-
Status: Closed (View Workflow)
-
Priority:
Major
-
Resolution: Fixed
-
Affects Version/s: 11.1.2
-
Fix Version/s: 11.2.1
-
Component/s: None
-
Labels:None
-
Funded by:
A RS can occur when a coach checks a checkbox in assessment tool:
stack of 1.cause::java.lang.NullPointerException: null -> at org.olat.course.nodes.cl.manager.CheckboxManagerImpl.loadCheck(CheckboxManagerImpl.java:349) at org.olat.course.nodes.cl.manager.CheckboxManagerImpl.check(CheckboxManagerImpl.java:309) at org.olat.course.nodes.cl.ui.AssessedIdentityCheckListController.doSave(AssessedIdentityCheckListController.java:279) at org.olat.course.nodes.cl.ui.AssessedIdentityCheckListController.formOK(AssessedIdentityCheckListController.java:193)
This patch fixes that:
diff --git a/src/main/java/org/olat/course/nodes/cl/manager/CheckboxManagerImpl.java b/src/main/java/org/olat/course/nodes/cl/manager/CheckboxManagerImpl.java --- a/src/main/java/org/olat/course/nodes/cl/manager/CheckboxManagerImpl.java +++ b/src/main/java/org/olat/course/nodes/cl/manager/CheckboxManagerImpl.java @@ -343,13 +343,15 @@ sb.append("select check from clcheck as check") .append(" where check.identity.key=:identityKey and check.checkbox.key=:checkboxKey"); - List<DBCheck> checks = dbInstance.getCurrentEntityManager() + List<DBCheck> checks = null; + if (checkbox != null) + checks = dbInstance.getCurrentEntityManager() .createQuery(sb.toString(), DBCheck.class) .setParameter("identityKey", identity.getKey()) .setParameter("checkboxKey", checkbox.getKey()) .getResultList(); - if(checks.isEmpty()) { + if(checks == null || checks.isEmpty()) { return null; } return checks.get(0);