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 +1 added lines, -1 removed lines

@@ -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)
 	{

Spacing +26 added lines, -26 removed lines

@@ -67,7 +67,7 @@ 
  * @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
@@ -133,9 +133,9 @@ 
 		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'));
+				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());
+				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;
@@ -175,7 +175,7 @@ 
 	 */
 	public function regenerate($deleteOld=false)
 	{
-		$old = $this->getSessionID();
+		$old=$this->getSessionID();
 		session_regenerate_id($deleteOld);
 		return $old;
 	}
@@ -227,7 +227,7 @@ 
 		else if(ctype_alnum($value))
 			session_name($value);
 		else
-			throw new TInvalidDataValueException('httpsession_sessionname_invalid',$value);
+			throw new TInvalidDataValueException('httpsession_sessionname_invalid', $value);
 	}
 
 	/**
@@ -249,7 +249,7 @@ 
 		else if(is_dir($value))
 			session_save_path($value);
 		else
-			throw new TInvalidDataValueException('httpsession_savepath_invalid',$value);
+			throw new TInvalidDataValueException('httpsession_savepath_invalid', $value);
 	}
 
 	/**
@@ -277,7 +277,7 @@ 
 	public function getCookie()
 	{
 		if($this->_cookie===null)
-			$this->_cookie=new THttpCookie($this->getSessionName(),$this->getSessionID());
+			$this->_cookie=new THttpCookie($this->getSessionName(), $this->getSessionID());
 		return $this->_cookie;
 	}
 
@@ -304,21 +304,21 @@ 
 			throw new TInvalidOperationException('httpsession_cookiemode_unchangeable');
 		else
 		{
-			$value=TPropertyValue::ensureEnum($value,'THttpSessionCookieMode');
+			$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)
 			{
-				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);
 			}
 		}
@@ -364,13 +364,13 @@ 
 		else
 		{
 			$value=TPropertyValue::ensureInteger($value);
-			if($value>=0 && $value<=100)
+			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);
 		}
 	}
 
@@ -392,9 +392,9 @@ 
 		else
 		{
 			$value=TPropertyValue::ensureBoolean($value);
-			if ($value && $this->getCookieMode()==THttpSessionCookieMode::Only)
+			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');
 		}
 	}
 
@@ -415,7 +415,7 @@ 
 		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,7 +540,7 @@ 
 	 * @param mixed session variable name
 	 * @param mixed session variable value
 	 */
-	public function add($key,$value)
+	public function add($key, $value)
 	{
 		$_SESSION[$key]=$value;
 	}
@@ -613,7 +613,7 @@ 
 	 * @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;
 	}
@@ -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;
 	}
 
 	/**

Braces +4 added lines, -8 removed lines

@@ -309,13 +309,11 @@ 
       {
 				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');
@@ -368,8 +366,7 @@ 
 			{
 				ini_set('session.gc_probability',$value);
 				ini_set('session.gc_divisor','100');
-			}
-			else
+			} else
 				throw new TInvalidDataValueException('httpsession_gcprobability_invalid',$value);
 		}
 	}
@@ -557,8 +554,7 @@ 
 			$value=$_SESSION[$key];
 			unset($_SESSION[$key]);
 			return $value;
-		}
-		else
+		} else
 			return null;
 	}
 

Indentation +10 added lines, -10 removed lines

@@ -1,13 +1,13 @@ 
 <?php
 /**
- * THttpSession class
- *
- * @author Qiang Xue <qiang.xue@gmail.com>
- * @link https://github.com/pradosoft/prado
- * @copyright Copyright &copy; 2005-2016 The PRADO Group
- * @license https://github.com/pradosoft/prado/blob/master/COPYRIGHT
- * @package System.Web
- */
+	 * THttpSession class
+	 *
+	 * @author Qiang Xue <qiang.xue@gmail.com>
+	 * @link https://github.com/pradosoft/prado
+	 * @copyright Copyright &copy; 2005-2016 The PRADO Group
+	 * @license https://github.com/pradosoft/prado/blob/master/COPYRIGHT
+	 * @package System.Web
+	 */
 
 /**
  * THttpSession class
@@ -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');

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 +10 added lines, -10 removed lines

@@ -36,7 +36,7 @@ 
 	/**
 	 * @var boolean true if validation is made during a callback request.
 	 */
-	private $_isCallback = false;
+	private $_isCallback=false;
 
 	/**
 	 * @return boolean true if validation is made during a callback request.
@@ -93,8 +93,8 @@ 
 	 */
  	public function raiseCallbackEvent($param)
 	{
-		$this->_isCallback = true;
-		$result = $this->onServerValidate($param->getCallbackParameter());
+		$this->_isCallback=true;
+		$result=$this->onServerValidate($param->getCallbackParameter());
 		$param->setResponseData($result);
 		$this->onCallback($param);
 	}
