summaryrefslogtreecommitdiff
path: root/program/include/rcmail_output_html.php
diff options
context:
space:
mode:
authorAndy Wermke <andy@dev.next-step-software.com>2013-04-08 14:34:17 +0200
committerAndy Wermke <andy@dev.next-step-software.com>2013-04-08 14:34:17 +0200
commitde6585eb771ef6a7e7a98c9a9b011da34cbf05b0 (patch)
treeb596b5ef85c740793e5f121960b2b2d11d46060f /program/include/rcmail_output_html.php
parentbf3084b608b8a1036f0b091140350813d54c3d0d (diff)
parent58e3a504b98f68595151fa908536b1e35b043b76 (diff)
Merge branch 'master' of git.next-step-software.com:/git-pub/roundcubemail
Diffstat (limited to 'program/include/rcmail_output_html.php')
-rw-r--r--program/include/rcmail_output_html.php30
1 files changed, 15 insertions, 15 deletions
diff --git a/program/include/rcmail_output_html.php b/program/include/rcmail_output_html.php
index 772bdccf7..0fba66080 100644
--- a/program/include/rcmail_output_html.php
+++ b/program/include/rcmail_output_html.php
@@ -759,14 +759,11 @@ class rcmail_output_html extends rcmail_output
/**
- * Parses expression and replaces variables
- *
+ * Parse & evaluate a given expression and return its result.
* @param string Expression statement
- * @return string Expression value
*/
- protected function parse_expression($expression)
- {
- return preg_replace(
+ protected function eval_expression ($expression) {
+ $expression = preg_replace(
array(
'/session:([a-z0-9_]+)/i',
'/config:([a-z0-9_]+)(:([a-z0-9_]+))?/i',
@@ -785,16 +782,19 @@ class rcmail_output_html extends rcmail_output
"\$browser->{'\\1'}",
$this->template_name,
),
- $expression);
- }
-
- /**
- * Evaluate a given expression and return its result.
- * @param string Expression statement
- */
- protected function eval_expression ($expression) {
- $expression = $this->parse_expression($expression);
+ $expression
+ );
+
$fn = create_function('$app,$browser,$env', "return ($expression);");
+ if(!$fn) {
+ rcube::raise_error(array(
+ 'code' => 505,
+ 'type' => 'php',
+ 'file' => __FILE__,
+ 'line' => __LINE__,
+ 'message' => "Expression parse error on: ($expression)"), true, false);
+ }
+
return $fn($this->app, $this->browser, $this->env);
}