From bafab0febc3b988f4a22ecefe446a462e81a2a23 Mon Sep 17 00:00:00 2001
From: srosse <none@none>
Date: Fri, 17 May 2013 11:31:31 +0200
Subject: [PATCH] OO-609: if the evaluation raise an error, denied access
instead of red screen and log the condition, it's not the fault of the user
---
.../course/condition/interpreter/ConditionInterpreter.java | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/main/java/org/olat/course/condition/interpreter/ConditionInterpreter.java b/src/main/java/org/olat/course/condition/interpreter/ConditionInterpreter.java
index 235f35d6a81..9b8a681daf4 100644
--- a/src/main/java/org/olat/course/condition/interpreter/ConditionInterpreter.java
+++ b/src/main/java/org/olat/course/condition/interpreter/ConditionInterpreter.java
@@ -385,8 +385,8 @@ public class ConditionInterpreter {
// if not null then: ok = evaluateCondition(Expression cachedExpression)
ok = doEvaluateCondition(condition);
} catch (ParseException e) {
- log.info("ParseException in evaluateCondition:" + e);
- throw new AssertException("parse error in:: " + condition + " exception=" + e.getMessage());
+ log.error("ParseException in evaluateCondition:" + condition, e);
+ ok = false;
}
return ok;
}
--
GitLab