pradosoft / prado

framework/Data/Common/TDbTableColumn.php

Doc Comments +1 added lines

@@ -136,6 +136,7 @@
 	 * @param mixed Name of the PHP variable to bind to the SQL statement parameter
 	 * @param int SQL data type of the parameter
 	 * @param int length of the data type
+	 * @param string $name
 	 * @see http://www.php.net/manual/en/function.PDOStatement-bindParam.php
 	 */
 	public function bindParameter($name, &$value, $dataType=null, $length=null)

Indentation +8 added lines, -8 removed lines

@@ -1,13 +1,13 @@
 <?php
 /**
- * TDbTableColumn class file.
- *
- * @author Wei Zhuo <weizhuo[at]gmail[dot]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.Data.Common
- */
+	 * TDbTableColumn class file.
+	 *
+	 * @author Wei Zhuo <weizhuo[at]gmail[dot]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.Data.Common
+	 */
 
 /**
  * TDbTableColumn class describes the column meta data of the schema for a database table.

Spacing +7 added lines, -7 removed lines

@@ -18,7 +18,7 @@ 
  */
 class TDbTableColumn extends TComponent
 {
-	const UNDEFINED_VALUE= INF; //use infinity for undefined value
+	const UNDEFINED_VALUE=INF; //use infinity for undefined value
 
 	private $_info=array();
 
@@ -36,7 +36,7 @@ 
 	 * @param mixed default value if information array value is null
 	 * @return mixed information array value.
 	 */
-	protected function getInfo($name,$default=null)
+	protected function getInfo($name, $default=null)
 	{
 		return isset($this->_info[$name]) ? $this->_info[$name] : $default;
 	}
@@ -45,7 +45,7 @@ 
 	 * @param string information array key name
 	 * @param mixed new information array value.
 	 */
-	protected function setInfo($name,$value)
+	protected function setInfo($name, $value)
 	{
 		$this->_info[$name]=$value;
 	}
@@ -118,7 +118,7 @@ 
 	 */
 	public function getAllowNull()
 	{
-		return $this->getInfo('AllowNull',false);
+		return $this->getInfo('AllowNull', false);
 	}
 
 	/**
@@ -150,7 +150,7 @@ 
 		if(($precision=$this->getNumericPrecision())!==null)
 		{
 			$scale=$this->getNumericScale();
-			return $scale===null ? pow(10,$precision) : pow(10,$precision-$scale);
+			return $scale===null ? pow(10, $precision) : pow(10, $precision - $scale);
 		}
 	}
 
@@ -159,7 +159,7 @@ 
 	 */
 	public function getIsPrimaryKey()
 	{
-		return $this->getInfo('IsPrimaryKey',false);
+		return $this->getInfo('IsPrimaryKey', false);
 	}
 
 	/**
@@ -167,7 +167,7 @@ 
 	 */
 	public function getIsForeignKey()
 	{
-		return $this->getInfo('IsForeignKey',false);
+		return $this->getInfo('IsForeignKey', false);
 	}
 
 	/**

framework/Data/SqlMap/Configuration/TParameterProperty.php

Doc Comments +1 added lines

@@ -113,6 +113,7 @@
 
 	/**
 	 * @param string button caption
+	 * @param string $value
 	 */
 	public function setButtonText($value)
 	{

Spacing +14 added lines, -14 removed lines

@@ -49,7 +49,7 @@ 
 	 */
 	public function setTypeHandler($value)
 	{
-		$this->_typeHandler = $value;
+		$this->_typeHandler=$value;
 	}
 
 	/**
@@ -65,7 +65,7 @@ 
 	 */
 	public function setType($value)
 	{
-		$this->_type = $value;
+		$this->_type=$value;
 	}
 
 	/**
@@ -81,7 +81,7 @@ 
 	 */
 	public function setColumn($value)
 	{
-		$this->_column = $value;
+		$this->_column=$value;
 	}
 
 	/**
@@ -97,7 +97,7 @@ 
 	 */
 	public function setDbType($value)
 	{
-		$this->_dbType = $value;
+		$this->_dbType=$value;
 	}
 
 	/**
@@ -113,7 +113,7 @@ 
 	 */
 	public function setProperty($value)
 	{
-		$this->_property = $value;
+		$this->_property=$value;
 	}
 
 	/**
@@ -130,19 +130,19 @@ 
 	 */
 	public function setNullValue($value)
 	{
-		$this->_nullValue = $value;
+		$this->_nullValue=$value;
 	}
 
 	public function __sleep()
 	{
-		$exprops = array(); $cn = 'TParameterProperty';
-		if ($this->_typeHandler===null) $exprops[] = "\0$cn\0_typeHandler";
-		if ($this->_type===null) $exprops[] = "\0$cn\0_type";
-		if ($this->_column===null) $exprops[] = "\0$cn\0_column";
-		if ($this->_dbType===null) $exprops[] = "\0$cn\0_dbType";
-		if ($this->_property===null) $exprops[] = "\0$cn\0_property";
-		if ($this->_nullValue===null) $exprops[] = "\0$cn\0_nullValue";
-		return array_diff(parent::__sleep(),$exprops);
+		$exprops=array(); $cn='TParameterProperty';
+		if($this->_typeHandler===null) $exprops[]="\0$cn\0_typeHandler";
+		if($this->_type===null) $exprops[]="\0$cn\0_type";
+		if($this->_column===null) $exprops[]="\0$cn\0_column";
+		if($this->_dbType===null) $exprops[]="\0$cn\0_dbType";
+		if($this->_property===null) $exprops[]="\0$cn\0_property";
+		if($this->_nullValue===null) $exprops[]="\0$cn\0_nullValue";
+		return array_diff(parent::__sleep(), $exprops);
 	}
 }
 

framework/Web/UI/ActiveControls/TActiveControlAdapter.php

Doc Comments +1 added lines

@@ -113,6 +113,7 @@
 
 	/**
 	 * @param string button caption
+	 * @param string $value
 	 */
 	public function setButtonText($value)
 	{

Indentation +2 added lines, -2 removed lines

@@ -78,8 +78,8 @@
 	}
 
  	/**
-	 * Publish the ajax script
-	 */
+ 	 * Publish the ajax script
+ 	 */
 	public function onPreRender($param)
 	{
 		parent::onPreRender($param);

Spacing +53 added lines, -53 removed lines

@@ -69,12 +69,12 @@ 
 		if($type===null)
 		{
 			if($this->getControl() instanceof ICallbackEventHandler)
-				$this->_activeControlType = 'TBaseActiveCallbackControl';
+				$this->_activeControlType='TBaseActiveCallbackControl';
 			else
-				$this->_activeControlType = 'TBaseActiveControl';
+				$this->_activeControlType='TBaseActiveControl';
 		}
 		else
-			$this->_activeControlType = $type;
+			$this->_activeControlType=$type;
 	}
 
  	/**
@@ -114,8 +114,8 @@ 
 	{
 		if($this->_baseActiveControl===null)
 		{
-			$type = $this->_activeControlType;
-			$this->_baseActiveControl = new $type($this->getControl());
+			$type=$this->_activeControlType;
+			$this->_baseActiveControl=new $type($this->getControl());
 		}
 		return $this->_baseActiveControl;
 	}
@@ -127,10 +127,10 @@ 
 	{
 		if($this->getPage()->getIsCallback())
 		{
-			$target = $this->getPage()->getCallbackEventTarget();
+			$target=$this->getPage()->getCallbackEventTarget();
 			if($target instanceof ICallbackEventHandler)
 			{
-				$client = $target->getActiveControl()->getClientSide();
+				$client=$target->getActiveControl()->getClientSide();
 				return $client->getEnablePageStateUpdate();
 			}
 		}
@@ -144,8 +144,8 @@ 
 	{
 		if($this->getIsTrackingPageState())
 		{
-		  $stateTrackerClass = $this->_stateTrackerClass;
-			$this->_stateTracker = new $stateTrackerClass($this->getControl());
+		  $stateTrackerClass=$this->_stateTrackerClass;
+			$this->_stateTracker=new $stateTrackerClass($this->getControl());
 			$this->_stateTracker->trackChanges();
 		}
 		parent::onLoad($param);
@@ -178,7 +178,7 @@ 
 	 */
 	public function setStateTracker($value)
 	{
-		$this->_stateTrackerClass = TPropertyValue::ensureString($value);
+		$this->_stateTrackerClass=TPropertyValue::ensureString($value);
 	}
 }
 