@@ -110,8 +110,8 @@ 
 		parent::setIsValid($value);
 		if($this->getActiveControl()->canUpdateClientSide())
 		{
-			$client = $this->getPage()->getCallbackClient();
-			$func = 'Prado.Validation.updateActiveCustomValidator';
+			$client=$this->getPage()->getCallbackClient();
+			$func='Prado.Validation.updateActiveCustomValidator';
 			$client->callClientFunction($func, array($this, $value));
 		}
 	}
@@ -135,7 +135,7 @@ 
 	protected function getClientScriptOptions()
 	{
 		$options=TBaseValidator::getClientScriptOptions();
-		$options['EventTarget'] = $this->getUniqueID();
+		$options['EventTarget']=$this->getUniqueID();
 		return $options;
 	}
 
@@ -145,15 +145,15 @@ 
 	 */
 	public function setErrorMessage($value)
 	{
-		if(parent::getErrorMessage() === $value)
+		if(parent::getErrorMessage()===$value)
 			return;
 
 
 		parent::setErrorMessage($value);
 		if($this->getActiveControl()->canUpdateClientSide())
 		{
-			$client = $this->getPage()->getCallbackClient();
-			$func = 'Prado.Validation.setErrorMessage';
+			$client=$this->getPage()->getCallbackClient();
+			$func='Prado.Validation.setErrorMessage';
 			$client->callClientFunction($func, array($this, $value));
 		}
 	}
