Index: lams_central/src/java/org/lamsfoundation/lams/authoring/LearningDesignValidator.java =================================================================== diff -u -r6df373e1d5922242a5ac307be415bf4f7a0b5214 -rb994875ec6fda8ed9af97c5e691b7ff555811567 --- lams_central/src/java/org/lamsfoundation/lams/authoring/LearningDesignValidator.java (.../LearningDesignValidator.java) (revision 6df373e1d5922242a5ac307be415bf4f7a0b5214) +++ lams_central/src/java/org/lamsfoundation/lams/authoring/LearningDesignValidator.java (.../LearningDesignValidator.java) (revision b994875ec6fda8ed9af97c5e691b7ff555811567) @@ -12,6 +12,8 @@ import java.util.Iterator; import java.util.Set; import java.util.Vector; + +import org.lamsfoundation.lams.util.MessageService; import org.lamsfoundation.lams.util.wddx.FlashMessage; /** @@ -32,14 +34,16 @@ protected LearningDesignDAO learningDesignDAO = null; private Vector listOfValidationErrorDTOs = null; private FlashMessage flashMessage; + protected MessageService messageService; /* * Default constructor * initialises the list of validation messages. */ - public LearningDesignValidator(LearningDesignDAO learningDesignDAO) + public LearningDesignValidator(LearningDesignDAO learningDesignDAO, MessageService messageService) { this.learningDesignDAO = learningDesignDAO; + this.messageService = messageService; listOfValidationErrorDTOs = new Vector(); } @@ -200,11 +204,15 @@ Iterator activityIterator = activities.iterator(); while (activityIterator.hasNext()) { + String errorMessage = null; Activity activity = (Activity)activityIterator.next(); checkIfGroupingRequired(activity); validateGroupingIfGroupingIsApplied(activity); validateOptionalActivity(activity); - validateOptionsActivityOrderId(activity); + validateOptionsActivityOrderId(activity); + if((errorMessage = activity.validateActivity()) != null) { + listOfValidationErrorDTOs.add(new ValidationErrorDTO(errorMessage, activity.getActivityUIID())); + } } }