pradosoft / prado

framework/Web/UI/WebControls/TRepeatInfo.php

Braces +8 added lines, -16 removed lines

@@ -191,13 +191,11 @@ 
 				$control->setCaption($this->_caption);
 				$control->setCaptionAlign($this->_captionAlign);
 			}
-		}
-		else if($this->_repeatLayout===TRepeatLayout::Raw)
+		} else if($this->_repeatLayout===TRepeatLayout::Raw)
 		{
 			$this->renderRawContents($writer,$user);
 			return;
-		}
-		else
+		} else
 			$control=new TWebControl;
 		$control->setID($user->getClientID());
 		$control->copyBaseAttributes($user);
@@ -295,8 +293,7 @@ 
 				}
 			}
 			$writer->renderEndTag();
-		}
-		else
+		} else
 		{
 			$column=0;
 			for($i=0;$i<$itemCount;++$i)
@@ -334,8 +331,7 @@ 
 			$rows=$itemCount;
 			$columns=1;
 			$lastColumns=1;
-		}
-		else
+		} else
 		{
 			$columns=$this->_repeatColumns;
 			$rows=(int)(($itemCount+$columns-1)/$columns);
@@ -391,8 +387,7 @@ 
 						$user->renderItem($writer,$this,'Separator',$index);
 						$writer->renderEndTag();
 						$writer->writeLine();
-					}
-					else if($columns>1)
+					} else if($columns>1)
 						$writer->write("<td></td>\n");
 				}
 				if($row==$rows-1)
@@ -407,8 +402,7 @@ 
 				$writer->writeLine();
 			}
 			$writer->renderEndTag();
-		}
-		else
+		} else
 		{
 			$renderedItems=0;
 			for($row=0;$row<$rows;++$row)
@@ -473,8 +467,7 @@ 
 			$writer->renderEndTag();
 			$writer->renderEndTag();
 			$writer->renderEndTag();
-		}
-		else
+		} else
 		{
 			$user->renderItem($writer,$this,'Header',-1);
 			if($needBreak)
@@ -505,8 +498,7 @@ 
 			$writer->renderEndTag();
 			$writer->renderEndTag();
 			$writer->renderEndTag();
-		}
-		else
+		} else
 			$user->renderItem($writer,$this,'Footer',-1);
 		$writer->writeLine();
 	}

Doc Comments +6 added lines

@@ -178,6 +178,7 @@ 
 	 * Renders contents in raw format.
 	 * @param THtmlWriter writer for the rendering purpose
 	 * @param IRepeatInfoUser repeat information user