@@ -263,7 +263,7 @@ 
 	 */
 	public function getObserveChanges()
 	{
-		$changes = $this->getOption('ObserveChanges');
+		$changes=$this->getOption('ObserveChanges');
 		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 +7 added lines, -7 removed lines

@@ -62,7 +62,7 @@ 
 	 */
 	protected function createListItemCollection()
 	{
-		$collection  = new TActiveListItemCollection;
+		$collection=new TActiveListItemCollection;
 		$collection->setControl($this);
 		return $collection;
 	}
@@ -85,15 +85,15 @@ 
 	 */
 	public function setSelectionMode($value)
 	{
-		if(parent::getSelectionMode() === $value)
+		if(parent::getSelectionMode()===$value)
 			return;
 
 		parent::setSelectionMode($value);
-		$multiple = $this->getIsMultiSelect();
-		$id = $this->getUniqueID(); $multi_id = $id.'[]';
+		$multiple=$this->getIsMultiSelect();
+		$id=$this->getUniqueID(); $multi_id=$id.'[]';
 		if($this->getActiveControl()->canUpdateClientSide())
 		{
-			$client = $this->getPage()->getCallbackClient();
+			$client=$this->getPage()->getCallbackClient();
 			$client->setAttribute($this, 'multiple', $multiple ? 'multiple' : false);
 			$client->setAttribute($this, 'name', $multiple ? $multi_id : $id);
 		}
@@ -146,8 +146,8 @@ 
 	protected function addAttributesToRender($writer)
 	{
 		parent::addAttributesToRender($writer);
-		$writer->addAttribute('id',$this->getClientID());
-		if ($this->getAutoPostBack())
+		$writer->addAttribute('id', $this->getClientID());
+		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 +49 added lines, -49 removed lines

@@ -32,40 +32,40 @@ 
 	/**
 	 * Callback response data header name.
 	 */
-	const CALLBACK_DATA_HEADER = 'X-PRADO-DATA';
+	const CALLBACK_DATA_HEADER='X-PRADO-DATA';
 	/**
 	 * Callback response client-side action header name.
 	 */
-	const CALLBACK_ACTION_HEADER = 'X-PRADO-ACTIONS';
+	const CALLBACK_ACTION_HEADER='X-PRADO-ACTIONS';
 	/**
 	 * Callback error header name.
 	 */
-	const CALLBACK_ERROR_HEADER = 'X-PRADO-ERROR';
+	const CALLBACK_ERROR_HEADER='X-PRADO-ERROR';
 	/**
 	 * Callback page state header name.
 	 */
-	const CALLBACK_PAGESTATE_HEADER = 'X-PRADO-PAGESTATE';
+	const CALLBACK_PAGESTATE_HEADER='X-PRADO-PAGESTATE';
 	/**
 	 * Script list header name.
 	 */
-	const CALLBACK_SCRIPTLIST_HEADER = 'X-PRADO-SCRIPTLIST';
+	const CALLBACK_SCRIPTLIST_HEADER='X-PRADO-SCRIPTLIST';
 	/**
 	 * Stylesheet list header name.
 	 */
-	const CALLBACK_STYLESHEETLIST_HEADER = 'X-PRADO-STYLESHEETLIST';
+	const CALLBACK_STYLESHEETLIST_HEADER='X-PRADO-STYLESHEETLIST';
 	/**
 	 * Stylesheet header name.
 	 */
-	const CALLBACK_STYLESHEET_HEADER = 'X-PRADO-STYLESHEET';
+	const CALLBACK_STYLESHEET_HEADER='X-PRADO-STYLESHEET';
 	/**
 	 * Hidden field list header name.
 	 */
-	const CALLBACK_HIDDENFIELDLIST_HEADER = 'X-PRADO-HIDDENFIELDLIST';
+	const CALLBACK_HIDDENFIELDLIST_HEADER='X-PRADO-HIDDENFIELDLIST';
 
 	/**
 	 * Callback redirect url header name.
 	 */
-	const CALLBACK_REDIRECT = 'X-PRADO-REDIRECT';
+	const CALLBACK_REDIRECT='X-PRADO-REDIRECT';
 
 	/**
 	 * @var ICallbackEventHandler callback event handler.
@@ -91,7 +91,7 @@ 
 		parent::__construct($control);
 
 		//TODO: can this be done later?
-		$response = $this->getApplication()->getResponse();
+		$response=$this->getApplication()->getResponse();
 		$response->setAdapter(new TCallbackResponseAdapter($response));
 
 		$this->trapCallbackErrorsExceptions();
@@ -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();
+		$id=$control->getUniqueID();
 		if(!isset($this->_controlsToRender[$id]))
-			$this->_controlsToRender[$id] = array($control,$writer);
+			$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,31 +157,31 @@ 
 	 */
 	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)
 			$forRender[0]->render($forRender[1]);
 
-		$response = $this->getResponse();
+		$response=$this->getResponse();
 
 		//send response data in header
 		if($response->getHasAdapter())
 		{
-			$responseData = $response->getAdapter()->getResponseData();
+			$responseData=$response->getAdapter()->getResponseData();
 			if($responseData!==null)
 			{
-				$data = TJavaScript::jsonEncode($responseData);
+				$data=TJavaScript::jsonEncode($responseData);
 
 				$this->appendContentPart($response, self::CALLBACK_DATA_HEADER, $data);
 			}
 		}
 
 		//sends page state in header
-		if(($handler = $this->getCallbackEventTarget()) !== null)
+		if(($handler=$this->getCallbackEventTarget())!==null)
 		{
 			if($handler->getActiveControl()->getClientSide()->getEnablePageStateUpdate())
 			{
-				$pagestate = $this->getPage()->getClientState();
+				$pagestate=$this->getPage()->getClientState();
 				$this->appendContentPart($response, self::CALLBACK_PAGESTATE_HEADER, $pagestate);
 			}
 		}
@@ -192,37 +192,37 @@ 
 		//output the end javascript
 		if($this->getPage()->getClientScript()->hasEndScripts())
 		{
-			$writer = $response->createHtmlWriter();
+			$writer=$response->createHtmlWriter();
 			$this->getPage()->getClientScript()->renderEndScriptsCallback($writer);
 			$this->getPage()->getCallbackClient()->evaluateScript($writer);
 		}
 
 		//output the actions
-		$executeJavascript = $this->getCallbackClientHandler()->getClientFunctionsToExecute();
-		$actions = TJavaScript::jsonEncode($executeJavascript);
+		$executeJavascript=$this->getCallbackClientHandler()->getClientFunctionsToExecute();
+		$actions=TJavaScript::jsonEncode($executeJavascript);
 		$this->appendContentPart($response, self::CALLBACK_ACTION_HEADER, $actions);
 
 
-		$cs = $this->Page->getClientScript();
+		$cs=$this->Page->getClientScript();
 
 		// collect all stylesheet file references
-		$stylesheets = $cs->getStyleSheetUrls();
-		if (count($stylesheets)>0)
+		$stylesheets=$cs->getStyleSheetUrls();
+		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)
+		$stylesheets=$cs->getStyleSheetCodes();
+		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)
+		$scripts=$cs->getScriptUrls();
+		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)
+		$fields=$cs->getHiddenFields();
+		if(count($fields) > 0)
 		$this->appendContentPart($response, self::CALLBACK_HIDDENFIELDLIST_HEADER, TJavaScript::jsonEncode($fields));
 	}
 
@@ -231,7 +231,7 @@ 
 	 */
 	private function appendContentPart($response, $delimiter, $data)
 	{
-		$content = $response->createHtmlWriter();
+		$content=$response->createHtmlWriter();
 		$content->getWriter()->setBoundary($delimiter);
 		$content->write($data);
 	}
@@ -248,8 +248,8 @@ 
 		 {
 			if($callbackHandler instanceof ICallbackEventHandler)
 			{
-				$param = $this->getCallbackEventParameter();
-				$result = new TCallbackEventParameter($this->getResponse(), $param);
+				$param=$this->getCallbackEventParameter();
+				$result=new TCallbackEventParameter($this->getResponse(), $param);
 				$callbackHandler->raiseCallbackEvent($result);
 			}
 			else
@@ -260,7 +260,7 @@ 
 		 }
 		 else
 		 {
-		 	$target = $this->getRequest()->itemAt(TPage::FIELD_CALLBACK_TARGET);
+		 	$target=$this->getRequest()->itemAt(TPage::FIELD_CALLBACK_TARGET);
 		 	throw new TInvalidCallbackException('callback_invalid_target', $target);
 		 }
 	}
