From a0ea4ac324c0eb0570c0217d07dbf7da7b51cca9 Mon Sep 17 00:00:00 2001
From: srosse <none@none>
Date: Thu, 11 Aug 2016 16:58:17 +0200
Subject: [PATCH] OO-2143: fix RS if a user copy a course with an empty check
list course element (patch S. Clemenz)
---
.../olat/course/nodes/CheckListCourseNode.java | 16 +++++++++-------
1 file changed, 9 insertions(+), 7 deletions(-)
diff --git a/src/main/java/org/olat/course/nodes/CheckListCourseNode.java b/src/main/java/org/olat/course/nodes/CheckListCourseNode.java
index 27dc734b4a7..8a3a655b44f 100644
--- a/src/main/java/org/olat/course/nodes/CheckListCourseNode.java
+++ b/src/main/java/org/olat/course/nodes/CheckListCourseNode.java
@@ -644,14 +644,16 @@ public class CheckListCourseNode extends AbstractAccessableCourseNode implements
ModuleConfiguration config = getModuleConfiguration();
CheckboxManager checkboxManager = CoreSpringFactory.getImpl(CheckboxManager.class);
CheckboxList list = (CheckboxList)config.get(CONFIG_KEY_CHECKBOX);
- for(Checkbox checkbox:list.getList()) {
- String sourceId = checkbox.getCheckboxId();
- String targetId = envMapper.getTargetUniqueKey(getIdent(), sourceId);
- if(targetId == null) {
- targetId = UUID.randomUUID().toString();
- envMapper.addUniqueKeyPair(getIdent(), sourceId, targetId);
+ if (list !=null && list.getList().size() > 0) {
+ for(Checkbox checkbox:list.getList()) {
+ String sourceId = checkbox.getCheckboxId();
+ String targetId = envMapper.getTargetUniqueKey(getIdent(), sourceId);
+ if(targetId == null) {
+ targetId = UUID.randomUUID().toString();
+ envMapper.addUniqueKeyPair(getIdent(), sourceId, targetId);
+ }
+ checkbox.setCheckboxId(targetId);
}
- checkbox.setCheckboxId(targetId);
}
// the ident of the course node is the same
--
GitLab