@@ -216,10 +216,10 @@ 
 	 */
 	public function __construct($control)
 	{
-		$this->_control = $control;
-		$this->_existingState = new TMap;
-		$this->_nullObject = new stdClass;
-		$this->_states = new TMap;
+		$this->_control=$control;
+		$this->_existingState=new TMap;
+		$this->_nullObject=new stdClass;
+		$this->_states=new TMap;
 		$this->addStatesToTrack();
 	}
 
@@ -233,14 +233,14 @@ 
 	 */
 	protected function addStatesToTrack()
 	{
-		$states = $this->getStatesToTrack();
-		$states['Visible'] = array('TScalarDiff', array($this, 'updateVisible'));
-		$states['Enabled'] = array('TScalarDiff', array($this, 'updateEnabled'));
-		$states['Attributes'] = array('TMapCollectionDiff', array($this, 'updateAttributes'));
-		$states['Style'] = array('TStyleDiff', array($this, 'updateStyle'));
-		$states['TabIndex'] = array('TScalarDiff', array($this, 'updateTabIndex'));
-		$states['ToolTip'] = array('TScalarDiff', array($this, 'updateToolTip'));
-		$states['AccessKey'] = array('TScalarDiff', array($this, 'updateAccessKey'));
+		$states=$this->getStatesToTrack();
+		$states['Visible']=array('TScalarDiff', array($this, 'updateVisible'));
+		$states['Enabled']=array('TScalarDiff', array($this, 'updateEnabled'));
+		$states['Attributes']=array('TMapCollectionDiff', array($this, 'updateAttributes'));
+		$states['Style']=array('TStyleDiff', array($this, 'updateStyle'));
+		$states['TabIndex']=array('TScalarDiff', array($this, 'updateTabIndex'));
+		$states['ToolTip']=array('TScalarDiff', array($this, 'updateToolTip'));
+		$states['AccessKey']=array('TScalarDiff', array($this, 'updateAccessKey'));
 	}
 
 	/**
@@ -259,8 +259,8 @@ 
 	{
 		foreach($this->_states as $name => $value)
 		{
-			$obj = $this->_control->getViewState($name);
-			$this->_existingState[$name] = is_object($obj) ? clone($obj) : $obj;
+			$obj=$this->_control->getViewState($name);
+			$this->_existingState[$name]=is_object($obj) ? clone($obj) : $obj;
 		}
 	}
 
@@ -269,16 +269,16 @@ 
 	 */
 	protected function getChanges()
 	{
-		$changes = array();
+		$changes=array();
 		foreach($this->_states as $name => $details)
 		{
-			$new = $this->_control->getViewState($name);
-			$old = $this->_existingState[$name];
-			if($new !== $old)
+			$new=$this->_control->getViewState($name);
+			$old=$this->_existingState[$name];
+			if($new!==$old)
 			{
-				$diff = new $details[0]($new, $old, $this->_nullObject);
-				if(($change = $diff->getDifference()) !== $this->_nullObject)
-					$changes[] = array($details[1],array($change));
+				$diff=new $details[0]($new, $old, $this->_nullObject);
+				if(($change=$diff->getDifference())!==$this->_nullObject)
+					$changes[]=array($details[1], array($change));
 			}
 		}
 		return $changes;
@@ -335,10 +335,10 @@ 
 	 */
 	protected function updateVisible($visible)
 	{
-		if($visible === false)
-			$this->client()->replaceContent($this->_control,"<span id=\"".$this->_control->getClientID()."\" style=\"display:none\" ></span>");
+		if($visible===false)
+			$this->client()->replaceContent($this->_control, "<span id=\"".$this->_control->getClientID()."\" style=\"display:none\" ></span>");
 		else
-			$this->client()->replaceContent($this->_control,$this->_control);
+			$this->client()->replaceContent($this->_control, $this->_control);
 	}
 
 	/**
@@ -403,9 +403,9 @@ 
 	 */
 	public function __construct($new, $old, $null)
 	{
-		$this->_new = $new;
-		$this->_old = $old;
-		$this->_null = $null;
+		$this->_new=$new;
+		$this->_old=$old;
+		$this->_null=$null;
 	}
 
 	/**
@@ -430,8 +430,8 @@ 
 	 */
 	public function getDifference()
 	{
-		if(gettype($this->_new) === gettype($this->_old)
-			&& $this->_new === $this->_old)
+		if(gettype($this->_new)===gettype($this->_old)
+			&& $this->_new===$this->_old)
 			return $this->_null;
 		else
 			return $this->_new;
@@ -457,10 +457,10 @@ 
 	{
 		if(!($obj instanceof TStyle))
 			return array();
-		$style = $obj->getStyleFields();
-		$style = array_merge($style,$this->getStyleFromString($obj->getCustomStyle()));
+		$style=$obj->getStyleFields();
+		$style=array_merge($style, $this->getStyleFromString($obj->getCustomStyle()));
 		if($obj->hasFont())
-			$style = array_merge($style, $this->getStyleFromString($obj->getFont()->toString()));
+			$style=array_merge($style, $this->getStyleFromString($obj->getFont()->toString()));
 		return $style;
 	}
 
@@ -470,14 +470,14 @@ 
 	 */
 	protected function getStyleFromString($string)
 	{
-		$style = array();
+		$style=array();
 		if(!is_string($string)) return $style;
 
-		foreach(explode(';',$string) as $sub)
+		foreach(explode(';', $string) as $sub)
 		{
-			$arr=explode(':',$sub);
+			$arr=explode(':', $sub);
 			if(isset($arr[1]) && trim($arr[0])!=='')
-				$style[trim($arr[0])] = trim($arr[1]);
+				$style[trim($arr[0])]=trim($arr[1]);
 		}
 		return $style;
 	}
@@ -494,7 +494,7 @@ 
 		}
 		else
 		{
-			return $this->_old->getCssClass() !== $this->_new->getCssClass() ?
+			return $this->_old->getCssClass()!==$this->_new->getCssClass() ?
 				$this->_new->getCssClass() : null;
 		}
 	}