@@ -297,7 +297,7 @@ 
 	{
 		if($this->_callbackEventParameter===null)
 		{
-			$param = $this->getRequest()->itemAt(TPage::FIELD_CALLBACK_PARAMETER);
+			$param=$this->getRequest()->itemAt(TPage::FIELD_CALLBACK_PARAMETER);
 			$this->_callbackEventParameter=$param;
 		}
 		return $this->_callbackEventParameter;
@@ -319,7 +319,7 @@ 
 	public function getCallbackClientHandler()
 	{
 		if($this->_callbackClient===null)
-			$this->_callbackClient = new TCallbackClientScript;
+			$this->_callbackClient=new TCallbackClientScript;
 		return $this->_callbackClient;
 	}
 }
@@ -347,22 +347,22 @@ 
 	{
 		if($this->getApplication()->getMode()===TApplication::STATE_DEBUG)
 		{
-			$response = $this->getApplication()->getResponse();
-			$trace = $this->getExceptionStackTrace($exception);
+			$response=$this->getApplication()->getResponse();
+			$trace=$this->getExceptionStackTrace($exception);
 			// avoid error on non-utf8 strings
 			try {
-				$trace = TJavaScript::jsonEncode($trace);
-			} catch (Exception $e) {
+				$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
 			try {
 				$response->setStatusCode(500, 'Internal Server Error');
-			} catch (Exception $e) { }
+			} catch(Exception $e) { }
 
-			$content = $response->createHtmlWriter();
+			$content=$response->createHtmlWriter();
 			$content->getWriter()->setBoundary(TActivePageAdapter::CALLBACK_ERROR_HEADER);
 			$content->write($trace);
 		}
@@ -397,7 +397,7 @@ 
 		$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['time']=@strftime('%Y-%m-%d %H:%M', time());
 		return $data;
 	}
 }

Braces +3 added lines, -6 removed lines

@@ -251,14 +251,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);
@@ -365,8 +363,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);

Indentation +9 added lines, -9 removed lines

@@ -1,14 +1,14 @@
 <?php
 /**
- * TActivePageAdapter, TCallbackErrorHandler and TInvalidCallbackException class file.
- *
- * @author Wei Zhuo <weizhuo[at]gamil[dot]com>
- * @author Gabor Berczi <gabor.berczi@devworx.hu> (lazyload additions & progressive rendering)
- * @link https://github.com/pradosoft/prado
- * @copyright Copyright &copy; 2005-2016 The PRADO Group
- * @license https://github.com/pradosoft/prado/blob/master/COPYRIGHT
- * @package System.Web.UI.ActiveControls
- */
+	 * TActivePageAdapter, TCallbackErrorHandler and TInvalidCallbackException class file.
+	 *
+	 * @author Wei Zhuo <weizhuo[at]gamil[dot]com>
+	 * @author Gabor Berczi <gabor.berczi@devworx.hu> (lazyload additions & progressive rendering)
+	 * @link https://github.com/pradosoft/prado
+	 * @copyright Copyright &copy; 2005-2016 The PRADO Group
+	 * @license https://github.com/pradosoft/prado/blob/master/COPYRIGHT
+	 * @package System.Web.UI.ActiveControls
+	 */
 
 /**
  * Load callback response adapter class.

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 +12 added lines, -12 removed lines

@@ -80,12 +80,12 @@ 
 	 */
 	public function setReadOnly($value)
 	{
-		if(parent::getReadOnly() === $value)
+		if(parent::getReadOnly()===$value)
 			return;
 
 		parent::setReadOnly($value);
-		$value = $this->getReadOnly();
-		$this->callClientFunction('setReadOnly',$value);
+		$value=$this->getReadOnly();
+		$this->callClientFunction('setReadOnly', $value);
 	}
 
 	/**
@@ -93,12 +93,12 @@ 
 	 */
 	public function setRating($value)
 	{
-		if(parent::getRating() === $value)
+		if(parent::getRating()===$value)
 			return;
 
 		parent::setRating($value);
-		$value = $this->getRating();
-		$this->callClientFunction('setRating',$value);
+		$value=$this->getRating();
+		$this->callClientFunction('setRating', $value);
 	}
 
 	/**
@@ -106,13 +106,13 @@ 
 	 * @param string static method name
 	 * @param mixed method parmaeter
 	 */
-	protected function callClientFunction($func,$value)
+	protected function callClientFunction($func, $value)
 	{
 		if($this->getActiveControl()->canUpdateClientSide())
 		{
-			$client = $this->getPage()->getCallbackClient();
-			$code = 'Prado.Registry[\''.$this->ClientID.'\'].'.$func.'('.$value.')';
-			$client->evaluateScript($code,array($value));
+			$client=$this->getPage()->getCallbackClient();
+			$code='Prado.Registry[\''.$this->ClientID.'\'].'.$func.'('.$value.')';
+			$client->evaluateScript($code, array($value));
 		}
 	}
 
