pradosoft / prado

framework/Web/THttpResponseAdapter.php

Doc Comments +1 added lines

@@ -31,6 +31,7 @@
 	/**
 	 * Constructor. Attach a response to be adapted.
 	 * @param THttpResponse the response object the adapter is to attach to.
+	 * @param THttpResponse $response
 	 */
 	public function __construct($response)
 	{

Spacing +2 added lines, -2 removed lines

@@ -34,7 +34,7 @@ 
 	 */
 	public function __construct($response)
 	{
-		$this->_response=$response;
+		$this->_response = $response;
 	}
 
 	/**
@@ -71,7 +71,7 @@ 
 	 */
 	public function createNewHtmlWriter($type, $writer)
 	{
-		return $this->_response->createNewHtmlWriter($type,$writer);
+		return $this->_response->createNewHtmlWriter($type, $writer);
 	}
 }
 

framework/Web/THttpSession.php

Doc Comments +3 added lines

@@ -265,6 +265,7 @@ 
 	 * If true, make sure the methods {@link _open}, {@link _close}, {@link _read},
 	 * {@link _write}, {@link _destroy}, and {@link _gc} are overridden in child
 	 * class, because they will be used as the callback handlers.
+	 * @param boolean $value
 	 */
 	public function setUseCustomStorage($value)
 	{
@@ -527,6 +528,7 @@ 
 	 * Returns the session variable value with the session variable name.
 	 * This method is exactly the same as {@link offsetGet}.
 	 * @param mixed the session variable name
+	 * @param string $key
 	 * @return mixed the session variable value, null if no such variable exists
 	 */
 	public function itemAt($key)
@@ -539,6 +541,7 @@ 
 	 * Note, if the specified name already exists, the old value will be removed first.
 	 * @param mixed session variable name
 	 * @param mixed session variable value
+	 * @param string $key
 	 */
 	public function add($key,$value)
 	{

Indentation +2 added lines, -2 removed lines

@@ -306,10 +306,10 @@
 		{
 			$value=TPropertyValue::ensureEnum($value,'THttpSessionCookieMode');
 			if($value===THttpSessionCookieMode::None) 
-      {
+	  {
 				ini_set('session.use_cookies','0');
 			  ini_set('session.use_only_cookies','0');
-      }
+	  }
 			else if($value===THttpSessionCookieMode::Allow)
 			{
 				ini_set('session.use_cookies','1');

Spacing +76 added lines, -76 removed lines

@@ -67,24 +67,24 @@ 
  * @package System.Web
  * @since 3.0
  */
-class THttpSession extends TApplicationComponent implements IteratorAggregate,ArrayAccess,Countable,IModule
+class THttpSession extends TApplicationComponent implements IteratorAggregate, ArrayAccess, Countable, IModule
 {
 	/**
 	 * @var boolean whether this module has been initialized
 	 */
-	private $_initialized=false;
+	private $_initialized = false;
 	/**
 	 * @var boolean whether the session has started
 	 */
-	private $_started=false;
+	private $_started = false;
 	/**
 	 * @var boolean whether the session should be started when the module is initialized
 	 */
-	private $_autoStart=false;
+	private $_autoStart = false;
 	/**
 	 * @var THttpCookie cookie to be used to store session ID and other data
 	 */
-	private $_cookie=null;
+	private $_cookie = null;
 	/**
 	 * @var string module id
 	 */
@@ -92,7 +92,7 @@ 
 	/**
 	 * @var boolean
 	 */
-	private $_customStorage=false;
+	private $_customStorage = false;
 
 	/**
 	 * @return string id of this module
@@ -107,7 +107,7 @@ 
 	 */
 	public function setID($value)
 	{
-		$this->_id=$value;
+		$this->_id = $value;
 	}
 
 	/**
@@ -118,9 +118,9 @@ 
 	 */
 	public function init($config)
 	{
-		if($this->_autoStart)
+		if ($this->_autoStart)
 			$this->open();
-		$this->_initialized=true;
+		$this->_initialized = true;
 		$this->getApplication()->setSession($this);
 		register_shutdown_function(array($this, "close"));
 	}
@@ -130,15 +130,15 @@ 
 	 */
 	public function open()
 	{
-		if(!$this->_started)
+		if (!$this->_started)
 		{
-			if($this->_customStorage)
-				session_set_save_handler(array($this,'_open'),array($this,'_close'),array($this,'_read'),array($this,'_write'),array($this,'_destroy'),array($this,'_gc'));
-			if($this->_cookie!==null)
-				session_set_cookie_params($this->_cookie->getExpire(),$this->_cookie->getPath(),$this->_cookie->getDomain(),$this->_cookie->getSecure(),$this->_cookie->getHttpOnly());
-			if(ini_get('session.auto_start')!=='1')
+			if ($this->_customStorage)
+				session_set_save_handler(array($this, '_open'), array($this, '_close'), array($this, '_read'), array($this, '_write'), array($this, '_destroy'), array($this, '_gc'));
+			if ($this->_cookie !== null)
+				session_set_cookie_params($this->_cookie->getExpire(), $this->_cookie->getPath(), $this->_cookie->getDomain(), $this->_cookie->getSecure(), $this->_cookie->getHttpOnly());
+			if (ini_get('session.auto_start') !== '1')
 				session_start();
-			$this->_started=true;
+			$this->_started = true;
 		}
 	}
 
@@ -147,10 +147,10 @@ 
 	 */
 	public function close()
 	{
-		if($this->_started)
+		if ($this->_started)
 		{
 			session_write_close();
-			$this->_started=false;
+			$this->_started = false;
 		}
 	}
 
@@ -159,10 +159,10 @@ 
 	 */
 	public function destroy()
 	{
-		if($this->_started)
+		if ($this->_started)
 		{
 			session_destroy();
-			$this->_started=false;
+			$this->_started = false;
 		}
 	}
 
@@ -173,7 +173,7 @@ 
 	 * @return string old session id
 	 * @link http://php.net/manual/en/function.session-regenerate-id.php
 	 */
-	public function regenerate($deleteOld=false)
+	public function regenerate($deleteOld = false)
 	{
 		$old = $this->getSessionID();
 		session_regenerate_id($deleteOld);
@@ -202,7 +202,7 @@ 
 	 */
 	public function setSessionID($value)
 	{
-		if($this->_started)
+		if ($this->_started)
 			throw new TInvalidOperationException('httpsession_sessionid_unchangeable');
 		else
 			session_id($value);
@@ -222,12 +222,12 @@ 
 	 */
 	public function setSessionName($value)
 	{
-		if($this->_started)
+		if ($this->_started)
 			throw new TInvalidOperationException('httpsession_sessionname_unchangeable');
-		else if(ctype_alnum($value))
+		else if (ctype_alnum($value))
 			session_name($value);
 		else
-			throw new TInvalidDataValueException('httpsession_sessionname_invalid',$value);
+			throw new TInvalidDataValueException('httpsession_sessionname_invalid', $value);
 	}
 
 	/**
@@ -244,12 +244,12 @@ 
 	 */
 	public function setSavePath($value)
 	{
-		if($this->_started)
+		if ($this->_started)
 			throw new TInvalidOperationException('httpsession_savepath_unchangeable');
-		else if(is_dir($value))
+		else if (is_dir($value))
 			session_save_path($value);
 		else
-			throw new TInvalidDataValueException('httpsession_savepath_invalid',$value);
+			throw new TInvalidDataValueException('httpsession_savepath_invalid', $value);
 	}
 
 	/**
@@ -268,7 +268,7 @@ 
 	 */
 	public function setUseCustomStorage($value)
 	{
-		$this->_customStorage=TPropertyValue::ensureBoolean($value);
+		$this->_customStorage = TPropertyValue::ensureBoolean($value);
 	}
 
 	/**
@@ -276,8 +276,8 @@ 
 	 */
 	public function getCookie()
 	{
-		if($this->_cookie===null)
-			$this->_cookie=new THttpCookie($this->getSessionName(),$this->getSessionID());
+		if ($this->_cookie === null)
+			$this->_cookie = new THttpCookie($this->getSessionName(), $this->getSessionID());
 		return $this->_cookie;
 	}
 
@@ -286,9 +286,9 @@ 
 	 */
 	public function getCookieMode()
 	{
-		if(ini_get('session.use_cookies')==='0')
+		if (ini_get('session.use_cookies') === '0')
 			return THttpSessionCookieMode::None;
-		else if(ini_get('session.use_only_cookies')==='0')
+		else if (ini_get('session.use_only_cookies') === '0')
 			return THttpSessionCookieMode::Allow;
 		else
 			return THttpSessionCookieMode::Only;
@@ -300,25 +300,25 @@ 
 	 */
 	public function setCookieMode($value)
 	{
-		if($this->_started)
+		if ($this->_started)
 			throw new TInvalidOperationException('httpsession_cookiemode_unchangeable');
 		else
 		{
-			$value=TPropertyValue::ensureEnum($value,'THttpSessionCookieMode');
-			if($value===THttpSessionCookieMode::None) 
+			$value = TPropertyValue::ensureEnum($value, 'THttpSessionCookieMode');
+			if ($value === THttpSessionCookieMode::None) 
       {
-				ini_set('session.use_cookies','0');
-			  ini_set('session.use_only_cookies','0');
+				ini_set('session.use_cookies', '0');
+			  ini_set('session.use_only_cookies', '0');
       }
-			else if($value===THttpSessionCookieMode::Allow)
+			else if ($value === THttpSessionCookieMode::Allow)
 			{
-				ini_set('session.use_cookies','1');
-				ini_set('session.use_only_cookies','0');
+				ini_set('session.use_cookies', '1');
+				ini_set('session.use_only_cookies', '0');
 			}
 			else
 			{
-				ini_set('session.use_cookies','1');
-				ini_set('session.use_only_cookies','1');
+				ini_set('session.use_cookies', '1');
+				ini_set('session.use_only_cookies', '1');
 				ini_set('session.use_trans_sid', 0);
 			}
 		}
@@ -338,10 +338,10 @@ 
 	 */
 	public function setAutoStart($value)
 	{
-		if($this->_initialized)
+		if ($this->_initialized)
 			throw new TInvalidOperationException('httpsession_autostart_unchangeable');
 		else
-			$this->_autoStart=TPropertyValue::ensureBoolean($value);
+			$this->_autoStart = TPropertyValue::ensureBoolean($value);
 	}
 
 	/**
@@ -359,18 +359,18 @@ 
 	 */
 	public function setGCProbability($value)
 	{
-		if($this->_started)
+		if ($this->_started)
 			throw new TInvalidOperationException('httpsession_gcprobability_unchangeable');
 		else
 		{
-			$value=TPropertyValue::ensureInteger($value);
-			if($value>=0 && $value<=100)
+			$value = TPropertyValue::ensureInteger($value);
+			if ($value >= 0 && $value <= 100)
 			{
-				ini_set('session.gc_probability',$value);
-				ini_set('session.gc_divisor','100');
+				ini_set('session.gc_probability', $value);
+				ini_set('session.gc_divisor', '100');
 			}
 			else
-				throw new TInvalidDataValueException('httpsession_gcprobability_invalid',$value);
+				throw new TInvalidDataValueException('httpsession_gcprobability_invalid', $value);
 		}
 	}
 
@@ -379,7 +379,7 @@ 
 	 */
 	public function getUseTransparentSessionID()
 	{
-		return ini_get('session.use_trans_sid')==='1';
+		return ini_get('session.use_trans_sid') === '1';
 	}
 
 	/**
@@ -387,14 +387,14 @@ 
 	 */
 	public function setUseTransparentSessionID($value)
 	{
-		if($this->_started)
+		if ($this->_started)
 			throw new TInvalidOperationException('httpsession_transid_unchangeable');
 		else
 		{
-			$value=TPropertyValue::ensureBoolean($value);
-			if ($value && $this->getCookieMode()==THttpSessionCookieMode::Only)
+			$value = TPropertyValue::ensureBoolean($value);
+			if ($value && $this->getCookieMode() == THttpSessionCookieMode::Only)
 					throw new TInvalidOperationException('httpsession_transid_cookieonly');
-			ini_set('session.use_trans_sid',$value?'1':'0');
+			ini_set('session.use_trans_sid', $value ? '1' : '0');
 		}
 	}
 
@@ -412,10 +412,10 @@ 
 	 */
 	public function setTimeout($value)
 	{
-		if($this->_started)
+		if ($this->_started)
 			throw new TInvalidOperationException('httpsession_maxlifetime_unchangeable');
 		else
-			ini_set('session.gc_maxlifetime',$value);
+			ini_set('session.gc_maxlifetime', $value);
 	}
 
 	/**
@@ -425,7 +425,7 @@ 
 	 * @param string session name
 	 * @return boolean whether session is opened successfully
 	 */
-	public function _open($savePath,$sessionName)
+	public function _open($savePath, $sessionName)
 	{
 		return true;
 	}
@@ -458,7 +458,7 @@ 
 	 * @param string session data
 	 * @return boolean whether session write is successful
 	 */
-	public function _write($id,$data)
+	public function _write($id, $data)
 	{
 		return true;
 	}
@@ -540,9 +540,9 @@ 
 	 * @param mixed session variable name
 	 * @param mixed session variable value
 	 */
-	public function add($key,$value)
+	public function add($key, $value)
 	{
-		$_SESSION[$key]=$value;
+		$_SESSION[$key] = $value;
 	}
 
 	/**
@@ -552,9 +552,9 @@ 
 	 */
 	public function remove($key)
 	{
-		if(isset($_SESSION[$key]))
+		if (isset($_SESSION[$key]))
 		{
-			$value=$_SESSION[$key];
+			$value = $_SESSION[$key];
 			unset($_SESSION[$key]);
 			return $value;
 		}
@@ -567,7 +567,7 @@ 
 	 */
 	public function clear()
 	{
-		foreach(array_keys($_SESSION) as $key)
+		foreach (array_keys($_SESSION) as $key)
 			unset($_SESSION[$key]);
 	}
 
@@ -613,9 +613,9 @@ 
 	 * @param integer the offset to set element
 	 * @param mixed the element value
 	 */
-	public function offsetSet($offset,$item)
+	public function offsetSet($offset, $item)
 	{
-		$_SESSION[$offset]=$item;
+		$_SESSION[$offset] = $item;
 	}
 
 	/**
@@ -657,7 +657,7 @@ 
 	 */
 	public function __construct()
 	{
-		$this->_keys=array_keys($_SESSION);
+		$this->_keys = array_keys($_SESSION);
 	}
 
 	/**
@@ -666,7 +666,7 @@ 
 	 */
 	public function rewind()
 	{
-		$this->_key=reset($this->_keys);
+		$this->_key = reset($this->_keys);
 	}
 
 	/**
@@ -686,7 +686,7 @@ 
 	 */
 	public function current()
 	{
-		return isset($_SESSION[$this->_key])?$_SESSION[$this->_key]:null;
+		return isset($_SESSION[$this->_key]) ? $_SESSION[$this->_key] : null;
 	}
 
 	/**
@@ -697,9 +697,9 @@ 
 	{
 		do
 		{
-			$this->_key=next($this->_keys);
+			$this->_key = next($this->_keys);
 		}
-		while(!isset($_SESSION[$this->_key]) && $this->_key!==false);
+		while (!isset($_SESSION[$this->_key]) && $this->_key !== false);
 	}
 
 	/**
@@ -709,7 +709,7 @@ 
 	 */
 	public function valid()
 	{
-		return $this->_key!==false;
+		return $this->_key !== false;
 	}
 }
 
@@ -730,8 +730,8 @@ 
  */
 class THttpSessionCookieMode extends TEnumerable
 {
-	const None='None';
-	const Allow='Allow';
-	const Only='Only';
+	const None = 'None';
+	const Allow = 'Allow';
+	const Only = 'Only';
 }
 

Braces +72 added lines, -61 removed lines

@@ -118,8 +118,9 @@ 
 	 */
 	public function init($config)
 	{
-		if($this->_autoStart)
-			$this->open();
+		if($this->_autoStart) {
+					$this->open();
+		}
 		$this->_initialized=true;
 		$this->getApplication()->setSession($this);
 		register_shutdown_function(array($this, "close"));
@@ -132,12 +133,15 @@ 
 	{
 		if(!$this->_started)
 		{
-			if($this->_customStorage)
-				session_set_save_handler(array($this,'_open'),array($this,'_close'),array($this,'_read'),array($this,'_write'),array($this,'_destroy'),array($this,'_gc'));
-			if($this->_cookie!==null)
-				session_set_cookie_params($this->_cookie->getExpire(),$this->_cookie->getPath(),$this->_cookie->getDomain(),$this->_cookie->getSecure(),$this->_cookie->getHttpOnly());
-			if(ini_get('session.auto_start')!=='1')
-				session_start();
+			if($this->_customStorage) {
+							session_set_save_handler(array($this,'_open'),array($this,'_close'),array($this,'_read'),array($this,'_write'),array($this,'_destroy'),array($this,'_gc'));
+			}
+			if($this->_cookie!==null) {
+							session_set_cookie_params($this->_cookie->getExpire(),$this->_cookie->getPath(),$this->_cookie->getDomain(),$this->_cookie->getSecure(),$this->_cookie->getHttpOnly());
+			}
+			if(ini_get('session.auto_start')!=='1') {
+							session_start();
+			}
 			$this->_started=true;
 		}
 	}
@@ -202,10 +206,11 @@ 
 	 */
 	public function setSessionID($value)
 	{
-		if($this->_started)
-			throw new TInvalidOperationException('httpsession_sessionid_unchangeable');
-		else
-			session_id($value);
+		if($this->_started) {
+					throw new TInvalidOperationException('httpsession_sessionid_unchangeable');
+		} else {
+					session_id($value);
+		}
 	}
 
 	/**
@@ -222,12 +227,13 @@ 
 	 */
 	public function setSessionName($value)
 	{
-		if($this->_started)
-			throw new TInvalidOperationException('httpsession_sessionname_unchangeable');
-		else if(ctype_alnum($value))
-			session_name($value);
-		else
-			throw new TInvalidDataValueException('httpsession_sessionname_invalid',$value);
+		if($this->_started) {
+					throw new TInvalidOperationException('httpsession_sessionname_unchangeable');
+		} else if(ctype_alnum($value)) {
+					session_name($value);
+		} else {
+					throw new TInvalidDataValueException('httpsession_sessionname_invalid',$value);
+		}
 	}
 
 	/**
@@ -244,12 +250,13 @@ 
 	 */
 	public function setSavePath($value)
 	{
-		if($this->_started)
-			throw new TInvalidOperationException('httpsession_savepath_unchangeable');
-		else if(is_dir($value))
-			session_save_path($value);
-		else
-			throw new TInvalidDataValueException('httpsession_savepath_invalid',$value);
+		if($this->_started) {
+					throw new TInvalidOperationException('httpsession_savepath_unchangeable');
+		} else if(is_dir($value)) {
+					session_save_path($value);
+		} else {
+					throw new TInvalidDataValueException('httpsession_savepath_invalid',$value);
+		}
 	}
 
 	/**
@@ -276,8 +283,9 @@ 
 	 */
 	public function getCookie()
 	{
-		if($this->_cookie===null)
-			$this->_cookie=new THttpCookie($this->getSessionName(),$this->getSessionID());
+		if($this->_cookie===null) {
+					$this->_cookie=new THttpCookie($this->getSessionName(),$this->getSessionID());
+		}
 		return $this->_cookie;
 	}
 
@@ -286,12 +294,13 @@ 
 	 */
 	public function getCookieMode()
 	{
-		if(ini_get('session.use_cookies')==='0')
-			return THttpSessionCookieMode::None;
-		else if(ini_get('session.use_only_cookies')==='0')
-			return THttpSessionCookieMode::Allow;
-		else
-			return THttpSessionCookieMode::Only;
+		if(ini_get('session.use_cookies')==='0') {
+					return THttpSessionCookieMode::None;
+		} else if(ini_get('session.use_only_cookies')==='0') {
+					return THttpSessionCookieMode::Allow;
+		} else {
+					return THttpSessionCookieMode::Only;
+		}
 	}
 
 	/**
@@ -300,22 +309,20 @@ 
 	 */
 	public function setCookieMode($value)
 	{
-		if($this->_started)
-			throw new TInvalidOperationException('httpsession_cookiemode_unchangeable');
-		else
+		if($this->_started) {
+					throw new TInvalidOperationException('httpsession_cookiemode_unchangeable');
+		} else
 		{
 			$value=TPropertyValue::ensureEnum($value,'THttpSessionCookieMode');
 			if($value===THttpSessionCookieMode::None) 
       {
 				ini_set('session.use_cookies','0');
 			  ini_set('session.use_only_cookies','0');
-      }
-			else if($value===THttpSessionCookieMode::Allow)
+      } else if($value===THttpSessionCookieMode::Allow)
 			{
 				ini_set('session.use_cookies','1');
 				ini_set('session.use_only_cookies','0');
-			}
-			else
+			} else
 			{
 				ini_set('session.use_cookies','1');
 				ini_set('session.use_only_cookies','1');
@@ -338,10 +345,11 @@ 
 	 */
 	public function setAutoStart($value)
 	{
-		if($this->_initialized)
-			throw new TInvalidOperationException('httpsession_autostart_unchangeable');
-		else
-			$this->_autoStart=TPropertyValue::ensureBoolean($value);
+		if($this->_initialized) {
+					throw new TInvalidOperationException('httpsession_autostart_unchangeable');
+		} else {
+					$this->_autoStart=TPropertyValue::ensureBoolean($value);
+		}
 	}
 
 	/**
@@ -359,18 +367,18 @@ 
 	 */
 	public function setGCProbability($value)
 	{
-		if($this->_started)
-			throw new TInvalidOperationException('httpsession_gcprobability_unchangeable');
-		else
+		if($this->_started) {
+					throw new TInvalidOperationException('httpsession_gcprobability_unchangeable');
+		} else
 		{
 			$value=TPropertyValue::ensureInteger($value);
 			if($value>=0 && $value<=100)
 			{
 				ini_set('session.gc_probability',$value);
 				ini_set('session.gc_divisor','100');
+			} else {
+							throw new TInvalidDataValueException('httpsession_gcprobability_invalid',$value);
 			}
-			else
-				throw new TInvalidDataValueException('httpsession_gcprobability_invalid',$value);
 		}
 	}
 
@@ -387,13 +395,14 @@ 
 	 */
 	public function setUseTransparentSessionID($value)
 	{
-		if($this->_started)
-			throw new TInvalidOperationException('httpsession_transid_unchangeable');
-		else
+		if($this->_started) {
+					throw new TInvalidOperationException('httpsession_transid_unchangeable');
+		} else
 		{
 			$value=TPropertyValue::ensureBoolean($value);
-			if ($value && $this->getCookieMode()==THttpSessionCookieMode::Only)
-					throw new TInvalidOperationException('httpsession_transid_cookieonly');
+			if ($value && $this->getCookieMode()==THttpSessionCookieMode::Only) {
+								throw new TInvalidOperationException('httpsession_transid_cookieonly');
+			}
 			ini_set('session.use_trans_sid',$value?'1':'0');
 		}
 	}
@@ -412,10 +421,11 @@ 
 	 */
 	public function setTimeout($value)
 	{
-		if($this->_started)
-			throw new TInvalidOperationException('httpsession_maxlifetime_unchangeable');
-		else
-			ini_set('session.gc_maxlifetime',$value);
+		if($this->_started) {
+					throw new TInvalidOperationException('httpsession_maxlifetime_unchangeable');
+		} else {
+					ini_set('session.gc_maxlifetime',$value);
+		}
 	}
 
 	/**
@@ -557,9 +567,9 @@ 
 			$value=$_SESSION[$key];
 			unset($_SESSION[$key]);
 			return $value;
+		} else {
+					return null;
 		}
-		else
-			return null;
 	}
 
 	/**
@@ -567,8 +577,9 @@ 
 	 */
 	public function clear()
 	{
-		foreach(array_keys($_SESSION) as $key)
-			unset($_SESSION[$key]);
+		foreach(array_keys($_SESSION) as $key) {
+					unset($_SESSION[$key]);
+		}
 	}
 
 	/**

framework/Web/THttpUtility.php

Doc Comments +1 added lines

@@ -39,6 +39,7 @@
 	 * HTML-decodes a string.
 	 * It is the inverse of {@link htmlEncode}.
 	 * @param string string to be decoded
+	 * @param string $s
 	 * @return string decoded string
 	 */
 	public static function htmlDecode($s)

Indentation +8 added lines, -8 removed lines

@@ -1,13 +1,13 @@
 <?php
 /**
- * THttpUtility class file
- *
- * @author Qiang Xue <qiang.xue@gmail.com>
- * @link https://github.com/pradosoft/prado
- * @copyright Copyright &copy; 2005-2015 The PRADO Group
- * @license https://github.com/pradosoft/prado/blob/master/COPYRIGHT
- * @package System.Web
- */
+	 * THttpUtility class file
+	 *
+	 * @author Qiang Xue <qiang.xue@gmail.com>
+	 * @link https://github.com/pradosoft/prado
+	 * @copyright Copyright &copy; 2005-2015 The PRADO Group
+	 * @license https://github.com/pradosoft/prado/blob/master/COPYRIGHT
+	 * @package System.Web
+	 */
 
 /**
  * THttpUtility class

Spacing +6 added lines, -6 removed lines

@@ -18,9 +18,9 @@ 
  */
 class THttpUtility
 {
-	private static $_encodeTable=array('<'=>'&lt;','>'=>'&gt;','"'=>'&quot;');
-	private static $_decodeTable=array('&lt;'=>'<','&gt;'=>'>','&quot;'=>'"');
-	private static $_stripTable=array('&lt;'=>'','&gt;'=>'','&quot;'=>'');
+	private static $_encodeTable = array('<'=>'&lt;', '>'=>'&gt;', '"'=>'&quot;');
+	private static $_decodeTable = array('&lt;'=>'<', '&gt;'=>'>', '&quot;'=>'"');
+	private static $_stripTable = array('&lt;'=>'', '&gt;'=>'', '&quot;'=>'');
 
 	/**
 	 * HTML-encodes a string.
@@ -32,7 +32,7 @@ 
 	 */
 	public static function htmlEncode($s)
 	{
-		return strtr($s,self::$_encodeTable);
+		return strtr($s, self::$_encodeTable);
 	}
 
 	/**
@@ -43,7 +43,7 @@ 
 	 */
 	public static function htmlDecode($s)
 	{
-		return strtr($s,self::$_decodeTable);
+		return strtr($s, self::$_decodeTable);
 	}
 
 	/**
@@ -54,7 +54,7 @@ 
 	 */
 	public static function htmlStrip($s)
 	{
-		return strtr($s,self::$_stripTable);
+		return strtr($s, self::$_stripTable);
 	}
 }
 

framework/Web/TUrlMapping.php

Doc Comments +4 added lines, -1 removed lines

@@ -96,6 +96,7 @@ 
 	 * Initializes this module.
 	 * This method is required by the IModule interface.
 	 * @param mixed configuration for this module, can be null
+	 * @param TXmlDocument $config
 	 * @throws TConfigurationException if module is configured in the global scope.
 	 */
 	public function init($config)
@@ -606,6 +607,7 @@ 
 
 	/**
 	 * @param string full regular expression mapping pattern.
+	 * @param string $value
 	 */
 	public function setRegularExpression($value)
 	{
@@ -715,6 +717,7 @@ 
 	 * Uses URL pattern (or full regular expression if available) to
 	 * match the given url path.
 	 * @param THttpRequest the request module
+	 * @param THttpRequest $request
 	 * @return array matched parameters, empty if no matches.
 	 */
 	public function getPatternMatches($request)
@@ -785,7 +788,7 @@ 
 	}
 
 	/**
-	 * @return THttpRequestUrlFormat the format of URLs. Defaults to THttpRequestUrlFormat::Get.
+	 * @return string the format of URLs. Defaults to THttpRequestUrlFormat::Get.
 	 */
 	public function getUrlFormat()
 	{

Spacing +159 added lines, -159 removed lines

@@ -70,7 +70,7 @@ 
 	/**
 	 * @var TUrlMappingPattern[] list of patterns.
 	 */
-	protected $_patterns=array();
+	protected $_patterns = array();
 	/**
 	 * @var TUrlMappingPattern matched pattern.
 	 */
@@ -78,19 +78,19 @@ 
 	/**
 	 * @var string external configuration file
 	 */
-	private $_configFile=null;
+	private $_configFile = null;
 	/**
 	 * @var boolean whether to enable custom contructUrl
 	 */
-	private $_customUrl=false;
+	private $_customUrl = false;
 	/**
 	 * @var array rules for constructing URLs
 	 */
-	protected $_constructRules=array();
+	protected $_constructRules = array();
 
-	private $_urlPrefix='';
+	private $_urlPrefix = '';
 
-	private $_defaultMappingClass='TUrlMappingPattern';
+	private $_defaultMappingClass = 'TUrlMappingPattern';
 
 	/**
 	 * Initializes this module.
@@ -101,22 +101,22 @@ 
 	public function init($config)
 	{
 		parent::init($config);
-		if($this->getRequest()->getRequestResolved())
+		if ($this->getRequest()->getRequestResolved())
 			throw new TConfigurationException('urlmapping_global_required');
-		if($this->_configFile!==null)
+		if ($this->_configFile !== null)
 			$this->loadConfigFile();
 		$this->loadUrlMappings($config);
-		if($this->_urlPrefix==='')
+		if ($this->_urlPrefix === '')
 		{
-			$request=$this->getRequest();
-			if($request->getUrlFormat()===THttpRequestUrlFormat::HiddenPath)
+			$request = $this->getRequest();
+			if ($request->getUrlFormat() === THttpRequestUrlFormat::HiddenPath)
 			{
-				$this->_urlPrefix=dirname($request->getApplicationUrl());
+				$this->_urlPrefix = dirname($request->getApplicationUrl());
 			} else {
-				$this->_urlPrefix=$request->getApplicationUrl();
+				$this->_urlPrefix = $request->getApplicationUrl();
 			}
 		}
-		$this->_urlPrefix=rtrim($this->_urlPrefix,'/');
+		$this->_urlPrefix = rtrim($this->_urlPrefix, '/');
 	}
 
 	/**
@@ -125,22 +125,22 @@ 
 	 */
 	protected function loadConfigFile()
 	{
-		if(is_file($this->_configFile))
+		if (is_file($this->_configFile))
 		{
-			if($this->getApplication()->getConfigurationType()==TApplication::CONFIG_TYPE_PHP)
+			if ($this->getApplication()->getConfigurationType() == TApplication::CONFIG_TYPE_PHP)
 			{
 				$config = include $this->_configFile;
 				$this->loadUrlMappings($dom);
 			}
 			else
 			{
-				$dom=new TXmlDocument;
+				$dom = new TXmlDocument;
 				$dom->loadFromFile($this->_configFile);
 				$this->loadUrlMappings($dom);
 			}
 		}
 		else
-			throw new TConfigurationException('urlmapping_configfile_inexistent',$this->_configFile);
+			throw new TConfigurationException('urlmapping_configfile_inexistent', $this->_configFile);
 	}
 
 	/**
@@ -164,7 +164,7 @@ 
 	 */
 	public function setEnableCustomUrl($value)
 	{
-		$this->_customUrl=TPropertyValue::ensureBoolean($value);
+		$this->_customUrl = TPropertyValue::ensureBoolean($value);
 	}
 
 	/**
@@ -183,7 +183,7 @@ 
 	 */
 	public function setUrlPrefix($value)
 	{
-		$this->_urlPrefix=$value;
+		$this->_urlPrefix = $value;
 	}
 
 	/**
@@ -201,8 +201,8 @@ 
 	 */
 	public function setConfigFile($value)
 	{
-		if(($this->_configFile=Prado::getPathOfNamespace($value,$this->getApplication()->getConfigurationFileExt()))===null)
-			throw new TConfigurationException('urlmapping_configfile_invalid',$value);
+		if (($this->_configFile = Prado::getPathOfNamespace($value, $this->getApplication()->getConfigurationFileExt())) === null)
+			throw new TConfigurationException('urlmapping_configfile_invalid', $value);
 	}
 
 	/**
@@ -223,7 +223,7 @@ 
 	 */
 	public function setDefaultMappingClass($value)
 	{
-		$this->_defaultMappingClass=$value;
+		$this->_defaultMappingClass = $value;
 	}
 
 	/**
@@ -235,43 +235,43 @@ 
 	{
 		$defaultClass = $this->getDefaultMappingClass();
 
-		if(is_array($config))
+		if (is_array($config))
 		{
-			if(isset($config['urls']) && is_array($config['urls']))
+			if (isset($config['urls']) && is_array($config['urls']))
 			{
-				foreach($config['urls'] as $url)
+				foreach ($config['urls'] as $url)
 				{
-					$class=isset($url['class'])?$url['class']:$defaultClass;
-					$properties = isset($url['properties'])?$url['properties']:array();
-					$this->buildUrlMapping($class,$properties,$url);
+					$class = isset($url['class']) ? $url['class'] : $defaultClass;
+					$properties = isset($url['properties']) ? $url['properties'] : array();
+					$this->buildUrlMapping($class, $properties, $url);
 				}
 			}
 		}
 		else
 		{
-			foreach($config->getElementsByTagName('url') as $url)
+			foreach ($config->getElementsByTagName('url') as $url)
 			{
-				$properties=$url->getAttributes();
-				if(($class=$properties->remove('class'))===null)
-					$class=$defaultClass;
-				$this->buildUrlMapping($class,$properties,$url);
+				$properties = $url->getAttributes();
+				if (($class = $properties->remove('class')) === null)
+					$class = $defaultClass;
+				$this->buildUrlMapping($class, $properties, $url);
 			}
 		}
 	}
 
 	private function buildUrlMapping($class, $properties, $url)
 	{
-		$pattern=Prado::createComponent($class,$this);
-		if(!($pattern instanceof TUrlMappingPattern))
+		$pattern = Prado::createComponent($class, $this);
+		if (!($pattern instanceof TUrlMappingPattern))
 			throw new TConfigurationException('urlmapping_urlmappingpattern_required');
-		foreach($properties as $name=>$value)
-			$pattern->setSubproperty($name,$value);
+		foreach ($properties as $name=>$value)
+			$pattern->setSubproperty($name, $value);
 
-		if($url instanceof TXmlElement) {
+		if ($url instanceof TXmlElement) {
 			$text = $url -> getValue();
-			if($text) {
+			if ($text) {
 				$text = preg_replace('/(\s+)/S', '', $text);
-				if(($regExp = $pattern->getRegularExpression()) !== '')
+				if (($regExp = $pattern->getRegularExpression()) !== '')
 				trigger_error(sPrintF('%s.RegularExpression property value "%s" for ServiceID="%s" and ServiceParameter="%s" was replaced by node value "%s"',
 				get_class($pattern),
 				$regExp,
@@ -283,11 +283,11 @@ 
 			}
 		}
 
-		$this->_patterns[]=$pattern;
+		$this->_patterns[] = $pattern;
 		$pattern->init($url);
 
-		$key=$pattern->getServiceID().':'.$pattern->getServiceParameter();
-		$this->_constructRules[$key][]=$pattern;
+		$key = $pattern->getServiceID() . ':' . $pattern->getServiceParameter();
+		$this->_constructRules[$key][] = $pattern;
 	}
 
 	/**
@@ -300,21 +300,21 @@ 
 	 */
 	public function parseUrl()
 	{
-		$request=$this->getRequest();
-		foreach($this->_patterns as $pattern)
+		$request = $this->getRequest();
+		foreach ($this->_patterns as $pattern)
 		{
-			$matches=$pattern->getPatternMatches($request);
-			if(count($matches)>0)
+			$matches = $pattern->getPatternMatches($request);
+			if (count($matches) > 0)
 			{
-				$this->_matched=$pattern;
-				$params=array();
-				foreach($matches as $key=>$value)
+				$this->_matched = $pattern;
+				$params = array();
+				foreach ($matches as $key=>$value)
 				{
-					if(is_string($key))
-						$params[$key]=$value;
+					if (is_string($key))
+						$params[$key] = $value;
 				}
 				if (!$pattern->getIsWildCardPattern())
-					$params[$pattern->getServiceID()]=$pattern->getServiceParameter();
+					$params[$pattern->getServiceID()] = $pattern->getServiceParameter();
 				return $params;
 			}
 		}
@@ -345,36 +345,36 @@ 
 	 * @see parseUrl
 	 * @since 3.1.1
 	 */
-	public function constructUrl($serviceID,$serviceParam,$getItems,$encodeAmpersand,$encodeGetItems)
+	public function constructUrl($serviceID, $serviceParam, $getItems, $encodeAmpersand, $encodeGetItems)
 	{
-		if($this->_customUrl)
+		if ($this->_customUrl)
 		{
-			if(!(is_array($getItems) || ($getItems instanceof Traversable)))
-				$getItems=array();
-			$key=$serviceID.':'.$serviceParam;
-			$wildCardKey = ($pos=strrpos($serviceParam,'.'))!==false ?
-				$serviceID.':'.substr($serviceParam,0,$pos).'.*' : $serviceID.':*';
-			if(isset($this->_constructRules[$key]))
+			if (!(is_array($getItems) || ($getItems instanceof Traversable)))
+				$getItems = array();
+			$key = $serviceID . ':' . $serviceParam;
+			$wildCardKey = ($pos = strrpos($serviceParam, '.')) !== false ?
+				$serviceID . ':' . substr($serviceParam, 0, $pos) . '.*' : $serviceID . ':*';
+			if (isset($this->_constructRules[$key]))
 			{
-				foreach($this->_constructRules[$key] as $rule)
+				foreach ($this->_constructRules[$key] as $rule)
 				{
-					if($rule->supportCustomUrl($getItems))
-						return $rule->constructUrl($getItems,$encodeAmpersand,$encodeGetItems);
+					if ($rule->supportCustomUrl($getItems))
+						return $rule->constructUrl($getItems, $encodeAmpersand, $encodeGetItems);
 				}
 			}
-			elseif(isset($this->_constructRules[$wildCardKey]))
+			elseif (isset($this->_constructRules[$wildCardKey]))
 			{
-				foreach($this->_constructRules[$wildCardKey] as $rule)
+				foreach ($this->_constructRules[$wildCardKey] as $rule)
 				{
-					if($rule->supportCustomUrl($getItems))
+					if ($rule->supportCustomUrl($getItems))
 					{
-						$getItems['*']= $pos ? substr($serviceParam,$pos+1) : $serviceParam;
-						return $rule->constructUrl($getItems,$encodeAmpersand,$encodeGetItems);
+						$getItems['*'] = $pos ? substr($serviceParam, $pos + 1) : $serviceParam;
+						return $rule->constructUrl($getItems, $encodeAmpersand, $encodeGetItems);
 					}
 				}
 			}
 		}
-		return parent::constructUrl($serviceID,$serviceParam,$getItems,$encodeAmpersand,$encodeGetItems);
+		return parent::constructUrl($serviceID, $serviceParam, $getItems, $encodeAmpersand, $encodeGetItems);
 	}
 
 	/**
@@ -494,7 +494,7 @@ 
 	/**
 	 * @var string service ID, default is 'page'.
 	 */
-	private $_serviceID='page';
+	private $_serviceID = 'page';
 	/**
 	 * @var string url pattern to match.
 	 */
@@ -510,19 +510,19 @@ 
 	/**
 	 * @var string regular expression pattern.
 	 */
-	private $_regexp='';
+	private $_regexp = '';
 
-	private $_customUrl=true;
+	private $_customUrl = true;
 
 	private $_manager;
 
-	private $_caseSensitive=true;
+	private $_caseSensitive = true;
 
-	private $_isWildCardPattern=false;
+	private $_isWildCardPattern = false;
 
-	private $_urlFormat=THttpRequestUrlFormat::Get;
+	private $_urlFormat = THttpRequestUrlFormat::Get;
 
-	private $_separator='/';
+	private $_separator = '/';
 
 	/**
 	 * @var TUrlMappingPatternSecureConnection
@@ -536,7 +536,7 @@ 
 	 */
 	public function __construct(TUrlManager $manager)
 	{
-		$this->_manager=$manager;
+		$this->_manager = $manager;
 	}
 
 	/**
@@ -554,10 +554,10 @@ 
 	 */
 	public function init($config)
 	{
-		if($this->_serviceParameter===null)
+		if ($this->_serviceParameter === null)
 			throw new TConfigurationException('urlmappingpattern_serviceparameter_required', $this->getPattern());
-		if(strpos($this->_serviceParameter,'*')!==false)
-				$this->_isWildCardPattern=true;
+		if (strpos($this->_serviceParameter, '*') !== false)
+				$this->_isWildCardPattern = true;
 	}
 
 	/**
@@ -567,32 +567,32 @@ 
 	 */
 	protected function getParameterizedPattern()
 	{
-		$params=array();
-		$values=array();
+		$params = array();
+		$values = array();
 		if ($this->_parameters)
 		{
-			foreach($this->_parameters as $key=>$value)
+			foreach ($this->_parameters as $key=>$value)
 			{
-				$params[]='{'.$key.'}';
-				$values[]='(?P<'.$key.'>'.$value.')';
+				$params[] = '{' . $key . '}';
+				$values[] = '(?P<' . $key . '>' . $value . ')';
 			}
 		}
 		if ($this->getIsWildCardPattern())
 		{
-				$params[]='{*}';
+				$params[] = '{*}';
 				// service parameter must not contain '=' and '/'
-				$values[]='(?P<'.$this->getServiceID().'>[^=/]+)';
+				$values[] = '(?P<' . $this->getServiceID() . '>[^=/]+)';
 		}
-		$params[]='/';
-		$values[]='\\/';
-		$regexp=str_replace($params,$values,trim($this->getPattern(),'/').'/');
-		if ($this->_urlFormat===THttpRequestUrlFormat::Get)
-				$regexp='/^'.$regexp.'$/u';
+		$params[] = '/';
+		$values[] = '\\/';
+		$regexp = str_replace($params, $values, trim($this->getPattern(), '/') . '/');
+		if ($this->_urlFormat === THttpRequestUrlFormat::Get)
+				$regexp = '/^' . $regexp . '$/u';
 		else
-				$regexp='/^'.$regexp.'(?P<urlparams>.*)$/u';
+				$regexp = '/^' . $regexp . '(?P<urlparams>.*)$/u';
 
-		if(!$this->getCaseSensitive())
-			$regexp.='i';
+		if (!$this->getCaseSensitive())
+			$regexp .= 'i';
 		return $regexp;
 	}
 
@@ -609,7 +609,7 @@ 
 	 */
 	public function setRegularExpression($value)
 	{
-		$this->_regexp=$value;
+		$this->_regexp = $value;
 	}
 
 	/**
@@ -625,7 +625,7 @@ 
 	 */
 	public function setCaseSensitive($value)
 	{
-		$this->_caseSensitive=TPropertyValue::ensureBoolean($value);
+		$this->_caseSensitive = TPropertyValue::ensureBoolean($value);
 	}
 
 	/**
@@ -633,7 +633,7 @@ 
 	 */
 	public function setServiceParameter($value)
 	{
-		$this->_serviceParameter=$value;
+		$this->_serviceParameter = $value;
 	}
 
 	/**
@@ -649,7 +649,7 @@ 
 	 */
 	public function setServiceID($value)
 	{
-		$this->_serviceID=$value;
+		$this->_serviceID = $value;
 	}
 
 	/**
@@ -683,7 +683,7 @@ 
 	{
 		if (!$this->_parameters)
 		{
-			$this->_parameters=new TAttributeCollection;
+			$this->_parameters = new TAttributeCollection;
 			$this->_parameters->setCaseSensitive(true);
 		}
 		return $this->_parameters;
@@ -694,7 +694,7 @@ 
 	 */
 	public function setParameters($value)
 	{
-		$this->_parameters=$value;
+		$this->_parameters = $value;
 	}
 
 	/**
@@ -719,38 +719,38 @@ 
 	 */
 	public function getPatternMatches($request)
 	{
-		$matches=array();
-		if(($pattern=$this->getRegularExpression())!=='')
-			preg_match($pattern,$request->getPathInfo(),$matches);
+		$matches = array();
+		if (($pattern = $this->getRegularExpression()) !== '')
+			preg_match($pattern, $request->getPathInfo(), $matches);
 		else
-			preg_match($this->getParameterizedPattern(),trim($request->getPathInfo(),'/').'/',$matches);
+			preg_match($this->getParameterizedPattern(), trim($request->getPathInfo(), '/') . '/', $matches);
 
-		if($this->getIsWildCardPattern() && isset($matches[$this->_serviceID]))
-			$matches[$this->_serviceID]=str_replace('*',$matches[$this->_serviceID],$this->_serviceParameter);
+		if ($this->getIsWildCardPattern() && isset($matches[$this->_serviceID]))
+			$matches[$this->_serviceID] = str_replace('*', $matches[$this->_serviceID], $this->_serviceParameter);
 
 		if (isset($matches['urlparams']))
 		{
-			$params=explode('/',$matches['urlparams']);
-			if ($this->_separator==='/')
+			$params = explode('/', $matches['urlparams']);
+			if ($this->_separator === '/')
 			{
-				while($key=array_shift($params))
-					$matches[$key]=($value=array_shift($params)) ? $value : '';
+				while ($key = array_shift($params))
+					$matches[$key] = ($value = array_shift($params)) ? $value : '';
 			}
 			else
 			{
 				array_pop($params);
-				foreach($params as $param)
+				foreach ($params as $param)
 				{
-					list($key,$value)=explode($this->_separator,$param,2);
-					$matches[$key]=$value;
+					list($key, $value) = explode($this->_separator, $param, 2);
+					$matches[$key] = $value;
 				}
 			}
 			unset($matches['urlparams']);
 		}
 
-		if(count($matches) > 0 && $this->_constants)
+		if (count($matches) > 0 && $this->_constants)
 		{
-			foreach($this->_constants->toArray() as $key=>$value)
+			foreach ($this->_constants->toArray() as $key=>$value)
 				$matches[$key] = $value;
 		}
 
@@ -773,7 +773,7 @@ 
 	 */
 	public function setEnableCustomUrl($value)
 	{
-		$this->_customUrl=TPropertyValue::ensureBoolean($value);
+		$this->_customUrl = TPropertyValue::ensureBoolean($value);
 	}
 
 	/**
@@ -805,7 +805,7 @@ 
 	 */
 	public function setUrlFormat($value)
 	{
-		$this->_urlFormat=TPropertyValue::ensureEnum($value,'THttpRequestUrlFormat');
+		$this->_urlFormat = TPropertyValue::ensureEnum($value, 'THttpRequestUrlFormat');
 	}
 
 	/**
@@ -822,8 +822,8 @@ 
 	 */
 	public function setUrlParamSeparator($value)
 	{
-		if(strlen($value)===1)
-			$this->_separator=$value;
+		if (strlen($value) === 1)
+			$this->_separator = $value;
 		else
 			throw new TInvalidDataValueException('httprequest_separator_invalid');
 	}
@@ -853,24 +853,24 @@ 
 	 */
 	public function supportCustomUrl($getItems)
 	{
-		if(!$this->_customUrl || $this->getPattern()===null)
+		if (!$this->_customUrl || $this->getPattern() === null)
 			return false;
 		if ($this->_parameters)
 		{
-			foreach($this->_parameters as $key=>$value)
+			foreach ($this->_parameters as $key=>$value)
 			{
-				if(!isset($getItems[$key]))
+				if (!isset($getItems[$key]))
 					return false;
 			}
 		}
 
 		if ($this->_constants)
 		{
-			foreach($this->_constants->toArray() as $key=>$value)
+			foreach ($this->_constants->toArray() as $key=>$value)
 			{
 				if (!isset($getItems[$key]))
 					return false;
-				if ($getItems[$key]!=$value)
+				if ($getItems[$key] != $value)
 					return false;
 			}
 		}
@@ -885,68 +885,68 @@ 
 	 * @return string the constructed URL
 	 * @since 3.1.1
 	 */
-	public function constructUrl($getItems,$encodeAmpersand,$encodeGetItems)
+	public function constructUrl($getItems, $encodeAmpersand, $encodeGetItems)
 	{
 		if ($this->_constants)
 		{
-			foreach($this->_constants->toArray() as $key=>$value)
+			foreach ($this->_constants->toArray() as $key=>$value)
 			{
 				unset($getItems[$key]);
 			}
 		}
 
-		$extra=array();
-		$replace=array();
+		$extra = array();
+		$replace = array();
 		// for the GET variables matching the pattern, put them in the URL path
-		foreach($getItems as $key=>$value)
+		foreach ($getItems as $key=>$value)
 		{
-			if(($this->_parameters && $this->_parameters->contains($key)) || ($key==='*' && $this->getIsWildCardPattern()))
-				$replace['{'.$key.'}']=$encodeGetItems ? rawurlencode($value) : $value;
+			if (($this->_parameters && $this->_parameters->contains($key)) || ($key === '*' && $this->getIsWildCardPattern()))
+				$replace['{' . $key . '}'] = $encodeGetItems ? rawurlencode($value) : $value;
 			else
-				$extra[$key]=$value;
+				$extra[$key] = $value;
 		}
 
-		$url=$this->_manager->getUrlPrefix().'/'.ltrim(strtr($this->getPattern(),$replace),'/');
+		$url = $this->_manager->getUrlPrefix() . '/' . ltrim(strtr($this->getPattern(), $replace), '/');
 
 		// for the rest of the GET variables, put them in the query string
-		if(count($extra)>0)
+		if (count($extra) > 0)
 		{
-			if ($this->_urlFormat===THttpRequestUrlFormat::Path && $this->getIsWildCardPattern()) {
+			if ($this->_urlFormat === THttpRequestUrlFormat::Path && $this->getIsWildCardPattern()) {
 				foreach ($extra as $name=>$value)
-					$url.='/'.$name.$this->_separator.($encodeGetItems?rawurlencode($value):$value);
+					$url .= '/' . $name . $this->_separator . ($encodeGetItems ? rawurlencode($value) : $value);
 				return $url;
 			}
 
-			$url2='';
-			$amp=$encodeAmpersand?'&amp;':'&';
-			if($encodeGetItems)
+			$url2 = '';
+			$amp = $encodeAmpersand ? '&amp;' : '&';
+			if ($encodeGetItems)
 			{
-				foreach($extra as $name=>$value)
+				foreach ($extra as $name=>$value)
 				{
-					if(is_array($value))
+					if (is_array($value))
 					{
-						$name=rawurlencode($name.'[]');
-						foreach($value as $v)
-							$url2.=$amp.$name.'='.rawurlencode($v);
+						$name = rawurlencode($name . '[]');
+						foreach ($value as $v)
+							$url2 .= $amp . $name . '=' . rawurlencode($v);
 					}
 					else
-						$url2.=$amp.rawurlencode($name).'='.rawurlencode($value);
+						$url2 .= $amp . rawurlencode($name) . '=' . rawurlencode($value);
 				}
 			}
 			else
 			{
-				foreach($extra as $name=>$value)
+				foreach ($extra as $name=>$value)
 				{
-					if(is_array($value))
+					if (is_array($value))
 					{
-						foreach($value as $v)
-							$url2.=$amp.$name.'[]='.$v;
+						foreach ($value as $v)
+							$url2 .= $amp . $name . '[]=' . $v;
 					}
 					else
-						$url2.=$amp.$name.'='.$value;
+						$url2 .= $amp . $name . '=' . $value;
 				}
 			}
-			$url=$url.'?'.substr($url2,strlen($amp));
+			$url = $url . '?' . substr($url2, strlen($amp));
 		}
 		return $this -> applySecureConnectionPrefix($url);
 	}
@@ -962,19 +962,19 @@ 
 	protected function applySecureConnectionPrefix($url)
 	{
 		static $request;
-		if($request === null) $request = Prado::getApplication() -> getRequest();
+		if ($request === null) $request = Prado::getApplication() -> getRequest();
 
 		static $isSecureConnection;
-		if($isSecureConnection === null) $isSecureConnection = $request -> getIsSecureConnection();
+		if ($isSecureConnection === null) $isSecureConnection = $request -> getIsSecureConnection();
 
-		switch($this -> getSecureConnection())
+		switch ($this -> getSecureConnection())
 		{
 			case TUrlMappingPatternSecureConnection::EnableIfNotSecure:
-				if($isSecureConnection) return $url;
+				if ($isSecureConnection) return $url;
 				return $request -> getBaseUrl(true) . $url;
 			break;
 			case TUrlMappingPatternSecureConnection::DisableIfSecure:
-				if(!$isSecureConnection) return $url;
+				if (!$isSecureConnection) return $url;
 				return $request -> getBaseUrl(false) . $url;
 			break;
 			case TUrlMappingPatternSecureConnection::Enable:

Braces +115 added lines, -84 removed lines

@@ -101,10 +101,12 @@ 
 	public function init($config)
 	{
 		parent::init($config);
-		if($this->getRequest()->getRequestResolved())
-			throw new TConfigurationException('urlmapping_global_required');
-		if($this->_configFile!==null)
-			$this->loadConfigFile();
+		if($this->getRequest()->getRequestResolved()) {
+					throw new TConfigurationException('urlmapping_global_required');
+		}
+		if($this->_configFile!==null) {
+					$this->loadConfigFile();
+		}
 		$this->loadUrlMappings($config);
 		if($this->_urlPrefix==='')
 		{
@@ -131,16 +133,15 @@ 
 			{
 				$config = include $this->_configFile;
 				$this->loadUrlMappings($dom);
-			}
-			else
+			} else
 			{
 				$dom=new TXmlDocument;
 				$dom->loadFromFile($this->_configFile);
 				$this->loadUrlMappings($dom);
 			}
+		} else {
+					throw new TConfigurationException('urlmapping_configfile_inexistent',$this->_configFile);
 		}
-		else
-			throw new TConfigurationException('urlmapping_configfile_inexistent',$this->_configFile);
 	}
 
 	/**
@@ -201,8 +202,9 @@ 
 	 */
 	public function setConfigFile($value)
 	{
-		if(($this->_configFile=Prado::getPathOfNamespace($value,$this->getApplication()->getConfigurationFileExt()))===null)
-			throw new TConfigurationException('urlmapping_configfile_invalid',$value);
+		if(($this->_configFile=Prado::getPathOfNamespace($value,$this->getApplication()->getConfigurationFileExt()))===null) {
+					throw new TConfigurationException('urlmapping_configfile_invalid',$value);
+		}
 	}
 
 	/**
@@ -246,14 +248,14 @@ 
 					$this->buildUrlMapping($class,$properties,$url);
 				}
 			}
-		}
-		else
+		} else
 		{
 			foreach($config->getElementsByTagName('url') as $url)
 			{
 				$properties=$url->getAttributes();
-				if(($class=$properties->remove('class'))===null)
-					$class=$defaultClass;
+				if(($class=$properties->remove('class'))===null) {
+									$class=$defaultClass;
+				}
 				$this->buildUrlMapping($class,$properties,$url);
 			}
 		}
@@ -262,23 +264,26 @@ 
 	private function buildUrlMapping($class, $properties, $url)
 	{
 		$pattern=Prado::createComponent($class,$this);
-		if(!($pattern instanceof TUrlMappingPattern))
-			throw new TConfigurationException('urlmapping_urlmappingpattern_required');
-		foreach($properties as $name=>$value)
-			$pattern->setSubproperty($name,$value);
+		if(!($pattern instanceof TUrlMappingPattern)) {
+					throw new TConfigurationException('urlmapping_urlmappingpattern_required');
+		}
+		foreach($properties as $name=>$value) {
+					$pattern->setSubproperty($name,$value);
+		}
 
 		if($url instanceof TXmlElement) {
 			$text = $url -> getValue();
 			if($text) {
 				$text = preg_replace('/(\s+)/S', '', $text);
-				if(($regExp = $pattern->getRegularExpression()) !== '')
-				trigger_error(sPrintF('%s.RegularExpression property value "%s" for ServiceID="%s" and ServiceParameter="%s" was replaced by node value "%s"',
+				if(($regExp = $pattern->getRegularExpression()) !== '') {
+								trigger_error(sPrintF('%s.RegularExpression property value "%s" for ServiceID="%s" and ServiceParameter="%s" was replaced by node value "%s"',
 				get_class($pattern),
 				$regExp,
 				$pattern->getServiceID(),
 				$pattern->getServiceParameter(),
 				$text),
 				E_USER_NOTICE);
+				}
 				$pattern->setRegularExpression($text);
 			}
 		}
@@ -310,11 +315,13 @@ 
 				$params=array();
 				foreach($matches as $key=>$value)
 				{
-					if(is_string($key))
-						$params[$key]=$value;
+					if(is_string($key)) {
+											$params[$key]=$value;
+					}
+				}
+				if (!$pattern->getIsWildCardPattern()) {
+									$params[$pattern->getServiceID()]=$pattern->getServiceParameter();
 				}
-				if (!$pattern->getIsWildCardPattern())
-					$params[$pattern->getServiceID()]=$pattern->getServiceParameter();
 				return $params;
 			}
 		}
@@ -349,8 +356,9 @@ 
 	{
 		if($this->_customUrl)
 		{
-			if(!(is_array($getItems) || ($getItems instanceof Traversable)))
-				$getItems=array();
+			if(!(is_array($getItems) || ($getItems instanceof Traversable))) {
+							$getItems=array();
+			}
 			$key=$serviceID.':'.$serviceParam;
 			$wildCardKey = ($pos=strrpos($serviceParam,'.'))!==false ?
 				$serviceID.':'.substr($serviceParam,0,$pos).'.*' : $serviceID.':*';
@@ -358,11 +366,11 @@ 
 			{
 				foreach($this->_constructRules[$key] as $rule)
 				{
-					if($rule->supportCustomUrl($getItems))
-						return $rule->constructUrl($getItems,$encodeAmpersand,$encodeGetItems);
+					if($rule->supportCustomUrl($getItems)) {
+											return $rule->constructUrl($getItems,$encodeAmpersand,$encodeGetItems);
+					}
 				}
-			}
-			elseif(isset($this->_constructRules[$wildCardKey]))
+			} elseif(isset($this->_constructRules[$wildCardKey]))
 			{
 				foreach($this->_constructRules[$wildCardKey] as $rule)
 				{
@@ -554,10 +562,12 @@ 
 	 */
 	public function init($config)
 	{
-		if($this->_serviceParameter===null)
-			throw new TConfigurationException('urlmappingpattern_serviceparameter_required', $this->getPattern());
-		if(strpos($this->_serviceParameter,'*')!==false)
-				$this->_isWildCardPattern=true;
+		if($this->_serviceParameter===null) {
+					throw new TConfigurationException('urlmappingpattern_serviceparameter_required', $this->getPattern());
+		}
+		if(strpos($this->_serviceParameter,'*')!==false) {
+						$this->_isWildCardPattern=true;
+		}
 	}
 
 	/**
@@ -586,13 +596,15 @@ 
 		$params[]='/';
 		$values[]='\\/';
 		$regexp=str_replace($params,$values,trim($this->getPattern(),'/').'/');
-		if ($this->_urlFormat===THttpRequestUrlFormat::Get)
-				$regexp='/^'.$regexp.'$/u';
-		else
-				$regexp='/^'.$regexp.'(?P<urlparams>.*)$/u';
+		if ($this->_urlFormat===THttpRequestUrlFormat::Get) {
+						$regexp='/^'.$regexp.'$/u';
+		} else {
+						$regexp='/^'.$regexp.'(?P<urlparams>.*)$/u';
+		}
 
-		if(!$this->getCaseSensitive())
-			$regexp.='i';
+		if(!$this->getCaseSensitive()) {
+					$regexp.='i';
+		}
 		return $regexp;
 	}
 
@@ -720,23 +732,25 @@ 
 	public function getPatternMatches($request)
 	{
 		$matches=array();
-		if(($pattern=$this->getRegularExpression())!=='')
-			preg_match($pattern,$request->getPathInfo(),$matches);
-		else
-			preg_match($this->getParameterizedPattern(),trim($request->getPathInfo(),'/').'/',$matches);
+		if(($pattern=$this->getRegularExpression())!=='') {
+					preg_match($pattern,$request->getPathInfo(),$matches);
+		} else {
+					preg_match($this->getParameterizedPattern(),trim($request->getPathInfo(),'/').'/',$matches);
+		}
 
-		if($this->getIsWildCardPattern() && isset($matches[$this->_serviceID]))
-			$matches[$this->_serviceID]=str_replace('*',$matches[$this->_serviceID],$this->_serviceParameter);
+		if($this->getIsWildCardPattern() && isset($matches[$this->_serviceID])) {
+					$matches[$this->_serviceID]=str_replace('*',$matches[$this->_serviceID],$this->_serviceParameter);
+		}
 
 		if (isset($matches['urlparams']))
 		{
 			$params=explode('/',$matches['urlparams']);
 			if ($this->_separator==='/')
 			{
-				while($key=array_shift($params))
-					$matches[$key]=($value=array_shift($params)) ? $value : '';
-			}
-			else
+				while($key=array_shift($params)) {
+									$matches[$key]=($value=array_shift($params)) ? $value : '';
+				}
+			} else
 			{
 				array_pop($params);
 				foreach($params as $param)
@@ -750,8 +764,9 @@ 
 
 		if(count($matches) > 0 && $this->_constants)
 		{
-			foreach($this->_constants->toArray() as $key=>$value)
-				$matches[$key] = $value;
+			foreach($this->_constants->toArray() as $key=>$value) {
+							$matches[$key] = $value;
+			}
 		}
 
 		return $matches;
@@ -822,10 +837,11 @@ 
 	 */
 	public function setUrlParamSeparator($value)
 	{
-		if(strlen($value)===1)
-			$this->_separator=$value;
-		else
-			throw new TInvalidDataValueException('httprequest_separator_invalid');
+		if(strlen($value)===1) {
+					$this->_separator=$value;
+		} else {
+					throw new TInvalidDataValueException('httprequest_separator_invalid');
+		}
 	}
 
 	/**
@@ -853,14 +869,16 @@ 
 	 */
 	public function supportCustomUrl($getItems)
 	{
-		if(!$this->_customUrl || $this->getPattern()===null)
-			return false;
+		if(!$this->_customUrl || $this->getPattern()===null) {
+					return false;
+		}
 		if ($this->_parameters)
 		{
 			foreach($this->_parameters as $key=>$value)
 			{
-				if(!isset($getItems[$key]))
-					return false;
+				if(!isset($getItems[$key])) {
+									return false;
+				}
 			}
 		}
 
@@ -868,10 +886,12 @@ 
 		{
 			foreach($this->_constants->toArray() as $key=>$value)
 			{
-				if (!isset($getItems[$key]))
-					return false;
-				if ($getItems[$key]!=$value)
-					return false;
+				if (!isset($getItems[$key])) {
+									return false;
+				}
+				if ($getItems[$key]!=$value) {
+									return false;
+				}
 			}
 		}
 		return true;
@@ -900,10 +920,11 @@ 
 		// for the GET variables matching the pattern, put them in the URL path
 		foreach($getItems as $key=>$value)
 		{
-			if(($this->_parameters && $this->_parameters->contains($key)) || ($key==='*' && $this->getIsWildCardPattern()))
-				$replace['{'.$key.'}']=$encodeGetItems ? rawurlencode($value) : $value;
-			else
-				$extra[$key]=$value;
+			if(($this->_parameters && $this->_parameters->contains($key)) || ($key==='*' && $this->getIsWildCardPattern())) {
+							$replace['{'.$key.'}']=$encodeGetItems ? rawurlencode($value) : $value;
+			} else {
+							$extra[$key]=$value;
+			}
 		}
 
 		$url=$this->_manager->getUrlPrefix().'/'.ltrim(strtr($this->getPattern(),$replace),'/');
@@ -912,8 +933,9 @@ 
 		if(count($extra)>0)
 		{
 			if ($this->_urlFormat===THttpRequestUrlFormat::Path && $this->getIsWildCardPattern()) {
-				foreach ($extra as $name=>$value)
-					$url.='/'.$name.$this->_separator.($encodeGetItems?rawurlencode($value):$value);
+				foreach ($extra as $name=>$value) {
+									$url.='/'.$name.$this->_separator.($encodeGetItems?rawurlencode($value):$value);
+				}
 				return $url;
 			}
 
@@ -926,24 +948,25 @@ 
 					if(is_array($value))
 					{
 						$name=rawurlencode($name.'[]');
-						foreach($value as $v)
-							$url2.=$amp.$name.'='.rawurlencode($v);
+						foreach($value as $v) {
+													$url2.=$amp.$name.'='.rawurlencode($v);
+						}
+					} else {
+											$url2.=$amp.rawurlencode($name).'='.rawurlencode($value);
 					}
-					else
-						$url2.=$amp.rawurlencode($name).'='.rawurlencode($value);
 				}
-			}
-			else
+			} else
 			{
 				foreach($extra as $name=>$value)
 				{
 					if(is_array($value))
 					{
-						foreach($value as $v)
-							$url2.=$amp.$name.'[]='.$v;
+						foreach($value as $v) {
+													$url2.=$amp.$name.'[]='.$v;
+						}
+					} else {
+											$url2.=$amp.$name.'='.$value;
 					}
-					else
-						$url2.=$amp.$name.'='.$value;
 				}
 			}
 			$url=$url.'?'.substr($url2,strlen($amp));
@@ -962,19 +985,27 @@ 
 	protected function applySecureConnectionPrefix($url)
 	{
 		static $request;
-		if($request === null) $request = Prado::getApplication() -> getRequest();
+		if($request === null) {
+			$request = Prado::getApplication() -> getRequest();
+		}
 
 		static $isSecureConnection;
-		if($isSecureConnection === null) $isSecureConnection = $request -> getIsSecureConnection();
+		if($isSecureConnection === null) {
+			$isSecureConnection = $request -> getIsSecureConnection();
+		}
 
 		switch($this -> getSecureConnection())
 		{
 			case TUrlMappingPatternSecureConnection::EnableIfNotSecure:
-				if($isSecureConnection) return $url;
+				if($isSecureConnection) {
+					return $url;
+				}
 				return $request -> getBaseUrl(true) . $url;
 			break;
 			case TUrlMappingPatternSecureConnection::DisableIfSecure:
-				if(!$isSecureConnection) return $url;
+				if(!$isSecureConnection) {
+					return $url;
+				}
 				return $request -> getBaseUrl(false) . $url;
 			break;
 			case TUrlMappingPatternSecureConnection::Enable:

framework/Web/UI/ActiveControls/TActiveCustomValidator.php

Doc Comments +1 added lines, -1 removed lines

@@ -252,7 +252,7 @@
 	}
 
 	/**
-	 * @return boolean true to observe changes.
+	 * @return boolean|string true to observe changes.
 	 */
 	public function getObserveChanges()
 	{

Spacing +4 added lines, -4 removed lines

@@ -105,7 +105,7 @@ 
 	public function setIsValid($value)
 	{
 		parent::setIsValid($value);
-		if($this->getActiveControl()->canUpdateClientSide())
+		if ($this->getActiveControl()->canUpdateClientSide())
 		{
 			$client = $this->getPage()->getCallbackClient();
 			$func = 'Prado.Validation.updateActiveCustomValidator';
@@ -131,7 +131,7 @@ 
 	 */
 	protected function getClientScriptOptions()
 	{
-		$options=TBaseValidator::getClientScriptOptions();
+		$options = TBaseValidator::getClientScriptOptions();
 		$options['EventTarget'] = $this->getUniqueID();
 		return $options;
 	}
@@ -143,7 +143,7 @@ 
 	public function setErrorMessage($value)
 	{
 		parent::setErrorMessage($value);
-		if($this->getActiveControl()->canUpdateClientSide())
+		if ($this->getActiveControl()->canUpdateClientSide())
 		{
 			$client = $this->getPage()->getCallbackClient();
 			$func = 'Prado.Validation.setErrorMessage';
@@ -257,6 +257,6 @@ 
 	public function getObserveChanges()
 	{
 		$changes = $this->getOption('ObserveChanges');
-		return ($changes===null) ? true : $changes;
+		return ($changes === null) ? true : $changes;
 	}
 }

framework/Web/UI/ActiveControls/TActiveListBox.php

Doc Comments +1 added lines, -1 removed lines

@@ -70,7 +70,7 @@
 	/**
 	 * Javascript client class for this control.
 	 * This method overrides the parent implementation.
-	 * @return null no javascript class name.
+	 * @return string no javascript class name.
 	 */
 	protected function getClientClassName()
 	{

Spacing +4 added lines, -4 removed lines

@@ -62,7 +62,7 @@ 
 	 */
 	protected function createListItemCollection()
 	{
-		$collection  = new TActiveListItemCollection;
+		$collection = new TActiveListItemCollection;
 		$collection->setControl($this);
 		return $collection;
 	}
@@ -87,8 +87,8 @@ 
 	{
 		parent::setSelectionMode($value);
 		$multiple = $this->getIsMultiSelect();
-		$id = $this->getUniqueID(); $multi_id = $id.'[]';
-		if($this->getActiveControl()->canUpdateClientSide())
+		$id = $this->getUniqueID(); $multi_id = $id . '[]';
+		if ($this->getActiveControl()->canUpdateClientSide())
 		{
 			$client = $this->getPage()->getCallbackClient();
 			$client->setAttribute($this, 'multiple', $multiple ? 'multiple' : false);
@@ -143,7 +143,7 @@ 
 	protected function addAttributesToRender($writer)
 	{
 		parent::addAttributesToRender($writer);
-		$writer->addAttribute('id',$this->getClientID());
+		$writer->addAttribute('id', $this->getClientID());
 		if ($this->getAutoPostBack())
 			$this->getActiveControl()->registerCallbackClientScript(
 				$this->getClientClassName(), $this->getPostBackOptions());

Braces +3 added lines, -2 removed lines

@@ -144,9 +144,10 @@
 	{
 		parent::addAttributesToRender($writer);
 		$writer->addAttribute('id',$this->getClientID());
-		if ($this->getAutoPostBack())
-			$this->getActiveControl()->registerCallbackClientScript(
+		if ($this->getAutoPostBack()) {
+					$this->getActiveControl()->registerCallbackClientScript(
 				$this->getClientClassName(), $this->getPostBackOptions());
+		}
 	}
 }
 

framework/Web/UI/ActiveControls/TActivePageAdapter.php

Doc Comments +2 added lines

@@ -228,6 +228,8 @@
 
 	/**
 	 * Appends data or javascript code to the body content surrounded with delimiters
+	 * @param THttpResponse $response
+	 * @param string $delimiter
 	 */
 	private function appendContentPart($response, $delimiter, $data)
 	{

Spacing +45 added lines, -45 removed lines

@@ -80,7 +80,7 @@ 
 	 */
 	private $_callbackClient;
 
-	private $_controlsToRender=array();
+	private $_controlsToRender = array();
 
 	/**
 	 * Constructor, trap errors and exception to let the callback response
@@ -103,7 +103,7 @@ 
 	 */
 	public function processCallbackEvent($writer)
 	{
-		Prado::trace("ActivePage raiseCallbackEvent()",'System.Web.UI.ActiveControls.TActivePageAdapter');
+		Prado::trace("ActivePage raiseCallbackEvent()", 'System.Web.UI.ActiveControls.TActivePageAdapter');
 		$this->raiseCallbackEvent();
 	}
 
@@ -112,11 +112,11 @@ 
 	 * @param TControl control for defered rendering
 	 * @param THtmlWriter the renderer
 	 */
-	public function registerControlToRender($control,$writer)
+	public function registerControlToRender($control, $writer)
 	{
 		$id = $control->getUniqueID();
-		if(!isset($this->_controlsToRender[$id]))
-			$this->_controlsToRender[$id] = array($control,$writer);
+		if (!isset($this->_controlsToRender[$id]))
+			$this->_controlsToRender[$id] = array($control, $writer);
 	}
 
 	/**
@@ -133,8 +133,8 @@ 
 	 */
 	public function renderCallbackResponse($writer)
 	{
-		Prado::trace("ActivePage renderCallbackResponse()",'System.Web.UI.ActiveControls.TActivePageAdapter');
-		if(($url = $this->getResponse()->getAdapter()->getRedirectedUrl())===null)
+		Prado::trace("ActivePage renderCallbackResponse()", 'System.Web.UI.ActiveControls.TActivePageAdapter');
+		if (($url = $this->getResponse()->getAdapter()->getRedirectedUrl()) === null)
 			$this->renderResponse($writer);
 		else
 			$this->redirect($url);
@@ -146,7 +146,7 @@ 
 	 */
 	protected function redirect($url)
 	{
-		Prado::trace("ActivePage redirect()",'System.Web.UI.ActiveControls.TActivePageAdapter');
+		Prado::trace("ActivePage redirect()", 'System.Web.UI.ActiveControls.TActivePageAdapter');
 		$this->appendContentPart($this->getResponse(), self::CALLBACK_REDIRECT, $url);
 	}
 
@@ -157,18 +157,18 @@ 
 	 */
 	protected function renderResponse($writer)
 	{
-		Prado::trace("ActivePage renderResponse()",'System.Web.UI.ActiveControls.TActivePageAdapter');
+		Prado::trace("ActivePage renderResponse()", 'System.Web.UI.ActiveControls.TActivePageAdapter');
 		//renders all the defered render() calls.
-		foreach($this->_controlsToRender as $rid => $forRender)
+		foreach ($this->_controlsToRender as $rid => $forRender)
 			$forRender[0]->render($forRender[1]);
 
 		$response = $this->getResponse();
 
 		//send response data in header
-		if($response->getHasAdapter())
+		if ($response->getHasAdapter())
 		{
 			$responseData = $response->getAdapter()->getResponseData();
-			if($responseData!==null)
+			if ($responseData !== null)
 			{
 				$data = TJavaScript::jsonEncode($responseData);
 
@@ -177,9 +177,9 @@ 
 		}
 
 		//sends page state in header
-		if(($handler = $this->getCallbackEventTarget()) !== null)
+		if (($handler = $this->getCallbackEventTarget()) !== null)
 		{
-			if($handler->getActiveControl()->getClientSide()->getEnablePageStateUpdate())
+			if ($handler->getActiveControl()->getClientSide()->getEnablePageStateUpdate())
 			{
 				$pagestate = $this->getPage()->getClientState();
 				$this->appendContentPart($response, self::CALLBACK_PAGESTATE_HEADER, $pagestate);
@@ -190,7 +190,7 @@ 
 		$writer->write(" ");
 
 		//output the end javascript
-		if($this->getPage()->getClientScript()->hasEndScripts())
+		if ($this->getPage()->getClientScript()->hasEndScripts())
 		{
 			$writer = $response->createHtmlWriter();
 			$this->getPage()->getClientScript()->renderEndScriptsCallback($writer);
@@ -207,22 +207,22 @@ 
 
 		// collect all stylesheet file references
 		$stylesheets = $cs->getStyleSheetUrls();
-		if (count($stylesheets)>0)
+		if (count($stylesheets) > 0)
 		$this->appendContentPart($response, self::CALLBACK_STYLESHEETLIST_HEADER, TJavaScript::jsonEncode($stylesheets));
 
 		// collect all stylesheet snippets references
 		$stylesheets = $cs->getStyleSheetCodes();
-		if (count($stylesheets)>0)
+		if (count($stylesheets) > 0)
 		$this->appendContentPart($response, self::CALLBACK_STYLESHEET_HEADER, TJavaScript::jsonEncode($stylesheets));
 
 		// collect all script file references
 		$scripts = $cs->getScriptUrls();
-		if (count($scripts)>0)
+		if (count($scripts) > 0)
 		$this->appendContentPart($response, self::CALLBACK_SCRIPTLIST_HEADER, TJavaScript::jsonEncode($scripts));
 
 		// collect all hidden field references
 		$fields = $cs->getHiddenFields();
-		if (count($fields)>0)
+		if (count($fields) > 0)
 		$this->appendContentPart($response, self::CALLBACK_HIDDENFIELDLIST_HEADER, TJavaScript::jsonEncode($fields));
 	}
 
@@ -244,9 +244,9 @@ 
 	 */
 	private function raiseCallbackEvent()
 	{
-		 if(($callbackHandler=$this->getCallbackEventTarget())!==null)
+		 if (($callbackHandler = $this->getCallbackEventTarget()) !== null)
 		 {
-			if($callbackHandler instanceof ICallbackEventHandler)
+			if ($callbackHandler instanceof ICallbackEventHandler)
 			{
 				$param = $this->getCallbackEventParameter();
 				$result = new TCallbackEventParameter($this->getResponse(), $param);
@@ -271,11 +271,11 @@ 
 	 */
 	public function getCallbackEventTarget()
 	{
-		if($this->_callbackEventTarget===null)
+		if ($this->_callbackEventTarget === null)
 		{
-			$eventTarget=$this->getRequest()->itemAt(TPage::FIELD_CALLBACK_TARGET);
-			if(!empty($eventTarget))
-				$this->_callbackEventTarget=$this->getPage()->findControl($eventTarget);
+			$eventTarget = $this->getRequest()->itemAt(TPage::FIELD_CALLBACK_TARGET);
+			if (!empty($eventTarget))
+				$this->_callbackEventTarget = $this->getPage()->findControl($eventTarget);
 		}
 		return $this->_callbackEventTarget;
 	}
@@ -286,7 +286,7 @@ 
 	 */
 	public function setCallbackEventTarget(TControl $control)
 	{
-		$this->_callbackEventTarget=$control;
+		$this->_callbackEventTarget = $control;
 	}
 
 	/**
@@ -295,10 +295,10 @@ 
 	 */
 	public function getCallbackEventParameter()
 	{
-		if($this->_callbackEventParameter===null)
+		if ($this->_callbackEventParameter === null)
 		{
 			$param = $this->getRequest()->itemAt(TPage::FIELD_CALLBACK_PARAMETER);
-			$this->_callbackEventParameter=$param;
+			$this->_callbackEventParameter = $param;
 		}
 		return $this->_callbackEventParameter;
 	}
@@ -308,7 +308,7 @@ 
 	 */
 	public function setCallbackEventParameter($value)
 	{
-		$this->_callbackEventParameter=$value;
+		$this->_callbackEventParameter = $value;
 	}
 
 	/**
@@ -318,7 +318,7 @@ 
 	 */
 	public function getCallbackClientHandler()
 	{
-		if($this->_callbackClient===null)
+		if ($this->_callbackClient === null)
 			$this->_callbackClient = new TCallbackClientScript;
 		return $this->_callbackClient;
 	}
@@ -345,7 +345,7 @@ 
 	 */
 	protected function displayException($exception)
 	{
-		if($this->getApplication()->getMode()===TApplication::STATE_DEBUG)
+		if ($this->getApplication()->getMode() === TApplication::STATE_DEBUG)
 		{
 			$response = $this->getApplication()->getResponse();
 			$trace = $this->getExceptionStackTrace($exception);
@@ -354,7 +354,7 @@ 
 				$trace = TJavaScript::jsonEncode($trace);
 			} catch (Exception $e) {
 				// strip everythin not 7bit ascii
-				$trace = preg_replace('/[^(\x20-\x7F)]*/','', serialize($trace));
+				$trace = preg_replace('/[^(\x20-\x7F)]*/', '', serialize($trace));
 			}
 
 			// avoid exception loop if headers have already been sent
@@ -368,7 +368,7 @@ 
 		}
 		else
 		{
-			error_log("Error happened while processing an existing error:\n".$exception->__toString());
+			error_log("Error happened while processing an existing error:\n" . $exception->__toString());
 			header('HTTP/1.0 500 Internal Server Error', true, 500);
 		}
 		$this->getApplication()->getResponse()->flush();
@@ -380,24 +380,24 @@ 
 	 */
 	private function getExceptionStackTrace($exception)
 	{
-		$data['code']=$exception->getCode() > 0 ? $exception->getCode() : 500;
-		$data['file']=$exception->getFile();
-		$data['line']=$exception->getLine();
-		$data['trace']=$exception->getTrace();
-		if($exception instanceof TPhpErrorException)
+		$data['code'] = $exception->getCode() > 0 ? $exception->getCode() : 500;
+		$data['file'] = $exception->getFile();
+		$data['line'] = $exception->getLine();
+		$data['trace'] = $exception->getTrace();
+		if ($exception instanceof TPhpErrorException)
 		{
 			// if PHP exception, we want to show the 2nd stack level context
 			// because the 1st stack level is of little use (it's in error handler)
-			if(isset($trace[0]) && isset($trace[0]['file']) && isset($trace[0]['line']))
+			if (isset($trace[0]) && isset($trace[0]['file']) && isset($trace[0]['line']))
 			{
-				$data['file']=$trace[0]['file'];
-				$data['line']=$trace[0]['line'];
+				$data['file'] = $trace[0]['file'];
+				$data['line'] = $trace[0]['line'];
 			}
 		}
-		$data['type']=get_class($exception);
-		$data['message']=$exception->getMessage();
-		$data['version']=$_SERVER['SERVER_SOFTWARE'].' '.Prado::getVersion();
-		$data['time']=@strftime('%Y-%m-%d %H:%M',time());
+		$data['type'] = get_class($exception);
+		$data['message'] = $exception->getMessage();
+		$data['version'] = $_SERVER['SERVER_SOFTWARE'] . ' ' . Prado::getVersion();
+		$data['time'] = @strftime('%Y-%m-%d %H:%M', time());
 		return $data;
 	}
 }

Braces +32 added lines, -26 removed lines

@@ -115,8 +115,9 @@ 
 	public function registerControlToRender($control,$writer)
 	{
 		$id = $control->getUniqueID();
-		if(!isset($this->_controlsToRender[$id]))
-			$this->_controlsToRender[$id] = array($control,$writer);
+		if(!isset($this->_controlsToRender[$id])) {
+					$this->_controlsToRender[$id] = array($control,$writer);
+		}
 	}
 
 	/**
@@ -134,10 +135,11 @@ 
 	public function renderCallbackResponse($writer)
 	{
 		Prado::trace("ActivePage renderCallbackResponse()",'System.Web.UI.ActiveControls.TActivePageAdapter');
-		if(($url = $this->getResponse()->getAdapter()->getRedirectedUrl())===null)
-			$this->renderResponse($writer);
-		else
-			$this->redirect($url);
+		if(($url = $this->getResponse()->getAdapter()->getRedirectedUrl())===null) {
+					$this->renderResponse($writer);
+		} else {
+					$this->redirect($url);
+		}
 	}
 
 	/**
@@ -159,8 +161,9 @@ 
 	{
 		Prado::trace("ActivePage renderResponse()",'System.Web.UI.ActiveControls.TActivePageAdapter');
 		//renders all the defered render() calls.
-		foreach($this->_controlsToRender as $rid => $forRender)
-			$forRender[0]->render($forRender[1]);
+		foreach($this->_controlsToRender as $rid => $forRender) {
+					$forRender[0]->render($forRender[1]);
+		}
 
 		$response = $this->getResponse();
 
@@ -207,23 +210,27 @@ 
 
 		// collect all stylesheet file references
 		$stylesheets = $cs->getStyleSheetUrls();
-		if (count($stylesheets)>0)
-		$this->appendContentPart($response, self::CALLBACK_STYLESHEETLIST_HEADER, TJavaScript::jsonEncode($stylesheets));
+		if (count($stylesheets)>0) {
+				$this->appendContentPart($response, self::CALLBACK_STYLESHEETLIST_HEADER, TJavaScript::jsonEncode($stylesheets));
+		}
 
 		// collect all stylesheet snippets references
 		$stylesheets = $cs->getStyleSheetCodes();
-		if (count($stylesheets)>0)
-		$this->appendContentPart($response, self::CALLBACK_STYLESHEET_HEADER, TJavaScript::jsonEncode($stylesheets));
+		if (count($stylesheets)>0) {
+				$this->appendContentPart($response, self::CALLBACK_STYLESHEET_HEADER, TJavaScript::jsonEncode($stylesheets));
+		}
 
 		// collect all script file references
 		$scripts = $cs->getScriptUrls();
-		if (count($scripts)>0)
-		$this->appendContentPart($response, self::CALLBACK_SCRIPTLIST_HEADER, TJavaScript::jsonEncode($scripts));
+		if (count($scripts)>0) {
+				$this->appendContentPart($response, self::CALLBACK_SCRIPTLIST_HEADER, TJavaScript::jsonEncode($scripts));
+		}
 
 		// collect all hidden field references
 		$fields = $cs->getHiddenFields();
-		if (count($fields)>0)
-		$this->appendContentPart($response, self::CALLBACK_HIDDENFIELDLIST_HEADER, TJavaScript::jsonEncode($fields));
+		if (count($fields)>0) {
+				$this->appendContentPart($response, self::CALLBACK_HIDDENFIELDLIST_HEADER, TJavaScript::jsonEncode($fields));
+		}
 	}
 
 	/**
@@ -251,14 +258,12 @@ 
 				$param = $this->getCallbackEventParameter();
 				$result = new TCallbackEventParameter($this->getResponse(), $param);
 				$callbackHandler->raiseCallbackEvent($result);
-			}
-			else
+			} else
 			{
 				throw new TInvalidCallbackException(
 					'callback_invalid_handler', $callbackHandler->getUniqueID());
 			}
-		 }
-		 else
+		 } else
 		 {
 		 	$target = $this->getRequest()->itemAt(TPage::FIELD_CALLBACK_TARGET);
 		 	throw new TInvalidCallbackException('callback_invalid_target', $target);
@@ -274,8 +279,9 @@ 
 		if($this->_callbackEventTarget===null)
 		{
 			$eventTarget=$this->getRequest()->itemAt(TPage::FIELD_CALLBACK_TARGET);
-			if(!empty($eventTarget))
-				$this->_callbackEventTarget=$this->getPage()->findControl($eventTarget);
+			if(!empty($eventTarget)) {
+							$this->_callbackEventTarget=$this->getPage()->findControl($eventTarget);
+			}
 		}
 		return $this->_callbackEventTarget;
 	}
@@ -318,8 +324,9 @@ 
 	 */
 	public function getCallbackClientHandler()
 	{
-		if($this->_callbackClient===null)
-			$this->_callbackClient = new TCallbackClientScript;
+		if($this->_callbackClient===null) {
+					$this->_callbackClient = new TCallbackClientScript;
+		}
 		return $this->_callbackClient;
 	}
 }
@@ -365,8 +372,7 @@ 
 			$content = $response->createHtmlWriter();
 			$content->getWriter()->setBoundary(TActivePageAdapter::CALLBACK_ERROR_HEADER);
 			$content->write($trace);
-		}
-		else
+		} else
 		{
 			error_log("Error happened while processing an existing error:\n".$exception->__toString());
 			header('HTTP/1.0 500 Internal Server Error', true, 500);

framework/Web/UI/ActiveControls/TActiveRatingList.php

Doc Comments +1 added lines

@@ -99,6 +99,7 @@
 	 * Calls the client-side static method for this control class.
 	 * @param string static method name
 	 * @param mixed method parmaeter
+	 * @param string $func
 	 */
 	protected function callClientFunction($func,$value)
 	{

Spacing +7 added lines, -7 removed lines

@@ -82,7 +82,7 @@ 
 	{
 		parent::setReadOnly($value);
 		$value = $this->getReadOnly();
-		$this->callClientFunction('setReadOnly',$value);
+		$this->callClientFunction('setReadOnly', $value);
 	}
 
 	/**
@@ -92,7 +92,7 @@ 
 	{
 		parent::setRating($value);
 		$value = $this->getRating();
-		$this->callClientFunction('setRating',$value);
+		$this->callClientFunction('setRating', $value);
 	}
 
 	/**
@@ -100,13 +100,13 @@ 
 	 * @param string static method name
 	 * @param mixed method parmaeter
 	 */
-	protected function callClientFunction($func,$value)
+	protected function callClientFunction($func, $value)
 	{
-		if($this->getActiveControl()->canUpdateClientSide())
+		if ($this->getActiveControl()->canUpdateClientSide())
 		{
 			$client = $this->getPage()->getCallbackClient();
-			$code = 'Prado.Registry[\''.$this->ClientID.'\'].'.$func.'('.$value.')';
-			$client->evaluateScript($code,array($value));
+			$code = 'Prado.Registry[\'' . $this->ClientID . '\'].' . $func . '(' . $value . ')';
+			$client->evaluateScript($code, array($value));
 		}
 	}
 
@@ -117,7 +117,7 @@ 
 	{
 		parent::setCaption($value);
 		// if it's an active control, this should not be needed.
-		$this->callClientFunction('setCaption',$value);
+		$this->callClientFunction('setCaption', $value);
 	}
 
 	/**

framework/Web/UI/ActiveControls/TActiveTableCell.php

Doc Comments +2 added lines

@@ -111,6 +111,7 @@ 
 	 * method, be sure to call the parent implementation so that the event
 	 * handler can be invoked.
 	 * @param TActiveTableCellEventParameter event parameter to be passed to the event handlers
+	 * @param TActiveTableCellEventParameter $param
 	 */
 	public function onCellSelected($param)
 	{
@@ -232,6 +233,7 @@ 
 
 	/**
 	 * Creates a new TActiveTableRowEventParameter.
+	 * @param THttpResponse $response
 	 */
 	public function __construct($response, $parameter, $index=-1)
 	{

Indentation +4 added lines, -4 removed lines

@@ -53,8 +53,8 @@ 
 {
 
 	/**
-	* @var TTable parent row control containing the cell
-	*/
+	 * @var TTable parent row control containing the cell
+	 */
 	private $_row;
 
 	/**
@@ -226,8 +226,8 @@ 
 {
 
 	/**
-	* @var integer the zero-based index of the cell.
-	*/
+	 * @var integer the zero-based index of the cell.
+	 */
 	private $_selectedCellIndex = -1;
 
 	/**

Braces +18 added lines, -12 removed lines

@@ -127,8 +127,9 @@ 
 	{
 		parent::addAttributesToRender($writer);
 		$writer->addAttribute('id', $this->getClientID());
-		if ($this->hasEventHandler('OnCellSelected'))
-			$this->getActiveControl()->registerCallbackClientScript($this->getClientClassName(), $this->getPostBackOptions());
+		if ($this->hasEventHandler('OnCellSelected')) {
+					$this->getActiveControl()->registerCallbackClientScript($this->getClientClassName(), $this->getPostBackOptions());
+		}
 	}
 
 	/**
@@ -143,17 +144,18 @@ 
 		if ($this->getHasPreRendered())
 		{
 			parent::render($writer);
-			if ($this->getActiveControl()->canUpdateClientSide())
-				$this->getPage()->getCallbackClient()->replaceContent($this, $writer);
-		}
-		else {
+			if ($this->getActiveControl()->canUpdateClientSide()) {
+							$this->getPage()->getCallbackClient()->replaceContent($this, $writer);
+			}
+		} else {
 			$this->getPage()->getAdapter()->registerControlToRender($this, $writer);
 			// If we update a TActiveTableCell on callback, we shouldn't update all childs,
 			// because the whole content will be replaced by the parent.
 			if ($this->getHasControls())
 			{
-				foreach ($this->findControlsByType('IActiveControl', false) as $control)
-					$control->getActiveControl()->setEnableUpdate(false);
+				foreach ($this->findControlsByType('IActiveControl', false) as $control) {
+									$control->getActiveControl()->setEnableUpdate(false);
+				}
 			}
 		}
 	}
@@ -178,8 +180,9 @@ 
 	 */
 	public function getCellIndex()
 	{
-		foreach ($this->getRow()->getCells() as $key => $row)
-			if ($row == $this) return $key;
+		foreach ($this->getRow()->getCells() as $key => $row) {
+					if ($row == $this) return $key;
+		}
 		throw new TConfigurationException('tactivetablecell_control_notincollection', get_class($this), $this->getUniqueID());
 	}
 
@@ -197,8 +200,11 @@ 
 			{
 				$row = $row->getParent();
 			}
-			if ($row instanceof TTableRow) $this->_row = $row;
-			else throw new TConfigurationException('tactivetablecell_control_outoftable', get_class($this), $this->getUniqueID());
+			if ($row instanceof TTableRow) {
+				$this->_row = $row;
+			} else {
+				throw new TConfigurationException('tactivetablecell_control_outoftable', get_class($this), $this->getUniqueID());
+			}
 		}
 		return $this->_row;
 	}

Spacing +1 added lines, -1 removed lines

@@ -251,7 +251,7 @@
 	/**
 	 * Creates a new TActiveTableRowEventParameter.
 	 */
-	public function __construct($response, $parameter, $index=-1)
+	public function __construct($response, $parameter, $index = -1)
 	{
 		parent::__construct($response, $parameter);
 		$this->_selectedRowIndex = $index;