pradosoft / prado

framework/Web/UI/JuiControls/TJuiControlAdapter.php

Doc Comments +1 added lines, -1 removed lines

@@ -80,7 +80,7 @@
 
 	/**
 	 * Publish the jQuery-ui style Css asset file.
-	 * @param file name
+	 * @param file string
 	 * @return string Css file url.
 	 */
 	public function publishJuiStyle($file)

Spacing +7 added lines, -7 removed lines

@@ -27,8 +27,8 @@ 
  */
 class TJuiControlAdapter extends TActiveControlAdapter
 {
-	const CSS_PATH = 'themes';
-	const BASE_CSS_FILENAME = 'jquery-ui.css';
+	const CSS_PATH='themes';
+	const BASE_CSS_FILENAME='jquery-ui.css';
 
 	/**
 	 * Replace default StateTracker with {@link TJuiCallbackPageStateTracker} for
@@ -71,10 +71,10 @@ 
 	 * @param string jQuery asset file in the jquery-ui directory.
 	 * @return string jQuery asset url.
 	 */
-	protected function getAssetUrl($file = '')
+	protected function getAssetUrl($file='')
 	{
-		$base = $this->getPage()->getClientScript()->getPradoScriptAssetUrl('jquery-ui');
-		return $base . '/' . $file;
+		$base=$this->getPage()->getClientScript()->getPradoScriptAssetUrl('jquery-ui');
+		return $base.'/'.$file;
 	}
 
 	/**
@@ -84,8 +84,8 @@ 
 	 */
 	public function publishJuiStyle($file)
 	{
-		$url = $this->getAssetUrl(self::CSS_PATH . '/' . $this->getJuiBaseStyle() . '/' . $file);
-		$cs = $this->getPage()->getClientScript();
+		$url=$this->getAssetUrl(self::CSS_PATH.'/'.$this->getJuiBaseStyle().'/'.$file);
+		$cs=$this->getPage()->getClientScript();
 		if(!$cs->isStyleSheetFileRegistered($url))
 			$cs->registerStyleSheetFile($url, $url);
 		return $url;

framework/Web/UI/JuiControls/TJuiEventParameter.php

Doc Comments +1 added lines, -1 removed lines

@@ -67,7 +67,7 @@
 	 * Gets a control instance named after a returned control id.
 	 * Example: if a $param->draggable control id is returned from clientside,
 	 * calling $param->DraggableControl will return the control instance
-	 * @return mixed control or null if not set.
+	 * @return null|TControl control or null if not set.
 	 */
 	public function __get($name)
 	{

Spacing +9 added lines, -9 removed lines

@@ -52,14 +52,14 @@ 
 	 */
 	public function getControl($id)
 	{
-		$control = null;
-		$service = Prado::getApplication()->getService();
-		if ($service instanceof TPageService)
+		$control=null;
+		$service=Prado::getApplication()->getService();
+		if($service instanceof TPageService)
 		{
 			// Find the control
 			// Warning, this will not work if you have a '_' in your control Id !
-			$controlId = str_replace(TControl::CLIENT_ID_SEPARATOR, TControl::ID_SEPARATOR, $id);
-			$control = $service->getRequestedPage()->findControl($controlId);
+			$controlId=str_replace(TControl::CLIENT_ID_SEPARATOR, TControl::ID_SEPARATOR, $id);
+			$control=$service->getRequestedPage()->findControl($controlId);
 		}
 		return $control;
 	}
@@ -72,11 +72,11 @@ 
 	 */
 	public function __get($name)
 	{
-		$pos = strpos($name, 'Control', 1);
-		$name = strtolower(substr($name, 0, $pos));
+		$pos=strpos($name, 'Control', 1);
+		$name=strtolower(substr($name, 0, $pos));
 
-		$cp = $this->getCallbackParameter();
-		if(!isset($cp->$name) || $cp->$name == '')
+		$cp=$this->getCallbackParameter();
+		if(!isset($cp->$name) || $cp->$name=='')
 			return null;
 
 		return $this->getControl($cp->$name);

framework/Web/UI/TCompositeLiteral.php

Doc Comments +1 added lines, -1 removed lines

@@ -59,7 +59,7 @@
 	}
 
 	/**
-	 * @return TComponent container of this component. It serves as the evaluation context of expressions and statements.
+	 * @return \Prado\TComponent container of this component. It serves as the evaluation context of expressions and statements.
 	 */
 	public function getContainer()
 	{

Spacing +24 added lines, -24 removed lines

@@ -23,14 +23,14 @@ 
  */
 class TCompositeLiteral extends \Prado\TComponent implements IRenderable, IBindable
 {
-	const TYPE_EXPRESSION = 0;
-	const TYPE_STATEMENTS = 1;
-	const TYPE_DATABINDING = 2;
+	const TYPE_EXPRESSION=0;
+	const TYPE_STATEMENTS=1;
+	const TYPE_DATABINDING=2;
 	private $_container;
-	private $_items = [];
-	private $_expressions = [];
-	private $_statements = [];
-	private $_bindings = [];
+	private $_items=[];
+	private $_expressions=[];
+	private $_statements=[];
+	private $_bindings=[];
 
 	/**
 	 * Constructor.
@@ -38,23 +38,23 @@ 
 	 */
 	public function __construct($items)
 	{
-		$this->_items = [];
-		$this->_expressions = [];
-		$this->_statements = [];
+		$this->_items=[];
+		$this->_expressions=[];
+		$this->_statements=[];
 		foreach($items as $id => $item)
 		{
 			if(is_array($item))
 			{
-				if($item[0] === self::TYPE_EXPRESSION)
-					$this->_expressions[$id] = $item[1];
-				elseif($item[0] === self::TYPE_STATEMENTS)
-					$this->_statements[$id] = $item[1];
-				elseif($item[0] === self::TYPE_DATABINDING)
-					$this->_bindings[$id] = $item[1];
-				$this->_items[$id] = '';
+				if($item[0]===self::TYPE_EXPRESSION)
+					$this->_expressions[$id]=$item[1];
+				elseif($item[0]===self::TYPE_STATEMENTS)
+					$this->_statements[$id]=$item[1];
+				elseif($item[0]===self::TYPE_DATABINDING)
+					$this->_bindings[$id]=$item[1];
+				$this->_items[$id]='';
 			}
 			else
-				$this->_items[$id] = $item;
+				$this->_items[$id]=$item;
 		}
 	}
 
@@ -71,7 +71,7 @@ 
 	 */
 	public function setContainer(\Prado\TComponent $value)
 	{
-		$this->_container = $value;
+		$this->_container=$value;
 	}
 
 	/**
@@ -79,11 +79,11 @@ 
 	 */
 	public function evaluateDynamicContent()
 	{
-		$context = $this->_container === null?$this:$this->_container;
+		$context=$this->_container===null ? $this : $this->_container;
 		foreach($this->_expressions as $id => $expression)
-			$this->_items[$id] = $context->evaluateExpression($expression);
+			$this->_items[$id]=$context->evaluateExpression($expression);
 		foreach($this->_statements as $id => $statement)
-			$this->_items[$id] = $context->evaluateStatements($statement);
+			$this->_items[$id]=$context->evaluateStatements($statement);
 	}
 
 	/**
@@ -92,9 +92,9 @@ 
 	 */
 	public function dataBind()
 	{
-		$context = $this->_container === null?$this:$this->_container;
+		$context=$this->_container===null ? $this : $this->_container;
 		foreach($this->_bindings as $id => $binding)
-			$this->_items[$id] = $context->evaluateExpression($binding);
+			$this->_items[$id]=$context->evaluateExpression($binding);
 	}
 
 	/**

Braces +1 added lines, -2 removed lines

@@ -52,8 +52,7 @@
 				elseif($item[0] === self::TYPE_DATABINDING)
 					$this->_bindings[$id] = $item[1];
 				$this->_items[$id] = '';
-			}
-			else
+			} else
 				$this->_items[$id] = $item;
 		}
 	}

framework/Web/UI/TTemplateManager.php

Doc Comments +1 added lines

@@ -68,6 +68,7 @@
 
 	/**
 	 * Loads the template from the specified file.
+	 * @param string $fileName
 	 * @return ITemplate template parsed from the specified file, null if the file doesn't exist.
 	 */
 	public function getTemplateByFileName($fileName)

Braces +1 added lines, -2 removed lines

@@ -107,8 +107,7 @@
 				$cache->set(self::TEMPLATE_CACHE_PREFIX . $fileName, [$template,$timestamps]);
 				return $template;
 			}
-		}
-		else
+		} else
 			return null;
 	}
 

Spacing +20 added lines, -20 removed lines

@@ -39,11 +39,11 @@ 
 	/**
 	 * Template file extension
 	 */
-	const TEMPLATE_FILE_EXT = '.tpl';
+	const TEMPLATE_FILE_EXT='.tpl';
 	/**
 	 * Prefix of the cache variable name for storing parsed templates
 	 */
-	const TEMPLATE_CACHE_PREFIX = 'prado:template:';
+	const TEMPLATE_CACHE_PREFIX='prado:template:';
 
 	/**
 	 * Initializes the module.
@@ -62,8 +62,8 @@ 
 	 */
 	public function getTemplateByClassName($className)
 	{
-		$class = new \ReflectionClass($className);
-		$tplFile = dirname($class->getFileName()) . DIRECTORY_SEPARATOR . $class->getShortName() . self::TEMPLATE_FILE_EXT;
+		$class=new \ReflectionClass($className);
+		$tplFile=dirname($class->getFileName()).DIRECTORY_SEPARATOR.$class->getShortName().self::TEMPLATE_FILE_EXT;
 		return $this->getTemplateByFileName($tplFile);
 	}
 
@@ -73,38 +73,38 @@ 
 	 */
 	public function getTemplateByFileName($fileName)
 	{
-		if(($fileName = $this->getLocalizedTemplate($fileName)) !== null)
+		if(($fileName=$this->getLocalizedTemplate($fileName))!==null)
 		{
 			Prado::trace("Loading template $fileName", '\Prado\Web\UI\TTemplateManager');
-			if(($cache = $this->getApplication()->getCache()) === null)
+			if(($cache=$this->getApplication()->getCache())===null)
 				return new TTemplate(file_get_contents($fileName), dirname($fileName), $fileName);
 			else
 			{
-				$array = $cache->get(self::TEMPLATE_CACHE_PREFIX . $fileName);
+				$array=$cache->get(self::TEMPLATE_CACHE_PREFIX.$fileName);
 				if(is_array($array))
 				{
-					list($template, $timestamps) = $array;
-					if($this->getApplication()->getMode() === TApplicationMode::Performance)
+					list($template, $timestamps)=$array;
+					if($this->getApplication()->getMode()===TApplicationMode::Performance)
 						return $template;
-					$cacheValid = true;
+					$cacheValid=true;
 					foreach($timestamps as $tplFile => $timestamp)
 					{
 						if(!is_file($tplFile) || filemtime($tplFile) > $timestamp)
 						{
-							$cacheValid = false;
+							$cacheValid=false;
 							break;
 						}
 					}
 					if($cacheValid)
 						return $template;
 				}
-				$template = new TTemplate(file_get_contents($fileName), dirname($fileName), $fileName);
-				$includedFiles = $template->getIncludedFiles();
-				$timestamps = [];
-				$timestamps[$fileName] = filemtime($fileName);
+				$template=new TTemplate(file_get_contents($fileName), dirname($fileName), $fileName);
+				$includedFiles=$template->getIncludedFiles();
+				$timestamps=[];
+				$timestamps[$fileName]=filemtime($fileName);
 				foreach($includedFiles as $includedFile)
-					$timestamps[$includedFile] = filemtime($includedFile);
-				$cache->set(self::TEMPLATE_CACHE_PREFIX . $fileName, [$template,$timestamps]);
+					$timestamps[$includedFile]=filemtime($includedFile);
+				$cache->set(self::TEMPLATE_CACHE_PREFIX.$fileName, [$template, $timestamps]);
 				return $template;
 			}
 		}
@@ -119,11 +119,11 @@ 
 	 */
 	protected function getLocalizedTemplate($filename)
 	{
-		if(($app = $this->getApplication()->getGlobalization(false)) === null)
-			return is_file($filename)?$filename:null;
+		if(($app=$this->getApplication()->getGlobalization(false))===null)
+			return is_file($filename) ? $filename : null;
 		foreach($app->getLocalizedResource($filename) as $file)
 		{
-			if(($file = realpath($file)) !== false && is_file($file))
+			if(($file=realpath($file))!==false && is_file($file))
 				return $file;
 		}
 		return null;

framework/Web/UI/WebControls/IDataSource.php

Doc Comments +4 added lines

@@ -22,5 +22,9 @@
 {
 	public function getView($viewName);
 	public function getViewNames();
+
+	/**
+	 * @return void
+	 */
 	public function onDataSourceChanged($param);
 }
\ No newline at end of file

framework/Web/UI/WebControls/IItemDataRenderer.php

Doc Comments +2 added lines

@@ -36,6 +36,7 @@ 
 	 * Sets the zero-based index for the item.
 	 * If the item is not in the item collection (e.g. it is a header item), -1 should be used.
 	 * @param integer zero-based index of the item.
+	 * @return void
 	 */
 	public function setItemIndex($value);
 
@@ -46,6 +47,7 @@ 
 
 	/**
 	 * @param TListItemType the item type.
+	 * @return void
 	 */
 	public function setItemType($value);
 }
\ No newline at end of file

framework/Web/UI/WebControls/IStyleable.php

Doc Comments +1 added lines

@@ -32,6 +32,7 @@
 	public function getStyle();
 	/**
 	 * Removes all styles associated with the object
+	 * @return void
 	 */
 	public function clearStyle();
 }
\ No newline at end of file

framework/Web/UI/WebControls/TBulletedList.php

Doc Comments +1 added lines

@@ -216,6 +216,7 @@
 	 * This method is invoked when the {@link getDisplayMode DisplayMode} is 'LinkButton'
 	 * and end-users click on one of the buttons.
 	 * @param TBulletedListEventParameter event parameter.
+	 * @param TBulletedListEventParameter $param
 	 */
 	public function onClick($param)
 	{

Spacing +27 added lines, -27 removed lines

@@ -62,7 +62,7 @@ 
 	{
 		if($this->getCausesValidation())
 			$this->getPage()->validate($this->getValidationGroup());
-		$this->onClick(new TBulletedListEventParameter((int)$param));
+		$this->onClick(new TBulletedListEventParameter((int) $param));
 	}
 
 	/**
@@ -99,32 +99,32 @@ 
 	 */
 	protected function addAttributesToRender($writer)
 	{
-		$needStart = false;
+		$needStart=false;
 		switch($this->getBulletStyle())
 		{
 			case TBulletStyle::None:
 				$writer->addStyleAttribute('list-style-type', 'none');
-				$needStart = true;
+				$needStart=true;
 				break;
 			case TBulletStyle::Numbered:
 				$writer->addStyleAttribute('list-style-type', 'decimal');
-				$needStart = true;
+				$needStart=true;
 				break;
 			case TBulletStyle::LowerAlpha:
 				$writer->addStyleAttribute('list-style-type', 'lower-alpha');
-				$needStart = true;
+				$needStart=true;
 				break;
 			case TBulletStyle::UpperAlpha:
 				$writer->addStyleAttribute('list-style-type', 'upper-alpha');
-				$needStart = true;
+				$needStart=true;
 				break;
 			case TBulletStyle::LowerRoman:
 				$writer->addStyleAttribute('list-style-type', 'lower-roman');
-				$needStart = true;
+				$needStart=true;
 				break;
 			case TBulletStyle::UpperRoman:
 				$writer->addStyleAttribute('list-style-type', 'upper-roman');
-				$needStart = true;
+				$needStart=true;
 				break;
 			case TBulletStyle::Disc:
 				$writer->addStyleAttribute('list-style-type', 'disc');
@@ -136,11 +136,11 @@ 
 				$writer->addStyleAttribute('list-style-type', 'square');
 				break;
 			case TBulletStyle::CustomImage:
-				$url = $this->getBulletImageUrl();
+				$url=$this->getBulletImageUrl();
 				$writer->addStyleAttribute('list-style-image', "url($url)");
 				break;
 		}
-		if($needStart && ($start = $this->getFirstBulletNumber()) != 1)
+		if($needStart && ($start=$this->getFirstBulletNumber())!=1)
 			$writer->addAttribute('start', "$start");
 		parent::addAttributesToRender($writer);
 	}
@@ -259,8 +259,8 @@ 
 	 */
 	public function renderContents($writer)
 	{
-		$this->_isEnabled = $this->getEnabled(true);
-		$this->_postBackOptions = $this->getPostBackOptions();
+		$this->_isEnabled=$this->getEnabled(true);
+		$this->_postBackOptions=$this->getPostBackOptions();
 		$writer->writeLine();
 		foreach($this->getItems() as $index => $item)
 		{
@@ -315,10 +315,10 @@ 
 		else
 		{
 			$writer->addAttribute('href', $item->getValue());
-			if(($target = $this->getTarget()) !== '')
+			if(($target=$this->getTarget())!=='')
 				$writer->addAttribute('target', $target);
 		}
-		if(($accesskey = $this->getAccessKey()) !== '')
+		if(($accesskey=$this->getAccessKey())!=='')
 			$writer->addAttribute('accesskey', $accesskey);
 		$writer->renderBeginTag('a');
 		$writer->write(THttpUtility::htmlEncode($item->getText()));
@@ -331,13 +331,13 @@ 
 			$writer->addAttribute('disabled', 'disabled');
 		else
 		{
-			$this->_currentRenderItemIndex = $index;
-			$writer->addAttribute('id', $this->getClientID() . $index);
-			$writer->addAttribute('href', "javascript:;//" . $this->getClientID() . $index);
-			$cs = $this->getPage()->getClientScript();
+			$this->_currentRenderItemIndex=$index;
+			$writer->addAttribute('id', $this->getClientID().$index);
+			$writer->addAttribute('href', "javascript:;//".$this->getClientID().$index);
+			$cs=$this->getPage()->getClientScript();
 			$cs->registerPostBackControl($this->getClientClassName(), $this->getPostBackOptions());
 		}
-		if(($accesskey = $this->getAccessKey()) !== '')
+		if(($accesskey=$this->getAccessKey())!=='')
 			$writer->addAttribute('accesskey', $accesskey);
 		$writer->renderBeginTag('a');
 		$writer->write(THttpUtility::htmlEncode($item->getText()));
@@ -349,19 +349,19 @@ 
 	 */
 	protected function getPostBackOptions()
 	{
-		$options['ValidationGroup'] = $this->getValidationGroup();
-		$options['CausesValidation'] = $this->getCausesValidation();
-		$options['EventTarget'] = $this->getUniqueID();
-		$options['EventParameter'] = $this->_currentRenderItemIndex;
-		$options['ID'] = $this->getClientID() . $this->_currentRenderItemIndex;
-		$options['StopEvent'] = true;
+		$options['ValidationGroup']=$this->getValidationGroup();
+		$options['CausesValidation']=$this->getCausesValidation();
+		$options['EventTarget']=$this->getUniqueID();
+		$options['EventParameter']=$this->_currentRenderItemIndex;
+		$options['ID']=$this->getClientID().$this->_currentRenderItemIndex;
+		$options['StopEvent']=true;
 		return $options;
 	}
 
 	protected function canCauseValidation()
 	{
-		$group = $this->getValidationGroup();
-		$hasValidators = $this->getPage()->getValidators($group)->getCount() > 0;
+		$group=$this->getValidationGroup();
+		$hasValidators=$this->getPage()->getValidators($group)->getCount() > 0;
 		return $this->getCausesValidation() && $hasValidators;
 	}
 

framework/Web/UI/WebControls/TBulletedListEventParameter.php

Doc Comments +1 added lines

@@ -31,6 +31,7 @@
 	/**
 	 * Constructor.
 	 * @param integer index of the item clicked
+	 * @param integer $index
 	 */
 	public function __construct($index)
 	{

Spacing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
 	 */
 	public function __construct($index)
 	{
-		$this->_index = $index;
+		$this->_index=$index;
 	}
 
 	/**