@@ -504,7 +504,7 @@ 
 	 */
 	protected function getStyleDiff()
 	{
-		$diff = array_diff_assoc(
+		$diff=array_diff_assoc(
 					$this->getCombinedStyle($this->_new),
 					$this->getCombinedStyle($this->_old));
 		return count($diff) > 0 ? $diff : null;
@@ -519,8 +519,8 @@ 
 			return $this->_null;
 		else
 		{
-			$css = $this->getCssClassDiff();
-			$style = $this->getStyleDiff();
+			$css=$this->getCssClassDiff();
+			$style=$this->getStyleDiff();
 			if(($css!==null) || ($style!==null))
 				return array('CssClass' => $css, 'Style' => $style);
 			else
@@ -551,9 +551,9 @@ 
 		}
 		else
 		{
-			$new = $this->_new->toArray();
-			$old = $this->_old->toArray();
-			$diff = array_diff_assoc($new, $old);
+			$new=$this->_new->toArray();
+			$old=$this->_old->toArray();
+			$diff=array_diff_assoc($new, $old);
 			return count($diff) > 0 ? $diff : $this->_null;
 		}
 	}

Braces +3 added lines, -6 removed lines

@@ -72,8 +72,7 @@ 
 				$this->_activeControlType = 'TBaseActiveCallbackControl';
 			else
 				$this->_activeControlType = 'TBaseActiveControl';
-		}
-		else
+		} else
 			$this->_activeControlType = $type;
 	}
 
