refactored CJavascriptExpression->_construct to get cleaner code

This commit is contained in:
mdomba (mdlap)
2012-08-07 18:35:06 +02:00
parent 65d5fdb7bd
commit 5973d891e0
10 changed files with 22 additions and 25 deletions

View File

@@ -25,12 +25,18 @@ class CJavaScriptExpression
/**
* @param string $code a javascript expression that is to be wrapped by this object
* @param boolean $keepBC if true removes "js:" from $code if it starts with it. Defaults to false.
* This parameter is used to keep BC (backward compatibility) with previous versions, when
* javascript expressions where prefixed with "js:". This functionality was deprecated in 1.1.11.
* in favor of this class for better security.
* @throws CException if argument is not a string
*/
public function __construct($code)
public function __construct($code, $keepBC=false)
{
if(!is_string($code))
throw new CException('Value passed to CJavaScriptExpression should be a string.');
if($keepBC && strpos($code, 'js:')===0)
$code=substr($code,3);
$this->code=$code;
}