@@ -121,12 +121,12 @@ 
 	 */
 	public function setCaption($value)
 	{
-		if(parent::getCaption() === $value)
+		if(parent::getCaption()===$value)
 			return;
 
 		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;
 
 	/**

Spacing +17 added lines, -17 removed lines

@@ -100,7 +100,7 @@ 
 	 */
 	public function raiseCallbackEvent($param)
 	{
-		$parameter = new TActiveTableCellEventParameter($this->getResponse(), $param->getCallbackParameter(), $this->getCellIndex());
+		$parameter=new TActiveTableCellEventParameter($this->getResponse(), $param->getCallbackParameter(), $this->getCellIndex());
 		$this->onCellSelected($parameter);
 		$this->raiseBubbleEvent($this, $parameter);
 	}
@@ -127,7 +127,7 @@ 
 	{
 		parent::addAttributesToRender($writer);
 		$writer->addAttribute('id', $this->getClientID());
-		if ($this->hasEventHandler('OnCellSelected'))
+		if($this->hasEventHandler('OnCellSelected'))
 			$this->getActiveControl()->registerCallbackClientScript($this->getClientClassName(), $this->getPostBackOptions());
 	}
 
@@ -140,19 +140,19 @@ 
 	 */
 	public function render($writer)
 	{
-		if ($this->getHasPreRendered())
+		if($this->getHasPreRendered())
 		{
 			parent::render($writer);
-			if ($this->getActiveControl()->canUpdateClientSide())
+			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())
+			if($this->getHasControls())
 			{
-				foreach ($this->findControlsByType('IActiveControl', false) as $control)
+				foreach($this->findControlsByType('IActiveControl', false) as $control)
 					$control->getActiveControl()->setEnableUpdate(false);
 			}
 		}
@@ -165,8 +165,8 @@ 
 	 */
 	protected function getPostBackOptions()
 	{
-		$options['ID'] = $this->getClientID();
-		$options['EventTarget'] = $this->getUniqueID();
+		$options['ID']=$this->getClientID();
+		$options['EventTarget']=$this->getUniqueID();
 		return $options;
 	}
 
@@ -178,8 +178,8 @@ 
 	 */
 	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());
 	}
 