@@ -491,8 +490,7 @@ 
 		{
 			return ($this->_new!==null) && $this->_new->hasCssClass()
 						? $this->_new->getCssClass() : null;
-		}
-		else
+		} else
 		{
 			return $this->_old->getCssClass() !== $this->_new->getCssClass() ?
 				$this->_new->getCssClass() : null;
@@ -548,8 +546,7 @@ 
 		if($this->_old===null)
 		{
 			return ($this->_new!==null) ? $this->_new->toArray() : $this->_null;
-		}
-		else
+		} else
 		{
 			$new = $this->_new->toArray();
 			$old = $this->_old->toArray();

framework/Web/UI/ActiveControls/TActiveDropDownList.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 +3 added lines, -3 removed lines

@@ -82,7 +82,7 @@ 
 	 */
 	protected function createListItemCollection()
 	{
-		$collection  = new TActiveListItemCollection;
+		$collection=new TActiveListItemCollection;
 		$collection->setControl($this);
 		return $collection;
 	}
@@ -102,8 +102,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/TActiveImageButton.php

Doc Comments +1 added lines

@@ -113,6 +113,7 @@
 
 	/**
 	 * @param string button caption
+	 * @param string $value
 	 */
 	public function setButtonText($value)
 	{

Indentation +8 added lines, -8 removed lines

@@ -1,13 +1,13 @@
 <?php
 /**
- * TActiveImageButton class file.
- *
- * @author Wei Zhuo <weizhuo[at]gamil[dot]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.UI.ActiveControls
- */
+   * TActiveImageButton class file.
+   *
+   * @author Wei Zhuo <weizhuo[at]gamil[dot]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.UI.ActiveControls
+   */
 
 /**
  * TActiveImageButton class.

Spacing +1 added lines, -1 removed lines

@@ -139,7 +139,7 @@
 	protected function addAttributesToRender($writer)
 	{
 		parent::addAttributesToRender($writer);
-		$writer->addAttribute('id',$this->getClientID());
+		$writer->addAttribute('id', $this->getClientID());
 		$this->getActiveControl()->registerCallbackClientScript(
 			$this->getClientClassName(), $this->getPostBackOptions());
 	}

framework/Web/UI/JuiControls/TJuiSelectable.php

Doc Comments +3 added lines

@@ -317,6 +317,9 @@
 {
 	private $_template;
 
+	/**
+	 * @param string $template
+	 */
 	public function __construct($template)
 	{
 		$this->_template = $template;

Spacing +15 added lines, -15 removed lines

@@ -65,7 +65,7 @@ 
 	 */
 	public function getWidgetID()
 	{
-	  return $this->getClientID() . '_0';
+	  return $this->getClientID().'_0';
 	}
 
 	/**
@@ -74,7 +74,7 @@ 
 	 */
 	public function getOptions()
 	{
-		if (($options=$this->getViewState('JuiOptions'))===null)
+		if(($options=$this->getViewState('JuiOptions'))===null)
 		{
 		  $options=new TJuiControlOptions($this);
 		  $this->setViewState('JuiOptions', $options);
@@ -105,7 +105,7 @@ 
 	 */
 	protected function getPostBackOptions()
 	{
-		$options = $this->getOptions()->toArray();
+		$options=$this->getOptions()->toArray();
 		// overload the "OnStop" event to add information about the current selected items
 		if(isset($options['stop']))
 		{
@@ -121,7 +121,7 @@ 
 	protected function addAttributesToRender($writer)
 	{
 		parent::addAttributesToRender($writer);
-		$writer->addAttribute('id',$this->getClientID());
+		$writer->addAttribute('id', $this->getClientID());
 		$options=TJavascript::encode($this->getPostBackOptions());
 		$cs=$this->getPage()->getClientScript();
 		$code="jQuery('#".$this->getWidgetID()."').".$this->getWidget()."(".$options.");";
@@ -142,7 +142,7 @@ 
 	 * Raises the OnCreate event
 	 * @param object $params event parameters
 	 */
-	public function onCreate ($params)
+	public function onCreate($params)
 	{
 		$this->raiseEvent('OnCreate', $this, $params);
 	}
@@ -151,7 +151,7 @@ 
 	 * Raises the OnSelected event
 	 * @param object $params event parameters
 	 */
-	public function onSelected ($params)
+	public function onSelected($params)
 	{
 		$this->raiseEvent('OnSelected', $this, $params);
 	}
@@ -160,7 +160,7 @@ 
 	 * Raises the OnSelecting event
 	 * @param object $params event parameters
 	 */
-	public function onSelecting ($params)
+	public function onSelecting($params)
 	{
 		$this->raiseEvent('OnSelecting', $this, $params);
 	}
@@ -169,7 +169,7 @@ 
 	 * Raises the OnStart event
 	 * @param object $params event parameters
 	 */
-	public function onStart ($params)
+	public function onStart($params)
 	{
 		$this->raiseEvent('OnStart', $this, $params);
 	}
@@ -178,7 +178,7 @@ 
 	 * Raises the OnStop event
 	 * @param object $params event parameters
 	 */
-	public function onStop ($params)
+	public function onStop($params)
 	{
 		$this->raiseEvent('OnStop', $this, $params);
 	}
@@ -187,7 +187,7 @@ 
 	 * Raises the OnUnselected event
 	 * @param object $params event parameters
 	 */
-	public function onUnselected ($params)
+	public function onUnselected($params)
 	{
 		$this->raiseEvent('OnUnselected', $this, $params);
 	}
@@ -196,7 +196,7 @@ 
 	 * Raises the OnUnselecting event
 	 * @param object $params event parameters
 	 */
-	public function onUnselecting ($params)
+	public function onUnselecting($params)
 	{
 		$this->raiseEvent('OnUnselecting', $this, $params);
 	}
@@ -231,7 +231,7 @@ 
 	public function getSelectables()
 	{
 		if($this->_repeater===null)
-			$this->_repeater = $this->createRepeater();
+			$this->_repeater=$this->createRepeater();
 		return $this->_repeater;
 	}
 
@@ -240,10 +240,10 @@ 
 	 */
 	protected function createRepeater()
 	{
-		$repeater = Prado::createComponent('System.Web.UI.WebControls.TRepeater');
+		$repeater=Prado::createComponent('System.Web.UI.WebControls.TRepeater');
 		$repeater->setHeaderTemplate(new TJuiSelectableTemplate('<ul id="'.$this->getWidgetID().'">'));
 		$repeater->setFooterTemplate(new TJuiSelectableTemplate('</ul>'));
-		$repeater->setItemTemplate(new TTemplate('<li id="<%# $this->ItemIndex %>"><%# $this->DataItem %></li>',null));
+		$repeater->setItemTemplate(new TTemplate('<li id="<%# $this->ItemIndex %>"><%# $this->DataItem %></li>', null));
 		$repeater->setEmptyTemplate(new TJuiSelectableTemplate('<ul></ul>'));
 		$this->getControls()->add($repeater);
 		return $repeater;
@@ -266,7 +266,7 @@ 
 
 	public function __construct($template)
 	{
-		$this->_template = $template;
+		$this->_template=$template;
 	}
 	/**
 	 * Instantiates the template.

framework/Web/UI/TForm.php

Doc Comments +1 added lines

@@ -113,6 +113,7 @@
 
 	/**
 	 * @param string button caption
+	 * @param string $value
 	 */
 	public function setButtonText($value)
 	{

Braces +1 added lines, -2 removed lines

@@ -58,8 +58,7 @@
 		if ($this->sourcepath === NULL)
 		{
 			$this->sourcepath = $sourcepath;
-		}
-		else
+		} else
 		{
 			$this->sourcepath->append($sourcepath);
 		}

Spacing +11 added lines, -11 removed lines

@@ -43,12 +43,12 @@ 
 	 */
 	protected function addAttributesToRender($writer)
 	{
-		$writer->addAttribute('id',$this->getClientID());
-		$writer->addAttribute('method',$this->getMethod());
+		$writer->addAttribute('id', $this->getClientID());
+		$writer->addAttribute('method', $this->getMethod());
 		$uri=$this->getRequest()->getRequestURI();
-		$writer->addAttribute('action',str_replace('&','&',str_replace('&','&',$uri)));
+		$writer->addAttribute('action', str_replace('&', '&', str_replace('&', '&', $uri)));
 		if(($enctype=$this->getEnctype())!=='')
-			$writer->addAttribute('enctype',$enctype);
+			$writer->addAttribute('enctype', $enctype);
 
 		$attributes=$this->getAttributes();
 		$attributes->remove('action');
@@ -59,7 +59,7 @@ 
 			if(($button=$this->findControl($butt))!==null)
 				$this->getPage()->getClientScript()->registerDefaultButton($this, $button);
 			else
-				throw new TInvalidDataValueException('form_defaultbutton_invalid',$butt);
+				throw new TInvalidDataValueException('form_defaultbutton_invalid', $butt);
 		}
 	}
 
@@ -108,7 +108,7 @@ 
 	 */
 	public function getDefaultButton()
 	{
-		return $this->getViewState('DefaultButton','');
+		return $this->getViewState('DefaultButton', '');
 	}
 
 	/**
@@ -119,7 +119,7 @@ 
 	 */
 	public function setDefaultButton($value)
 	{
-		$this->setViewState('DefaultButton',$value,'');
+		$this->setViewState('DefaultButton', $value, '');
 	}
 
 	/**
@@ -127,7 +127,7 @@ 
 	 */
 	public function getMethod()
 	{
-		return $this->getViewState('Method','post');
+		return $this->getViewState('Method', 'post');
 	}
 
 	/**
@@ -135,7 +135,7 @@ 
 	 */
 	public function setMethod($value)
 	{
-		$this->setViewState('Method',TPropertyValue::ensureEnum($value,'post','get'),'post');
+		$this->setViewState('Method', TPropertyValue::ensureEnum($value, 'post', 'get'), 'post');
 	}
 
 	/**
@@ -143,7 +143,7 @@ 
 	 */
 	public function getEnctype()
 	{
-		return $this->getViewState('Enctype','');
+		return $this->getViewState('Enctype', '');
 	}
 
 	/**
@@ -155,7 +155,7 @@ 
 	 */
 	public function setEnctype($value)
 	{
-		$this->setViewState('Enctype',$value,'');
+		$this->setViewState('Enctype', $value, '');
 	}
 
 	/**

framework/Web/UI/WebControls/TDataTypeValidator.php

Doc Comments +1 added lines

@@ -113,6 +113,7 @@
 
 	/**
 	 * @param string button caption
+	 * @param string $value
 	 */
 	public function setButtonText($value)
 	{

Indentation +3 added lines, -3 removed lines

@@ -124,9 +124,9 @@
 	}
 
 	/**
-     * Sets the date format for a date validation
-     * @param string the date format value
-     */
+	 * Sets the date format for a date validation
+	 * @param string the date format value
+	 */
 	public function setDateFormat($value)
 	{
 		$this->setViewState('DateFormat', $value, '');

Spacing +7 added lines, -7 removed lines

@@ -49,7 +49,7 @@ 
 	 */
 	public function getDataType()
 	{
-		return $this->getViewState('DataType','String');
+		return $this->getViewState('DataType', 'String');
 	}
 
 	/**
@@ -58,7 +58,7 @@ 
 	 */
 	public function setDataType($value)
 	{
-		$this->setViewState('DataType',TPropertyValue::ensureEnum($value,'TValidationDataType'),TValidationDataType::String);
+		$this->setViewState('DataType', TPropertyValue::ensureEnum($value, 'TValidationDataType'), TValidationDataType::String);
 	}
 
 	/**
@@ -92,14 +92,14 @@ 
 		switch($this->getDataType())
 		{
 			case TValidationDataType::Integer:
-				return preg_match('/^[-+]?[0-9]+$/',trim($value));
+				return preg_match('/^[-+]?[0-9]+$/', trim($value));
 			case TValidationDataType::Float:
-				return preg_match('/^[-+]?([0-9]*\.)?[0-9]+([eE][-+]?[0-9]+)?$/',trim($value));
+				return preg_match('/^[-+]?([0-9]*\.)?[0-9]+([eE][-+]?[0-9]+)?$/', trim($value));
 			case TValidationDataType::Date:
-				$dateFormat = $this->getDateFormat();
+				$dateFormat=$this->getDateFormat();
 				if(strlen($dateFormat))
 				{
-					$formatter = Prado::createComponent('System.Util.TSimpleDateFormatter',$dateFormat);
+					$formatter=Prado::createComponent('System.Util.TSimpleDateFormatter', $dateFormat);
 					return $formatter->isValidDate($value);
 				}
 				else
@@ -114,7 +114,7 @@ 
 	 */
 	protected function getClientScriptOptions()
 	{
-		$options = parent::getClientScriptOptions();
+		$options=parent::getClientScriptOptions();
 		$options['DataType']=$this->getDataType();
 		if(($dateFormat=$this->getDateFormat())!=='')
 			$options['DateFormat']=$dateFormat;

Braces +1 added lines, -2 removed lines

@@ -70,8 +70,7 @@
 				{
 					$domain=substr($value,$pos+1);
 					return $domain===''?false:checkdnsrr($domain,'MX');
-				}
-				else
+				} else
 					return false;
 			}
 		}

framework/Web/UI/WebControls/TDropDownList.php

Doc Comments +1 added lines, -1 removed lines

@@ -213,7 +213,7 @@
 	/**
 	 * Returns the value to be validated.
 	 * This methid is required by IValidatable interface.
-	 * @return mixed the value of the property to be validated.
+	 * @return string the value of the property to be validated.
 	 */
 	public function getValidationPropertyValue()
 	{

Indentation +2 added lines, -2 removed lines

@@ -463,14 +463,14 @@
 	 */
 	public function getIsValid()
 	{
-	    return $this->_isValid;
+		return $this->_isValid;
 	}
 	/**
 	 * @param bool wether this control is valid.
 	 */
 	public function setIsValid($value)
 	{
-	    $this->_isValid=TPropertyValue::ensureBoolean($value);
+		$this->_isValid=TPropertyValue::ensureBoolean($value);
 	}
 
 	/**

Spacing +4 added lines, -4 removed lines

@@ -52,7 +52,7 @@ 
 	 */
 	protected function addAttributesToRender($writer)
 	{
-		$writer->addAttribute('name',$this->getUniqueID());
+		$writer->addAttribute('name', $this->getUniqueID());
 		parent::addAttributesToRender($writer);
 	}
 
@@ -73,15 +73,15 @@ 
 	 * @param array the input data collection
 	 * @return boolean whether the data of the component has been changed
 	 */
-	public function loadPostData($key,$values)
+	public function loadPostData($key, $values)
 	{
 		if(!$this->getEnabled(true))
 			return false;
 		$this->ensureDataBound();
-		$selection=isset($values[$key])?$values[$key]:null;
+		$selection=isset($values[$key]) ? $values[$key] : null;
 		if($selection!==null)
 		{
-			$index=$this->getItems()->findIndexByValue($selection,false);
+			$index=$this->getItems()->findIndexByValue($selection, false);
 			if($this->getSelectedIndex()!==$index)
 			{
 				$this->setSelectedIndex($index);