RED

refs 2490

org.fortiss.tooling.kernel/trunk/src/org/fortiss/tooling/kernel/extension/ITutorialStepCompletionChecker.java

@@ -17,11 +17,15 @@ $Id: codetemplates.xml 1 2011-01-01 00:00:01Z hoelzl $
 +--------------------------------------------------------------------------*/
 package org.fortiss.tooling.kernel.extension;
 
+import java.util.function.Predicate;
+
 import org.eclipse.emf.ecore.EObject;
 
 /**
  * Instances of this class provide completion checkers for tutorial steps.
  * 
+ * TODO(VA) Is really an interface needed for this? {@link Predicate} would do the job...
+ * 
  * @author hoelzl
  * @author $Author: hoelzl $
  * @version $Rev: 18709 $

org.fortiss.tooling.kernel/trunk/src/org/fortiss/tooling/kernel/extension/data/TutorialAtomicStep.java

@@ -17,6 +17,8 @@ $Id: codetemplates.xml 1 2011-01-01 00:00:01Z hoelzl $
 +--------------------------------------------------------------------------*/
 package org.fortiss.tooling.kernel.extension.data;
 
+import java.util.function.Predicate;
+
 import org.fortiss.tooling.kernel.extension.ITutorialStepCompletionChecker;
 
 /**
@@ -41,6 +43,9 @@ public abstract class TutorialAtomicStep extends TutorialStepBase {
 	/**
 	 * Returns the completion checker for this tutorial step. Sub-classes MUST NOT return
 	 * <code>null</code>.
+	 * 
+	 * TODO(VA) As mentioned in {@link ITutorialStepCompletionChecker}, wouldn't here returning
+	 * {@link Predicate} be enough?
 	 */
 	public abstract ITutorialStepCompletionChecker getCompletionChecker();
 }

org.fortiss.tooling.kernel/trunk/src/org/fortiss/tooling/kernel/extension/data/TutorialCompositeStep.java

@@ -23,7 +23,7 @@ import java.util.LinkedList;
 import java.util.List;
 
 /**
- * A composite step, which encapsulates other tutorial steps.
+ * A composite tutorial step, i.e., a step which encapsulates other tutorial steps.
  * 
  * @author hoelzl
  * @author $Author: hoelzl $
@@ -34,6 +34,8 @@ public class TutorialCompositeStep extends TutorialStepBase {
 	/** The list of contained steps. */
 	private final List<TutorialStepBase> steps = new LinkedList<TutorialStepBase>();
 
+	// TODO(VA) Do we need absolutely a first step? Why isn't there a constructor to build an empty
+	// step first?
 	/** Constructor. */
 	public TutorialCompositeStep(String title, String url, TutorialStepBase firstStep) {
 		super(title, url);