diff --git a/src/main/java/org/olat/modules/portfolio/ui/AssignmentEditController.java b/src/main/java/org/olat/modules/portfolio/ui/AssignmentEditController.java index a107399662ff7448330c21f232d7b47a844e5c42..7682fdd973d0312a5f65c65572e167fdf75dce43 100644 --- a/src/main/java/org/olat/modules/portfolio/ui/AssignmentEditController.java +++ b/src/main/java/org/olat/modules/portfolio/ui/AssignmentEditController.java @@ -220,6 +220,7 @@ public class AssignmentEditController extends FormBasicController { String editPage = Util.getPackageVelocityRoot(getClass()) + "/assignment_select_form.html"; selectFormLayout = FormLayoutContainer.createCustomFormLayout("selectFormLayout", getTranslator(), editPage); selectFormLayout.setLabel("assignment.evaluation.form.entry", null); + selectFormLayout.setMandatory(true); formLayout.add(selectFormLayout); if(formEntry != null) { String displayName = StringHelper.escapeHtml(formEntry.getDisplayname()); @@ -358,6 +359,21 @@ public class AssignmentEditController extends FormBasicController { allOk &= false; } } + + typeEl.clearError(); + selectFormLayout.clearError(); + if(typeEl.isOneSelected()) { + AssignmentType type = AssignmentType.valueOf(typeEl.getSelectedKey()); + if(type == AssignmentType.form) { + if(formEntry == null) { + selectFormLayout.setErrorKey("form.legende.mandatory", null); + allOk &= false; + } + } + } else { + typeEl.setErrorKey("form.legende.mandatory", null); + allOk &= false; + } return allOk & super.validateFormLogic(ureq); }