@@ -190,14 +190,14 @@ 
 	 */
 	public function getRow()
 	{
-		if ($this->_row === null)
+		if($this->_row===null)
 		{
-			$row = $this->getParent();
-			while (!($row instanceof TTableRow) && $row !== null)
+			$row=$this->getParent();
+			while(!($row instanceof TTableRow) && $row!==null)
 			{
-				$row = $row->getParent();
+				$row=$row->getParent();
 			}
-			if ($row instanceof TTableRow) $this->_row = $row;
+			if($row instanceof TTableRow) $this->_row=$row;
 			else throw new TConfigurationException('tactivetablecell_control_outoftable', get_class($this), $this->getUniqueID());
 		}
 		return $this->_row;
@@ -228,7 +228,7 @@ 
 	/**
 	* @var integer the zero-based index of the cell.
 	*/
-	private $_selectedCellIndex = -1;
+	private $_selectedCellIndex=-1;
 
 	/**
 	 * Creates a new TActiveTableRowEventParameter.
@@ -236,7 +236,7 @@ 
 	public function __construct($response, $parameter, $index=-1)
 	{
 		parent::__construct($response, $parameter);
-		$this->_selectedCellIndex = $index;
+		$this->_selectedCellIndex=$index;
 	}
 
 	/**

Braces +1 added lines, -2 removed lines

@@ -95,8 +95,7 @@
 		if($this->getHasPreRendered()) {
 			$this->renderRepeater($writer);
 			if($this->getActiveControl()->canUpdateClientSide()) $this->getPage()->getCallbackClient()->replaceContent($this->getSurroundingTagId(),$writer);
-		}
-		else {
+		} else {
 			$this->getPage()->getAdapter()->registerControlToRender($this,$writer);
 		}
 	}

framework/Web/UI/ActiveControls/TActiveTableRow.php

Doc Comments +3 added lines

@@ -98,6 +98,7 @@ 
 	 * TActiveTableRow.
 	 * This method is mainly used by framework and control developers.
 	 * @param TCallbackEventParameter the event parameter
+	 * @param TActiveTableCellEventParameter $param
 	 */
 	public function raiseCallbackEvent($param)
 	{
@@ -129,6 +130,7 @@ 
 	 * method, be sure to call the parent implementation so that the event
 	 * handler can be invoked.
 	 * @param TActiveTableRowEventParameter event parameter to be passed to the event handlers
+	 * @param TActiveTableRowEventParameter $param
 	 */
 	public function onRowSelected($param)
 	{
@@ -250,6 +252,7 @@ 
 
 	/**
 	 * Creates a new TActiveTableRowEventParameter.
+	 * @param THttpResponse $response
 	 */
 	public function __construct($response, $parameter, $index=-1)
 	{

Spacing +18 added lines, -18 removed lines

@@ -101,7 +101,7 @@ 
 	 */
 	public function raiseCallbackEvent($param)
 	{
-		$parameter = new TActiveTableRowEventParameter($this->getResponse(), $param->getCallbackParameter(), $this->getRowIndex());
+		$parameter=new TActiveTableRowEventParameter($this->getResponse(), $param->getCallbackParameter(), $this->getRowIndex());
 		$this->onRowSelected($parameter);
 	}
 
@@ -115,7 +115,7 @@ 
 	 */
 	public function bubbleEvent($sender, $param)
 	{
-		if ($param instanceof TActiveTableCellEventParameter)
+		if($param instanceof TActiveTableCellEventParameter)
 		{
 			$this->raiseCallbackEvent($param);
 			return true;
@@ -145,7 +145,7 @@ 
 	{
 		parent::addAttributesToRender($writer);
 		$writer->addAttribute('id', $this->getClientID());
-		if ($this->hasEventHandler('OnRowSelected'))
+		if($this->hasEventHandler('OnRowSelected'))
 			$this->getActiveControl()->registerCallbackClientScript($this->getClientClassName(), $this->getPostBackOptions());
 	}
 
@@ -158,10 +158,10 @@ 
 	 */
 	public function render($writer)
 	{
-		if ($this->getHasPreRendered())
+		if($this->getHasPreRendered())
 		{
 			parent::render($writer);
-			if ($this->getActiveControl()->canUpdateClientSide())
+			if($this->getActiveControl()->canUpdateClientSide())
 				$this->getPage()->getCallbackClient()->replaceContent($this, $writer);
 		}
 		else
@@ -169,9 +169,9 @@ 
 			$this->getPage()->getAdapter()->registerControlToRender($this, $writer);
 			// If we update a TActiveTableRow on callback, we shouldn't update all childs,
 			// because the whole content will be replaced by the parent.
-			if ($this->getHasControls())
+			if($this->getHasControls())
 			{
-				foreach ($this->findControlsByType('IActiveControl', false) as $control)
+				foreach($this->findControlsByType('IActiveControl', false) as $control)
 					$control->getActiveControl()->setEnableUpdate(false);
 			}
 		}
@@ -184,8 +184,8 @@ 
 	 */
 	protected function getPostBackOptions()
 	{
-		$options['ID'] = $this->getClientID();
-		$options['EventTarget'] = $this->getUniqueID();
+		$options['ID']=$this->getClientID();
+		$options['EventTarget']=$this->getUniqueID();
 		return $options;
 	}
 
@@ -197,8 +197,8 @@ 
 	 */
 	public function getRowIndex()
 	{
-		foreach ($this->getTable()->getRows() as $key => $row)
-			if ($row == $this) return $key;
+		foreach($this->getTable()->getRows() as $key => $row)
+			if($row==$this) return $key;
 		throw new TConfigurationException('tactivetablerow_control_notincollection', get_class($this), $this->getUniqueID());
 	}
 
@@ -209,14 +209,14 @@ 
 	 */
 	public function getTable()
 	{
-		if ($this->_table === null)
+		if($this->_table===null)
 		{
-			$table = $this->getParent();
-			while (!($table instanceof TTable) && $table !== null)
+			$table=$this->getParent();
+			while(!($table instanceof TTable) && $table!==null)
 			{
-				$table = $table->getParent();
+				$table=$table->getParent();
 			}
-			if ($table instanceof TTable) $this->_table = $table;
+			if($table instanceof TTable) $this->_table=$table;
 			else throw new TConfigurationException('tactivetablerow_control_outoftable', get_class($this), $this->getUniqueID());
 		}
 		return $this->_table;
@@ -246,7 +246,7 @@ 
 	/**
 	* @var integer the zero-based index of the row.
 	*/
-	private $_selectedRowIndex = -1;
+	private $_selectedRowIndex=-1;
 
 	/**
 	 * Creates a new TActiveTableRowEventParameter.
@@ -254,7 +254,7 @@ 
 	public function __construct($response, $parameter, $index=-1)
 	{
 		parent::__construct($response, $parameter);
-		$this->_selectedRowIndex = $index;
+		$this->_selectedRowIndex=$index;
 	}
 
 	/**

Braces +2 added lines, -4 removed lines

@@ -119,8 +119,7 @@ 
 		{
 			$this->raiseCallbackEvent($param);
 			return true;
-		}
-		else return false;
+		} else return false;
 	}
 
 	/**
@@ -163,8 +162,7 @@ 
 			parent::render($writer);
 			if ($this->getActiveControl()->canUpdateClientSide())
 				$this->getPage()->getCallbackClient()->replaceContent($this, $writer);
-		}
-		else
+		} else
 		{
 			$this->getPage()->getAdapter()->registerControlToRender($this, $writer);
 			// If we update a TActiveTableRow on callback, we shouldn't update all childs,

Indentation +11 added lines, -11 removed lines

@@ -1,14 +1,14 @@ 
 <?php
 /**
- * TActiveTableRow and TActiveTableRowEventParameter class file
- *
- * @author LANDWEHR Computer und Software GmbH <programmierung@landwehr-software.de>
- * @link http://www.landwehr-software.de/
- * @copyright Copyright &copy; 2009 LANDWEHR Computer und Software GmbH
- * @license https://github.com/pradosoft/prado/blob/master/COPYRIGHT
- * @package System.Web.UI.ActiveControls
- * @version $Id$
- */
+	 * TActiveTableRow and TActiveTableRowEventParameter class file
+	 *
+	 * @author LANDWEHR Computer und Software GmbH <programmierung@landwehr-software.de>
+	 * @link http://www.landwehr-software.de/
+	 * @copyright Copyright &copy; 2009 LANDWEHR Computer und Software GmbH
+	 * @license https://github.com/pradosoft/prado/blob/master/COPYRIGHT
+	 * @package System.Web.UI.ActiveControls
+	 * @version $Id$
+	 */
 
 /**
  * Includes the following used classes
@@ -244,8 +244,8 @@ 
 class TActiveTableRowEventParameter extends TCallbackEventParameter
 {
 	/**
-	* @var integer the zero-based index of the row.
-	*/
+	 * @var integer the zero-based index of the row.
+	 */
 	private $_selectedRowIndex = -1;
 
 	/**

framework/Web/UI/ActiveControls/TAutoComplete.php

Doc Comments +6 added lines

@@ -191,6 +191,7 @@ 
 	 * method, be sure to call the parent implementation so that the event
 	 * handler can be invoked.
 	 * @param TCallbackEventParameter event parameter to be passed to the event handlers
+	 * @param TAutoCompleteEventParameter $param
 	 */
 	public function onSuggest($param)
 	{
@@ -203,6 +204,7 @@ 
 	 * method, be sure to call the parent implementation so that the event
 	 * handler can be invoked.
 	 * @param TCallbackEventParameter event parameter to be passed to the event handlers
+	 * @param TAutoCompleteEventParameter $param
 	 */
 	public function onSuggestionSelected($param)
 	{
@@ -383,6 +385,7 @@ 
 
 	/**
 	 * Creates a new TCallbackEventParameter.
+	 * @param THttpResponse $response
 	 */
 	public function __construct($response, $parameter, $index=-1)
 	{
@@ -421,6 +424,9 @@ 
 {
 	private $_template;
 
+	/**
+	 * @param string $template
+	 */
 	public function __construct($template)
 	{
 		$this->_template = $template;

Braces +2 added lines, -4 removed lines

@@ -174,14 +174,12 @@
 			{
 				$parameter = new TAutoCompleteEventParameter($this->getResponse(), $token[0]);
 				$this->onSuggest($parameter);
-			}
-			else if($token[1] === '__TAutoComplete_onSuggestionSelected__')
+			} else if($token[1] === '__TAutoComplete_onSuggestionSelected__')
 			{
 				$parameter = new TAutoCompleteEventParameter($this->getResponse(), null, $token[0]);
 				$this->onSuggestionSelected($parameter);
 			}
-		}
-		else if($this->getAutoPostBack())
+		} else if($this->getAutoPostBack())
 			parent::raiseCallbackEvent($param);
 	}
 

Spacing +31 added lines, -31 removed lines

@@ -120,7 +120,7 @@ 
 	 */
 	public function setFrequency($value)
 	{
-		$this->setViewState('frequency', TPropertyValue::ensureFloat($value),'');
+		$this->setViewState('frequency', TPropertyValue::ensureFloat($value), '');
 	}
 
 	/**
@@ -128,7 +128,7 @@ 
 	 */
 	public function getMinChars()
 	{
-		return $this->getViewState('minChars','');
+		return $this->getViewState('minChars', '');
 	}
 
 	/**
@@ -167,17 +167,17 @@ 
 	 */
  	public function raiseCallbackEvent($param)
 	{
-		$token = $param->getCallbackParameter();
-		if(is_array($token) && count($token) == 2)
+		$token=$param->getCallbackParameter();
+		if(is_array($token) && count($token)==2)
 		{
-			if($token[1] === '__TAutoComplete_onSuggest__')
+			if($token[1]==='__TAutoComplete_onSuggest__')
 			{
-				$parameter = new TAutoCompleteEventParameter($this->getResponse(), $token[0]);
+				$parameter=new TAutoCompleteEventParameter($this->getResponse(), $token[0]);
 				$this->onSuggest($parameter);
 			}
-			else if($token[1] === '__TAutoComplete_onSuggestionSelected__')
+			else if($token[1]==='__TAutoComplete_onSuggestionSelected__')
 			{
-				$parameter = new TAutoCompleteEventParameter($this->getResponse(), null, $token[0]);
+				$parameter=new TAutoCompleteEventParameter($this->getResponse(), null, $token[0]);
 				$this->onSuggestionSelected($parameter);
 			}
 		}
@@ -234,7 +234,7 @@ 
 	public function getResultPanel()
 	{
 		if($this->_resultPanel===null)
-			$this->_resultPanel = $this->createResultPanel();
+			$this->_resultPanel=$this->createResultPanel();
 		return $this->_resultPanel;
 	}
 
@@ -243,7 +243,7 @@ 
 	 */
 	protected function createResultPanel()
 	{
-		$panel = Prado::createComponent('System.Web.UI.WebControls.TPanel');
+		$panel=Prado::createComponent('System.Web.UI.WebControls.TPanel');
 		$this->getControls()->add($panel);
 		$panel->setID('result');
 		return $panel;
@@ -255,7 +255,7 @@ 
 	public function getSuggestions()
 	{
 		if($this->_repeater===null)
-			$this->_repeater = $this->createRepeater();
+			$this->_repeater=$this->createRepeater();
 		return $this->_repeater;
 	}
 
@@ -264,10 +264,10 @@ 
 	 */
 	protected function createRepeater()
 	{
-		$repeater = Prado::createComponent('System.Web.UI.WebControls.TRepeater');
+		$repeater=Prado::createComponent('System.Web.UI.WebControls.TRepeater');
 		$repeater->setHeaderTemplate(new TAutoCompleteTemplate('<ul>'));
 		$repeater->setFooterTemplate(new TAutoCompleteTemplate('</ul>'));
-		$repeater->setItemTemplate(new TTemplate('<li><%# $this->DataItem %></li>',null));
+		$repeater->setItemTemplate(new TTemplate('<li><%# $this->DataItem %></li>', null));
 		$repeater->setEmptyTemplate(new TAutoCompleteTemplate('<ul></ul>'));
 		$this->getControls()->add($repeater);
 		return $repeater;
@@ -311,7 +311,7 @@ 
 		if($this->getActiveControl()->canUpdateClientSide())
 		{
 			$this->getSuggestions()->render($writer);
-			$boundary = $writer->getWriter()->getBoundary();
+			$boundary=$writer->getWriter()->getBoundary();
 			$this->getResponse()->getAdapter()->setResponseData($boundary);
 		}
 	}
@@ -323,29 +323,29 @@ 
 	{
 		//disallow page state update ?
 		//$this->getActiveControl()->getClientSide()->setEnablePageStateUpdate(false);
-		$options = array();
-		if(strlen($string = $this->getSeparator()))
+		$options=array();
+		if(strlen($string=$this->getSeparator()))
 		{
-			$string = strtr($string,array('\t'=>"\t",'\n'=>"\n",'\r'=>"\r"));
-			$token = preg_split('//', $string, -1, PREG_SPLIT_NO_EMPTY);
-			$options['tokens'] = $token;
+			$string=strtr($string, array('\t'=>"\t", '\n'=>"\n", '\r'=>"\r"));
+			$token=preg_split('//', $string, -1, PREG_SPLIT_NO_EMPTY);
+			$options['tokens']=$token;
 		}
 		if($this->getAutoPostBack())
 		{
-			$options = array_merge($options,parent::getPostBackOptions());
-			$options['AutoPostBack'] = true;
+			$options=array_merge($options, parent::getPostBackOptions());
+			$options['AutoPostBack']=true;
 		}
-		if(strlen($select = $this->getTextCssClass()))
-			$options['select'] = $select;
-		$options['ResultPanel'] = $this->getResultPanel()->getClientID();
-		$options['ID'] = $this->getClientID();
-		$options['EventTarget'] = $this->getUniqueID();
+		if(strlen($select=$this->getTextCssClass()))
+			$options['select']=$select;
+		$options['ResultPanel']=$this->getResultPanel()->getClientID();
+		$options['ID']=$this->getClientID();
+		$options['EventTarget']=$this->getUniqueID();
 		if(($minchars=$this->getMinChars())!=='')
-			$options['minChars'] = $minchars;
+			$options['minChars']=$minchars;
 		if(($frequency=$this->getFrequency())!=='')
-			$options['frequency'] = $frequency;
-		$options['CausesValidation'] = $this->getCausesValidation();
-		$options['ValidationGroup'] = $this->getValidationGroup();
+			$options['frequency']=$frequency;
+		$options['CausesValidation']=$this->getCausesValidation();
+		$options['ValidationGroup']=$this->getValidationGroup();
 		return $options;
 	}
 
@@ -423,7 +423,7 @@ 
 
 	public function __construct($template)
 	{
-		$this->_template = $template;
+		$this->_template=$template;
 	}
 	/**
 	 * Instantiates the template.