From 22a5fa0cba763bd59f62fe80596b0eac61bf80a6 Mon Sep 17 00:00:00 2001
From: srosse <none@none>
Date: Fri, 5 Jul 2013 13:58:04 +0200
Subject: [PATCH] OO-619: fix validation of the height element in layout
 options

---
 .../generic/iframe/DeliveryOptionsConfigurationController.java | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/main/java/org/olat/core/gui/control/generic/iframe/DeliveryOptionsConfigurationController.java b/src/main/java/org/olat/core/gui/control/generic/iframe/DeliveryOptionsConfigurationController.java
index aaf515103b8..8b423d76f8a 100644
--- a/src/main/java/org/olat/core/gui/control/generic/iframe/DeliveryOptionsConfigurationController.java
+++ b/src/main/java/org/olat/core/gui/control/generic/iframe/DeliveryOptionsConfigurationController.java
@@ -325,7 +325,8 @@ public class DeliveryOptionsConfigurationController extends FormBasicController
 			}
 			
 			heightEl.clearError();
-			if(standardModeEl.isSelected(0) || jsOptionEl.isSelected(0)) {
+			if(heightEl.isSelected(0)
+					&& (standardModeEl.isSelected(0) || jsOptionEl.isSelected(0))) {
 				heightEl.setErrorKey("automatic.need.js", null);
 				allOk = false;
 			}
-- 
GitLab