+	 * @param IRepeatInfoUser $user
 	 */
 	protected function renderRawContents($writer,$user)
 	{
@@ -201,6 +202,7 @@ 
 	 * Renders contents in horizontal repeat direction.
 	 * @param THtmlWriter writer for the rendering purpose
 	 * @param IRepeatInfoUser repeat information user
+	 * @param IRepeatInfoUser $user
 	 */
 	protected function renderHorizontalContents($writer,$user)
 	{
@@ -283,6 +285,7 @@ 
 	 * Renders contents in veritcal repeat direction.
 	 * @param THtmlWriter writer for the rendering purpose
 	 * @param IRepeatInfoUser repeat information user
+	 * @param IRepeatInfoUser $user
 	 */
 	protected function renderVerticalContents($writer,$user)
 	{
@@ -416,6 +419,8 @@ 
 	 * @param boolean whether to render using table layout
 	 * @param integer number of columns to be rendered
 	 * @param boolean if a line break is needed at the end
+	 * @param boolean $tableLayout
+	 * @param boolean $needBreak
 	 */
 	protected function renderHeader($writer,$user,$tableLayout,$columns,$needBreak)
 	{
@@ -449,6 +454,7 @@ 
 	 * @param IRepeatInfoUser repeat information user
 	 * @param boolean whether to render using table layout
 	 * @param integer number of columns to be rendered
+	 * @param boolean $tableLayout
 	 */
 	protected function renderFooter($writer,$user,$tableLayout,$columns)
 	{

Spacing +82 added lines, -82 removed lines

@@ -83,7 +83,7 @@ 
 	 */
 	public function setCaptionAlign($value)
 	{
-		$this->_captionAlign=TPropertyValue::ensureEnum($value,'Prado\\Web\\UI\\WebControls\\TTableCaptionAlign');
+		$this->_captionAlign=TPropertyValue::ensureEnum($value, 'Prado\\Web\\UI\\WebControls\\TTableCaptionAlign');
 	}
 
 	/**
@@ -99,7 +99,7 @@ 
 	 */
 	public function setRepeatColumns($value)
 	{
-		if(($value=TPropertyValue::ensureInteger($value))<0)
+		if(($value=TPropertyValue::ensureInteger($value)) < 0)
 			throw new TInvalidDataValueException('repeatinfo_repeatcolumns_invalid');
 		$this->_repeatColumns=$value;
 	}
@@ -117,7 +117,7 @@ 
 	 */
 	public function setRepeatDirection($value)
 	{
-		$this->_repeatDirection=TPropertyValue::ensureEnum($value,'Prado\\Web\\UI\\WebControls\\TRepeatDirection');
+		$this->_repeatDirection=TPropertyValue::ensureEnum($value, 'Prado\\Web\\UI\\WebControls\\TRepeatDirection');
 	}
 
 	/**
@@ -133,7 +133,7 @@ 
 	 */
 	public function setRepeatLayout($value)
 	{
-		$this->_repeatLayout=TPropertyValue::ensureEnum($value,'Prado\\Web\\UI\\WebControls\\TRepeatLayout');
+		$this->_repeatLayout=TPropertyValue::ensureEnum($value, 'Prado\\Web\\UI\\WebControls\\TRepeatLayout');
 	}
 
 	/**
@@ -154,7 +154,7 @@ 
 		}
 		else if($this->_repeatLayout===TRepeatLayout::Raw)
 		{
-			$this->renderRawContents($writer,$user);
+			$this->renderRawContents($writer, $user);
 			return;
 		}
 		else
@@ -167,9 +167,9 @@ 
 		$writer->writeLine();
 
 		if($this->_repeatDirection===TRepeatDirection::Vertical)
-			$this->renderVerticalContents($writer,$user);
+			$this->renderVerticalContents($writer, $user);
 		else
-			$this->renderHorizontalContents($writer,$user);
+			$this->renderHorizontalContents($writer, $user);
 
 		$control->renderEndTag($writer);
 	}
@@ -179,22 +179,22 @@ 
 	 * @param THtmlWriter writer for the rendering purpose
 	 * @param IRepeatInfoUser repeat information user
 	 */
-	protected function renderRawContents($writer,$user)
+	protected function renderRawContents($writer, $user)
 	{
 		if($user->getHasHeader())
-			$user->renderItem($writer,$this,'Header',-1);
+			$user->renderItem($writer, $this, 'Header', -1);
 
 		// render items
 		$hasSeparators=$user->getHasSeparators();
 		$itemCount=$user->getItemCount();
-		for($i=0;$i<$itemCount;++$i)
+		for($i=0; $i < $itemCount; ++$i)
 		{
-			$user->renderItem($writer,$this,'Item',$i);
-			if($hasSeparators && $i!=$itemCount-1)
-				$user->renderItem($writer,$this,'Separator',$i);
+			$user->renderItem($writer, $this, 'Item', $i);
+			if($hasSeparators && $i!=$itemCount - 1)
+				$user->renderItem($writer, $this, 'Separator', $i);
 		}
 		if($user->getHasFooter())
-			$user->renderItem($writer,$this,'Footer',-1);
+			$user->renderItem($writer, $this, 'Footer', -1);
 	}
 
 	/**
@@ -202,52 +202,52 @@ 
 	 * @param THtmlWriter writer for the rendering purpose
 	 * @param IRepeatInfoUser repeat information user
 	 */
-	protected function renderHorizontalContents($writer,$user)
+	protected function renderHorizontalContents($writer, $user)
 	{
 		$tableLayout=($this->_repeatLayout===TRepeatLayout::Table);
 		$hasSeparators=$user->getHasSeparators();
 		$itemCount=$user->getItemCount();
-		$columns=$this->_repeatColumns===0?$itemCount:$this->_repeatColumns;
-		$totalColumns=$hasSeparators?$columns+$columns:$columns;
-		$needBreak=$columns<$itemCount;
+		$columns=$this->_repeatColumns===0 ? $itemCount : $this->_repeatColumns;
+		$totalColumns=$hasSeparators ? $columns + $columns : $columns;
+		$needBreak=$columns < $itemCount;
 
 		if($user->getHasHeader())
-			$this->renderHeader($writer,$user,$tableLayout,$totalColumns,$needBreak);
+			$this->renderHeader($writer, $user, $tableLayout, $totalColumns, $needBreak);
 
 		// render items
 		if($tableLayout)
 		{
 			$writer->renderBeginTag('tbody');
 			$column=0;
-			for($i=0;$i<$itemCount;++$i)
+			for($i=0; $i < $itemCount; ++$i)
 			{
 				if($column==0)
 					$writer->renderBeginTag('tr');
-				if(($style=$user->generateItemStyle('Item',$i))!==null)
+				if(($style=$user->generateItemStyle('Item', $i))!==null)
 					$style->addAttributesToRender($writer);
 				$writer->renderBeginTag('td');
-				$user->renderItem($writer,$this,'Item',$i);
+				$user->renderItem($writer, $this, 'Item', $i);
 				$writer->renderEndTag();
 				$writer->writeLine();
-				if($hasSeparators && $i!=$itemCount-1)
+				if($hasSeparators && $i!=$itemCount - 1)
 				{
-					if(($style=$user->generateItemStyle('Separator',$i))!==null)
+					if(($style=$user->generateItemStyle('Separator', $i))!==null)
 						$style->addAttributesToRender($writer);
 					$writer->renderBeginTag('td');
-					$user->renderItem($writer,$this,'Separator',$i);
+					$user->renderItem($writer, $this, 'Separator', $i);
 					$writer->renderEndTag();
 					$writer->writeLine();
 				}
 				$column++;
-				if($i==$itemCount-1)
+				if($i==$itemCount - 1)
 				{
-					$restColumns=$columns-$column;
+					$restColumns=$columns - $column;
 					if($hasSeparators)
-						$restColumns=$restColumns?$restColumns+$restColumns+1:1;
-					for($j=0;$j<$restColumns;++$j)
+						$restColumns=$restColumns ? $restColumns + $restColumns + 1 : 1;
+					for($j=0; $j < $restColumns; ++$j)
 						$writer->write("<td></td>\n");
 				}
-				if($column==$columns || $i==$itemCount-1)
+				if($column==$columns || $i==$itemCount - 1)
 				{
 					$writer->renderEndTag();
 					$writer->writeLine();
@@ -259,13 +259,13 @@ 
 		else
 		{
 			$column=0;
-			for($i=0;$i<$itemCount;++$i)
+			for($i=0; $i < $itemCount; ++$i)
 			{
-				$user->renderItem($writer,$this,'Item',$i);
-				if($hasSeparators && $i!=$itemCount-1)
-					$user->renderItem($writer,$this,'Separator',$i);
+				$user->renderItem($writer, $this, 'Item', $i);
+				if($hasSeparators && $i!=$itemCount - 1)
+					$user->renderItem($writer, $this, 'Separator', $i);
 				$column++;
-				if($column==$columns || $i==$itemCount-1)
+				if($column==$columns || $i==$itemCount - 1)
 				{
 					if($needBreak)
 						$writer->writeBreak();
@@ -276,7 +276,7 @@ 
 		}
 
 		if($user->getHasFooter())
-			$this->renderFooter($writer,$user,$tableLayout,$totalColumns,$needBreak);
+			$this->renderFooter($writer, $user, $tableLayout, $totalColumns, $needBreak);
 	}
 
 	/**
@@ -284,12 +284,12 @@ 
 	 * @param THtmlWriter writer for the rendering purpose
 	 * @param IRepeatInfoUser repeat information user
 	 */
-	protected function renderVerticalContents($writer,$user)
+	protected function renderVerticalContents($writer, $user)
 	{
 		$tableLayout=($this->_repeatLayout===TRepeatLayout::Table);
 		$hasSeparators=$user->getHasSeparators();
 		$itemCount=$user->getItemCount();
-		if($this->_repeatColumns<=1)
+		if($this->_repeatColumns <= 1)
 		{
 			$rows=$itemCount;
 			$columns=1;
@@ -298,69 +298,69 @@ 
 		else
 		{
 			$columns=$this->_repeatColumns;
-			$rows=(int)(($itemCount+$columns-1)/$columns);
-			if($rows==0 && $itemCount>0)
+			$rows=(int) (($itemCount + $columns - 1) / $columns);
+			if($rows==0 && $itemCount > 0)
 				$rows=1;
-			if(($lastColumns=$itemCount%$columns)==0)
+			if(($lastColumns=$itemCount % $columns)==0)
 				$lastColumns=$columns;
 		}
-		$totalColumns=$hasSeparators?$columns+$columns:$columns;
+		$totalColumns=$hasSeparators ? $columns + $columns : $columns;
 
 		if($user->getHasHeader())
-			$this->renderHeader($writer,$user,$tableLayout,$totalColumns,false);
+			$this->renderHeader($writer, $user, $tableLayout, $totalColumns, false);
 
 		if($tableLayout)
 		{
 			$writer->renderBeginTag('tbody');
 			$renderedItems=0;
-			for($row=0;$row<$rows;++$row)
+			for($row=0; $row < $rows; ++$row)
 			{
 				$index=$row;
 				$writer->renderBeginTag('tr');
-				for($col=0;$col<$columns;++$col)
+				for($col=0; $col < $columns; ++$col)
 				{
-					if($renderedItems>=$itemCount)
+					if($renderedItems >= $itemCount)
 						break;
-					if($col>0)
+					if($col > 0)
 					{
 						$index+=$rows;
-						if($col-1>=$lastColumns)
+						if($col - 1 >= $lastColumns)
 							$index--;
 					}
-					if($index>=$itemCount)
+					if($index >= $itemCount)
 						continue;
 					$renderedItems++;
-					if(($style=$user->generateItemStyle('Item',$index))!==null)
+					if(($style=$user->generateItemStyle('Item', $index))!==null)
 						$style->addAttributesToRender($writer);
 					$writer->renderBeginTag('td');
-					$user->renderItem($writer,$this,'Item',$index);
+					$user->renderItem($writer, $this, 'Item', $index);
 					$writer->renderEndTag();
 					$writer->writeLine();
 					if(!$hasSeparators)
 						continue;
-					if($renderedItems<$itemCount-1)
+					if($renderedItems < $itemCount - 1)
 					{
 						if($columns==1)
 						{
 							$writer->renderEndTag();
 							$writer->renderBeginTag('tr');
 						}
-						if(($style=$user->generateItemStyle('Separator',$index))!==null)
+						if(($style=$user->generateItemStyle('Separator', $index))!==null)
 							$style->addAttributesToRender($writer);
 						$writer->renderBeginTag('td');
-						$user->renderItem($writer,$this,'Separator',$index);
+						$user->renderItem($writer, $this, 'Separator', $index);
 						$writer->renderEndTag();
 						$writer->writeLine();
 					}
-					else if($columns>1)
+					else if($columns > 1)
 						$writer->write("<td></td>\n");
 				}
-				if($row==$rows-1)
+				if($row==$rows - 1)
 				{
-					$restColumns=$columns-$lastColumns;
+					$restColumns=$columns - $lastColumns;
 					if($hasSeparators)
 						$restColumns+=$restColumns;
-					for($col=0;$col<$restColumns;++$col)
+					for($col=0; $col < $restColumns; ++$col)
 						$writer->write("<td></td>\n");
 				}
 				$writer->renderEndTag();
@@ -371,41 +371,41 @@ 
 		else
 		{
 			$renderedItems=0;
-			for($row=0;$row<$rows;++$row)
+			for($row=0; $row < $rows; ++$row)
 			{
 				$index=$row;
-				for($col=0;$col<$columns;++$col)
+				for($col=0; $col < $columns; ++$col)
 				{
-					if($renderedItems>=$itemCount)
+					if($renderedItems >= $itemCount)
 						break;
-					if($col>0)
+					if($col > 0)
 					{
 						$index+=$rows;
-						if($col-1>=$lastColumns)
+						if($col - 1 >= $lastColumns)
 							$index--;
 					}
-					if($index>=$itemCount)
+					if($index >= $itemCount)
 						continue;
 					$renderedItems++;
-					$user->renderItem($writer,$this,'Item',$index);
+					$user->renderItem($writer, $this, 'Item', $index);
 					$writer->writeLine();
 					if(!$hasSeparators)
 						continue;
-					if($renderedItems<$itemCount-1)
+					if($renderedItems < $itemCount - 1)
 					{
 						if($columns==1)
 							$writer->writeBreak();
-						$user->renderItem($writer,$this,'Separator',$index);
+						$user->renderItem($writer, $this, 'Separator', $index);
 					}
 					$writer->writeLine();
 				}
-				if($row<$rows-1 || $user->getHasFooter())
+				if($row < $rows - 1 || $user->getHasFooter())
 					$writer->writeBreak();
 			}
 		}
 
 		if($user->getHasFooter())
-			$this->renderFooter($writer,$user,$tableLayout,$totalColumns,false);
+			$this->renderFooter($writer, $user, $tableLayout, $totalColumns, false);
 
 	}
 
@@ -417,26 +417,26 @@ 
 	 * @param integer number of columns to be rendered
 	 * @param boolean if a line break is needed at the end
 	 */
-	protected function renderHeader($writer,$user,$tableLayout,$columns,$needBreak)
+	protected function renderHeader($writer, $user, $tableLayout, $columns, $needBreak)
 	{
 		if($tableLayout)
 		{
 			$writer->renderBeginTag('thead');
 			$writer->renderBeginTag('tr');
-			if($columns>1)
-				$writer->addAttribute('colspan',"$columns");
-			$writer->addAttribute('scope','col');
-			if(($style=$user->generateItemStyle('Header',-1))!==null)
+			if($columns > 1)
+				$writer->addAttribute('colspan', "$columns");
+			$writer->addAttribute('scope', 'col');
+			if(($style=$user->generateItemStyle('Header', -1))!==null)
 				$style->addAttributesToRender($writer);
 			$writer->renderBeginTag('th');
-			$user->renderItem($writer,$this,'Header',-1);
+			$user->renderItem($writer, $this, 'Header', -1);
 			$writer->renderEndTag();
 			$writer->renderEndTag();
 			$writer->renderEndTag();
 		}
 		else
 		{
-			$user->renderItem($writer,$this,'Header',-1);
+			$user->renderItem($writer, $this, 'Header', -1);
 			if($needBreak)
 				$writer->writeBreak();
 		}
@@ -450,24 +450,24 @@ 
 	 * @param boolean whether to render using table layout
 	 * @param integer number of columns to be rendered
 	 */
-	protected function renderFooter($writer,$user,$tableLayout,$columns)
+	protected function renderFooter($writer, $user, $tableLayout, $columns)
 	{
 		if($tableLayout)
 		{
 			$writer->renderBeginTag('tfoot');
 			$writer->renderBeginTag('tr');
-			if($columns>1)
-				$writer->addAttribute('colspan',"$columns");
-			if(($style=$user->generateItemStyle('Footer',-1))!==null)
+			if($columns > 1)
+				$writer->addAttribute('colspan', "$columns");
+			if(($style=$user->generateItemStyle('Footer', -1))!==null)
 				$style->addAttributesToRender($writer);
 			$writer->renderBeginTag('td');
-			$user->renderItem($writer,$this,'Footer',-1);
+			$user->renderItem($writer, $this, 'Footer', -1);
 			$writer->renderEndTag();
 			$writer->renderEndTag();
 			$writer->renderEndTag();
 		}
 		else
-			$user->renderItem($writer,$this,'Footer',-1);
+			$user->renderItem($writer, $this, 'Footer', -1);
 		$writer->writeLine();
 	}
 }
\ No newline at end of file

framework/Web/UI/WebControls/TButton.php

Braces +2 added lines, -4 removed lines

@@ -106,8 +106,7 @@ 
 		{
 			if($this->getEnableClientScript() && $this->needPostBackScript())
 				$this->renderClientControlScript($writer);
-		}
-		else if($this->getEnabled()) // in this case, parent will not render 'disabled'
+		} else if($this->getEnabled()) // in this case, parent will not render 'disabled'
 			$writer->addAttribute('disabled','disabled');
 		parent::addAttributesToRender($writer);
 	}
@@ -141,8 +140,7 @@ 
 		{
 			$group=$this->getValidationGroup();
 			return $this->getPage()->getValidators($group)->getCount()>0;
-		}
-		else
+		} else
 			return false;
 	}
 

Spacing +29 added lines, -29 removed lines

@@ -64,7 +64,7 @@ 
 	 */
 	public function getButtonTag()
 	{
-		return $this->getViewState('ButtonTag',TButtonTag::Input);
+		return $this->getViewState('ButtonTag', TButtonTag::Input);
 	}
 
 	/**
@@ -72,7 +72,7 @@ 
 	 */
 	public function setButtonTag($value)
 	{
-		$this->setViewState('ButtonTag',TPropertyValue::ensureEnum($value,'TButtonTag'),TButtonTag::Input);
+		$this->setViewState('ButtonTag', TPropertyValue::ensureEnum($value, 'TButtonTag'), TButtonTag::Input);
 	}
 
 	/**
@@ -80,7 +80,7 @@ 
 	 */
 	public function getEnableClientScript()
 	{
-		return $this->getViewState('EnableClientScript',true);
+		return $this->getViewState('EnableClientScript', true);
 	}
 
 	/**
@@ -88,7 +88,7 @@ 
 	 */
 	public function setEnableClientScript($value)
 	{
-		$this->setViewState('EnableClientScript',TPropertyValue::ensureBoolean($value),true);
+		$this->setViewState('EnableClientScript', TPropertyValue::ensureBoolean($value), true);
 	}
 
 	/**
@@ -100,20 +100,20 @@ 
 	{
 		$page=$this->getPage();
 		$page->ensureRenderInForm($this);
-		$writer->addAttribute('type',strtolower($this->getButtonType()));
+		$writer->addAttribute('type', strtolower($this->getButtonType()));
 		if(($uniqueID=$this->getUniqueID())!=='')
-			$writer->addAttribute('name',$uniqueID);
+			$writer->addAttribute('name', $uniqueID);
 		if($this->getButtonTag()===TButtonTag::Button)
 		  $this->addParsedObject($this->getText());
 		else
-		  $writer->addAttribute('value',$this->getText());
+		  $writer->addAttribute('value', $this->getText());
 		if($this->getEnabled(true))
 		{
 			if($this->getEnableClientScript() && $this->needPostBackScript())
 				$this->renderClientControlScript($writer);
 		}
 		else if($this->getEnabled()) // in this case, parent will not render 'disabled'
-			$writer->addAttribute('disabled','disabled');
+			$writer->addAttribute('disabled', 'disabled');
 
 		parent::addAttributesToRender($writer);
 	}
@@ -123,8 +123,8 @@ 
 	 */
 	protected function renderClientControlScript($writer)
 	{
-		$writer->addAttribute('id',$this->getClientID());
-		$this->getPage()->getClientScript()->registerPostBackControl($this->getClientClassName(),$this->getPostBackOptions());
+		$writer->addAttribute('id', $this->getClientID());
+		$this->getPage()->getClientScript()->registerPostBackControl($this->getClientClassName(), $this->getPostBackOptions());
 	}
 
 	/**
@@ -145,7 +145,7 @@ 
 		if($this->getCausesValidation())
 		{
 			$group=$this->getValidationGroup();
-			return $this->getPage()->getValidators($group)->getCount()>0;
+			return $this->getPage()->getValidators($group)->getCount() > 0;
 		}
 		else
 			return false;
@@ -156,7 +156,7 @@ 
 	 */
 	public function setIsDefaultButton($value)
 	{
-		$this->setViewState('IsDefaultButton', TPropertyValue::ensureBoolean($value),false);
+		$this->setViewState('IsDefaultButton', TPropertyValue::ensureBoolean($value), false);
 	}
 
 	/**
@@ -186,7 +186,7 @@ 
 	{
 		$options['ID']=$this->getClientID();
 		$options['CausesValidation']=$this->getCausesValidation();
-		$options['EventTarget'] = $this->getUniqueID();
+		$options['EventTarget']=$this->getUniqueID();
 		$options['ValidationGroup']=$this->getValidationGroup();
 
 		return $options;
@@ -213,7 +213,7 @@ 
 	 */
 	public function onClick($param)
 	{
-		$this->raiseEvent('OnClick',$this,$param);
+		$this->raiseEvent('OnClick', $this, $param);
 	}
 
 	/**
@@ -225,8 +225,8 @@ 
 	 */
 	public function onCommand($param)
 	{
-		$this->raiseEvent('OnCommand',$this,$param);
-		$this->raiseBubbleEvent($this,$param);
+		$this->raiseEvent('OnCommand', $this, $param);
+		$this->raiseBubbleEvent($this, $param);
 	}
 
 	/**
@@ -243,7 +243,7 @@ 
 		if($this->getCausesValidation())
 			$this->getPage()->validate($this->getValidationGroup());
 		$this->onClick(null);
-		$this->onCommand(new \Prado\Web\UI\TCommandEventParameter($this->getCommandName(),$this->getCommandParameter()));
+		$this->onCommand(new \Prado\Web\UI\TCommandEventParameter($this->getCommandName(), $this->getCommandParameter()));
 	}
 
 	/**
@@ -251,7 +251,7 @@ 
 	 */
 	public function getText()
 	{
-		return $this->getViewState('Text','');
+		return $this->getViewState('Text', '');
 	}
 
 	/**
@@ -259,7 +259,7 @@ 
 	 */
 	public function setText($value)
 	{
-		$this->setViewState('Text',$value,'');
+		$this->setViewState('Text', $value, '');
 	}
 
 	/**
@@ -293,7 +293,7 @@ 
 	 */
 	public function getCausesValidation()
 	{
-		return $this->getViewState('CausesValidation',true);
+		return $this->getViewState('CausesValidation', true);
 	}
 
 	/**
@@ -301,7 +301,7 @@ 
 	 */
 	public function setCausesValidation($value)
 	{
-		$this->setViewState('CausesValidation',TPropertyValue::ensureBoolean($value),true);
+		$this->setViewState('CausesValidation', TPropertyValue::ensureBoolean($value), true);
 	}
 
 	/**
@@ -309,7 +309,7 @@ 
 	 */
 	public function getCommandName()
 	{
-		return $this->getViewState('CommandName','');
+		return $this->getViewState('CommandName', '');
 	}
 
 	/**
@@ -317,7 +317,7 @@ 
 	 */
 	public function setCommandName($value)
 	{
-		$this->setViewState('CommandName',$value,'');
+		$this->setViewState('CommandName', $value, '');
 	}
 
 	/**
@@ -325,7 +325,7 @@ 
 	 */
 	public function getCommandParameter()
 	{
-		return $this->getViewState('CommandParameter','');
+		return $this->getViewState('CommandParameter', '');
 	}
 
 	/**
@@ -333,7 +333,7 @@ 
 	 */
 	public function setCommandParameter($value)
 	{
-		$this->setViewState('CommandParameter',$value,'');
+		$this->setViewState('CommandParameter', $value, '');
 	}
 
 	/**
@@ -341,7 +341,7 @@ 
 	 */
 	public function getValidationGroup()
 	{
-		return $this->getViewState('ValidationGroup','');
+		return $this->getViewState('ValidationGroup', '');
 	}
 
 	/**
@@ -349,7 +349,7 @@ 
 	 */
 	public function setValidationGroup($value)
 	{
-		$this->setViewState('ValidationGroup',$value,'');
+		$this->setViewState('ValidationGroup', $value, '');
 	}
 
 	/**
@@ -357,7 +357,7 @@ 
 	 */
 	public function getButtonType()
 	{
-		return $this->getViewState('ButtonType',TButtonType::Submit);
+		return $this->getViewState('ButtonType', TButtonType::Submit);
 	}
 
 	/**
@@ -365,6 +365,6 @@ 
 	 */
 	public function setButtonType($value)
 	{
-		$this->setViewState('ButtonType',TPropertyValue::ensureEnum($value,'Prado\\Web\\UI\\WebControls\\TButtonType'),TButtonType::Submit);
+		$this->setViewState('ButtonType', TPropertyValue::ensureEnum($value, 'Prado\\Web\\UI\\WebControls\\TButtonType'), TButtonType::Submit);
 	}
 }

Doc Comments +1 added lines

@@ -222,6 +222,7 @@
 	 * If you override this method, be sure to call the parent implementation
 	 * so that the event handlers can be invoked.
 	 * @param \Prado\Web\UI\TCommandEventParameter event parameter to be passed to the event handlers
+	 * @param TCommandEventParameter $param
 	 */
 	public function onCommand($param)
 	{

framework/Web/UI/WebControls/TRequiredFieldValidator.php

Spacing +15 added lines, -15 removed lines

@@ -51,7 +51,7 @@ 
 	 */
 	public function getInitialValue()
 	{
-		return $this->getViewState('InitialValue',$this->getControlPromptValue());
+		return $this->getViewState('InitialValue', $this->getControlPromptValue());
 	}
 
 	/**
@@ -61,7 +61,7 @@ 
 	 */
 	protected function getControlPromptValue()
 	{
-		$control = $this->getValidationTarget();
+		$control=$this->getValidationTarget();
 		if($control instanceof TListControl)
 			return $control->getPromptValue();
 		return '';
@@ -73,7 +73,7 @@ 
 	 */
 	public function setInitialValue($value)
 	{
-		$this->setViewState('InitialValue',TPropertyValue::ensureString($value),'');
+		$this->setViewState('InitialValue', TPropertyValue::ensureString($value), '');
 	}
 
 	/**
@@ -89,7 +89,7 @@ 
 	 */
 	protected function evaluateIsValid()
 	{
-		$control = $this->getValidationTarget();
+		$control=$this->getValidationTarget();
 		if($control instanceof TListControl)
 			return $this->validateListControl($control);
 		else if($control instanceof TRadioButton && strlen($control->getGroupName()) > 0)
@@ -100,11 +100,11 @@ 
 
 	private function validateListControl($control)
 	{
-		$initial = trim($this->getInitialValue());
-		$count = 0;
+		$initial=trim($this->getInitialValue());
+		$count=0;
 		foreach($control->getItems() as $item)
 		{
-			if($item->getSelected() && $item->getValue() != $initial)
+			if($item->getSelected() && $item->getValue()!=$initial)
 				$count++;
 		}
 		return $count > 0;
@@ -112,13 +112,13 @@ 
 
 	private function validateRadioButtonGroup($control)
 	{
-		$initial = trim($this->getInitialValue());
+		$initial=trim($this->getInitialValue());
 		foreach($control->getRadioButtonsInGroup() as $radio)
 		{
 			if($radio->getChecked())
 			{
-				if(strlen($value = $radio->getValue()) > 0)
-					return $value !== $initial;
+				if(strlen($value=$radio->getValue()) > 0)
+					return $value!==$initial;
 				else
 					return true;
 			}
@@ -128,7 +128,7 @@ 
 
 	private function validateStandardControl($control)
 	{
-		$initial = trim($this->getInitialValue());
+		$initial=trim($this->getInitialValue());
 		$value=$this->getValidationValue($control);
 		return (is_bool($value) && $value) || trim($value)!==$initial;
 	}
@@ -139,13 +139,13 @@ 
 	 */
 	protected function getClientScriptOptions()
 	{
-		$options = parent::getClientScriptOptions();
+		$options=parent::getClientScriptOptions();
 		$options['InitialValue']=$this->getInitialValue();
-		$control = $this->getValidationTarget();
+		$control=$this->getValidationTarget();
 		if($control instanceof TListControl)
-			$options['TotalItems'] = $control->getItemCount();
+			$options['TotalItems']=$control->getItemCount();
 		if($control instanceof TRadioButton && strlen($control->getGroupName()) > 0)
-			$options['GroupName'] = $control->getGroupName();
+			$options['GroupName']=$control->getGroupName();
 		return $options;
 	}
 }

Doc Comments +9 added lines

@@ -96,6 +96,9 @@ 
 			return $this->validateStandardControl($control);
 	}
 
+	/**
+	 * @param TListControl $control
+	 */
 	private function validateListControl($control)
 	{
 		$initial = trim($this->getInitialValue());
@@ -108,6 +111,9 @@ 
 		return $count > 0;
 	}
 
+	/**
+	 * @param TRadioButton $control
+	 */
 	private function validateRadioButtonGroup($control)
 	{
 		$initial = trim($this->getInitialValue());
@@ -124,6 +130,9 @@ 
 		return false;
 	}
 
+	/**
+	 * @param \Prado\Web\UI\TControl $control
+	 */
 	private function validateStandardControl($control)
 	{
 		$initial = trim($this->getInitialValue());

framework/Web/UI/WebControls/TReCaptchaValidator.php

Spacing +18 added lines, -18 removed lines

@@ -30,7 +30,7 @@ 
  */
 class TReCaptchaValidator extends TBaseValidator
 {
-	protected $_isvalid = null;
+	protected $_isvalid=null;
 
 	/**
 	 * Gets the name of the javascript class responsible for performing validation for this control.
@@ -49,18 +49,18 @@ 
 
 	protected function getCaptchaControl()
 	{
-		$control = $this->getValidationTarget();
-		if (!$control)
+		$control=$this->getValidationTarget();
+		if(!$control)
 			throw new Exception('No target control specified for TReCaptchaValidator');
-		if (!($control instanceof TReCaptcha))
+		if(!($control instanceof TReCaptcha))
 			throw new Exception('TReCaptchaValidator only works with TReCaptcha controls');
 		return $control;
 	}
 
 	public function getClientScriptOptions()
 	{
-		$options = parent::getClientScriptOptions();
-		$options['ResponseFieldName'] = $this->getCaptchaControl()->getResponseFieldName();
+		$options=parent::getClientScriptOptions();
+		$options['ResponseFieldName']=$this->getCaptchaControl()->getResponseFieldName();
 		return $options;
 	}
 
@@ -74,10 +74,10 @@ 
 	protected function evaluateIsValid()
 	{
 		// check validity only once (if trying to evaulate multiple times, all redundant checks would fail)
-		if (is_null($this->_isvalid))
+		if(is_null($this->_isvalid))
 		{
-			$control = $this->getCaptchaControl();
-			$this->_isvalid = $control->validate();
+			$control=$this->getCaptchaControl();
+			$this->_isvalid=$control->validate();
 		}
 		return ($this->_isvalid==true);
 	}
@@ -86,23 +86,23 @@ 
 	{
 		parent::onPreRender($param);
 
-		$cs = $this->Page->getClientScript();
+		$cs=$this->Page->getClientScript();
 		$cs->registerPradoScript('validator');
 
 		// communicate validation status to the client side
-		$value = $this->_isvalid===false ? '0' : '1';
-		$cs->registerHiddenField($this->getClientID().'_1',$value);
+		$value=$this->_isvalid===false ? '0' : '1';
+		$cs->registerHiddenField($this->getClientID().'_1', $value);
 		
 		// update validator display
-		if ($control = $this->getValidationTarget())
+		if($control=$this->getValidationTarget())
 		{
-			$fn = 'captchaUpdateValidatorStatus_'.$this->getClientID();
+			$fn='captchaUpdateValidatorStatus_'.$this->getClientID();
 
 			// check if we need to request a new captcha too
-			if ($this->Page->IsCallback)
+			if($this->Page->IsCallback)
 			{
-				if ($control->getVisible(true))
-					if (!is_null($this->_isvalid))
+				if($control->getVisible(true))
+					if(!is_null($this->_isvalid))
 					{
 						// if the response has been tested and we reach the pre-render phase 
 						// then we need to regenerate the token, because it won't test positive
@@ -112,7 +112,7 @@ 
 					}
 			}
 
-			$cs->registerEndScript($this->getClientID().'::validate', implode(' ',array(
+			$cs->registerEndScript($this->getClientID().'::validate', implode(' ', array(
 				// this function will be used to update the validator
 				'function '.$fn.'(valid)',
 				'{',

framework/Web/UI/WebControls/TJavascriptLogger.php

Spacing +10 added lines, -10 removed lines

@@ -30,7 +30,7 @@ 
  */
 class TJavascriptLogger extends TWebControl
 {
-	private static $_keyCodes = array(
+	private static $_keyCodes=array(
 		'0'=>48, '1'=>49, '2'=>50, '3'=>51, '4'=>52, '5'=>53, '6'=>54, '7'=>55, '8'=>56, '9'=>57,
 		'a'=>65, 'b'=>66, 'c'=>67, 'd'=>68, 'e'=>69, 'f'=>70, 'g'=>71, 'h'=>72,
 		'i'=>73, 'j'=>74, 'k'=>75, 'l'=>76, 'm'=>77, 'n'=>78, 'o'=>79, 'p'=>80,
@@ -66,11 +66,11 @@ 
 	 */
 	public function onPreRender($param)
 	{
-		$key = strtolower($this->getToggleKey());
-		$code = isset(self::$_keyCodes[$key]) ? self::$_keyCodes[$key] : 74;
-		$js = "var logConsole; jQuery(function() { logConsole = new LogConsole($code)}); ";
-		$cs = $this->getPage()->getClientScript();
-		$cs->registerBeginScript($this->getClientID(),$js);
+		$key=strtolower($this->getToggleKey());
+		$code=isset(self::$_keyCodes[$key]) ? self::$_keyCodes[$key] : 74;
+		$js="var logConsole; jQuery(function() { logConsole = new LogConsole($code)}); ";
+		$cs=$this->getPage()->getClientScript();
+		$cs->registerBeginScript($this->getClientID(), $js);
 		$cs->registerPradoScript('logger');
 	}
 
@@ -81,10 +81,10 @@ 
 	 */
 	public function renderContents($writer)
 	{
-		$code = strtoupper($this->getToggleKey());
-		$info = '(<a href="http://web.archive.org/web/20060512041505/gleepglop.com/javascripts/logger/" target="_blank">more info</a>).';
-		$link = '<a href="javascript:if(logConsole)logConsole.toggle()">toggle the javascript log console.</a>';
-		$usage = 'Press ALT-'.$code.' (Or CTRL-'.$code.' on OS X) to';
+		$code=strtoupper($this->getToggleKey());
+		$info='(<a href="http://web.archive.org/web/20060512041505/gleepglop.com/javascripts/logger/" target="_blank">more info</a>).';
+		$link='<a href="javascript:if(logConsole)logConsole.toggle()">toggle the javascript log console.</a>';
+		$usage='Press ALT-'.$code.' (Or CTRL-'.$code.' on OS X) to';
 		$writer->write("{$usage} {$link} {$info}");
 	}
 }

framework/Web/UI/WebControls/TImage.php

Spacing +12 added lines, -12 removed lines

@@ -39,12 +39,12 @@ 
 	 */
 	protected function addAttributesToRender($writer)
 	{
-		$writer->addAttribute('src',$this->getImageUrl());
-		$writer->addAttribute('alt',$this->getAlternateText());
+		$writer->addAttribute('src', $this->getImageUrl());
+		$writer->addAttribute('alt', $this->getAlternateText());
 		if(($desc=$this->getDescriptionUrl())!=='')
-			$writer->addAttribute('longdesc',$desc);
+			$writer->addAttribute('longdesc', $desc);
 		if(($align=$this->getImageAlign())!=='')
-			$writer->addAttribute('align',$align);
+			$writer->addAttribute('align', $align);
 		parent::addAttributesToRender($writer);
 	}
 
@@ -62,7 +62,7 @@ 
 	 */
 	public function getAlternateText()
 	{
-		return $this->getViewState('AlternateText','');
+		return $this->getViewState('AlternateText', '');
 	}
 
 	/**
@@ -71,7 +71,7 @@ 
 	 */
 	public function setAlternateText($value)
 	{
-		$this->setViewState('AlternateText',$value,'');
+		$this->setViewState('AlternateText', $value, '');
 	}
 
 	/**
@@ -79,7 +79,7 @@ 
 	 */
 	public function getImageAlign()
 	{
-		return $this->getViewState('ImageAlign','');
+		return $this->getViewState('ImageAlign', '');
 	}
 
 	/**
@@ -91,7 +91,7 @@ 
 	 */
 	public function setImageAlign($value)
 	{
-		$this->setViewState('ImageAlign',$value,'');
+		$this->setViewState('ImageAlign', $value, '');
 	}
 
 	/**
@@ -99,7 +99,7 @@ 
 	 */
 	public function getImageUrl()
 	{
-		return $this->getViewState('ImageUrl','');
+		return $this->getViewState('ImageUrl', '');
 	}
 
 	/**
@@ -107,7 +107,7 @@ 
 	 */
 	public function setImageUrl($value)
 	{
-		$this->setViewState('ImageUrl',$value,'');
+		$this->setViewState('ImageUrl', $value, '');
 	}
 
 	/**
@@ -141,7 +141,7 @@ 
 	 */
 	public function getDescriptionUrl()
 	{
-		return $this->getViewState('DescriptionUrl','');
+		return $this->getViewState('DescriptionUrl', '');
 	}
 
 	/**
@@ -149,7 +149,7 @@ 
 	 */
 	public function setDescriptionUrl($value)
 	{
-		$this->setViewState('DescriptionUrl',$value,'');
+		$this->setViewState('DescriptionUrl', $value, '');
 	}
 }
 

framework/Web/UI/WebControls/TContent.php

Spacing +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
 	{
 		if(($id=$this->getID())==='')
 			throw new TConfigurationException('content_id_required');
-		$this->getTemplateControl()->registerContent($id,$this);
+		$this->getTemplateControl()->registerContent($id, $this);
 	}
 }
 

framework/Web/UI/WebControls/TRadioButtonList.php

Spacing +2 added lines, -2 removed lines

@@ -57,9 +57,9 @@
 	 * @param array the input data collection
 	 * @return boolean whether the data of the control has been changed
 	 */
-	public function loadPostData($key,$values)
+	public function loadPostData($key, $values)
 	{
-		$value=isset($values[$key])?$values[$key]:'';
+		$value=isset($values[$key]) ? $values[$key] : '';
 		$oldSelection=$this->getSelectedIndex();
 		$this->ensureDataBound();
 		foreach($this->getItems() as $index=>$item)

Doc Comments +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
 
 	/**
 	 * Creates a control used for repetition (used as a template).
-	 * @return TControl the control to be repeated
+	 * @return TRadioButtonItem the control to be repeated
 	 */
 	protected function createRepeatedControl()
 	{

framework/Web/UI/WebControls/TDataBoundControl.php

Spacing +36 added lines, -36 removed lines

@@ -82,7 +82,7 @@ 
 	 */
 	public function getDataSourceID()
 	{
-		return $this->getViewState('DataSourceID','');
+		return $this->getViewState('DataSourceID', '');
 	}
 
 	/**
@@ -91,10 +91,10 @@ 
 	 */
 	public function setDataSourceID($value)
 	{
-		$dsid=$this->getViewState('DataSourceID','');
+		$dsid=$this->getViewState('DataSourceID', '');
 		if($dsid!=='' && $value==='')
 			$this->_requiresBindToNull=true;
-		$this->setViewState('DataSourceID',$value,'');
+		$this->setViewState('DataSourceID', $value, '');
 		$this->onDataSourceChanged();
 	}
 
@@ -170,7 +170,7 @@ 
 	 */
 	public function getAllowPaging()
 	{
-		return $this->getViewState('AllowPaging',false);
+		return $this->getViewState('AllowPaging', false);
 	}
 
 	/**
@@ -178,7 +178,7 @@ 
 	 */
 	public function setAllowPaging($value)
 	{
-		$this->setViewState('AllowPaging',TPropertyValue::ensureBoolean($value),false);
+		$this->setViewState('AllowPaging', TPropertyValue::ensureBoolean($value), false);
 	}
 
 	/**
@@ -186,7 +186,7 @@ 
 	 */
 	public function getAllowCustomPaging()
 	{
-		return $this->getViewState('AllowCustomPaging',false);
+		return $this->getViewState('AllowCustomPaging', false);
 	}
 
 	/**
@@ -198,7 +198,7 @@ 
 	 */
 	public function setAllowCustomPaging($value)
 	{
-		$this->setViewState('AllowCustomPaging',TPropertyValue::ensureBoolean($value),false);
+		$this->setViewState('AllowCustomPaging', TPropertyValue::ensureBoolean($value), false);
 	}
 
 	/**
@@ -206,7 +206,7 @@ 
 	 */
 	public function getCurrentPageIndex()
 	{
-		return $this->getViewState('CurrentPageIndex',0);
+		return $this->getViewState('CurrentPageIndex', 0);
 	}
 
 	/**
@@ -215,9 +215,9 @@ 
 	 */
 	public function setCurrentPageIndex($value)
 	{
-		if(($value=TPropertyValue::ensureInteger($value))<0)
+		if(($value=TPropertyValue::ensureInteger($value)) < 0)
 			$value=0;
-		$this->setViewState('CurrentPageIndex',$value,0);
+		$this->setViewState('CurrentPageIndex', $value, 0);
 	}
 
 	/**
@@ -225,7 +225,7 @@ 
 	 */
 	public function getPageSize()
 	{
-		return $this->getViewState('PageSize',10);
+		return $this->getViewState('PageSize', 10);
 	}
 
 	/**
@@ -234,9 +234,9 @@ 
 	 */
 	public function setPageSize($value)
 	{
-		if(($value=TPropertyValue::ensureInteger($value))<1)
-			throw new TInvalidDataValueException('databoundcontrol_pagesize_invalid',get_class($this));
-		$this->setViewState('PageSize',TPropertyValue::ensureInteger($value),10);
+		if(($value=TPropertyValue::ensureInteger($value)) < 1)
+			throw new TInvalidDataValueException('databoundcontrol_pagesize_invalid', get_class($this));
+		$this->setViewState('PageSize', TPropertyValue::ensureInteger($value), 10);
 	}
 
 	/**
@@ -244,7 +244,7 @@ 
 	 */
 	public function getPageCount()
 	{
-		return $this->getViewState('PageCount',1);
+		return $this->getViewState('PageCount', 1);
 	}
 
 	/**
@@ -253,7 +253,7 @@ 
 	 */
 	public function getVirtualItemCount()
 	{
-		return $this->getViewState('VirtualItemCount',0);
+		return $this->getViewState('VirtualItemCount', 0);
 	}
 
 	/**
@@ -263,9 +263,9 @@ 
 	 */
 	public function setVirtualItemCount($value)
 	{
-		if(($value=TPropertyValue::ensureInteger($value))<0)
-			throw new TInvalidDataValueException('databoundcontrol_virtualitemcount_invalid',get_class($this));
-		$this->setViewState('VirtualItemCount',$value,0);
+		if(($value=TPropertyValue::ensureInteger($value)) < 0)
+			throw new TInvalidDataValueException('databoundcontrol_virtualitemcount_invalid', get_class($this));
+		$this->setViewState('VirtualItemCount', $value, 0);
 	}
 
 	/**
@@ -338,10 +338,10 @@ 
 			{
 				$ds=$this->createPagedDataSource();
 				$ds->setDataSource($data);
-				$this->setViewState('PageCount',$ds->getPageCount());
-				if($ds->getCurrentPageIndex()>=$ds->getPageCount())
+				$this->setViewState('PageCount', $ds->getPageCount());
+				if($ds->getCurrentPageIndex() >= $ds->getPageCount())
 				{
-					$ds->setCurrentPageIndex($ds->getPageCount()-1);
+					$ds->setCurrentPageIndex($ds->getPageCount() - 1);
 					$this->setCurrentPageIndex($ds->getCurrentPageIndex());
 				}
 				$this->performDataBinding($ds);
@@ -356,7 +356,7 @@ 
 		$this->onDataBound(null);
 	}
 
-	public function dataSourceViewChanged($sender,$param)
+	public function dataSourceViewChanged($sender, $param)
 	{
 		if(!$this->_ignoreDataSourceViewChanged)
 			$this->setRequiresDataBinding(true);
@@ -367,13 +367,13 @@ 
 		if(!$this->_currentViewValid)
 		{
 			if($this->_currentView && $this->_currentViewIsFromDataSourceID)
-				$this->_currentView->detachEventHandler('DataSourceViewChanged',array($this,'dataSourceViewChanged'));
+				$this->_currentView->detachEventHandler('DataSourceViewChanged', array($this, 'dataSourceViewChanged'));
 			if(($dataSource=$this->determineDataSource())!==null)
 			{
 				if(($view=$dataSource->getView($this->getDataMember()))===null)
-					throw new TInvalidDataValueException('databoundcontrol_datamember_invalid',$this->getDataMember());
+					throw new TInvalidDataValueException('databoundcontrol_datamember_invalid', $this->getDataMember());
 				if($this->_currentViewIsFromDataSourceID=$this->getUsingDataSourceID())
-					$view->attachEventHandler('OnDataSourceViewChanged',array($this,'dataSourceViewChanged'));
+					$view->attachEventHandler('OnDataSourceViewChanged', array($this, 'dataSourceViewChanged'));
 				$this->_currentView=$view;
 			}
 			else
@@ -390,14 +390,14 @@ 
 			if(($dsid=$this->getDataSourceID())!=='')
 			{
 				if(($dataSource=$this->getNamingContainer()->findControl($dsid))===null)
-					throw new TInvalidDataValueException('databoundcontrol_datasourceid_inexistent',$dsid);
+					throw new TInvalidDataValueException('databoundcontrol_datasourceid_inexistent', $dsid);
 				else if(!($dataSource instanceof IDataSource))
-					throw new TInvalidDataValueException('databoundcontrol_datasourceid_invalid',$dsid);
+					throw new TInvalidDataValueException('databoundcontrol_datasourceid_invalid', $dsid);
 				else
 					$this->_currentDataSource=$dataSource;
 			}
 			else if(($dataSource=$this->getDataSource())!==null)
-				$this->_currentDataSource=new TReadOnlyDataSource($dataSource,$this->getDataMember());
+				$this->_currentDataSource=new TReadOnlyDataSource($dataSource, $this->getDataMember());
 			else
 				$this->_currentDataSource=null;
 			$this->_currentDataSourceValid=true;
@@ -414,7 +414,7 @@ 
 	 */
 	public function onDataBound($param)
 	{
-		$this->raiseEvent('OnDataBound',$this,$param);
+		$this->raiseEvent('OnDataBound', $this, $param);
 	}
 
 	/**
@@ -428,7 +428,7 @@ 
 	{
 		parent::onInit($param);
 		$page=$this->getPage();
-		$page->attachEventHandler('OnPreLoad',array($this,'pagePreLoad'));
+		$page->attachEventHandler('OnPreLoad', array($this, 'pagePreLoad'));
 	}
 
 	/**
@@ -437,7 +437,7 @@ 
 	 * @param mixed event sender
 	 * @param TEventParameter event parameter
 	 */
-	public function pagePreLoad($sender,$param)
+	public function pagePreLoad($sender, $param)
 	{
 		$this->_initialized=true;
 		$isPostBack=$this->getPage()->getIsPostBack();
@@ -474,7 +474,7 @@ 
 				if(is_array($value))
 					$list->add($value);
 				else
-					$list->add(array($value,is_string($key)?$key:$value));
+					$list->add(array($value, is_string($key) ? $key : $value));
 			}
 			return $list;
 		}
@@ -487,17 +487,17 @@ 
 		else if(($value instanceof Traversable) || $value===null)
 			return $value;
 		else
-			throw new TInvalidDataTypeException('databoundcontrol_datasource_invalid',get_class($this));
+			throw new TInvalidDataTypeException('databoundcontrol_datasource_invalid', get_class($this));
 	}
 
 	public function getDataMember()
 	{
-		return $this->getViewState('DataMember','');
+		return $this->getViewState('DataMember', '');
 	}
 
 	public function setDataMember($value)
 	{
-		$this->setViewState('DataMember',$value,'');
+		$this->setViewState('DataMember', $value, '');
 	}
 
 	public function getSelectParameters()

Doc Comments +6 added lines

@@ -156,6 +156,7 @@ 
 
 	/**
 	 * @param boolean if databind has been invoked in this page request
+	 * @param boolean $value
 	 */
 	protected function setIsDataBound($value)
 	{
@@ -216,6 +217,7 @@ 
 
 	/**
 	 * @param integer the zero-based index of the current page
+	 * @param integer $value
 	 * @throws TInvalidDataValueException if the value is less than 0
 	 */
 	public function setCurrentPageIndex($value)
@@ -278,6 +280,7 @@ 
 	 * If true and the control has been prerendered while it uses the data source
 	 * specified by {@link setDataSourceID}, a databind call will be called by this method.
 	 * @param boolean whether a databind call is required.
+	 * @param boolean $value
 	 */
 	protected function setRequiresDataBinding($value)
 	{
@@ -410,6 +413,9 @@ 
 		return $this->_currentDataSource;
 	}
 
+	/**
+	 * @param \Traversable $data
+	 */
 	abstract protected function performDataBinding($data);
 
 	/**

Braces +6 added lines, -12 removed lines

@@ -286,8 +286,7 @@ 
 		{
 			$this->_requiresDataBinding=true;
 			$this->ensureDataBound();
-		}
-		else
+		} else
 			$this->_requiresDataBinding=$value;
 	}
 
@@ -350,8 +349,7 @@ 
 					$this->setCurrentPageIndex($ds->getCurrentPageIndex());
 				}
 				$this->performDataBinding($ds);
-			}
-			else
+			} else
 			{
 				$this->clearViewState('PageCount');
 				$this->performDataBinding($data);
@@ -380,8 +378,7 @@ 
 				if($this->_currentViewIsFromDataSourceID=$this->getUsingDataSourceID())
 					$view->attachEventHandler('OnDataSourceViewChanged',array($this,'dataSourceViewChanged'));
 				$this->_currentView=$view;
-			}
-			else
+			} else
 				$this->_currentView=null;
 			$this->_currentViewValid=true;
 		}
@@ -400,8 +397,7 @@ 
 					throw new TInvalidDataValueException('databoundcontrol_datasourceid_invalid',$dsid);
 				else
 					$this->_currentDataSource=$dataSource;
-			}
-			else if(($dataSource=$this->getDataSource())!==null)
+			} else if(($dataSource=$this->getDataSource())!==null)
 				$this->_currentDataSource=new TReadOnlyDataSource($dataSource,$this->getDataMember());
 			else
 				$this->_currentDataSource=null;
@@ -482,14 +478,12 @@ 
 					$list->add(array($value,is_string($key)?$key:$value));
 			}
 			return $list;
-		}
-		else if(is_array($value))
+		} else if(is_array($value))
 			return new TMap($value);
 		else if($value instanceof TDbDataReader) {
 			// read array from TDbDataReader since it's forward-only stream and can only be traversed once
 			return $value->readAll();
-		}
-		else if(($value instanceof \Traversable) || $value===null)
+		} else if(($value instanceof \Traversable) || $value===null)
 			return $value;
 		else
 			throw new TInvalidDataTypeException('databoundcontrol_datasource_invalid',get_class($this));