From 718e98853fcdd927f72521010616631485679974 Mon Sep 17 00:00:00 2001
From: gnaegi <none@none>
Date: Thu, 18 Jul 2013 16:01:08 +0200
Subject: [PATCH] OO-558 show LTI nodes in asessment tool only when score
transfering is enabled, analog to SCORM element
---
.../olat/course/nodes/BasicLTICourseNode.java | 23 +++++++++++++++----
1 file changed, 18 insertions(+), 5 deletions(-)
diff --git a/src/main/java/org/olat/course/nodes/BasicLTICourseNode.java b/src/main/java/org/olat/course/nodes/BasicLTICourseNode.java
index ae186b4dac2..5f3b0a37029 100644
--- a/src/main/java/org/olat/course/nodes/BasicLTICourseNode.java
+++ b/src/main/java/org/olat/course/nodes/BasicLTICourseNode.java
@@ -38,6 +38,7 @@ import org.olat.core.logging.OLATRuntimeException;
import org.olat.core.util.Util;
import org.olat.course.ICourse;
import org.olat.course.assessment.AssessmentManager;
+import org.olat.course.auditing.UserNodeAuditManager;
import org.olat.course.editor.CourseEditorEnv;
import org.olat.course.editor.NodeEditController;
import org.olat.course.editor.StatusDescription;
@@ -263,12 +264,18 @@ public class BasicLTICourseNode extends AbstractAccessableCourseNode implements
@Override
public boolean hasAttemptsConfigured() {
- return false;
+ // having score defined means the node is assessable
+ ModuleConfiguration config = getModuleConfiguration();
+ Boolean score = config.getBooleanEntry(CONFIG_KEY_HAS_SCORE_FIELD);
+ return (score == null) ? false : score.booleanValue();
}
@Override
public boolean hasDetails() {
- return true;
+ // having score defined means the node is assessable
+ ModuleConfiguration config = getModuleConfiguration();
+ Boolean score = config.getBooleanEntry(CONFIG_KEY_HAS_SCORE_FIELD);
+ return (score == null) ? false : score.booleanValue();
}
@Override
@@ -278,7 +285,10 @@ public class BasicLTICourseNode extends AbstractAccessableCourseNode implements
@Override
public boolean isEditableConfigured() {
- return true;
+ // having score defined means the node is assessable
+ ModuleConfiguration config = getModuleConfiguration();
+ Boolean score = config.getBooleanEntry(CONFIG_KEY_HAS_SCORE_FIELD);
+ return (score == null) ? false : score.booleanValue();
}
@Override
@@ -314,8 +324,11 @@ public class BasicLTICourseNode extends AbstractAccessableCourseNode implements
@Override
public String getUserLog(UserCourseEnvironment userCourseEnvironment) {
- // TODO Auto-generated method stub
- return null;
+ // having score defined means the node is assessable
+ UserNodeAuditManager am = userCourseEnvironment.getCourseEnvironment().getAuditManager();
+ Identity mySelf = userCourseEnvironment.getIdentityEnvironment().getIdentity();
+ String logValue = am.getUserNodeLog(this, mySelf);
+ return logValue;
}
@Override
--
GitLab