OO-4207: Do not update fully assessed if conventional course node

src/main/java/org/olat/course/condition/ConditionNodeAccessProvider.java

@@ -83,6 +83,11 @@ public class ConditionNodeAccessProvider implements NodeAccessProvider {
 		return new ConditionCourseTreeModelBuilder(userCourseEnv);
 	}
 
+	@Override
+	public boolean onNodeVisited(CourseNode courseNode, UserCourseEnvironment userCourseEnvironment) {
+		return false;
+	}
+
 	@Override
 	public boolean isAssessmentConfirmationEnabled(CourseNode courseNode, UserCourseEnvironment userCourseEnv) {
 		return false;

src/main/java/org/olat/course/learningpath/manager/LearningPathNodeAccessProvider.java

@@ -34,7 +34,6 @@ import org.olat.course.learningpath.ui.LearningPathNodeConfigController;
 import org.olat.course.learningpath.ui.TabbableLeaningPathNodeConfigController;
 import org.olat.course.nodeaccess.NodeAccessProvider;
 import org.olat.course.nodes.CourseNode;
-import org.olat.course.run.navigation.NodeVisitedListener;
 import org.olat.course.run.userview.CourseTreeModelBuilder;
 import org.olat.course.run.userview.UserCourseEnvironment;
 import org.olat.course.tree.CourseEditorTreeModel;
@@ -51,7 +50,7 @@ import org.springframework.stereotype.Service;
  *
  */
 @Service
-public class LearningPathNodeAccessProvider implements NodeAccessProvider, NodeVisitedListener {
+public class LearningPathNodeAccessProvider implements NodeAccessProvider {
 
 	public static final String TYPE = "learningpath";
 	

src/main/java/org/olat/course/nodeaccess/NodeAccessProvider.java

@@ -44,6 +44,8 @@ public interface NodeAccessProvider extends NodeAccessProviderIdentifier {
 
 	public CourseTreeModelBuilder getCourseTreeModelBuilder(UserCourseEnvironment userCourseEnv);
 
+	public boolean onNodeVisited(CourseNode courseNode, UserCourseEnvironment userCourseEnvironment);
+
 	public boolean isAssessmentConfirmationEnabled(CourseNode courseNode, UserCourseEnvironment userCourseEnv);
 
 	public void onAssessmentConfirmed(CourseNode courseNode, UserCourseEnvironment userCourseEnv);

src/main/java/org/olat/course/nodeaccess/manager/NodeAccessServiceImpl.java

@@ -31,6 +31,7 @@ import org.olat.course.nodeaccess.NodeAccessProviderIdentifier;
 import org.olat.course.nodeaccess.NodeAccessService;
 import org.olat.course.nodeaccess.NodeAccessType;
 import org.olat.course.nodes.CourseNode;
+import org.olat.course.run.navigation.NodeVisitedListener;
 import org.olat.course.run.userview.CourseTreeModelBuilder;
 import org.olat.course.run.userview.UserCourseEnvironment;
 import org.olat.course.tree.CourseEditorTreeModel;
@@ -46,7 +47,7 @@ import org.springframework.stereotype.Service;
  *
  */
 @Service
-public class NodeAccessServiceImpl implements NodeAccessService {
+public class NodeAccessServiceImpl implements NodeAccessService, NodeVisitedListener {
 
 	private static final Logger log = Tracing.createLoggerFor(NodeAccessServiceImpl.class);
 
@@ -109,4 +110,10 @@ public class NodeAccessServiceImpl implements NodeAccessService {
 		getNodeAccessProvider(type).onCompletionUpdate(courseNode, userCourseEnv, completion, status, by);
 	}
 
+	@Override
+	public boolean onNodeVisited(CourseNode courseNode, UserCourseEnvironment userCourseEnvironment) {
+		NodeAccessType type = NodeAccessType.of(userCourseEnvironment);
+		return getNodeAccessProvider(type).onNodeVisited(courseNode, userCourseEnvironment);
+	}
+
 }