pradosoft / prado

framework/Web/UI/TTemplate.php

Spacing +193 added lines, -193 removed lines

@@ -192,38 +192,38 @@ 
 	public function instantiateIn($tplControl, $parentControl = null)
 	{
 		$this->_tplControl = $tplControl;
-		if($parentControl === null)
+		if ($parentControl === null)
 			$parentControl = $tplControl;
-		if(($page = $tplControl->getPage()) === null)
+		if (($page = $tplControl->getPage()) === null)
 			$page = $this->getService()->getRequestedPage();
 		$controls = [];
 		$directChildren = [];
-		foreach($this->_tpl as $key => $object)
+		foreach ($this->_tpl as $key => $object)
 		{
-			if($object[0] === -1)
+			if ($object[0] === -1)
 				$parent = $parentControl;
-			elseif(isset($controls[$object[0]]))
+			elseif (isset($controls[$object[0]]))
 				$parent = $controls[$object[0]];
 			else
 				continue;
-			if(isset($object[2]))	// component
+			if (isset($object[2]))	// component
 			{
 				$component = Prado::createComponent($object[1]);
 				$properties = &$object[2];
-				if($component instanceof TControl)
+				if ($component instanceof TControl)
 				{
-					if($component instanceof \Prado\Web\UI\WebControls\TOutputCache)
-						$component->setCacheKeyPrefix($this->_hashCode . $key);
+					if ($component instanceof \Prado\Web\UI\WebControls\TOutputCache)
+						$component->setCacheKeyPrefix($this->_hashCode.$key);
 					$component->setTemplateControl($tplControl);
-					if(isset($properties['id']))
+					if (isset($properties['id']))
 					{
-						if(is_array($properties['id']))
+						if (is_array($properties['id']))
 							$properties['id'] = $component->evaluateExpression($properties['id'][1]);
 						$tplControl->registerObject($properties['id'], $component);
 					}
-					if(isset($properties['skinid']))
+					if (isset($properties['skinid']))
 					{
-						if(is_array($properties['skinid']))
+						if (is_array($properties['skinid']))
 							$component->setSkinID($component->evaluateExpression($properties['skinid'][1]));
 						else
 							$component->setSkinID($properties['skinid']);
@@ -233,32 +233,32 @@ 
 					$component->trackViewState(false);
 
 					$component->applyStyleSheetSkin($page);
-					foreach($properties as $name => $value)
+					foreach ($properties as $name => $value)
 						$this->configureControl($component, $name, $value);
 
 					$component->trackViewState(true);
 
-					if($parent === $parentControl)
+					if ($parent === $parentControl)
 						$directChildren[] = $component;
 					else
 						$component->createdOnTemplate($parent);
-					if($component->getAllowChildControls())
+					if ($component->getAllowChildControls())
 						$controls[$key] = $component;
 				}
-				elseif($component instanceof TComponent)
+				elseif ($component instanceof TComponent)
 				{
 					$controls[$key] = $component;
-					if(isset($properties['id']))
+					if (isset($properties['id']))
 					{
-						if(is_array($properties['id']))
+						if (is_array($properties['id']))
 							$properties['id'] = $component->evaluateExpression($properties['id'][1]);
 						$tplControl->registerObject($properties['id'], $component);
-						if(!$component->hasProperty('id'))
+						if (!$component->hasProperty('id'))
 							unset($properties['id']);
 					}
-					foreach($properties as $name => $value)
+					foreach ($properties as $name => $value)
 						$this->configureComponent($component, $name, $value);
-					if($parent === $parentControl)
+					if ($parent === $parentControl)
 						$directChildren[] = $component;
 					else
 						$component->createdOnTemplate($parent);
@@ -266,19 +266,19 @@ 
 			}
 			else
 			{
-				if($object[1] instanceof TCompositeLiteral)
+				if ($object[1] instanceof TCompositeLiteral)
 				{
 					// need to clone a new object because the one in template is reused
 					$o = clone $object[1];
 					$o->setContainer($tplControl);
-					if($parent === $parentControl)
+					if ($parent === $parentControl)
 						$directChildren[] = $o;
 					else
 						$parent->addParsedObject($o);
 				}
 				else
 				{
-					if($parent === $parentControl)
+					if ($parent === $parentControl)
 						$directChildren[] = $object[1];
 					else
 						$parent->addParsedObject($object[1]);
@@ -288,9 +288,9 @@ 
 		// delay setting parent till now because the parent may cause
 		// the child to do lifecycle catchup which may cause problem
 		// if the child needs its own child controls.
-		foreach($directChildren as $control)
+		foreach ($directChildren as $control)
 		{
-			if($control instanceof TComponent)
+			if ($control instanceof TComponent)
 				$control->createdOnTemplate($parentControl);
 			else
 				$parentControl->addParsedObject($control);
@@ -305,9 +305,9 @@ 
 	 */
 	protected function configureControl($control, $name, $value)
 	{
-		if(strncasecmp($name, 'on', 2) === 0)		// is an event
+		if (strncasecmp($name, 'on', 2) === 0)		// is an event
 			$this->configureEvent($control, $name, $value, $control);
-		elseif(($pos = strrpos($name, '.')) === false)	// is a simple property or custom attribute
+		elseif (($pos = strrpos($name, '.')) === false)	// is a simple property or custom attribute
 			$this->configureProperty($control, $name, $value);
 		else	// is a subproperty
 			$this->configureSubProperty($control, $name, $value);
@@ -321,7 +321,7 @@ 
 	 */
 	protected function configureComponent($component, $name, $value)
 	{
-		if(strpos($name, '.') === false)	// is a simple property or custom attribute
+		if (strpos($name, '.') === false)	// is a simple property or custom attribute
 			$this->configureProperty($component, $name, $value);
 		else	// is a subproperty
 			$this->configureSubProperty($component, $name, $value);
@@ -336,10 +336,10 @@ 
 	 */
 	protected function configureEvent($control, $name, $value, $contextControl)
 	{
-		if(strpos($value, '.') === false)
-			$control->attachEventHandler($name, [$contextControl,'TemplateControl.' . $value]);
+		if (strpos($value, '.') === false)
+			$control->attachEventHandler($name, [$contextControl, 'TemplateControl.'.$value]);
 		else
-			$control->attachEventHandler($name, [$contextControl,$value]);
+			$control->attachEventHandler($name, [$contextControl, $value]);
 	}
 
 	/**
@@ -350,37 +350,37 @@ 
 	 */
 	protected function configureProperty($component, $name, $value)
 	{
-		if(is_array($value))
+		if (is_array($value))
 		{
-			switch($value[0])
+			switch ($value[0])
 			{
 				case self::CONFIG_DATABIND:
 					$component->bindProperty($name, $value[1]);
 					break;
 				case self::CONFIG_EXPRESSION:
-					if($component instanceof TControl)
+					if ($component instanceof TControl)
 						$component->autoBindProperty($name, $value[1]);
 					else
 					{
-						$setter = 'set' . $name;
+						$setter = 'set'.$name;
 						$component->$setter($this->_tplControl->evaluateExpression($value[1]));
 					}
 					break;
 				case self::CONFIG_TEMPLATE:
-					$setter = 'set' . $name;
+					$setter = 'set'.$name;
 					$component->$setter($value[1]);
 					break;
 				case self::CONFIG_ASSET:		// asset URL
-					$setter = 'set' . $name;
-					$url = $this->publishFilePath($this->_contextPath . DIRECTORY_SEPARATOR . $value[1]);
+					$setter = 'set'.$name;
+					$url = $this->publishFilePath($this->_contextPath.DIRECTORY_SEPARATOR.$value[1]);
 					$component->$setter($url);
 					break;
 				case self::CONFIG_PARAMETER:		// application parameter
-					$setter = 'set' . $name;
+					$setter = 'set'.$name;
 					$component->$setter($this->getApplication()->getParameters()->itemAt($value[1]));
 					break;
 				case self::CONFIG_LOCALIZATION:
-					$setter = 'set' . $name;
+					$setter = 'set'.$name;
 					$component->$setter(Prado::localize($value[1]));
 					break;
 				default:	// an error if reaching here
@@ -393,7 +393,7 @@ 
 			if (substr($name, 0, 2) == 'js')
 				if ($value and !($value instanceof TJavaScriptLiteral))
 					$value = new TJavaScriptLiteral($value);
-			$setter = 'set' . $name;
+			$setter = 'set'.$name;
 			$component->$setter($value);
 		}
 	}
@@ -406,15 +406,15 @@ 
 	 */
 	protected function configureSubProperty($component, $name, $value)
 	{
-		if(is_array($value))
+		if (is_array($value))
 		{
-			switch($value[0])
+			switch ($value[0])
 			{
 				case self::CONFIG_DATABIND:		// databinding
 					$component->bindProperty($name, $value[1]);
 					break;
 				case self::CONFIG_EXPRESSION:		// expression
-					if($component instanceof TControl)
+					if ($component instanceof TControl)
 						$component->autoBindProperty($name, $value[1]);
 					else
 						$component->setSubProperty($name, $this->_tplControl->evaluateExpression($value[1]));
@@ -423,7 +423,7 @@ 
 					$component->setSubProperty($name, $value[1]);
 					break;
 				case self::CONFIG_ASSET:		// asset URL
-					$url = $this->publishFilePath($this->_contextPath . DIRECTORY_SEPARATOR . $value[1]);
+					$url = $this->publishFilePath($this->_contextPath.DIRECTORY_SEPARATOR.$value[1]);
 					$component->setSubProperty($name, $url);
 					break;
 				case self::CONFIG_PARAMETER:		// application parameter
@@ -476,105 +476,105 @@ 
 		$this->_directive = null;
 		try
 		{
-			for($i = 0;$i < $n;++$i)
+			for ($i = 0; $i < $n; ++$i)
 			{
 				$match = &$matches[$i];
 				$str = $match[0][0];
 				$matchStart = $match[0][1];
 				$matchEnd = $matchStart + strlen($str) - 1;
-				if(strpos($str, '<com:') === 0)	// opening component tag
+				if (strpos($str, '<com:') === 0)	// opening component tag
 				{
-					if($expectPropEnd)
+					if ($expectPropEnd)
 						continue;
-					if($matchStart > $textStart)
-						$tpl[$c++] = [$container,substr($input, $textStart, $matchStart - $textStart)];
+					if ($matchStart > $textStart)
+						$tpl[$c++] = [$container, substr($input, $textStart, $matchStart - $textStart)];
 					$textStart = $matchEnd + 1;
 					$type = $match[1][0];
 					$attributes = $this->parseAttributes($match[2][0], $match[2][1]);
 					$class = $this->validateAttributes($type, $attributes);
-					$tpl[$c++] = [$container,$class,$attributes];
-					if($str[strlen($str) - 2] !== '/')  // open tag
+					$tpl[$c++] = [$container, $class, $attributes];
+					if ($str[strlen($str) - 2] !== '/')  // open tag
 					{
 						$stack[] = $type;
 						$container = $c - 1;
 					}
 				}
-				elseif(strpos($str, '</com:') === 0)	// closing component tag
+				elseif (strpos($str, '</com:') === 0)	// closing component tag
 				{
-					if($expectPropEnd)
+					if ($expectPropEnd)
 						continue;
-					if($matchStart > $textStart)
-						$tpl[$c++] = [$container,substr($input, $textStart, $matchStart - $textStart)];
+					if ($matchStart > $textStart)
+						$tpl[$c++] = [$container, substr($input, $textStart, $matchStart - $textStart)];
 					$textStart = $matchEnd + 1;
 					$type = $match[1][0];
 
-					if(empty($stack))
+					if (empty($stack))
 						throw new TConfigurationException('template_closingtag_unexpected', "</com:$type>");
 
 					$name = array_pop($stack);
-					if($name !== $type)
+					if ($name !== $type)
 					{
-						$tag = $name[0] === '@' ? '</prop:' . substr($name, 1) . '>' : "</com:$name>";
+						$tag = $name[0] === '@' ? '</prop:'.substr($name, 1).'>' : "</com:$name>";
 						throw new TConfigurationException('template_closingtag_expected', $tag, "</com:$type>");
 					}
 					$container = $tpl[$container][0];
 				}
-				elseif(strpos($str, '<%@') === 0)	// directive
+				elseif (strpos($str, '<%@') === 0)	// directive
 				{
-					if($expectPropEnd)
+					if ($expectPropEnd)
 						continue;
-					if($matchStart > $textStart)
-						$tpl[$c++] = [$container,substr($input, $textStart, $matchStart - $textStart)];
+					if ($matchStart > $textStart)
+						$tpl[$c++] = [$container, substr($input, $textStart, $matchStart - $textStart)];
 					$textStart = $matchEnd + 1;
-					if(isset($tpl[0]) || $this->_directive !== null)
+					if (isset($tpl[0]) || $this->_directive !== null)
 						throw new TConfigurationException('template_directive_nonunique');
 					$this->_directive = $this->parseAttributes($match[4][0], $match[4][1]);
 				}
-				elseif(strpos($str, '<%') === 0)	// expression
+				elseif (strpos($str, '<%') === 0)	// expression
 				{
-					if($expectPropEnd)
+					if ($expectPropEnd)
 						continue;
-					if($matchStart > $textStart)
-						$tpl[$c++] = [$container,substr($input, $textStart, $matchStart - $textStart)];
+					if ($matchStart > $textStart)
+						$tpl[$c++] = [$container, substr($input, $textStart, $matchStart - $textStart)];
 					$textStart = $matchEnd + 1;
 					$literal = trim($match[5][0]);
-					if($str[2] === '=')	// expression
-						$tpl[$c++] = [$container,[TCompositeLiteral::TYPE_EXPRESSION,$literal]];
-					elseif($str[2] === '%')  // statements
-						$tpl[$c++] = [$container,[TCompositeLiteral::TYPE_STATEMENTS,$literal]];
-					elseif($str[2] === '#')
-						$tpl[$c++] = [$container,[TCompositeLiteral::TYPE_DATABINDING,$literal]];
-					elseif($str[2] === '$')
-						$tpl[$c++] = [$container,[TCompositeLiteral::TYPE_EXPRESSION,"\$this->getApplication()->getParameters()->itemAt('$literal')"]];
-					elseif($str[2] === '~')
-						$tpl[$c++] = [$container,[TCompositeLiteral::TYPE_EXPRESSION,"\$this->publishFilePath('$this->_contextPath/$literal')"]];
-					elseif($str[2] === '/')
-						$tpl[$c++] = [$container,[TCompositeLiteral::TYPE_EXPRESSION,"rtrim(dirname(\$this->getApplication()->getRequest()->getApplicationUrl()), '\/').'/$literal'"]];
-					elseif($str[2] === '[')
+					if ($str[2] === '=')	// expression
+						$tpl[$c++] = [$container, [TCompositeLiteral::TYPE_EXPRESSION, $literal]];
+					elseif ($str[2] === '%')  // statements
+						$tpl[$c++] = [$container, [TCompositeLiteral::TYPE_STATEMENTS, $literal]];
+					elseif ($str[2] === '#')
+						$tpl[$c++] = [$container, [TCompositeLiteral::TYPE_DATABINDING, $literal]];
+					elseif ($str[2] === '$')
+						$tpl[$c++] = [$container, [TCompositeLiteral::TYPE_EXPRESSION, "\$this->getApplication()->getParameters()->itemAt('$literal')"]];
+					elseif ($str[2] === '~')
+						$tpl[$c++] = [$container, [TCompositeLiteral::TYPE_EXPRESSION, "\$this->publishFilePath('$this->_contextPath/$literal')"]];
+					elseif ($str[2] === '/')
+						$tpl[$c++] = [$container, [TCompositeLiteral::TYPE_EXPRESSION, "rtrim(dirname(\$this->getApplication()->getRequest()->getApplicationUrl()), '\/').'/$literal'"]];
+					elseif ($str[2] === '[')
 					{
-						$literal = strtr(trim(substr($literal, 0, strlen($literal) - 1)), ["'" => "\'","\\" => "\\\\"]);
-						$tpl[$c++] = [$container,[TCompositeLiteral::TYPE_EXPRESSION,"Prado::localize('$literal')"]];
+						$literal = strtr(trim(substr($literal, 0, strlen($literal) - 1)), ["'" => "\'", "\\" => "\\\\"]);
+						$tpl[$c++] = [$container, [TCompositeLiteral::TYPE_EXPRESSION, "Prado::localize('$literal')"]];
 					}
 				}
-				elseif(strpos($str, '<prop:') === 0)	// opening property
+				elseif (strpos($str, '<prop:') === 0)	// opening property
 				{
-					if(strrpos($str, '/>') === strlen($str) - 2)  //subproperties
+					if (strrpos($str, '/>') === strlen($str) - 2)  //subproperties
 					{
-						if($expectPropEnd)
+						if ($expectPropEnd)
 							continue;
-						if($matchStart > $textStart)
-							$tpl[$c++] = [$container,substr($input, $textStart, $matchStart - $textStart)];
+						if ($matchStart > $textStart)
+							$tpl[$c++] = [$container, substr($input, $textStart, $matchStart - $textStart)];
 						$textStart = $matchEnd + 1;
 						$prop = strtolower($match[6][0]);
 						$attrs = $this->parseAttributes($match[7][0], $match[7][1]);
 						$attributes = [];
-						foreach($attrs as $name => $value)
-							$attributes[$prop . '.' . $name] = $value;
+						foreach ($attrs as $name => $value)
+							$attributes[$prop.'.'.$name] = $value;
 						$type = $tpl[$container][1];
 						$this->validateAttributes($type, $attributes);
-						foreach($attributes as $name => $value)
+						foreach ($attributes as $name => $value)
 						{
-							if(isset($tpl[$container][2][$name]))
+							if (isset($tpl[$container][2][$name]))
 								throw new TConfigurationException('template_property_duplicated', $name);
 							$tpl[$container][2][$name] = $value;
 						}
@@ -582,41 +582,41 @@ 
 					else  // regular property
 					{
 						$prop = strtolower($match[3][0]);
-						$stack[] = '@' . $prop;
-						if(!$expectPropEnd)
+						$stack[] = '@'.$prop;
+						if (!$expectPropEnd)
 						{
-							if($matchStart > $textStart)
-								$tpl[$c++] = [$container,substr($input, $textStart, $matchStart - $textStart)];
+							if ($matchStart > $textStart)
+								$tpl[$c++] = [$container, substr($input, $textStart, $matchStart - $textStart)];
 							$textStart = $matchEnd + 1;
 							$expectPropEnd = true;
 						}
 					}
 				}
-				elseif(strpos($str, '</prop:') === 0)	// closing property
+				elseif (strpos($str, '</prop:') === 0)	// closing property
 				{
 					$prop = strtolower($match[3][0]);
-					if(empty($stack))
+					if (empty($stack))
 						throw new TConfigurationException('template_closingtag_unexpected', "</prop:$prop>");
 					$name = array_pop($stack);
-					if($name !== '@' . $prop)
+					if ($name !== '@'.$prop)
 					{
-						$tag = $name[0] === '@' ? '</prop:' . substr($name, 1) . '>' : "</com:$name>";
+						$tag = $name[0] === '@' ? '</prop:'.substr($name, 1).'>' : "</com:$name>";
 						throw new TConfigurationException('template_closingtag_expected', $tag, "</prop:$prop>");
 					}
-					if(($last = count($stack)) < 1 || $stack[$last - 1][0] !== '@')
+					if (($last = count($stack)) < 1 || $stack[$last - 1][0] !== '@')
 					{
-						if($matchStart > $textStart)
+						if ($matchStart > $textStart)
 						{
 							$value = substr($input, $textStart, $matchStart - $textStart);
-							if(substr($prop, -8, 8) === 'template')
+							if (substr($prop, -8, 8) === 'template')
 								$value = $this->parseTemplateProperty($value, $textStart);
 							else
 								$value = $this->parseAttribute($value);
-							if($container >= 0)
+							if ($container >= 0)
 							{
 								$type = $tpl[$container][1];
 								$this->validateAttributes($type, [$prop => $value]);
-								if(isset($tpl[$container][2][$prop]))
+								if (isset($tpl[$container][2][$prop]))
 									throw new TConfigurationException('template_property_duplicated', $prop);
 								$tpl[$container][2][$prop] = $value;
 							}
@@ -627,56 +627,56 @@ 
 						$expectPropEnd = false;
 					}
 				}
-				elseif(strpos($str, '<!--') === 0)	// comments
+				elseif (strpos($str, '<!--') === 0)	// comments
 				{
-					if($expectPropEnd)
+					if ($expectPropEnd)
 						throw new TConfigurationException('template_comments_forbidden');
-					if($matchStart > $textStart)
-						$tpl[$c++] = [$container,substr($input, $textStart, $matchStart - $textStart)];
+					if ($matchStart > $textStart)
+						$tpl[$c++] = [$container, substr($input, $textStart, $matchStart - $textStart)];
 					$textStart = $matchEnd + 1;
 				}
 				else
 					throw new TConfigurationException('template_matching_unexpected', $match);
 			}
-			if(!empty($stack))
+			if (!empty($stack))
 			{
 				$name = array_pop($stack);
-				$tag = $name[0] === '@' ? '</prop:' . substr($name, 1) . '>' : "</com:$name>";
+				$tag = $name[0] === '@' ? '</prop:'.substr($name, 1).'>' : "</com:$name>";
 				throw new TConfigurationException('template_closingtag_expected', $tag, "nothing");
 			}
-			if($textStart < strlen($input))
-				$tpl[$c++] = [$container,substr($input, $textStart)];
+			if ($textStart < strlen($input))
+				$tpl[$c++] = [$container, substr($input, $textStart)];
 		}
-		catch(\Exception $e)
+		catch (\Exception $e)
 		{
-			if(($e instanceof TException) && ($e instanceof TTemplateException))
+			if (($e instanceof TException) && ($e instanceof TTemplateException))
 				throw $e;
-			if($matchEnd === 0)
+			if ($matchEnd === 0)
 				$line = $this->_startingLine + 1;
 			else
 				$line = $this->_startingLine + count(explode("\n", substr($input, 0, $matchEnd + 1)));
 			$this->handleException($e, $line, $input);
 		}
 
-		if($this->_directive === null)
+		if ($this->_directive === null)
 			$this->_directive = [];
 
 		// optimization by merging consecutive strings, expressions, statements and bindings
 		$objects = [];
 		$parent = null;
 		$merged = [];
-		foreach($tpl as $id => $object)
+		foreach ($tpl as $id => $object)
 		{
-			if(isset($object[2]) || $object[0] !== $parent)
+			if (isset($object[2]) || $object[0] !== $parent)
 			{
-				if($parent !== null)
+				if ($parent !== null)
 				{
-					if(count($merged[1]) === 1 && is_string($merged[1][0]))
-						$objects[$id - 1] = [$merged[0],$merged[1][0]];
+					if (count($merged[1]) === 1 && is_string($merged[1][0]))
+						$objects[$id - 1] = [$merged[0], $merged[1][0]];
 					else
-						$objects[$id - 1] = [$merged[0],new TCompositeLiteral($merged[1])];
+						$objects[$id - 1] = [$merged[0], new TCompositeLiteral($merged[1])];
 				}
-				if(isset($object[2]))
+				if (isset($object[2]))
 				{
 					$parent = null;
 					$objects[$id] = $object;
@@ -684,18 +684,18 @@ 
 				else
 				{
 					$parent = $object[0];
-					$merged = [$parent,[$object[1]]];
+					$merged = [$parent, [$object[1]]];
 				}
 			}
 			else
 				$merged[1][] = $object[1];
 		}
-		if($parent !== null)
+		if ($parent !== null)
 		{
-			if(count($merged[1]) === 1 && is_string($merged[1][0]))
-				$objects[$id] = [$merged[0],$merged[1][0]];
+			if (count($merged[1]) === 1 && is_string($merged[1][0]))
+				$objects[$id] = [$merged[0], $merged[1][0]];
 			else
-				$objects[$id] = [$merged[0],new TCompositeLiteral($merged[1])];
+				$objects[$id] = [$merged[0], new TCompositeLiteral($merged[1])];
 		}
 		$tpl = $objects;
 		return $objects;
@@ -708,28 +708,28 @@ 
 	 */
 	protected function parseAttributes($str, $offset)
 	{
-		if($str === '')
+		if ($str === '')
 			return [];
 		$pattern = '/([\w\.\-]+)\s*=\s*(\'.*?\'|".*?"|<%.*?%>)/msS';
 		$attributes = [];
 		$n = preg_match_all($pattern, $str, $matches, PREG_SET_ORDER | PREG_OFFSET_CAPTURE);
-		for($i = 0;$i < $n;++$i)
+		for ($i = 0; $i < $n; ++$i)
 		{
 			$match = &$matches[$i];
 			$name = strtolower($match[1][0]);
-			if(isset($attributes[$name]))
+			if (isset($attributes[$name]))
 				throw new TConfigurationException('template_property_duplicated', $name);
 			$value = $match[2][0];
-			if(substr($name, -8, 8) === 'template')
+			if (substr($name, -8, 8) === 'template')
 			{
-				if($value[0] === '\'' || $value[0] === '"')
+				if ($value[0] === '\'' || $value[0] === '"')
 					$attributes[$name] = $this->parseTemplateProperty(substr($value, 1, strlen($value) - 2), $match[2][1] + 1);
 				else
 					$attributes[$name] = $this->parseTemplateProperty($value, $match[2][1]);
 			}
 			else
 			{
-				if($value[0] === '\'' || $value[0] === '"')
+				if ($value[0] === '\'' || $value[0] === '"')
 					$attributes[$name] = $this->parseAttribute(substr($value, 1, strlen($value) - 2));
 				else
 					$attributes[$name] = $this->parseAttribute($value);
@@ -741,7 +741,7 @@ 
 	protected function parseTemplateProperty($content, $offset)
 	{
 		$line = $this->_startingLine + count(explode("\n", substr($this->_content, 0, $offset))) - 1;
-		return [self::CONFIG_TEMPLATE,new TTemplate($content, $this->_contextPath, $this->_tplFile, $line, false)];
+		return [self::CONFIG_TEMPLATE, new TTemplate($content, $this->_contextPath, $this->_tplFile, $line, false)];
 	}
 
 	/**
@@ -751,44 +751,44 @@ 
 	 */
 	protected function parseAttribute($value)
 	{
-		if(($n = preg_match_all('/<%[#=].*?%>/msS', $value, $matches, PREG_OFFSET_CAPTURE)) > 0)
+		if (($n = preg_match_all('/<%[#=].*?%>/msS', $value, $matches, PREG_OFFSET_CAPTURE)) > 0)
 		{
 			$isDataBind = false;
 			$textStart = 0;
 			$expr = '';
-			for($i = 0;$i < $n;++$i)
+			for ($i = 0; $i < $n; ++$i)
 			{
 				$match = $matches[0][$i];
 				$token = $match[0];
 				$offset = $match[1];
 				$length = strlen($token);
-				if($token[2] === '#')
+				if ($token[2] === '#')
 					$isDataBind = true;
-				if($offset > $textStart)
-					$expr .= ".'" . strtr(substr($value, $textStart, $offset - $textStart), ["'" => "\\'","\\" => "\\\\"]) . "'";
-				$expr .= '.(' . substr($token, 3, $length - 5) . ')';
+				if ($offset > $textStart)
+					$expr .= ".'".strtr(substr($value, $textStart, $offset - $textStart), ["'" => "\\'", "\\" => "\\\\"])."'";
+				$expr .= '.('.substr($token, 3, $length - 5).')';
 				$textStart = $offset + $length;
 			}
 			$length = strlen($value);
-			if($length > $textStart)
-				$expr .= ".'" . strtr(substr($value, $textStart, $length - $textStart), ["'" => "\\'","\\" => "\\\\"]) . "'";
-			if($isDataBind)
-				return [self::CONFIG_DATABIND,ltrim($expr, '.')];
+			if ($length > $textStart)
+				$expr .= ".'".strtr(substr($value, $textStart, $length - $textStart), ["'" => "\\'", "\\" => "\\\\"])."'";
+			if ($isDataBind)
+				return [self::CONFIG_DATABIND, ltrim($expr, '.')];
 			else
-				return [self::CONFIG_EXPRESSION,ltrim($expr, '.')];
+				return [self::CONFIG_EXPRESSION, ltrim($expr, '.')];
 		}
-		elseif(preg_match('/\\s*(<%~.*?%>|<%\\$.*?%>|<%\\[.*?\\]%>|<%\/.*?%>)\\s*/msS', $value, $matches) && $matches[0] === $value)
+		elseif (preg_match('/\\s*(<%~.*?%>|<%\\$.*?%>|<%\\[.*?\\]%>|<%\/.*?%>)\\s*/msS', $value, $matches) && $matches[0] === $value)
 		{
 			$value = $matches[1];
-			if($value[2] === '~')
-				return [self::CONFIG_ASSET,trim(substr($value, 3, strlen($value) - 5))];
-			elseif($value[2] === '[')
-				return [self::CONFIG_LOCALIZATION,trim(substr($value, 3, strlen($value) - 6))];
-			elseif($value[2] === '$')
-				return [self::CONFIG_PARAMETER,trim(substr($value, 3, strlen($value) - 5))];
-			elseif($value[2] === '/') {
+			if ($value[2] === '~')
+				return [self::CONFIG_ASSET, trim(substr($value, 3, strlen($value) - 5))];
+			elseif ($value[2] === '[')
+				return [self::CONFIG_LOCALIZATION, trim(substr($value, 3, strlen($value) - 6))];
+			elseif ($value[2] === '$')
+				return [self::CONFIG_PARAMETER, trim(substr($value, 3, strlen($value) - 5))];
+			elseif ($value[2] === '/') {
 				$literal = trim(substr($value, 3, strlen($value) - 5));
-				return [self::CONFIG_EXPRESSION,"rtrim(dirname(\$this->getApplication()->getRequest()->getApplicationUrl()), '\/').'/$literal'"];
+				return [self::CONFIG_EXPRESSION, "rtrim(dirname(\$this->getApplication()->getRequest()->getApplicationUrl()), '\/').'/$literal'"];
 			}
 		}
 		else
@@ -798,71 +798,71 @@ 
 	protected function validateAttributes($type, $attributes)
 	{
 		Prado::using($type);
-		if(($pos = strrpos($type, '.')) !== false)
+		if (($pos = strrpos($type, '.')) !== false)
 			$className = substr($type, $pos + 1);
 		else
 			$className = $type;
 		$class = new \ReflectionClass($className);
-		if(is_subclass_of($className, '\Prado\Web\UI\TControl') || $className === '\Prado\Web\UI\TControl')
+		if (is_subclass_of($className, '\Prado\Web\UI\TControl') || $className === '\Prado\Web\UI\TControl')
 		{
-			foreach($attributes as $name => $att)
+			foreach ($attributes as $name => $att)
 			{
-				if(($pos = strpos($name, '.')) !== false)
+				if (($pos = strpos($name, '.')) !== false)
 				{
 					// a subproperty, so the first segment must be readable
 					$subname = substr($name, 0, $pos);
-					if(!$class->hasMethod('get' . $subname))
+					if (!$class->hasMethod('get'.$subname))
 						throw new TConfigurationException('template_property_unknown', $type, $subname);
 				}
-				elseif(strncasecmp($name, 'on', 2) === 0)
+				elseif (strncasecmp($name, 'on', 2) === 0)
 				{
 					// an event
-					if(!$class->hasMethod($name))
+					if (!$class->hasMethod($name))
 						throw new TConfigurationException('template_event_unknown', $type, $name);
-					elseif(!is_string($att))
+					elseif (!is_string($att))
 						throw new TConfigurationException('template_eventhandler_invalid', $type, $name);
 				}
 				else
 				{
 					// a simple property
-					if (! ($class->hasMethod('set' . $name) || $class->hasMethod('setjs' . $name) || $this->isClassBehaviorMethod($class, 'set' . $name)))
+					if (!($class->hasMethod('set'.$name) || $class->hasMethod('setjs'.$name) || $this->isClassBehaviorMethod($class, 'set'.$name)))
 					{
-						if ($class->hasMethod('get' . $name) || $class->hasMethod('getjs' . $name))
+						if ($class->hasMethod('get'.$name) || $class->hasMethod('getjs'.$name))
 							throw new TConfigurationException('template_property_readonly', $type, $name);
 						else
 							throw new TConfigurationException('template_property_unknown', $type, $name);
 					}
-					elseif(is_array($att) && $att[0] !== self::CONFIG_EXPRESSION)
+					elseif (is_array($att) && $att[0] !== self::CONFIG_EXPRESSION)
 					{
-						if(strcasecmp($name, 'id') === 0)
+						if (strcasecmp($name, 'id') === 0)
 							throw new TConfigurationException('template_controlid_invalid', $type);
-						elseif(strcasecmp($name, 'skinid') === 0)
+						elseif (strcasecmp($name, 'skinid') === 0)
 							throw new TConfigurationException('template_controlskinid_invalid', $type);
 					}
 				}
 			}
 		}
-		elseif(is_subclass_of($className, '\Prado\TComponent') || $className === '\Prado\TComponent')
+		elseif (is_subclass_of($className, '\Prado\TComponent') || $className === '\Prado\TComponent')
 		{
-			foreach($attributes as $name => $att)
+			foreach ($attributes as $name => $att)
 			{
-				if(is_array($att) && ($att[0] === self::CONFIG_DATABIND))
+				if (is_array($att) && ($att[0] === self::CONFIG_DATABIND))
 					throw new TConfigurationException('template_databind_forbidden', $type, $name);
-				if(($pos = strpos($name, '.')) !== false)
+				if (($pos = strpos($name, '.')) !== false)
 				{
 					// a subproperty, so the first segment must be readable
 					$subname = substr($name, 0, $pos);
-					if(!$class->hasMethod('get' . $subname))
+					if (!$class->hasMethod('get'.$subname))
 						throw new TConfigurationException('template_property_unknown', $type, $subname);
 				}
-				elseif(strncasecmp($name, 'on', 2) === 0)
+				elseif (strncasecmp($name, 'on', 2) === 0)
 					throw new TConfigurationException('template_event_forbidden', $type, $name);
 				else
 				{
 					// id is still alowed for TComponent, even if id property doesn't exist
-					if(strcasecmp($name, 'id') !== 0 && !($class->hasMethod('set' . $name) || $this->isClassBehaviorMethod($class, 'set' . $name)))
+					if (strcasecmp($name, 'id') !== 0 && !($class->hasMethod('set'.$name) || $this->isClassBehaviorMethod($class, 'set'.$name)))
 					{
-						if($class->hasMethod('get' . $name))
+						if ($class->hasMethod('get'.$name))
 							throw new TConfigurationException('template_property_readonly', $type, $name);
 						else
 							throw new TConfigurationException('template_property_unknown', $type, $name);
@@ -895,13 +895,13 @@ 
 	{
 		$srcFile = $this->_tplFile;
 
-		if(($n = count($this->_includedFiles)) > 0) // need to adjust error row number and file name
+		if (($n = count($this->_includedFiles)) > 0) // need to adjust error row number and file name
 		{
-			for($i = $n - 1;$i >= 0;--$i)
+			for ($i = $n - 1; $i >= 0; --$i)
 			{
-				if($this->_includeAtLine[$i] <= $line)
+				if ($this->_includeAtLine[$i] <= $line)
 				{
-					if($line < $this->_includeAtLine[$i] + $this->_includeLines[$i])
+					if ($line < $this->_includeAtLine[$i] + $this->_includeLines[$i])
 					{
 						$line = $line - $this->_includeAtLine[$i] + 1;
 						$srcFile = $this->_includedFiles[$i];
@@ -914,7 +914,7 @@ 
 		}
 		$exception = new TTemplateException('template_format_invalid', $e->getMessage());
 		$exception->setLineNumber($line);
-		if(!empty($srcFile))
+		if (!empty($srcFile))
 			$exception->setTemplateFile($srcFile);
 		else
 			$exception->setTemplateSource($input);
@@ -928,12 +928,12 @@ 
 	 */
 	protected function preprocess($input)
 	{
-		if($n = preg_match_all('/<%include(.*?)%>/', $input, $matches, PREG_SET_ORDER | PREG_OFFSET_CAPTURE))
+		if ($n = preg_match_all('/<%include(.*?)%>/', $input, $matches, PREG_SET_ORDER | PREG_OFFSET_CAPTURE))
 		{
-			for($i = 0;$i < $n;++$i)
+			for ($i = 0; $i < $n; ++$i)
 			{
 				$filePath = Prado::getPathOfNamespace(trim($matches[$i][1][0]), TTemplateManager::TEMPLATE_FILE_EXT);
-				if($filePath !== null && is_file($filePath))
+				if ($filePath !== null && is_file($filePath))
 					$this->_includedFiles[] = $filePath;
 				else
 				{
@@ -942,7 +942,7 @@ 
 				}
 			}
 			$base = 0;
-			for($i = 0;$i < $n;++$i)
+			for ($i = 0; $i < $n; ++$i)
 			{
 				$ext = file_get_contents($this->_includedFiles[$i]);
 				$length = strlen($matches[$i][0][0]);
@@ -967,14 +967,14 @@ 
 	{
 	  $component = new \ReflectionClass('\Prado\TComponent');
 	  $behaviors = $component->getStaticProperties();
-	  if(!isset($behaviors['_um']))
+	  if (!isset($behaviors['_um']))
 		return false;
-	  foreach($behaviors['_um'] as $name => $list)
+	  foreach ($behaviors['_um'] as $name => $list)
 	  {
-		if(strtolower($class->getShortName()) !== $name && !$class->isSubclassOf($name)) continue;
-		foreach($list as $param)
+		if (strtolower($class->getShortName()) !== $name && !$class->isSubclassOf($name)) continue;
+		foreach ($list as $param)
 		{
-		  if(method_exists($param->getBehavior(), $method))
+		  if (method_exists($param->getBehavior(), $method))
 			return true;
 		}
 	  }

Braces +325 added lines, -254 removed lines

@@ -192,96 +192,110 @@ 
 	public function instantiateIn($tplControl, $parentControl = null)
 	{
 		$this->_tplControl = $tplControl;
-		if($parentControl === null)
-			$parentControl = $tplControl;
-		if(($page = $tplControl->getPage()) === null)
-			$page = $this->getService()->getRequestedPage();
+		if($parentControl === null) {
+					$parentControl = $tplControl;
+		}
+		if(($page = $tplControl->getPage()) === null) {
+					$page = $this->getService()->getRequestedPage();
+		}
 		$controls = [];
 		$directChildren = [];
 		foreach($this->_tpl as $key => $object)
 		{
-			if($object[0] === -1)
-				$parent = $parentControl;
-			elseif(isset($controls[$object[0]]))
-				$parent = $controls[$object[0]];
-			else
-				continue;
-			if(isset($object[2]))	// component
+			if($object[0] === -1) {
+							$parent = $parentControl;
+			} elseif(isset($controls[$object[0]])) {
+							$parent = $controls[$object[0]];
+			} else {
+							continue;
+			}
+			if(isset($object[2])) {
+				// component
 			{
 				$component = Prado::createComponent($object[1]);
+			}
 				$properties = &$object[2];
 				if($component instanceof TControl)
 				{
-					if($component instanceof \Prado\Web\UI\WebControls\TOutputCache)
-						$component->setCacheKeyPrefix($this->_hashCode . $key);
+					if($component instanceof \Prado\Web\UI\WebControls\TOutputCache) {
+											$component->setCacheKeyPrefix($this->_hashCode . $key);
+					}
 					$component->setTemplateControl($tplControl);
 					if(isset($properties['id']))
 					{
-						if(is_array($properties['id']))
-							$properties['id'] = $component->evaluateExpression($properties['id'][1]);
+						if(is_array($properties['id'])) {
+													$properties['id'] = $component->evaluateExpression($properties['id'][1]);
+						}
 						$tplControl->registerObject($properties['id'], $component);
 					}
 					if(isset($properties['skinid']))
 					{
-						if(is_array($properties['skinid']))
-							$component->setSkinID($component->evaluateExpression($properties['skinid'][1]));
-						else
-							$component->setSkinID($properties['skinid']);
+						if(is_array($properties['skinid'])) {
+													$component->setSkinID($component->evaluateExpression($properties['skinid'][1]));
+						} else {
+													$component->setSkinID($properties['skinid']);
+						}
 						unset($properties['skinid']);
 					}
 
 					$component->trackViewState(false);
 
 					$component->applyStyleSheetSkin($page);
-					foreach($properties as $name => $value)
-						$this->configureControl($component, $name, $value);
+					foreach($properties as $name => $value) {
+											$this->configureControl($component, $name, $value);
+					}
 
 					$component->trackViewState(true);
 
-					if($parent === $parentControl)
-						$directChildren[] = $component;
-					else
-						$component->createdOnTemplate($parent);
-					if($component->getAllowChildControls())
-						$controls[$key] = $component;
-				}
-				elseif($component instanceof TComponent)
+					if($parent === $parentControl) {
+											$directChildren[] = $component;
+					} else {
+											$component->createdOnTemplate($parent);
+					}
+					if($component->getAllowChildControls()) {
+											$controls[$key] = $component;
+					}
+				} elseif($component instanceof TComponent)
 				{
 					$controls[$key] = $component;
 					if(isset($properties['id']))
 					{
-						if(is_array($properties['id']))
-							$properties['id'] = $component->evaluateExpression($properties['id'][1]);
+						if(is_array($properties['id'])) {
+													$properties['id'] = $component->evaluateExpression($properties['id'][1]);
+						}
 						$tplControl->registerObject($properties['id'], $component);
-						if(!$component->hasProperty('id'))
-							unset($properties['id']);
-					}
-					foreach($properties as $name => $value)
-						$this->configureComponent($component, $name, $value);
-					if($parent === $parentControl)
-						$directChildren[] = $component;
-					else
-						$component->createdOnTemplate($parent);
+						if(!$component->hasProperty('id')) {
+													unset($properties['id']);
+						}
+					}
+					foreach($properties as $name => $value) {
+											$this->configureComponent($component, $name, $value);
+					}
+					if($parent === $parentControl) {
+											$directChildren[] = $component;
+					} else {
+											$component->createdOnTemplate($parent);
+					}
 				}
-			}
-			else
+			} else
 			{
 				if($object[1] instanceof TCompositeLiteral)
 				{
 					// need to clone a new object because the one in template is reused
 					$o = clone $object[1];
 					$o->setContainer($tplControl);
-					if($parent === $parentControl)
-						$directChildren[] = $o;
-					else
-						$parent->addParsedObject($o);
-				}
-				else
+					if($parent === $parentControl) {
+											$directChildren[] = $o;
+					} else {
+											$parent->addParsedObject($o);
+					}
+				} else
 				{
-					if($parent === $parentControl)
-						$directChildren[] = $object[1];
-					else
-						$parent->addParsedObject($object[1]);
+					if($parent === $parentControl) {
+											$directChildren[] = $object[1];
+					} else {
+											$parent->addParsedObject($object[1]);
+					}
 				}
 			}
 		}
@@ -290,10 +304,11 @@ 
 		// if the child needs its own child controls.
 		foreach($directChildren as $control)
 		{
-			if($control instanceof TComponent)
-				$control->createdOnTemplate($parentControl);
-			else
-				$parentControl->addParsedObject($control);
+			if($control instanceof TComponent) {
+							$control->createdOnTemplate($parentControl);
+			} else {
+							$parentControl->addParsedObject($control);
+			}
 		}
 	}
 
@@ -305,12 +320,16 @@ 
 	 */
 	protected function configureControl($control, $name, $value)
 	{
-		if(strncasecmp($name, 'on', 2) === 0)		// is an event
+		if(strncasecmp($name, 'on', 2) === 0) {
+			// is an event
 			$this->configureEvent($control, $name, $value, $control);
-		elseif(($pos = strrpos($name, '.')) === false)	// is a simple property or custom attribute
+		} elseif(($pos = strrpos($name, '.')) === false) {
+			// is a simple property or custom attribute
 			$this->configureProperty($control, $name, $value);
-		else	// is a subproperty
+		} else {
+			// is a subproperty
 			$this->configureSubProperty($control, $name, $value);
+		}
 	}
 
 	/**
@@ -321,10 +340,13 @@ 
 	 */
 	protected function configureComponent($component, $name, $value)
 	{
-		if(strpos($name, '.') === false)	// is a simple property or custom attribute
+		if(strpos($name, '.') === false) {
+			// is a simple property or custom attribute
 			$this->configureProperty($component, $name, $value);
-		else	// is a subproperty
+		} else {
+			// is a subproperty
 			$this->configureSubProperty($component, $name, $value);
+		}
 	}
 
 	/**
@@ -336,10 +358,11 @@ 
 	 */
 	protected function configureEvent($control, $name, $value, $contextControl)
 	{
-		if(strpos($value, '.') === false)
-			$control->attachEventHandler($name, [$contextControl,'TemplateControl.' . $value]);
-		else
-			$control->attachEventHandler($name, [$contextControl,$value]);
+		if(strpos($value, '.') === false) {
+					$control->attachEventHandler($name, [$contextControl,'TemplateControl.' . $value]);
+		} else {
+					$control->attachEventHandler($name, [$contextControl,$value]);
+		}
 	}
 
 	/**
@@ -358,9 +381,9 @@ 
 					$component->bindProperty($name, $value[1]);
 					break;
 				case self::CONFIG_EXPRESSION:
-					if($component instanceof TControl)
-						$component->autoBindProperty($name, $value[1]);
-					else
+					if($component instanceof TControl) {
+											$component->autoBindProperty($name, $value[1]);
+					} else
 					{
 						$setter = 'set' . $name;
 						$component->$setter($this->_tplControl->evaluateExpression($value[1]));
@@ -387,12 +410,12 @@ 
 					throw new TConfigurationException('template_tag_unexpected', $name, $value[1]);
 					break;
 			}
-		}
-		else
+		} else
 		{
-			if (substr($name, 0, 2) == 'js')
-				if ($value and !($value instanceof TJavaScriptLiteral))
+			if (substr($name, 0, 2) == 'js') {
+							if ($value and !($value instanceof TJavaScriptLiteral))
 					$value = new TJavaScriptLiteral($value);
+			}
 			$setter = 'set' . $name;
 			$component->$setter($value);
 		}
@@ -414,10 +437,11 @@ 
 					$component->bindProperty($name, $value[1]);
 					break;
 				case self::CONFIG_EXPRESSION:		// expression
-					if($component instanceof TControl)
-						$component->autoBindProperty($name, $value[1]);
-					else
-						$component->setSubProperty($name, $this->_tplControl->evaluateExpression($value[1]));
+					if($component instanceof TControl) {
+											$component->autoBindProperty($name, $value[1]);
+					} else {
+											$component->setSubProperty($name, $this->_tplControl->evaluateExpression($value[1]));
+					}
 					break;
 				case self::CONFIG_TEMPLATE:
 					$component->setSubProperty($name, $value[1]);
@@ -436,9 +460,9 @@ 
 					throw new TConfigurationException('template_tag_unexpected', $name, $value[1]);
 					break;
 			}
+		} else {
+					$component->setSubProperty($name, $value);
 		}
-		else
-			$component->setSubProperty($name, $value);
 	}
 
 	/**
@@ -482,34 +506,42 @@ 
 				$str = $match[0][0];
 				$matchStart = $match[0][1];
 				$matchEnd = $matchStart + strlen($str) - 1;
-				if(strpos($str, '<com:') === 0)	// opening component tag
+				if(strpos($str, '<com:') === 0) {
+					// opening component tag
 				{
 					if($expectPropEnd)
 						continue;
-					if($matchStart > $textStart)
-						$tpl[$c++] = [$container,substr($input, $textStart, $matchStart - $textStart)];
+				}
+					if($matchStart > $textStart) {
+											$tpl[$c++] = [$container,substr($input, $textStart, $matchStart - $textStart)];
+					}
 					$textStart = $matchEnd + 1;
 					$type = $match[1][0];
 					$attributes = $this->parseAttributes($match[2][0], $match[2][1]);
 					$class = $this->validateAttributes($type, $attributes);
 					$tpl[$c++] = [$container,$class,$attributes];
-					if($str[strlen($str) - 2] !== '/')  // open tag
+					if($str[strlen($str) - 2] !== '/') {
+						// open tag
 					{
 						$stack[] = $type;
+					}
 						$container = $c - 1;
 					}
-				}
-				elseif(strpos($str, '</com:') === 0)	// closing component tag
+				} elseif(strpos($str, '</com:') === 0) {
+					// closing component tag
 				{
 					if($expectPropEnd)
 						continue;
-					if($matchStart > $textStart)
-						$tpl[$c++] = [$container,substr($input, $textStart, $matchStart - $textStart)];
+				}
+					if($matchStart > $textStart) {
+											$tpl[$c++] = [$container,substr($input, $textStart, $matchStart - $textStart)];
+					}
 					$textStart = $matchEnd + 1;
 					$type = $match[1][0];
 
-					if(empty($stack))
-						throw new TConfigurationException('template_closingtag_unexpected', "</com:$type>");
+					if(empty($stack)) {
+											throw new TConfigurationException('template_closingtag_unexpected', "</com:$type>");
+					}
 
 					$name = array_pop($stack);
 					if($name !== $type)
@@ -518,85 +550,98 @@ 
 						throw new TConfigurationException('template_closingtag_expected', $tag, "</com:$type>");
 					}
 					$container = $tpl[$container][0];
-				}
-				elseif(strpos($str, '<%@') === 0)	// directive
+				} elseif(strpos($str, '<%@') === 0) {
+					// directive
 				{
 					if($expectPropEnd)
 						continue;
-					if($matchStart > $textStart)
-						$tpl[$c++] = [$container,substr($input, $textStart, $matchStart - $textStart)];
+				}
+					if($matchStart > $textStart) {
+											$tpl[$c++] = [$container,substr($input, $textStart, $matchStart - $textStart)];
+					}
 					$textStart = $matchEnd + 1;
-					if(isset($tpl[0]) || $this->_directive !== null)
-						throw new TConfigurationException('template_directive_nonunique');
+					if(isset($tpl[0]) || $this->_directive !== null) {
+											throw new TConfigurationException('template_directive_nonunique');
+					}
 					$this->_directive = $this->parseAttributes($match[4][0], $match[4][1]);
-				}
-				elseif(strpos($str, '<%') === 0)	// expression
+				} elseif(strpos($str, '<%') === 0) {
+					// expression
 				{
 					if($expectPropEnd)
 						continue;
-					if($matchStart > $textStart)
-						$tpl[$c++] = [$container,substr($input, $textStart, $matchStart - $textStart)];
+				}
+					if($matchStart > $textStart) {
+											$tpl[$c++] = [$container,substr($input, $textStart, $matchStart - $textStart)];
+					}
 					$textStart = $matchEnd + 1;
 					$literal = trim($match[5][0]);
-					if($str[2] === '=')	// expression
+					if($str[2] === '=') {
+						// expression
 						$tpl[$c++] = [$container,[TCompositeLiteral::TYPE_EXPRESSION,$literal]];
-					elseif($str[2] === '%')  // statements
+					} elseif($str[2] === '%') {
+						// statements
 						$tpl[$c++] = [$container,[TCompositeLiteral::TYPE_STATEMENTS,$literal]];
-					elseif($str[2] === '#')
-						$tpl[$c++] = [$container,[TCompositeLiteral::TYPE_DATABINDING,$literal]];
-					elseif($str[2] === '$')
-						$tpl[$c++] = [$container,[TCompositeLiteral::TYPE_EXPRESSION,"\$this->getApplication()->getParameters()->itemAt('$literal')"]];
-					elseif($str[2] === '~')
-						$tpl[$c++] = [$container,[TCompositeLiteral::TYPE_EXPRESSION,"\$this->publishFilePath('$this->_contextPath/$literal')"]];
-					elseif($str[2] === '/')
-						$tpl[$c++] = [$container,[TCompositeLiteral::TYPE_EXPRESSION,"rtrim(dirname(\$this->getApplication()->getRequest()->getApplicationUrl()), '\/').'/$literal'"]];
-					elseif($str[2] === '[')
+					} elseif($str[2] === '#') {
+											$tpl[$c++] = [$container,[TCompositeLiteral::TYPE_DATABINDING,$literal]];
+					} elseif($str[2] === '$') {
+											$tpl[$c++] = [$container,[TCompositeLiteral::TYPE_EXPRESSION,"\$this->getApplication()->getParameters()->itemAt('$literal')"]];
+					} elseif($str[2] === '~') {
+											$tpl[$c++] = [$container,[TCompositeLiteral::TYPE_EXPRESSION,"\$this->publishFilePath('$this->_contextPath/$literal')"]];
+					} elseif($str[2] === '/') {
+											$tpl[$c++] = [$container,[TCompositeLiteral::TYPE_EXPRESSION,"rtrim(dirname(\$this->getApplication()->getRequest()->getApplicationUrl()), '\/').'/$literal'"]];
+					} elseif($str[2] === '[')
 					{
 						$literal = strtr(trim(substr($literal, 0, strlen($literal) - 1)), ["'" => "\'","\\" => "\\\\"]);
 						$tpl[$c++] = [$container,[TCompositeLiteral::TYPE_EXPRESSION,"Prado::localize('$literal')"]];
 					}
-				}
-				elseif(strpos($str, '<prop:') === 0)	// opening property
+				} elseif(strpos($str, '<prop:') === 0) {
+					// opening property
 				{
 					if(strrpos($str, '/>') === strlen($str) - 2)  //subproperties
 					{
 						if($expectPropEnd)
 							continue;
-						if($matchStart > $textStart)
-							$tpl[$c++] = [$container,substr($input, $textStart, $matchStart - $textStart)];
+				}
+						if($matchStart > $textStart) {
+													$tpl[$c++] = [$container,substr($input, $textStart, $matchStart - $textStart)];
+						}
 						$textStart = $matchEnd + 1;
 						$prop = strtolower($match[6][0]);
 						$attrs = $this->parseAttributes($match[7][0], $match[7][1]);
 						$attributes = [];
-						foreach($attrs as $name => $value)
-							$attributes[$prop . '.' . $name] = $value;
+						foreach($attrs as $name => $value) {
+													$attributes[$prop . '.' . $name] = $value;
+						}
 						$type = $tpl[$container][1];
 						$this->validateAttributes($type, $attributes);
 						foreach($attributes as $name => $value)
 						{
-							if(isset($tpl[$container][2][$name]))
-								throw new TConfigurationException('template_property_duplicated', $name);
+							if(isset($tpl[$container][2][$name])) {
+															throw new TConfigurationException('template_property_duplicated', $name);
+							}
 							$tpl[$container][2][$name] = $value;
 						}
-					}
-					else  // regular property
+					} else  // regular property
 					{
 						$prop = strtolower($match[3][0]);
 						$stack[] = '@' . $prop;
 						if(!$expectPropEnd)
 						{
-							if($matchStart > $textStart)
-								$tpl[$c++] = [$container,substr($input, $textStart, $matchStart - $textStart)];
+							if($matchStart > $textStart) {
+															$tpl[$c++] = [$container,substr($input, $textStart, $matchStart - $textStart)];
+							}
 							$textStart = $matchEnd + 1;
 							$expectPropEnd = true;
 						}
 					}
-				}
-				elseif(strpos($str, '</prop:') === 0)	// closing property
+				} elseif(strpos($str, '</prop:') === 0) {
+					// closing property
 				{
 					$prop = strtolower($match[3][0]);
-					if(empty($stack))
-						throw new TConfigurationException('template_closingtag_unexpected', "</prop:$prop>");
+				}
+					if(empty($stack)) {
+											throw new TConfigurationException('template_closingtag_unexpected', "</prop:$prop>");
+					}
 					$name = array_pop($stack);
 					if($name !== '@' . $prop)
 					{
@@ -608,35 +653,40 @@ 
 						if($matchStart > $textStart)
 						{
 							$value = substr($input, $textStart, $matchStart - $textStart);
-							if(substr($prop, -8, 8) === 'template')
-								$value = $this->parseTemplateProperty($value, $textStart);
-							else
-								$value = $this->parseAttribute($value);
+							if(substr($prop, -8, 8) === 'template') {
+															$value = $this->parseTemplateProperty($value, $textStart);
+							} else {
+															$value = $this->parseAttribute($value);
+							}
 							if($container >= 0)
 							{
 								$type = $tpl[$container][1];
 								$this->validateAttributes($type, [$prop => $value]);
-								if(isset($tpl[$container][2][$prop]))
-									throw new TConfigurationException('template_property_duplicated', $prop);
+								if(isset($tpl[$container][2][$prop])) {
+																	throw new TConfigurationException('template_property_duplicated', $prop);
+								}
 								$tpl[$container][2][$prop] = $value;
-							}
-							else	// a property for the template control
+							} else {
+								// a property for the template control
 								$this->_directive[$prop] = $value;
+							}
 							$textStart = $matchEnd + 1;
 						}
 						$expectPropEnd = false;
 					}
-				}
-				elseif(strpos($str, '<!--') === 0)	// comments
+				} elseif(strpos($str, '<!--') === 0) {
+					// comments
 				{
 					if($expectPropEnd)
 						throw new TConfigurationException('template_comments_forbidden');
-					if($matchStart > $textStart)
-						$tpl[$c++] = [$container,substr($input, $textStart, $matchStart - $textStart)];
+				}
+					if($matchStart > $textStart) {
+											$tpl[$c++] = [$container,substr($input, $textStart, $matchStart - $textStart)];
+					}
 					$textStart = $matchEnd + 1;
+				} else {
+									throw new TConfigurationException('template_matching_unexpected', $match);
 				}
-				else
-					throw new TConfigurationException('template_matching_unexpected', $match);
 			}
 			if(!empty($stack))
 			{
@@ -644,22 +694,25 @@ 
 				$tag = $name[0] === '@' ? '</prop:' . substr($name, 1) . '>' : "</com:$name>";
 				throw new TConfigurationException('template_closingtag_expected', $tag, "nothing");
 			}
-			if($textStart < strlen($input))
-				$tpl[$c++] = [$container,substr($input, $textStart)];
-		}
-		catch(\Exception $e)
+			if($textStart < strlen($input)) {
+							$tpl[$c++] = [$container,substr($input, $textStart)];
+			}
+		} catch(\Exception $e)
 		{
-			if(($e instanceof TException) && ($e instanceof TTemplateException))
-				throw $e;
-			if($matchEnd === 0)
-				$line = $this->_startingLine + 1;
-			else
-				$line = $this->_startingLine + count(explode("\n", substr($input, 0, $matchEnd + 1)));
+			if(($e instanceof TException) && ($e instanceof TTemplateException)) {
+							throw $e;
+			}
+			if($matchEnd === 0) {
+							$line = $this->_startingLine + 1;
+			} else {
+							$line = $this->_startingLine + count(explode("\n", substr($input, 0, $matchEnd + 1)));
+			}
 			$this->handleException($e, $line, $input);
 		}
 
-		if($this->_directive === null)
-			$this->_directive = [];
+		if($this->_directive === null) {
+					$this->_directive = [];
+		}
 
 		// optimization by merging consecutive strings, expressions, statements and bindings
 		$objects = [];
@@ -671,31 +724,32 @@ 
 			{
 				if($parent !== null)
 				{
-					if(count($merged[1]) === 1 && is_string($merged[1][0]))
-						$objects[$id - 1] = [$merged[0],$merged[1][0]];
-					else
-						$objects[$id - 1] = [$merged[0],new TCompositeLiteral($merged[1])];
+					if(count($merged[1]) === 1 && is_string($merged[1][0])) {
+											$objects[$id - 1] = [$merged[0],$merged[1][0]];
+					} else {
+											$objects[$id - 1] = [$merged[0],new TCompositeLiteral($merged[1])];
+					}
 				}
 				if(isset($object[2]))
 				{
 					$parent = null;
 					$objects[$id] = $object;
-				}
-				else
+				} else
 				{
 					$parent = $object[0];
 					$merged = [$parent,[$object[1]]];
 				}
+			} else {
+							$merged[1][] = $object[1];
 			}
-			else
-				$merged[1][] = $object[1];
 		}
 		if($parent !== null)
 		{
-			if(count($merged[1]) === 1 && is_string($merged[1][0]))
-				$objects[$id] = [$merged[0],$merged[1][0]];
-			else
-				$objects[$id] = [$merged[0],new TCompositeLiteral($merged[1])];
+			if(count($merged[1]) === 1 && is_string($merged[1][0])) {
+							$objects[$id] = [$merged[0],$merged[1][0]];
+			} else {
+							$objects[$id] = [$merged[0],new TCompositeLiteral($merged[1])];
+			}
 		}
 		$tpl = $objects;
 		return $objects;
@@ -708,8 +762,9 @@ 
 	 */
 	protected function parseAttributes($str, $offset)
 	{
-		if($str === '')
-			return [];
+		if($str === '') {
+					return [];
+		}
 		$pattern = '/([\w\.\-]+)\s*=\s*(\'.*?\'|".*?"|<%.*?%>)/msS';
 		$attributes = [];
 		$n = preg_match_all($pattern, $str, $matches, PREG_SET_ORDER | PREG_OFFSET_CAPTURE);
@@ -717,22 +772,24 @@ 
 		{
 			$match = &$matches[$i];
 			$name = strtolower($match[1][0]);
-			if(isset($attributes[$name]))
-				throw new TConfigurationException('template_property_duplicated', $name);
+			if(isset($attributes[$name])) {
+							throw new TConfigurationException('template_property_duplicated', $name);
+			}
 			$value = $match[2][0];
 			if(substr($name, -8, 8) === 'template')
 			{
-				if($value[0] === '\'' || $value[0] === '"')
-					$attributes[$name] = $this->parseTemplateProperty(substr($value, 1, strlen($value) - 2), $match[2][1] + 1);
-				else
-					$attributes[$name] = $this->parseTemplateProperty($value, $match[2][1]);
-			}
-			else
+				if($value[0] === '\'' || $value[0] === '"') {
+									$attributes[$name] = $this->parseTemplateProperty(substr($value, 1, strlen($value) - 2), $match[2][1] + 1);
+				} else {
+									$attributes[$name] = $this->parseTemplateProperty($value, $match[2][1]);
+				}
+			} else
 			{
-				if($value[0] === '\'' || $value[0] === '"')
-					$attributes[$name] = $this->parseAttribute(substr($value, 1, strlen($value) - 2));
-				else
-					$attributes[$name] = $this->parseAttribute($value);
+				if($value[0] === '\'' || $value[0] === '"') {
+									$attributes[$name] = $this->parseAttribute(substr($value, 1, strlen($value) - 2));
+				} else {
+									$attributes[$name] = $this->parseAttribute($value);
+				}
 			}
 		}
 		return $attributes;
@@ -762,46 +819,50 @@ 
 				$token = $match[0];
 				$offset = $match[1];
 				$length = strlen($token);
-				if($token[2] === '#')
-					$isDataBind = true;
-				if($offset > $textStart)
-					$expr .= ".'" . strtr(substr($value, $textStart, $offset - $textStart), ["'" => "\\'","\\" => "\\\\"]) . "'";
+				if($token[2] === '#') {
+									$isDataBind = true;
+				}
+				if($offset > $textStart) {
+									$expr .= ".'" . strtr(substr($value, $textStart, $offset - $textStart), ["'" => "\\'","\\" => "\\\\"]) . "'";
+				}
 				$expr .= '.(' . substr($token, 3, $length - 5) . ')';
 				$textStart = $offset + $length;
 			}
 			$length = strlen($value);
-			if($length > $textStart)
-				$expr .= ".'" . strtr(substr($value, $textStart, $length - $textStart), ["'" => "\\'","\\" => "\\\\"]) . "'";
-			if($isDataBind)
-				return [self::CONFIG_DATABIND,ltrim($expr, '.')];
-			else
-				return [self::CONFIG_EXPRESSION,ltrim($expr, '.')];
-		}
-		elseif(preg_match('/\\s*(<%~.*?%>|<%\\$.*?%>|<%\\[.*?\\]%>|<%\/.*?%>)\\s*/msS', $value, $matches) && $matches[0] === $value)
+			if($length > $textStart) {
+							$expr .= ".'" . strtr(substr($value, $textStart, $length - $textStart), ["'" => "\\'","\\" => "\\\\"]) . "'";
+			}
+			if($isDataBind) {
+							return [self::CONFIG_DATABIND,ltrim($expr, '.')];
+			} else {
+							return [self::CONFIG_EXPRESSION,ltrim($expr, '.')];
+			}
+		} elseif(preg_match('/\\s*(<%~.*?%>|<%\\$.*?%>|<%\\[.*?\\]%>|<%\/.*?%>)\\s*/msS', $value, $matches) && $matches[0] === $value)
 		{
 			$value = $matches[1];
-			if($value[2] === '~')
-				return [self::CONFIG_ASSET,trim(substr($value, 3, strlen($value) - 5))];
-			elseif($value[2] === '[')
-				return [self::CONFIG_LOCALIZATION,trim(substr($value, 3, strlen($value) - 6))];
-			elseif($value[2] === '$')
-				return [self::CONFIG_PARAMETER,trim(substr($value, 3, strlen($value) - 5))];
-			elseif($value[2] === '/') {
+			if($value[2] === '~') {
+							return [self::CONFIG_ASSET,trim(substr($value, 3, strlen($value) - 5))];
+			} elseif($value[2] === '[') {
+							return [self::CONFIG_LOCALIZATION,trim(substr($value, 3, strlen($value) - 6))];
+			} elseif($value[2] === '$') {
+							return [self::CONFIG_PARAMETER,trim(substr($value, 3, strlen($value) - 5))];
+			} elseif($value[2] === '/') {
 				$literal = trim(substr($value, 3, strlen($value) - 5));
 				return [self::CONFIG_EXPRESSION,"rtrim(dirname(\$this->getApplication()->getRequest()->getApplicationUrl()), '\/').'/$literal'"];
 			}
+		} else {
+					return $value;
 		}
-		else
-			return $value;
 	}
 
 	protected function validateAttributes($type, $attributes)
 	{
 		Prado::using($type);
-		if(($pos = strrpos($type, '.')) !== false)
-			$className = substr($type, $pos + 1);
-		else
-			$className = $type;
+		if(($pos = strrpos($type, '.')) !== false) {
+					$className = substr($type, $pos + 1);
+		} else {
+					$className = $type;
+		}
 		$class = new \ReflectionClass($className);
 		if(is_subclass_of($className, '\Prado\Web\UI\TControl') || $className === '\Prado\Web\UI\TControl')
 		{
@@ -811,67 +872,69 @@ 
 				{
 					// a subproperty, so the first segment must be readable
 					$subname = substr($name, 0, $pos);
-					if(!$class->hasMethod('get' . $subname))
-						throw new TConfigurationException('template_property_unknown', $type, $subname);
-				}
-				elseif(strncasecmp($name, 'on', 2) === 0)
+					if(!$class->hasMethod('get' . $subname)) {
+											throw new TConfigurationException('template_property_unknown', $type, $subname);
+					}
+				} elseif(strncasecmp($name, 'on', 2) === 0)
 				{
 					// an event
-					if(!$class->hasMethod($name))
-						throw new TConfigurationException('template_event_unknown', $type, $name);
-					elseif(!is_string($att))
-						throw new TConfigurationException('template_eventhandler_invalid', $type, $name);
-				}
-				else
+					if(!$class->hasMethod($name)) {
+											throw new TConfigurationException('template_event_unknown', $type, $name);
+					} elseif(!is_string($att)) {
+											throw new TConfigurationException('template_eventhandler_invalid', $type, $name);
+					}
+				} else
 				{
 					// a simple property
 					if (! ($class->hasMethod('set' . $name) || $class->hasMethod('setjs' . $name) || $this->isClassBehaviorMethod($class, 'set' . $name)))
 					{
-						if ($class->hasMethod('get' . $name) || $class->hasMethod('getjs' . $name))
-							throw new TConfigurationException('template_property_readonly', $type, $name);
-						else
-							throw new TConfigurationException('template_property_unknown', $type, $name);
-					}
-					elseif(is_array($att) && $att[0] !== self::CONFIG_EXPRESSION)
+						if ($class->hasMethod('get' . $name) || $class->hasMethod('getjs' . $name)) {
+													throw new TConfigurationException('template_property_readonly', $type, $name);
+						} else {
+													throw new TConfigurationException('template_property_unknown', $type, $name);
+						}
+					} elseif(is_array($att) && $att[0] !== self::CONFIG_EXPRESSION)
 					{
-						if(strcasecmp($name, 'id') === 0)
-							throw new TConfigurationException('template_controlid_invalid', $type);
-						elseif(strcasecmp($name, 'skinid') === 0)
-							throw new TConfigurationException('template_controlskinid_invalid', $type);
+						if(strcasecmp($name, 'id') === 0) {
+													throw new TConfigurationException('template_controlid_invalid', $type);
+						} elseif(strcasecmp($name, 'skinid') === 0) {
+													throw new TConfigurationException('template_controlskinid_invalid', $type);
+						}
 					}
 				}
 			}
-		}
-		elseif(is_subclass_of($className, '\Prado\TComponent') || $className === '\Prado\TComponent')
+		} elseif(is_subclass_of($className, '\Prado\TComponent') || $className === '\Prado\TComponent')
 		{
 			foreach($attributes as $name => $att)
 			{
-				if(is_array($att) && ($att[0] === self::CONFIG_DATABIND))
-					throw new TConfigurationException('template_databind_forbidden', $type, $name);
+				if(is_array($att) && ($att[0] === self::CONFIG_DATABIND)) {
+									throw new TConfigurationException('template_databind_forbidden', $type, $name);
+				}
 				if(($pos = strpos($name, '.')) !== false)
 				{
 					// a subproperty, so the first segment must be readable
 					$subname = substr($name, 0, $pos);
-					if(!$class->hasMethod('get' . $subname))
-						throw new TConfigurationException('template_property_unknown', $type, $subname);
-				}
-				elseif(strncasecmp($name, 'on', 2) === 0)
-					throw new TConfigurationException('template_event_forbidden', $type, $name);
-				else
+					if(!$class->hasMethod('get' . $subname)) {
+											throw new TConfigurationException('template_property_unknown', $type, $subname);
+					}
+				} elseif(strncasecmp($name, 'on', 2) === 0) {
+									throw new TConfigurationException('template_event_forbidden', $type, $name);
+				} else
 				{
 					// id is still alowed for TComponent, even if id property doesn't exist
 					if(strcasecmp($name, 'id') !== 0 && !($class->hasMethod('set' . $name) || $this->isClassBehaviorMethod($class, 'set' . $name)))
 					{
-						if($class->hasMethod('get' . $name))
-							throw new TConfigurationException('template_property_readonly', $type, $name);
-						else
-							throw new TConfigurationException('template_property_unknown', $type, $name);
+						if($class->hasMethod('get' . $name)) {
+													throw new TConfigurationException('template_property_readonly', $type, $name);
+						} else {
+													throw new TConfigurationException('template_property_unknown', $type, $name);
+						}
 					}
 				}
 			}
+		} else {
+					throw new TConfigurationException('template_component_required', $type);
 		}
-		else
-			throw new TConfigurationException('template_component_required', $type);
 		return $class->getName();
 	}
 
@@ -895,9 +958,12 @@ 
 	{
 		$srcFile = $this->_tplFile;
 
-		if(($n = count($this->_includedFiles)) > 0) // need to adjust error row number and file name
+		if(($n = count($this->_includedFiles)) > 0) {
+			// need to adjust error row number and file name
 		{
-			for($i = $n - 1;$i >= 0;--$i)
+			for($i = $n - 1;
+		}
+		$i >= 0;--$i)
 			{
 				if($this->_includeAtLine[$i] <= $line)
 				{
@@ -906,18 +972,19 @@ 
 						$line = $line - $this->_includeAtLine[$i] + 1;
 						$srcFile = $this->_includedFiles[$i];
 						break;
+					} else {
+											$line = $line - $this->_includeLines[$i] + 1;
 					}
-					else
-						$line = $line - $this->_includeLines[$i] + 1;
 				}
 			}
 		}
 		$exception = new TTemplateException('template_format_invalid', $e->getMessage());
 		$exception->setLineNumber($line);
-		if(!empty($srcFile))
-			$exception->setTemplateFile($srcFile);
-		else
-			$exception->setTemplateSource($input);
+		if(!empty($srcFile)) {
+					$exception->setTemplateFile($srcFile);
+		} else {
+					$exception->setTemplateSource($input);
+		}
 		throw $exception;
 	}
 
@@ -933,9 +1000,9 @@ 
 			for($i = 0;$i < $n;++$i)
 			{
 				$filePath = Prado::getPathOfNamespace(trim($matches[$i][1][0]), TTemplateManager::TEMPLATE_FILE_EXT);
-				if($filePath !== null && is_file($filePath))
-					$this->_includedFiles[] = $filePath;
-				else
+				if($filePath !== null && is_file($filePath)) {
+									$this->_includedFiles[] = $filePath;
+				} else
 				{
 					$errorLine = count(explode("\n", substr($input, 0, $matches[$i][0][1] + 1)));
 					$this->handleException(new TConfigurationException('template_include_invalid', trim($matches[$i][1][0])), $errorLine, $input);
@@ -967,15 +1034,19 @@ 
 	{
 	  $component = new \ReflectionClass('\Prado\TComponent');
 	  $behaviors = $component->getStaticProperties();
-	  if(!isset($behaviors['_um']))
-		return false;
+	  if(!isset($behaviors['_um'])) {
+	  		return false;
+	  }
 	  foreach($behaviors['_um'] as $name => $list)
 	  {
-		if(strtolower($class->getShortName()) !== $name && !$class->isSubclassOf($name)) continue;
+		if(strtolower($class->getShortName()) !== $name && !$class->isSubclassOf($name)) {
+			continue;
+		}
 		foreach($list as $param)
 		{
-		  if(method_exists($param->getBehavior(), $method))
-			return true;
+		  if(method_exists($param->getBehavior(), $method)) {
+		  			return true;
+		  }
 		}
 	  }
 	  return false;

framework/Web/UI/TEmptyControlCollection.php

Spacing +2 added lines, -2 removed lines

@@ -40,7 +40,7 @@
 	 */
 	public function insertAt($index, $item)
 	{
-		if(!is_string($item))  // string is possible if property tag is used. we simply ignore it in this case
-			parent::insertAt($index, $item);  // this will generate an exception in parent implementation
+		if (!is_string($item))  // string is possible if property tag is used. we simply ignore it in this case
+			parent::insertAt($index, $item); // this will generate an exception in parent implementation
 	}
 }
\ No newline at end of file

Braces +5 added lines, -2 removed lines

@@ -40,7 +40,10 @@
 	 */
 	public function insertAt($index, $item)
 	{
-		if(!is_string($item))  // string is possible if property tag is used. we simply ignore it in this case
-			parent::insertAt($index, $item);  // this will generate an exception in parent implementation
+		if(!is_string($item)) {
+			// string is possible if property tag is used. we simply ignore it in this case
+			parent::insertAt($index, $item);
+		}
+		// this will generate an exception in parent implementation
 	}
 }
\ No newline at end of file

framework/Web/UI/TCachePageStatePersister.php

Spacing +9 added lines, -9 removed lines

@@ -110,15 +110,15 @@ 
 	 */
 	public function getCache()
 	{
-		if($this->_cache === null)
+		if ($this->_cache === null)
 		{
-			if($this->_cacheModuleID !== '')
+			if ($this->_cacheModuleID !== '')
 				$cache = Prado::getApplication()->getModule($this->_cacheModuleID);
 			else
 				$cache = Prado::getApplication()->getCache();
-			if($cache === null || !($cache instanceof ICache))
+			if ($cache === null || !($cache instanceof ICache))
 			{
-				if($this->_cacheModuleID !== '')
+				if ($this->_cacheModuleID !== '')
 					throw new TConfigurationException('cachepagestatepersister_cachemoduleid_invalid', $this->_cacheModuleID);
 				else
 					throw new TConfigurationException('cachepagestatepersister_cache_required');
@@ -142,7 +142,7 @@ 
 	 */
 	public function setCacheTimeout($value)
 	{
-		if(($value = TPropertyValue::ensureInteger($value)) >= 0)
+		if (($value = TPropertyValue::ensureInteger($value)) >= 0)
 			$this->_timeout = $value;
 		else
 			throw new TInvalidDataValueException('cachepagestatepersister_timeout_invalid');
@@ -170,7 +170,7 @@ 
 	 */
 	protected function calculateKey($timestamp)
 	{
-		return $this->getKeyPrefix() . ':'
+		return $this->getKeyPrefix().':'
 			. $this->_page->getRequest()->getUserHostAddress()
 			. $this->_page->getPagePath()
 			. $timestamp;
@@ -182,7 +182,7 @@ 
 	 */
 	public function save($data)
 	{
-		$timestamp = (string)microtime(true);
+		$timestamp = (string) microtime(true);
 		$key = $this->calculateKey($timestamp);
 		$this->getCache()->add($key, $data, $this->_timeout);
 		$this->_page->setClientState(TPageStateFormatter::serialize($this->_page, $timestamp));
@@ -195,10 +195,10 @@ 
 	 */
 	public function load()
 	{
-		if(($timestamp = TPageStateFormatter::unserialize($this->_page, $this->_page->getRequestClientState())) !== null)
+		if (($timestamp = TPageStateFormatter::unserialize($this->_page, $this->_page->getRequestClientState())) !== null)
 		{
 			$key = $this->calculateKey($timestamp);
-			if(($data = $this->getCache()->get($key)) !== false)
+			if (($data = $this->getCache()->get($key)) !== false)
 				return $data;
 		}
 		throw new THttpException(400, 'cachepagestatepersister_pagestate_corrupted');

Braces +18 added lines, -14 removed lines

@@ -112,16 +112,18 @@ 
 	{
 		if($this->_cache === null)
 		{
-			if($this->_cacheModuleID !== '')
-				$cache = Prado::getApplication()->getModule($this->_cacheModuleID);
-			else
-				$cache = Prado::getApplication()->getCache();
+			if($this->_cacheModuleID !== '') {
+							$cache = Prado::getApplication()->getModule($this->_cacheModuleID);
+			} else {
+							$cache = Prado::getApplication()->getCache();
+			}
 			if($cache === null || !($cache instanceof ICache))
 			{
-				if($this->_cacheModuleID !== '')
-					throw new TConfigurationException('cachepagestatepersister_cachemoduleid_invalid', $this->_cacheModuleID);
-				else
-					throw new TConfigurationException('cachepagestatepersister_cache_required');
+				if($this->_cacheModuleID !== '') {
+									throw new TConfigurationException('cachepagestatepersister_cachemoduleid_invalid', $this->_cacheModuleID);
+				} else {
+									throw new TConfigurationException('cachepagestatepersister_cache_required');
+				}
 			}
 			$this->_cache = $cache;
 		}
@@ -142,10 +144,11 @@ 
 	 */
 	public function setCacheTimeout($value)
 	{
-		if(($value = TPropertyValue::ensureInteger($value)) >= 0)
-			$this->_timeout = $value;
-		else
-			throw new TInvalidDataValueException('cachepagestatepersister_timeout_invalid');
+		if(($value = TPropertyValue::ensureInteger($value)) >= 0) {
+					$this->_timeout = $value;
+		} else {
+					throw new TInvalidDataValueException('cachepagestatepersister_timeout_invalid');
+		}
 	}
 
 	/**
@@ -198,8 +201,9 @@ 
 		if(($timestamp = TPageStateFormatter::unserialize($this->_page, $this->_page->getRequestClientState())) !== null)
 		{
 			$key = $this->calculateKey($timestamp);
-			if(($data = $this->getCache()->get($key)) !== false)
-				return $data;
+			if(($data = $this->getCache()->get($key)) !== false) {
+							return $data;
+			}
 		}
 		throw new THttpException(400, 'cachepagestatepersister_pagestate_corrupted');
 	}

framework/Web/UI/TPageStateFormatter.php

Spacing +9 added lines, -9 removed lines

@@ -38,13 +38,13 @@ 
 	public static function serialize($page, $data)
 	{
 		$sm = $page->getApplication()->getSecurityManager();
-		if($page->getEnableStateValidation())
+		if ($page->getEnableStateValidation())
 			$str = $sm->hashData(serialize($data));
 		else
 			$str = serialize($data);
-		if($page->getEnableStateCompression() && extension_loaded('zlib'))
+		if ($page->getEnableStateCompression() && extension_loaded('zlib'))
 			$str = gzcompress($str);
-		if($page->getEnableStateEncryption())
+		if ($page->getEnableStateEncryption())
 			$str = $sm->encrypt($str);
 		return base64_encode($str);
 	}
@@ -57,18 +57,18 @@ 
 	public static function unserialize($page, $data)
 	{
 		$str = base64_decode($data);
-		if($str === '')
+		if ($str === '')
 			return null;
-		if($str !== false)
+		if ($str !== false)
 		{
 			$sm = $page->getApplication()->getSecurityManager();
-			if($page->getEnableStateEncryption())
+			if ($page->getEnableStateEncryption())
 				$str = $sm->decrypt($str);
-			if($page->getEnableStateCompression() && extension_loaded('zlib'))
+			if ($page->getEnableStateCompression() && extension_loaded('zlib'))
 				$str = @gzuncompress($str);
-			if($page->getEnableStateValidation())
+			if ($page->getEnableStateValidation())
 			{
-				if(($str = $sm->validateData($str)) !== false)
+				if (($str = $sm->validateData($str)) !== false)
 					return unserialize($str);
 			}
 			else

Braces +25 added lines, -18 removed lines

@@ -38,14 +38,17 @@ 
 	public static function serialize($page, $data)
 	{
 		$sm = $page->getApplication()->getSecurityManager();
-		if($page->getEnableStateValidation())
-			$str = $sm->hashData(serialize($data));
-		else
-			$str = serialize($data);
-		if($page->getEnableStateCompression() && extension_loaded('zlib'))
-			$str = gzcompress($str);
-		if($page->getEnableStateEncryption())
-			$str = $sm->encrypt($str);
+		if($page->getEnableStateValidation()) {
+					$str = $sm->hashData(serialize($data));
+		} else {
+					$str = serialize($data);
+		}
+		if($page->getEnableStateCompression() && extension_loaded('zlib')) {
+					$str = gzcompress($str);
+		}
+		if($page->getEnableStateEncryption()) {
+					$str = $sm->encrypt($str);
+		}
 		return base64_encode($str);
 	}
 
@@ -57,22 +60,26 @@ 
 	public static function unserialize($page, $data)
 	{
 		$str = base64_decode($data);
-		if($str === '')
-			return null;
+		if($str === '') {
+					return null;
+		}
 		if($str !== false)
 		{
 			$sm = $page->getApplication()->getSecurityManager();
-			if($page->getEnableStateEncryption())
-				$str = $sm->decrypt($str);
-			if($page->getEnableStateCompression() && extension_loaded('zlib'))
-				$str = @gzuncompress($str);
+			if($page->getEnableStateEncryption()) {
+							$str = $sm->decrypt($str);
+			}
+			if($page->getEnableStateCompression() && extension_loaded('zlib')) {
+							$str = @gzuncompress($str);
+			}
 			if($page->getEnableStateValidation())
 			{
-				if(($str = $sm->validateData($str)) !== false)
-					return unserialize($str);
+				if(($str = $sm->validateData($str)) !== false) {
+									return unserialize($str);
+				}
+			} else {
+							return unserialize($str);
 			}
-			else
-				return unserialize($str);
 		}
 		return null;
 	}

framework/Web/UI/WebControls/THotSpot.php

Spacing +15 added lines, -15 removed lines

@@ -40,7 +40,7 @@ 
 	 */
 	protected function getViewState($key, $defaultValue = null)
 	{
-		return isset($this->_viewState[$key])?$this->_viewState[$key]:$defaultValue;
+		return isset($this->_viewState[$key]) ? $this->_viewState[$key] : $defaultValue;
 	}
 
 	/**
@@ -55,7 +55,7 @@ 
 	 */
 	protected function setViewState($key, $value, $defaultValue = null)
 	{
-		if($value === $defaultValue)
+		if ($value === $defaultValue)
 			unset($this->_viewState[$key]);
 		else
 			$this->_viewState[$key] = $value;
@@ -221,7 +221,7 @@ 
 	 */
 	public function getHasAttributes()
 	{
-		if($attributes = $this->getViewState('Attributes', null))
+		if ($attributes = $this->getViewState('Attributes', null))
 			return $attributes->getCount() > 0;
 		else
 			return false;
@@ -235,7 +235,7 @@ 
 	 */
 	public function getAttributes()
 	{
-		if($attributes = $this->getViewState('Attributes', null))
+		if ($attributes = $this->getViewState('Attributes', null))
 			return $attributes;
 		else
 		{
@@ -250,7 +250,7 @@ 
 	 */
 	public function hasAttribute($name)
 	{
-		if($attributes = $this->getViewState('Attributes', null))
+		if ($attributes = $this->getViewState('Attributes', null))
 			return $attributes->contains($name);
 		else
 			return false;
@@ -261,7 +261,7 @@ 
 	 */
 	public function getAttribute($name)
 	{
-		if($attributes = $this->getViewState('Attributes', null))
+		if ($attributes = $this->getViewState('Attributes', null))
 			return $attributes->itemAt($name);
 		else
 			return null;
@@ -284,7 +284,7 @@ 
 	 */
 	public function removeAttribute($name)
 	{
-		if($attributes = $this->getViewState('Attributes', null))
+		if ($attributes = $this->getViewState('Attributes', null))
 			return $attributes->remove($name);
 		else
 			return null;
@@ -298,26 +298,26 @@ 
 	{
 		$writer->addAttribute('shape', $this->getShape());
 		$writer->addAttribute('coords', $this->getCoordinates());
-		if(($mode = $this->getHotSpotMode()) === THotSpotMode::NotSet)
+		if (($mode = $this->getHotSpotMode()) === THotSpotMode::NotSet)
 			$mode = THotSpotMode::Navigate;
-		if($mode === THotSpotMode::Navigate)
+		if ($mode === THotSpotMode::Navigate)
 		{
 			$writer->addAttribute('href', $this->getNavigateUrl());
-			if(($target = $this->getTarget()) !== '')
+			if (($target = $this->getTarget()) !== '')
 				$writer->addAttribute('target', $target);
 		}
-		elseif($mode === THotSpotMode::Inactive)
+		elseif ($mode === THotSpotMode::Inactive)
 			$writer->addAttribute('nohref', 'true');
 		$text = $this->getAlternateText();
 		$writer->addAttribute('title', $text);
 		$writer->addAttribute('alt', $text);
-		if(($accessKey = $this->getAccessKey()) !== '')
+		if (($accessKey = $this->getAccessKey()) !== '')
 			$writer->addAttribute('accesskey', $accessKey);
-		if(($tabIndex = $this->getTabIndex()) !== 0)
+		if (($tabIndex = $this->getTabIndex()) !== 0)
 			$writer->addAttribute('tabindex', "$tabIndex");
-		if($this->getHasAttributes())
+		if ($this->getHasAttributes())
 		{
-			foreach($this->getAttributes() as $name => $value)
+			foreach ($this->getAttributes() as $name => $value)
 				$writer->addAttribute($name, $value);
 		}
 		$writer->renderBeginTag('area');

Braces +45 added lines, -35 removed lines

@@ -55,10 +55,11 @@ 
 	 */
 	protected function setViewState($key, $value, $defaultValue = null)
 	{
-		if($value === $defaultValue)
-			unset($this->_viewState[$key]);
-		else
-			$this->_viewState[$key] = $value;
+		if($value === $defaultValue) {
+					unset($this->_viewState[$key]);
+		} else {
+					$this->_viewState[$key] = $value;
+		}
 	}
 
 	/**
@@ -221,10 +222,11 @@ 
 	 */
 	public function getHasAttributes()
 	{
-		if($attributes = $this->getViewState('Attributes', null))
-			return $attributes->getCount() > 0;
-		else
-			return false;
+		if($attributes = $this->getViewState('Attributes', null)) {
+					return $attributes->getCount() > 0;
+		} else {
+					return false;
+		}
 	}
 
 	/**
@@ -235,9 +237,9 @@ 
 	 */
 	public function getAttributes()
 	{
-		if($attributes = $this->getViewState('Attributes', null))
-			return $attributes;
-		else
+		if($attributes = $this->getViewState('Attributes', null)) {
+					return $attributes;
+		} else
 		{
 			$attributes = new TAttributeCollection;
 			$this->setViewState('Attributes', $attributes, null);
@@ -250,10 +252,11 @@ 
 	 */
 	public function hasAttribute($name)
 	{
-		if($attributes = $this->getViewState('Attributes', null))
-			return $attributes->contains($name);
-		else
-			return false;
+		if($attributes = $this->getViewState('Attributes', null)) {
+					return $attributes->contains($name);
+		} else {
+					return false;
+		}
 	}
 
 	/**
@@ -261,10 +264,11 @@ 
 	 */
 	public function getAttribute($name)
 	{
-		if($attributes = $this->getViewState('Attributes', null))
-			return $attributes->itemAt($name);
-		else
-			return null;
+		if($attributes = $this->getViewState('Attributes', null)) {
+					return $attributes->itemAt($name);
+		} else {
+					return null;
+		}
 	}
 
 	/**
@@ -284,10 +288,11 @@ 
 	 */
 	public function removeAttribute($name)
 	{
-		if($attributes = $this->getViewState('Attributes', null))
-			return $attributes->remove($name);
-		else
-			return null;
+		if($attributes = $this->getViewState('Attributes', null)) {
+					return $attributes->remove($name);
+		} else {
+					return null;
+		}
 	}
 
 	/**
@@ -298,27 +303,32 @@ 
 	{
 		$writer->addAttribute('shape', $this->getShape());
 		$writer->addAttribute('coords', $this->getCoordinates());
-		if(($mode = $this->getHotSpotMode()) === THotSpotMode::NotSet)
-			$mode = THotSpotMode::Navigate;
+		if(($mode = $this->getHotSpotMode()) === THotSpotMode::NotSet) {
+					$mode = THotSpotMode::Navigate;
+		}
 		if($mode === THotSpotMode::Navigate)
 		{
 			$writer->addAttribute('href', $this->getNavigateUrl());
-			if(($target = $this->getTarget()) !== '')
-				$writer->addAttribute('target', $target);
+			if(($target = $this->getTarget()) !== '') {
+							$writer->addAttribute('target', $target);
+			}
+		} elseif($mode === THotSpotMode::Inactive) {
+					$writer->addAttribute('nohref', 'true');
 		}
-		elseif($mode === THotSpotMode::Inactive)
-			$writer->addAttribute('nohref', 'true');
 		$text = $this->getAlternateText();
 		$writer->addAttribute('title', $text);
 		$writer->addAttribute('alt', $text);
-		if(($accessKey = $this->getAccessKey()) !== '')
-			$writer->addAttribute('accesskey', $accessKey);
-		if(($tabIndex = $this->getTabIndex()) !== 0)
-			$writer->addAttribute('tabindex', "$tabIndex");
+		if(($accessKey = $this->getAccessKey()) !== '') {
+					$writer->addAttribute('accesskey', $accessKey);
+		}
+		if(($tabIndex = $this->getTabIndex()) !== 0) {
+					$writer->addAttribute('tabindex', "$tabIndex");
+		}
 		if($this->getHasAttributes())
 		{
-			foreach($this->getAttributes() as $name => $value)
-				$writer->addAttribute($name, $value);
+			foreach($this->getAttributes() as $name => $value) {
+							$writer->addAttribute($name, $value);
+			}
 		}
 		$writer->renderBeginTag('area');
 		$writer->renderEndTag();

framework/Web/UI/WebControls/TStyle.php

Spacing +35 added lines, -35 removed lines

@@ -56,9 +56,9 @@ 
 		parent::_getZappableSleepProps($exprops);
 		if ($this->_fields === [])
 			$exprops[] = "\0Prado\Web\UI\WebControls\TStyle\0_fields";
-		if($this->_font === null)
+		if ($this->_font === null)
 			$exprops[] = "\0Prado\Web\UI\WebControls\TStyle\0_font";
-		if($this->_class === null)
+		if ($this->_class === null)
 			$exprops[] = "\0Prado\Web\UI\WebControls\TStyle\0_class";
 		if ($this->_customStyle === null)
 			$exprops[] = "\0Prado\Web\UI\WebControls\TStyle\0_customStyle";
@@ -73,7 +73,7 @@ 
 	public function __construct($style = null)
 	{
 	parent::__construct();
-		if($style !== null)
+		if ($style !== null)
 			$this->copyFrom($style);
 	}
 
@@ -82,7 +82,7 @@ 
 	 */
 	public function __clone()
 	{
-		if($this->_font !== null)
+		if ($this->_font !== null)
 			$this->_font = clone($this->_font);
 	}
 
@@ -91,7 +91,7 @@ 
 	 */
 	public function getBackColor()
 	{
-		return isset($this->_fields['background-color'])?$this->_fields['background-color']:'';
+		return isset($this->_fields['background-color']) ? $this->_fields['background-color'] : '';
 	}
 
 	/**
@@ -99,7 +99,7 @@ 
 	 */
 	public function setBackColor($value)
 	{
-		if(trim($value) === '')
+		if (trim($value) === '')
 			unset($this->_fields['background-color']);
 		else
 			$this->_fields['background-color'] = $value;
@@ -110,7 +110,7 @@ 
 	 */
 	public function getBorderColor()
 	{
-		return isset($this->_fields['border-color'])?$this->_fields['border-color']:'';
+		return isset($this->_fields['border-color']) ? $this->_fields['border-color'] : '';
 	}
 
 	/**
@@ -118,7 +118,7 @@ 
 	 */
 	public function setBorderColor($value)
 	{
-		if(trim($value) === '')
+		if (trim($value) === '')
 			unset($this->_fields['border-color']);
 		else
 			$this->_fields['border-color'] = $value;
@@ -129,7 +129,7 @@ 
 	 */
 	public function getBorderStyle()
 	{
-		return isset($this->_fields['border-style'])?$this->_fields['border-style']:'';
+		return isset($this->_fields['border-style']) ? $this->_fields['border-style'] : '';
 	}
 
 	/**
@@ -138,7 +138,7 @@ 
 	 */
 	public function setBorderStyle($value)
 	{
-		if(trim($value) === '')
+		if (trim($value) === '')
 			unset($this->_fields['border-style']);
 		else
 			$this->_fields['border-style'] = $value;
@@ -149,7 +149,7 @@ 
 	 */
 	public function getBorderWidth()
 	{
-		return isset($this->_fields['border-width'])?$this->_fields['border-width']:'';
+		return isset($this->_fields['border-width']) ? $this->_fields['border-width'] : '';
 	}
 
 	/**
@@ -157,7 +157,7 @@ 
 	 */
 	public function setBorderWidth($value)
 	{
-		if(trim($value) === '')
+		if (trim($value) === '')
 			unset($this->_fields['border-width']);
 		else
 			$this->_fields['border-width'] = $value;
@@ -168,7 +168,7 @@ 
 	 */
 	public function getCssClass()
 	{
-		return $this->_class === null?'':$this->_class;
+		return $this->_class === null ? '' : $this->_class;
 	}
 
 	/**
@@ -192,7 +192,7 @@ 
 	 */
 	public function getFont()
 	{
-		if($this->_font === null)
+		if ($this->_font === null)
 			$this->_font = new TFont;
 		return $this->_font;
 	}
@@ -211,7 +211,7 @@ 
 	public function setDisplayStyle($value)
 	{
 		$this->_displayStyle = TPropertyValue::ensureEnum($value, 'Prado\\Web\\UI\\WebControls\\TDisplayStyle');
-		switch($this->_displayStyle)
+		switch ($this->_displayStyle)
 		{
 			case TDisplayStyle::None:
 				$this->_fields['display'] = 'none';
@@ -241,7 +241,7 @@ 
 	 */
 	public function getForeColor()
 	{
-		return isset($this->_fields['color'])?$this->_fields['color']:'';
+		return isset($this->_fields['color']) ? $this->_fields['color'] : '';
 	}
 
 	/**
@@ -249,7 +249,7 @@ 
 	 */
 	public function setForeColor($value)
 	{
-		if(trim($value) === '')
+		if (trim($value) === '')
 			unset($this->_fields['color']);
 		else
 			$this->_fields['color'] = $value;
@@ -260,7 +260,7 @@ 
 	 */
 	public function getHeight()
 	{
-		return isset($this->_fields['height'])?$this->_fields['height']:'';
+		return isset($this->_fields['height']) ? $this->_fields['height'] : '';
 	}
 
 	/**
@@ -268,7 +268,7 @@ 
 	 */
 	public function setHeight($value)
 	{
-		if(trim($value) === '')
+		if (trim($value) === '')
 			unset($this->_fields['height']);
 		else
 			$this->_fields['height'] = $value;
@@ -279,7 +279,7 @@ 
 	 */
 	public function getCustomStyle()
 	{
-		return $this->_customStyle === null?'':$this->_customStyle;
+		return $this->_customStyle === null ? '' : $this->_customStyle;
 	}
 
 	/**
@@ -297,7 +297,7 @@ 
 	 */
 	public function getStyleField($name)
 	{
-		return isset($this->_fields[$name])?$this->_fields[$name]:'';
+		return isset($this->_fields[$name]) ? $this->_fields[$name] : '';
 	}
 
 	/**
@@ -333,7 +333,7 @@ 
 	 */
 	public function getWidth()
 	{
-		return isset($this->_fields['width'])?$this->_fields['width']:'';
+		return isset($this->_fields['width']) ? $this->_fields['width'] : '';
 	}
 
 	/**
@@ -363,14 +363,14 @@ 
 	 */
 	public function copyFrom($style)
 	{
-		if($style instanceof TStyle)
+		if ($style instanceof TStyle)
 		{
 			$this->_fields = array_merge($this->_fields, $style->_fields);
-			if($style->_class !== null)
+			if ($style->_class !== null)
 				$this->_class = $style->_class;
-			if($style->_customStyle !== null)
+			if ($style->_customStyle !== null)
 				$this->_customStyle = $style->_customStyle;
-			if($style->_font !== null)
+			if ($style->_font !== null)
 				$this->getFont()->copyFrom($style->_font);
 		}
 	}
@@ -383,14 +383,14 @@ 
 	 */
 	public function mergeWith($style)
 	{
-		if($style instanceof TStyle)
+		if ($style instanceof TStyle)
 		{
 			$this->_fields = array_merge($style->_fields, $this->_fields);
-			if($this->_class === null)
+			if ($this->_class === null)
 				$this->_class = $style->_class;
-			if($this->_customStyle === null)
+			if ($this->_customStyle === null)
 				$this->_customStyle = $style->_customStyle;
-			if($style->_font !== null)
+			if ($style->_font !== null)
 				$this->getFont()->mergeWith($style->_font);
 		}
 	}
@@ -401,19 +401,19 @@ 
 	 */
 	public function addAttributesToRender($writer)
 	{
-		if($this->_customStyle !== null)
+		if ($this->_customStyle !== null)
 		{
-			foreach(explode(';', $this->_customStyle) as $style)
+			foreach (explode(';', $this->_customStyle) as $style)
 			{
 				$arr = explode(':', $style, 2);
-				if(isset($arr[1]) && trim($arr[0]) !== '')
+				if (isset($arr[1]) && trim($arr[0]) !== '')
 					$writer->addStyleAttribute(trim($arr[0]), trim($arr[1]));
 			}
 		}
 		$writer->addStyleAttributes($this->_fields);
-		if($this->_font !== null)
+		if ($this->_font !== null)
 			$this->_font->addAttributesToRender($writer);
-		if($this->_class !== null)
+		if ($this->_class !== null)
 			$writer->addAttribute('class', $this->_class);
 	}
 

Braces +81 added lines, -58 removed lines

@@ -54,16 +54,21 @@ 
 	protected function _getZappableSleepProps(&$exprops)
 	{
 		parent::_getZappableSleepProps($exprops);
-		if ($this->_fields === [])
-			$exprops[] = "\0Prado\Web\UI\WebControls\TStyle\0_fields";
-		if($this->_font === null)
-			$exprops[] = "\0Prado\Web\UI\WebControls\TStyle\0_font";
-		if($this->_class === null)
-			$exprops[] = "\0Prado\Web\UI\WebControls\TStyle\0_class";
-		if ($this->_customStyle === null)
-			$exprops[] = "\0Prado\Web\UI\WebControls\TStyle\0_customStyle";
-		if ($this->_displayStyle === 'Fixed')
-			$exprops[] = "\0Prado\Web\UI\WebControls\TStyle\0_displayStyle";
+		if ($this->_fields === []) {
+					$exprops[] = "\0Prado\Web\UI\WebControls\TStyle\0_fields";
+		}
+		if($this->_font === null) {
+					$exprops[] = "\0Prado\Web\UI\WebControls\TStyle\0_font";
+		}
+		if($this->_class === null) {
+					$exprops[] = "\0Prado\Web\UI\WebControls\TStyle\0_class";
+		}
+		if ($this->_customStyle === null) {
+					$exprops[] = "\0Prado\Web\UI\WebControls\TStyle\0_customStyle";
+		}
+		if ($this->_displayStyle === 'Fixed') {
+					$exprops[] = "\0Prado\Web\UI\WebControls\TStyle\0_displayStyle";
+		}
 	}
 
 	/**
@@ -73,8 +78,9 @@ 
 	public function __construct($style = null)
 	{
 	parent::__construct();
-		if($style !== null)
-			$this->copyFrom($style);
+		if($style !== null) {
+					$this->copyFrom($style);
+		}
 	}
 
 	/**
@@ -82,8 +88,9 @@ 
 	 */
 	public function __clone()
 	{
-		if($this->_font !== null)
-			$this->_font = clone($this->_font);
+		if($this->_font !== null) {
+					$this->_font = clone($this->_font);
+		}
 	}
 
 	/**
@@ -99,10 +106,11 @@ 
 	 */
 	public function setBackColor($value)
 	{
-		if(trim($value) === '')
-			unset($this->_fields['background-color']);
-		else
-			$this->_fields['background-color'] = $value;
+		if(trim($value) === '') {
+					unset($this->_fields['background-color']);
+		} else {
+					$this->_fields['background-color'] = $value;
+		}
 	}
 
 	/**
@@ -118,10 +126,11 @@ 
 	 */
 	public function setBorderColor($value)
 	{
-		if(trim($value) === '')
-			unset($this->_fields['border-color']);
-		else
-			$this->_fields['border-color'] = $value;
+		if(trim($value) === '') {
+					unset($this->_fields['border-color']);
+		} else {
+					$this->_fields['border-color'] = $value;
+		}
 	}
 
 	/**
@@ -138,10 +147,11 @@ 
 	 */
 	public function setBorderStyle($value)
 	{
-		if(trim($value) === '')
-			unset($this->_fields['border-style']);
-		else
-			$this->_fields['border-style'] = $value;
+		if(trim($value) === '') {
+					unset($this->_fields['border-style']);
+		} else {
+					$this->_fields['border-style'] = $value;
+		}
 	}
 
 	/**
@@ -157,10 +167,11 @@ 
 	 */
 	public function setBorderWidth($value)
 	{
-		if(trim($value) === '')
-			unset($this->_fields['border-width']);
-		else
-			$this->_fields['border-width'] = $value;
+		if(trim($value) === '') {
+					unset($this->_fields['border-width']);
+		} else {
+					$this->_fields['border-width'] = $value;
+		}
 	}
 
 	/**
@@ -192,8 +203,9 @@ 
 	 */
 	public function getFont()
 	{
-		if($this->_font === null)
-			$this->_font = new TFont;
+		if($this->_font === null) {
+					$this->_font = new TFont;
+		}
 		return $this->_font;
 	}
 
@@ -249,10 +261,11 @@ 
 	 */
 	public function setForeColor($value)
 	{
-		if(trim($value) === '')
-			unset($this->_fields['color']);
-		else
-			$this->_fields['color'] = $value;
+		if(trim($value) === '') {
+					unset($this->_fields['color']);
+		} else {
+					$this->_fields['color'] = $value;
+		}
 	}
 
 	/**
@@ -268,10 +281,11 @@ 
 	 */
 	public function setHeight($value)
 	{
-		if(trim($value) === '')
-			unset($this->_fields['height']);
-		else
-			$this->_fields['height'] = $value;
+		if(trim($value) === '') {
+					unset($this->_fields['height']);
+		} else {
+					$this->_fields['height'] = $value;
+		}
 	}
 
 	/**
@@ -366,12 +380,15 @@ 
 		if($style instanceof TStyle)
 		{
 			$this->_fields = array_merge($this->_fields, $style->_fields);
-			if($style->_class !== null)
-				$this->_class = $style->_class;
-			if($style->_customStyle !== null)
-				$this->_customStyle = $style->_customStyle;
-			if($style->_font !== null)
-				$this->getFont()->copyFrom($style->_font);
+			if($style->_class !== null) {
+							$this->_class = $style->_class;
+			}
+			if($style->_customStyle !== null) {
+							$this->_customStyle = $style->_customStyle;
+			}
+			if($style->_font !== null) {
+							$this->getFont()->copyFrom($style->_font);
+			}
 		}
 	}
 
@@ -386,12 +403,15 @@ 
 		if($style instanceof TStyle)
 		{
 			$this->_fields = array_merge($style->_fields, $this->_fields);
-			if($this->_class === null)
-				$this->_class = $style->_class;
-			if($this->_customStyle === null)
-				$this->_customStyle = $style->_customStyle;
-			if($style->_font !== null)
-				$this->getFont()->mergeWith($style->_font);
+			if($this->_class === null) {
+							$this->_class = $style->_class;
+			}
+			if($this->_customStyle === null) {
+							$this->_customStyle = $style->_customStyle;
+			}
+			if($style->_font !== null) {
+							$this->getFont()->mergeWith($style->_font);
+			}
 		}
 	}
 
@@ -406,15 +426,18 @@ 
 			foreach(explode(';', $this->_customStyle) as $style)
 			{
 				$arr = explode(':', $style, 2);
-				if(isset($arr[1]) && trim($arr[0]) !== '')
-					$writer->addStyleAttribute(trim($arr[0]), trim($arr[1]));
+				if(isset($arr[1]) && trim($arr[0]) !== '') {
+									$writer->addStyleAttribute(trim($arr[0]), trim($arr[1]));
+				}
 			}
 		}
 		$writer->addStyleAttributes($this->_fields);
-		if($this->_font !== null)
-			$this->_font->addAttributesToRender($writer);
-		if($this->_class !== null)
-			$writer->addAttribute('class', $this->_class);
+		if($this->_font !== null) {
+					$this->_font->addAttributesToRender($writer);
+		}
+		if($this->_class !== null) {
+					$writer->addAttribute('class', $this->_class);
+		}
 	}
 
 	/**

framework/Web/UI/WebControls/TWizardNavigationTemplate.php

Spacing +1 added lines, -1 removed lines

@@ -62,7 +62,7 @@
 	 */
 	protected function createNavigationButton($buttonStyle, $causesValidation, $commandName)
 	{
-		switch($buttonStyle->getButtonType())
+		switch ($buttonStyle->getButtonType())
 		{
 			case TWizardNavigationButtonType::Button:
 				$button = new TButton;

framework/Web/UI/WebControls/TContentPlaceHolder.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
 	 */
 	public function createdOnTemplate($parent)
 	{
-		if(($id = $this->getID()) === '')
+		if (($id = $this->getID()) === '')
 			throw new TConfigurationException('contentplaceholder_id_required');
 		$this->getTemplateControl()->registerContentPlaceHolder($id, $this);
 		$parent->getControls()->add($this);

Braces +3 added lines, -2 removed lines

@@ -41,8 +41,9 @@
 	 */
 	public function createdOnTemplate($parent)
 	{
-		if(($id = $this->getID()) === '')
-			throw new TConfigurationException('contentplaceholder_id_required');
+		if(($id = $this->getID()) === '') {
+					throw new TConfigurationException('contentplaceholder_id_required');
+		}
 		$this->getTemplateControl()->registerContentPlaceHolder($id, $this);
 		$parent->getControls()->add($this);
 	}

framework/Web/UI/WebControls/TDataSourceSelectParameters.php

Spacing +3 added lines, -3 removed lines

@@ -34,7 +34,7 @@ 
 
 	public function setStartRowIndex($value)
 	{
-		if(($value = TPropertyValue::ensureInteger($value)) < 0)
+		if (($value = TPropertyValue::ensureInteger($value)) < 0)
 			$value = 0;
 		$this->_startRowIndex = $value;
 	}
@@ -46,7 +46,7 @@ 
 
 	public function setMaximumRows($value)
 	{
-		if(($value = TPropertyValue::ensureInteger($value)) < 0)
+		if (($value = TPropertyValue::ensureInteger($value)) < 0)
 			$value = 0;
 		$this->_maximumRows = $value;
 	}
@@ -68,7 +68,7 @@ 
 
 	public function setTotalRowCount($value)
 	{
-		if(($value = TPropertyValue::ensureInteger($value)) < 0)
+		if (($value = TPropertyValue::ensureInteger($value)) < 0)
 			$value = 0;
 		$this->_totalRowCount = $value;
 	}

Braces +9 added lines, -6 removed lines

@@ -34,8 +34,9 @@ 
 
 	public function setStartRowIndex($value)
 	{
-		if(($value = TPropertyValue::ensureInteger($value)) < 0)
-			$value = 0;
+		if(($value = TPropertyValue::ensureInteger($value)) < 0) {
+					$value = 0;
+		}
 		$this->_startRowIndex = $value;
 	}
 
@@ -46,8 +47,9 @@ 
 
 	public function setMaximumRows($value)
 	{
-		if(($value = TPropertyValue::ensureInteger($value)) < 0)
-			$value = 0;
+		if(($value = TPropertyValue::ensureInteger($value)) < 0) {
+					$value = 0;
+		}
 		$this->_maximumRows = $value;
 	}
 
@@ -68,8 +70,9 @@ 
 
 	public function setTotalRowCount($value)
 	{
-		if(($value = TPropertyValue::ensureInteger($value)) < 0)
-			$value = 0;
+		if(($value = TPropertyValue::ensureInteger($value)) < 0) {
+					$value = 0;
+		}
 		$this->_totalRowCount = $value;
 	}
 }
\ No newline at end of file