pradosoft / prado

framework/Web/UI/WebControls/TBaseValidator.php

Braces +2 added lines, -4 removed lines

@@ -518,16 +518,14 @@
 				{
 					$this->setIsValid(true);
 					$this->onValidationSuccess();
-				}
-				else
+				} else
 				{
 					if($target)
 						$target->setIsValid(false);
 					$this->setIsValid(false);
 					$this->onValidationError();
 				}
-			}
-			else
+			} else
 			{
 				$this->evaluateIsValid();
 				$this->setIsValid(true);

Doc Comments +5 added lines, -2 removed lines

@@ -185,6 +185,7 @@ 
 	 * TBaseValidator::$_clientClass, be sure to update the corresponding
 	 * "Javascript/validation3.js" file as well.
 	 * @param TControl control to validate.
+	 * @param \Prado\Web\UI\TControl $control
 	 * @return string control type for client-side validation.
 	 */
 	private function getClientControlClass($control)
@@ -263,7 +264,7 @@ 
 	/**
 	 * Update the ControlToValidate component's css class depending
 	 * if the ControlCssClass property is set, and whether this is valid.
-	 * @return boolean true if change, false otherwise.
+	 * @return boolean|null true if change, false otherwise.
 	 */
 	protected function updateControlCssClass()
 	{
@@ -388,6 +389,7 @@ 
 	 * The ID path is the dot-connected IDs of the controls reaching from
 	 * the validator's naming container to the target control.
 	 * @param string the ID path
+	 * @param string $value
 	 */
 	public function setControlToValidate($value)
 	{
@@ -476,7 +478,7 @@ 
 	}
 
 	/**
-	 * @return TControl control to be validated. Null if no control is found.
+	 * @return \Prado\Web\UI\TControl control to be validated. Null if no control is found.
 	 * @throws TConfigurationException if {@link getControlToValidate
 	 * ControlToValidate} is empty or does not point to a valid control
 	 */
@@ -553,6 +555,7 @@ 
 
 	/**
 	 * @param string the css class that is applied to the control being validated in case the validation fails
+	 * @param string $value
 	 */
 	public function setControlCssClass($value)
 	{

Spacing +54 added lines, -54 removed lines

@@ -94,7 +94,7 @@ 
 	 * them specially.
 	 * @var array list of control class names
 	 */
-	private static $_clientClass = array('THtmlArea', 'THtmlArea4', 'TDatePicker', 'TListBox', 'TCheckBoxList');
+	private static $_clientClass=array('THtmlArea', 'THtmlArea4', 'TDatePicker', 'TListBox', 'TCheckBoxList');
 
 	/**
 	 * Constructor.
@@ -139,10 +139,10 @@ 
 		$display=$this->getDisplay();
 		$visible=$this->getEnabled(true) && !$this->getIsValid();
 		if($display===TValidatorDisplayStyle::None || (!$visible && $display===TValidatorDisplayStyle::Dynamic))
-			$writer->addStyleAttribute('display','none');
+			$writer->addStyleAttribute('display', 'none');
 		else if(!$visible)
-			$writer->addStyleAttribute('visibility','hidden');
-		$writer->addAttribute('id',$this->getClientID());
+			$writer->addStyleAttribute('visibility', 'hidden');
+		$writer->addAttribute('id', $this->getClientID());
 		parent::addAttributesToRender($writer);
 		$this->renderClientControlScript($writer);
 	}
@@ -153,29 +153,29 @@ 
 	 */
 	protected function getClientScriptOptions()
 	{
-		$control = $this->getValidationTarget();
-		$options['ID'] = $this->getClientID();
-		$options['FormID'] = $this->getPage()->getForm()->getClientID();
-		$options['Display'] = $this->getDisplay();
-		$options['ErrorMessage'] = $this->getErrorMessage();
+		$control=$this->getValidationTarget();
+		$options['ID']=$this->getClientID();
+		$options['FormID']=$this->getPage()->getForm()->getClientID();
+		$options['Display']=$this->getDisplay();
+		$options['ErrorMessage']=$this->getErrorMessage();
 		if($this->getFocusOnError())
 		{
-			$options['FocusOnError'] = $this->getFocusOnError();
-			$options['FocusElementID'] = $this->getFocusElementID();
+			$options['FocusOnError']=$this->getFocusOnError();
+			$options['FocusElementID']=$this->getFocusElementID();
 		}
-		$options['ValidationGroup'] = $this->getValidationGroup();
+		$options['ValidationGroup']=$this->getValidationGroup();
 		if($control)
-			$options['ControlToValidate'] = $control->getClientID();
-		$options['ControlCssClass'] = $this->getControlCssClass();
+			$options['ControlToValidate']=$control->getClientID();
+		$options['ControlCssClass']=$this->getControlCssClass();
 
-		$options['ControlType'] = $this->getClientControlClass($control);
-		$options['Enabled'] = $this->getEnabled(true);
+		$options['ControlType']=$this->getClientControlClass($control);
+		$options['Enabled']=$this->getEnabled(true);
 
 		//get date format from date picker target control
 		if($control instanceof TDatePicker)
-			$options['DateFormat'] = $control->getDateFormat();
+			$options['DateFormat']=$control->getDateFormat();
 
-		$options = array_merge($options,$this->getClientSide()->getOptions()->toArray());
+		$options=array_merge($options, $this->getClientSide()->getOptions()->toArray());
 
 		return $options;
 	}
@@ -192,7 +192,7 @@ 
 		foreach(self::$_clientClass as $type)
 			if($control instanceof $type)
 				return $type;
-		$reflectionClass = new \ReflectionClass($control);
+		$reflectionClass=new \ReflectionClass($control);
 		return $reflectionClass->getShortName();
 	}
 
@@ -215,7 +215,7 @@ 
 	public function getClientSide()
 	{
 		if($this->_clientSide===null)
-			$this->_clientSide = $this->createClientSide();
+			$this->_clientSide=$this->createClientSide();
 		return $this->_clientSide;
 	}
 
@@ -235,15 +235,15 @@ 
 	 */
 	public function renderClientControlScript($writer)
 	{
-		$scripts = $this->getPage()->getClientScript();
-		if ($this->getEnableClientScript())
+		$scripts=$this->getPage()->getClientScript();
+		if($this->getEnableClientScript())
 			$scripts->registerPradoScript('validator');
 		$formID=$this->getPage()->getForm()->getClientID();
-		$scriptKey = "TBaseValidator:$formID";
+		$scriptKey="TBaseValidator:$formID";
 		if($this->getEnableClientScript() && !$scripts->isEndScriptRegistered($scriptKey))
 		{
-			$manager['FormID'] = $formID;
-			$options = TJavaScript::encode($manager);
+			$manager['FormID']=$formID;
+			$options=TJavaScript::encode($manager);
 			$scripts->registerEndScript($scriptKey, "new Prado.ValidationManager({$options});");
 		}
 		if($this->getEnableClientScript())
@@ -254,7 +254,7 @@ 
 	 * Override parent implementation to update the control CSS Class before
 	 * the validated control is rendered
 	 */
-	public function onPreRender ($param)
+	public function onPreRender($param)
 	{
 		parent::onPreRender($param);
 		$this->updateControlCssClass();
@@ -272,12 +272,12 @@ 
 			$control=$this->getValidationTarget();
 			if($control instanceof TWebControl)
 			{
-				$class = preg_replace ('/ '.preg_quote($cssClass).'/', '',$control->getCssClass());
+				$class=preg_replace('/ '.preg_quote($cssClass).'/', '', $control->getCssClass());
 				if(!$this->getIsValid())
 				{
-					$class .= ' '.$cssClass;
+					$class.=' '.$cssClass;
 					$control->setCssClass($class);
-				} elseif ($control->getIsValid())
+				} elseif($control->getIsValid())
 					$control->setCssClass($class);
 			}
 		}
@@ -288,11 +288,11 @@ 
 	 */
 	protected function registerClientScriptValidator()
 	{
-		$key = 'prado:'.$this->getClientID();
+		$key='prado:'.$this->getClientID();
 		if(!$this->getPage()->getClientScript()->isEndScriptRegistered($key))
 		{
-			$options = TJavaScript::encode($this->getClientScriptOptions());
-			$script = 'new '.$this->getClientClassName().'('.$options.');';
+			$options=TJavaScript::encode($this->getClientScriptOptions());
+			$script='new '.$this->getClientClassName().'('.$options.');';
 			$this->getPage()->getClientScript()->registerEndScript($key, $script);
 		}
 	}
@@ -311,7 +311,7 @@ 
 	 */
 	public function setForControl($value)
 	{
-		throw new TNotSupportedException('basevalidator_forcontrol_unsupported',get_class($this));
+		throw new TNotSupportedException('basevalidator_forcontrol_unsupported', get_class($this));
 	}
 
 	/**
@@ -331,7 +331,7 @@ 
 	 */
 	public function getDisplay()
 	{
-		return $this->getViewState('Display',TValidatorDisplayStyle::Fixed);
+		return $this->getViewState('Display', TValidatorDisplayStyle::Fixed);
 	}
 
 	/**
@@ -339,7 +339,7 @@ 
 	 */
 	public function setDisplay($value)
 	{
-		$this->setViewState('Display',TPropertyValue::ensureEnum($value,'Prado\\Web\\UI\\WebControls\\TValidatorDisplayStyle'),TValidatorDisplayStyle::Fixed);
+		$this->setViewState('Display', TPropertyValue::ensureEnum($value, 'Prado\\Web\\UI\\WebControls\\TValidatorDisplayStyle'), TValidatorDisplayStyle::Fixed);
 	}
 
 	/**
@@ -347,7 +347,7 @@ 
 	 */
 	public function getEnableClientScript()
 	{
-		return $this->getViewState('EnableClientScript',true);
+		return $this->getViewState('EnableClientScript', true);
 	}
 
 	/**
@@ -355,7 +355,7 @@ 
 	 */
 	public function setEnableClientScript($value)
 	{
-		$this->setViewState('EnableClientScript',TPropertyValue::ensureBoolean($value),true);
+		$this->setViewState('EnableClientScript', TPropertyValue::ensureBoolean($value), true);
 	}
 
 	/**
@@ -363,7 +363,7 @@ 
 	 */
 	public function getErrorMessage()
 	{
-		return $this->getViewState('ErrorMessage','');
+		return $this->getViewState('ErrorMessage', '');
 	}
 
 	/**
@@ -372,7 +372,7 @@ 
 	 */
 	public function setErrorMessage($value)
 	{
-		$this->setViewState('ErrorMessage',$value,'');
+		$this->setViewState('ErrorMessage', $value, '');
 	}
 
 	/**
@@ -380,7 +380,7 @@ 
 	 */
 	public function getControlToValidate()
 	{
-		return $this->getViewState('ControlToValidate','');
+		return $this->getViewState('ControlToValidate', '');
 	}
 
 	/**
@@ -391,7 +391,7 @@ 
 	 */
 	public function setControlToValidate($value)
 	{
-		$this->setViewState('ControlToValidate',$value,'');
+		$this->setViewState('ControlToValidate', $value, '');
 	}
 
 	/**
@@ -399,7 +399,7 @@ 
 	 */
 	public function getFocusOnError()
 	{
-		return $this->getViewState('FocusOnError',false);
+		return $this->getViewState('FocusOnError', false);
 	}
 
 	/**
@@ -407,7 +407,7 @@ 
 	 */
 	public function setFocusOnError($value)
 	{
-		$this->setViewState('FocusOnError',TPropertyValue::ensureBoolean($value),false);
+		$this->setViewState('FocusOnError', TPropertyValue::ensureBoolean($value), false);
 	}
 
 	/**
@@ -417,13 +417,13 @@ 
 	 */
 	public function getFocusElementID()
 	{
-		if(($id=$this->getViewState('FocusElementID',''))==='')
+		if(($id=$this->getViewState('FocusElementID', ''))==='')
 		{
 			$target=$this->getValidationTarget();
 			/* Workaround: TCheckBoxList and TRadioButtonList nests the actual
 			 * inputs inside a table; we ensure the first input gets focused
 			 */
-			if($target instanceof TCheckBoxList && $target->getItemCount()>0)
+			if($target instanceof TCheckBoxList && $target->getItemCount() > 0)
 			{
 				$id=$target->getClientID().'_c0';
 			} else {
@@ -447,7 +447,7 @@ 
 	 */
 	public function getValidationGroup()
 	{
-		return $this->getViewState('ValidationGroup','');
+		return $this->getViewState('ValidationGroup', '');
 	}
 
 	/**
@@ -455,7 +455,7 @@ 
 	 */
 	public function setValidationGroup($value)
 	{
-		$this->setViewState('ValidationGroup',$value,'');
+		$this->setViewState('ValidationGroup', $value, '');
 	}
 
 	/**
@@ -485,7 +485,7 @@ 
 		if(($id=$this->getControlToValidate())!=='' && ($control=$this->findControl($id))!==null)
 			return $control;
 		else
-			throw new TConfigurationException('basevalidator_controltovalidate_invalid',get_class($this));
+			throw new TConfigurationException('basevalidator_controltovalidate_invalid', get_class($this));
 	}
 
 	/**
@@ -499,7 +499,7 @@ 
 		if($control instanceof \Prado\Web\UI\IValidatable)
 			return $control->getValidationPropertyValue();
 		else
-			throw new TInvalidDataTypeException('basevalidator_validatable_required',get_class($this));
+			throw new TInvalidDataTypeException('basevalidator_validatable_required', get_class($this));
 	}
 
 	/**
@@ -548,7 +548,7 @@ 
 	 */
 	public function getControlCssClass()
 	{
-		return $this->getViewState('ControlCssClass','');
+		return $this->getViewState('ControlCssClass', '');
 	}
 
 	/**
@@ -556,7 +556,7 @@ 
 	 */
 	public function setControlCssClass($value)
 	{
-		$this->setViewState('ControlCssClass',$value,'');
+		$this->setViewState('ControlCssClass', $value, '');
 	}
 
 	/**
@@ -571,7 +571,7 @@ 
 	 */
 	public function onValidationSuccess()
 	{
-		$this->raiseEvent('OnValidationSuccess',$this,null);
+		$this->raiseEvent('OnValidationSuccess', $this, null);
 	}
 
 	/**
@@ -579,7 +579,7 @@ 
 	 */
 	public function onValidationError()
 	{
-		$this->raiseEvent('OnValidationError',$this,null);
+		$this->raiseEvent('OnValidationError', $this, null);
 	}
 
 	/**
@@ -590,7 +590,7 @@ 
 	 */
 	public function onValidate()
 	{
-		$this->raiseEvent('OnValidate',$this,null);
+		$this->raiseEvent('OnValidate', $this, null);
 	}
 
 	/**

framework/Web/UI/WebControls/TClientScript.php

Spacing +5 added lines, -5 removed lines

@@ -105,11 +105,11 @@ 
 	public function onPreRender($param)
 	{
 		parent::onPreRender($param);
-		$scripts = preg_split('/,|\s+/', $this->getPradoScripts());
-		$cs = $this->getPage()->getClientScript();
+		$scripts=preg_split('/,|\s+/', $this->getPradoScripts());
+		$cs=$this->getPage()->getClientScript();
 		foreach($scripts as $script)
 		{
-			if(($script = trim($script))!=='')
+			if(($script=trim($script))!=='')
 				$cs->registerPradoScript($script);
 		}
 	}
@@ -122,7 +122,7 @@ 
 	 */
 	public function render($writer)
 	{
-		if ($this->getFlushScriptFiles())
+		if($this->getFlushScriptFiles())
 			$this->getPage()->getClientScript()->flushScriptFiles($writer, $this);
 		$this->renderCustomScriptFile($writer);
 		$this->renderCustomScript($writer);
@@ -134,7 +134,7 @@ 
 	 */
 	protected function renderCustomScriptFile($writer)
 	{
-		if(($scriptUrl = $this->getScriptUrl())!=='')
+		if(($scriptUrl=$this->getScriptUrl())!=='')
 			$writer->write("<script type=\"text/javascript\" src=\"$scriptUrl\"></script>\n");
 	}
 

framework/Web/UI/WebControls/TMarkdown.php

Spacing +2 added lines, -2 removed lines

@@ -51,7 +51,7 @@ 
 	 */
 	public function processText($text)
 	{
-		$result = Parsedown::instance()->parse($text);
+		$result=Parsedown::instance()->parse($text);
 		return preg_replace_callback(
 				'/<pre><code class="language-(\w+)">((.|\n)*?)<\\/code><\\/pre>/im',
 				array($this, 'highlightCode'), $result);
@@ -64,7 +64,7 @@ 
 	 */
 	protected function highlightCode($matches)
 	{
-		$text = html_entity_decode($matches[2],ENT_QUOTES,'UTF-8');
+		$text=html_entity_decode($matches[2], ENT_QUOTES, 'UTF-8');
 		$this->setLanguage($matches[1]);
 		return parent::processText($text);
 	}

framework/Web/UI/WebControls/TLabel.php

Spacing +7 added lines, -7 removed lines

@@ -37,7 +37,7 @@ 
 	 */
 	protected function getTagName()
 	{
-		return ($this->getForControl()==='')?'span':'label';
+		return ($this->getForControl()==='') ? 'span' : 'label';
 	}
 
 	/**
@@ -48,7 +48,7 @@ 
 	protected function addAttributesToRender($writer)
 	{
 		if($this->_forControl!=='')
-			$writer->addAttribute('for',$this->_forControl);
+			$writer->addAttribute('for', $this->_forControl);
 		parent::addAttributesToRender($writer);
 	}
 
@@ -71,7 +71,7 @@ 
 				}
 			}
 			else
-				throw new TInvalidDataValueException('label_associatedcontrol_invalid',$aid);
+				throw new TInvalidDataValueException('label_associatedcontrol_invalid', $aid);
 		}
 		else
 			parent::render($writer);
@@ -94,7 +94,7 @@ 
 	 */
 	public function getText()
 	{
-		return $this->getViewState('Text','');
+		return $this->getViewState('Text', '');
 	}
 
 	/**
@@ -102,7 +102,7 @@ 
 	 */
 	public function setText($value)
 	{
-		$this->setViewState('Text',TPropertyValue::ensureString($value),'');
+		$this->setViewState('Text', TPropertyValue::ensureString($value), '');
 	}
 
 	/**
@@ -136,7 +136,7 @@ 
 	 */
 	public function getForControl()
 	{
-		return $this->getViewState('ForControl','');
+		return $this->getViewState('ForControl', '');
 	}
 
 	/**
@@ -146,7 +146,7 @@ 
 	 */
 	public function setForControl($value)
 	{
-		$this->setViewState('ForControl',$value,'');
+		$this->setViewState('ForControl', $value, '');
 	}
 }
 

Braces +2 added lines, -4 removed lines

@@ -233,8 +233,7 @@ 
 				$newid=substr($id,$pos+1);
 				if ($control!==null)
 					$control=$control->$newid;
-			}
-			else
+			} else
 			{
 				// TCheckBoxList overrides findControl() with a fake implementation
 				// but accepts a second parameter to use the standard one
@@ -296,8 +295,7 @@ 
 		{
 			$group=$this->getValidationGroup();
 			return $this->getPage()->getValidators($group)->getCount()>0;
-		}
-		else
+		} else
 			return false;
 	}
 

framework/Web/UI/WebControls/TListControlValidator.php

Spacing +36 added lines, -36 removed lines

@@ -76,7 +76,7 @@ 
 	 */
 	public function getMinSelection()
 	{
-		return $this->getViewState('MinSelection',-1);
+		return $this->getViewState('MinSelection', -1);
 	}
 
 	/**
@@ -84,9 +84,9 @@ 
 	 */
 	public function setMinSelection($value)
 	{
-		if(($value=TPropertyValue::ensureInteger($value))<0)
+		if(($value=TPropertyValue::ensureInteger($value)) < 0)
 			$value=-1;
-		$this->setViewState('MinSelection',$value,-1);
+		$this->setViewState('MinSelection', $value, -1);
 	}
 
 	/**
@@ -94,7 +94,7 @@ 
 	 */
 	public function getMaxSelection()
 	{
-		return $this->getViewState('MaxSelection',-1);
+		return $this->getViewState('MaxSelection', -1);
 	}
 
 	/**
@@ -102,9 +102,9 @@ 
 	 */
 	public function setMaxSelection($value)
 	{
-		if(($value=TPropertyValue::ensureInteger($value))<0)
+		if(($value=TPropertyValue::ensureInteger($value)) < 0)
 			$value=-1;
-		$this->setViewState('MaxSelection',$value,-1);
+		$this->setViewState('MaxSelection', $value, -1);
 	}
 
 	/**
@@ -113,7 +113,7 @@ 
 	 */
 	public function getRequiredSelections()
 	{
-		return $this->getViewState('RequiredSelections','');
+		return $this->getViewState('RequiredSelections', '');
 	}
 
 	/**
@@ -122,7 +122,7 @@ 
 	 */
 	public function setRequiredSelections($value)
 	{
-		$this->setViewState('RequiredSelections',$value,'');
+		$this->setViewState('RequiredSelections', $value, '');
 	}
 
 	/**
@@ -135,28 +135,28 @@ 
 	{
 		$control=$this->getValidationTarget();
 
-		$exists = true;
-		$values = $this->getSelection($control);
-		$count = count($values);
-		$required = $this->getRequiredValues();
+		$exists=true;
+		$values=$this->getSelection($control);
+		$count=count($values);
+		$required=$this->getRequiredValues();
 
 		//if required, check the values
 		if(!empty($required))
 		{
-			if($count < count($required) )
+			if($count < count($required))
 				return false;
 			foreach($required as $require)
-				$exists = $exists && in_array($require, $values);
+				$exists=$exists && in_array($require, $values);
 		}
 
-		$min = $this->getMinSelection();
-		$max = $this->getMaxSelection();
+		$min=$this->getMinSelection();
+		$max=$this->getMaxSelection();
 
-		if($min !== -1 && $max !== -1)
+		if($min!==-1 && $max!==-1)
 			return $exists && $count >= $min && $count <= $max;
-		else if($min === -1 && $max !== -1)
+		else if($min===-1 && $max!==-1)
 			return $exists && $count <= $max;
-		else if($min !== -1 && $max === -1)
+		else if($min!==-1 && $max===-1)
 			return $exists && $count >= $min;
 		else
 			return $exists;
@@ -168,13 +168,13 @@ 
 	 */
 	protected function getSelection($control)
 	{
-		$values = array();
+		$values=array();
 
 		//get the data
 		foreach($control->getItems() as $item)
 		{
 			if($item->getSelected())
-				$values[] = $item->getValue();
+				$values[]=$item->getValue();
 		}
 		return $values;
 	}
@@ -184,10 +184,10 @@ 
 	 */
 	protected function getRequiredValues()
 	{
-		$required = array();
-		$string = $this->getRequiredSelections();
+		$required=array();
+		$string=$this->getRequiredSelections();
 		if(!empty($string))
-			$required = preg_split('/,\s*/', $string);
+			$required=preg_split('/,\s*/', $string);
 		return $required;
 	}
 
@@ -197,26 +197,26 @@ 
 	 */
 	protected function getClientScriptOptions()
 	{
-		$options = parent::getClientScriptOptions();
-		$control = $this->getValidationTarget();
+		$options=parent::getClientScriptOptions();
+		$control=$this->getValidationTarget();
 
 		if(!$control instanceof TListControl)
 		{
 			throw new TConfigurationException(
 				'listcontrolvalidator_invalid_control',
-				$this->getID(),$this->getControlToValidate(), get_class($control));
+				$this->getID(), $this->getControlToValidate(), get_class($control));
 		}
 
-		$min = $this->getMinSelection();
-		$max = $this->getMaxSelection();
-		if($min !== -1)
-			$options['Min']= $min;
-		if($max !== -1)
-			$options['Max']= $max;
-		$required = $this->getRequiredSelections();
+		$min=$this->getMinSelection();
+		$max=$this->getMaxSelection();
+		if($min!==-1)
+			$options['Min']=$min;
+		if($max!==-1)
+			$options['Max']=$max;
+		$required=$this->getRequiredSelections();
 		if(strlen($required) > 0)
-			$options['Required']= $required;
-		$options['TotalItems'] = $control->getItemCount();
+			$options['Required']=$required;
+		$options['TotalItems']=$control->getItemCount();
 
 		return $options;
 	}

Doc Comments +1 added lines

@@ -163,6 +163,7 @@
 
 	/**
 	 * @param TListControl control to validate
+	 * @param \Prado\Web\UI\TControl $control
 	 * @return array number of selected values and its values.
 	 */
 	protected function getSelection($control)

framework/Web/UI/WebControls/TListItem.php

Spacing +17 added lines, -17 removed lines

@@ -53,7 +53,7 @@ 
 	 * @param boolean whether the item is enabled
 	 * @param boolean whether the item is selected
 	 */
-	public function __construct($text='',$value='',$enabled=true,$selected=false)
+	public function __construct($text='', $value='', $enabled=true, $selected=false)
 	{
 		$this->setText($text);
 		$this->setValue($value);
@@ -70,16 +70,16 @@ 
 	protected function __getZappableSleepProps(&$exprops)
 	{
 		parent::__getZappableSleepProps($exprops);
-		if ($this->_attributes===null)
-			$exprops[] = "\0TListItem\0_attributes";
+		if($this->_attributes===null)
+			$exprops[]="\0TListItem\0_attributes";
 		if($this->_text==='')
-			$exprops[] = "\0TListItem\0_text";
+			$exprops[]="\0TListItem\0_text";
 		if($this->_value==='')
-			$exprops[] = "\0TListItem\0_value";
-		if ($this->_enabled===true)
-			$exprops[] = "\0TListItem\0_enabled";
-		if ($this->_selected===false)
-			$exprops[] = "\0TListItem\0_selected";
+			$exprops[]="\0TListItem\0_value";
+		if($this->_enabled===true)
+			$exprops[]="\0TListItem\0_enabled";
+		if($this->_selected===false)
+			$exprops[]="\0TListItem\0_selected";
 	}
 
 	/**
@@ -119,7 +119,7 @@ 
 	 */
 	public function getText()
 	{
-		return $this->_text===''?$this->_value:$this->_text;
+		return $this->_text==='' ? $this->_value : $this->_text;
 	}
 
 	/**
@@ -135,7 +135,7 @@ 
 	 */
 	public function getValue()
 	{
-		return $this->_value===''?$this->_text:$this->_value;
+		return $this->_value==='' ? $this->_text : $this->_value;
 	}
 
 	/**
@@ -161,7 +161,7 @@ 
 	 */
 	public function getHasAttributes()
 	{
-		return $this->_attributes && $this->_attributes->getCount()>0;
+		return $this->_attributes && $this->_attributes->getCount() > 0;
 	}
 
 	/**
@@ -170,7 +170,7 @@ 
 	 */
 	public function hasAttribute($name)
 	{
-		return $this->_attributes?$this->_attributes->contains($name):false;
+		return $this->_attributes ? $this->_attributes->contains($name) : false;
 	}
 
 	/**
@@ -178,16 +178,16 @@ 
 	 */
 	public function getAttribute($name)
 	{
-		return $this->_attributes?$this->_attributes->itemAt($name):null;
+		return $this->_attributes ? $this->_attributes->itemAt($name) : null;
 	}
 
 	/**
 	 * @param string attribute name
 	 * @param string value of the attribute
 	 */
-	public function setAttribute($name,$value)
+	public function setAttribute($name, $value)
 	{
-		$this->getAttributes()->add($name,$value);
+		$this->getAttributes()->add($name, $value);
 	}
 
 	/**
@@ -197,7 +197,7 @@ 
 	 */
 	public function removeAttribute($name)
 	{
-		return $this->_attributes?$this->_attributes->remove($name):null;
+		return $this->_attributes ? $this->_attributes->remove($name) : null;
 	}
 }
 

Doc Comments +3 added lines

@@ -97,6 +97,7 @@ 
 
 	/**
 	 * @param boolean whether the item is enabled
+	 * @param boolean $value
 	 */
 	public function setEnabled($value)
 	{
@@ -113,6 +114,7 @@ 
 
 	/**
 	 * @param boolean whether the item is selected
+	 * @param boolean $value
 	 */
 	public function setSelected($value)
 	{
@@ -145,6 +147,7 @@ 
 
 	/**
 	 * @param string value of the item
+	 * @param string $value
 	 */
 	public function setValue($value)
 	{

framework/Web/UI/TSessionPageStatePersister.php

Spacing +8 added lines, -8 removed lines

@@ -79,7 +79,7 @@ 
 	 */
 	public function setHistorySize($value)
 	{
-		if(($value=TPropertyValue::ensureInteger($value))>0)
+		if(($value=TPropertyValue::ensureInteger($value)) > 0)
 			$this->_historySize=$value;
 		else
 			throw new TInvalidDataValueException('sessionpagestatepersister_historysize_invalid');
@@ -93,19 +93,19 @@ 
 		$session=$this->_page->getSession();
 		$session->open();
 		$data=serialize($state);
-		$timestamp=(string)microtime(true);
+		$timestamp=(string) microtime(true);
 		$key=self::STATE_SESSION_KEY.$timestamp;
-		$session->add($key,$data);
+		$session->add($key, $data);
 		if(($queue=$session->itemAt(self::QUEUE_SESSION_KEY))===null)
 			$queue=array();
 		$queue[]=$key;
-		if(count($queue)>$this->getHistorySize())
+		if(count($queue) > $this->getHistorySize())
 		{
 			$expiredKey=array_shift($queue);
 			$session->remove($expiredKey);
 		}
-		$session->add(self::QUEUE_SESSION_KEY,$queue);
-		$this->_page->setClientState(TPageStateFormatter::serialize($this->_page,$timestamp));
+		$session->add(self::QUEUE_SESSION_KEY, $queue);
+		$this->_page->setClientState(TPageStateFormatter::serialize($this->_page, $timestamp));
 	}
 
 	/**
@@ -115,7 +115,7 @@ 
 	 */
 	public function load()
 	{
-		if(($timestamp=TPageStateFormatter::unserialize($this->_page,$this->_page->getRequestClientState()))!==null)
+		if(($timestamp=TPageStateFormatter::unserialize($this->_page, $this->_page->getRequestClientState()))!==null)
 		{
 			$session=$this->_page->getSession();
 			$session->open();
@@ -123,7 +123,7 @@ 
 			if(($data=$session->itemAt($key))!==null)
 				return unserialize($data);
 		}
-		throw new THttpException(400,'sessionpagestatepersister_pagestate_corrupted');
+		throw new THttpException(400, 'sessionpagestatepersister_pagestate_corrupted');
 	}
 }
 

framework/Web/UI/TControlAdapter.php

Spacing +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
 	 */
 	public function getPage()
 	{
-		return $this->_control?$this->_control->getPage():null;
+		return $this->_control ? $this->_control->getPage() : null;
 	}
 
 	/**

framework/Web/UI/THtmlWriter.php

Spacing +5 added lines, -5 removed lines

@@ -88,7 +88,7 @@ 
 
 	public function setWriter($writer)
 	{
-		$this->_writer = $writer;
+		$this->_writer=$writer;
 	}
 	/**
 	 * Adds a list of attributes to be rendered.
@@ -105,7 +105,7 @@ 
 	 * @param string name of the attribute
 	 * @param string value of the attribute
 	 */
-	public function addAttribute($name,$value)
+	public function addAttribute($name, $value)
 	{
 		$this->_attributes[THttpUtility::htmlStrip($name)]=THttpUtility::htmlEncode($value);
 	}
@@ -134,7 +134,7 @@ 
 	 * @param string stylesheet attribute name
 	 * @param string stylesheet attribute value
 	 */
-	public function addStyleAttribute($name,$value)
+	public function addStyleAttribute($name, $value)
 	{
 		$this->_styles[THttpUtility::htmlStrip($name)]=THttpUtility::htmlEncode($value);
 	}
@@ -203,12 +203,12 @@ 
 		if(isset(self::$_simpleTags[$tagName]))
 		{
 			$str.=' />';
-			$this->_openTags[] = '';
+			$this->_openTags[]='';
 		}
 		else
 		{
 			$str.='>';
-			$this->_openTags[] = $tagName;
+			$this->_openTags[]=$tagName;
 		}
 		$this->_writer->write($str);
 		$this->_attributes=array();

Braces +1 added lines, -2 removed lines

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