pradosoft / prado

framework/Web/UI/TCompositeLiteral.php

Braces +10 added lines, -8 removed lines

@@ -52,8 +52,7 @@ 
 				elseif($item[0] === self::TYPE_DATABINDING)
 					$this->_bindings[$id] = $item[1];
 				$this->_items[$id] = '';
-			}
-			else
+			} else
 				$this->_items[$id] = $item;
 		}
 	}
@@ -80,10 +79,12 @@ 
 	public function evaluateDynamicContent()
 	{
 		$context = $this->_container === null?$this:$this->_container;
-		foreach($this->_expressions as $id => $expression)
-			$this->_items[$id] = $context->evaluateExpression($expression);
-		foreach($this->_statements as $id => $statement)
-			$this->_items[$id] = $context->evaluateStatements($statement);
+		foreach($this->_expressions as $id => $expression) {
+					$this->_items[$id] = $context->evaluateExpression($expression);
+		}
+		foreach($this->_statements as $id => $statement) {
+					$this->_items[$id] = $context->evaluateStatements($statement);
+		}
 	}
 
 	/**
@@ -93,8 +94,9 @@ 
 	public function dataBind()
 	{
 		$context = $this->_container === null?$this:$this->_container;
-		foreach($this->_bindings as $id => $binding)
-			$this->_items[$id] = $context->evaluateExpression($binding);
+		foreach($this->_bindings as $id => $binding) {
+					$this->_items[$id] = $context->evaluateExpression($binding);
+		}
 	}
 
 	/**

framework/Web/UI/JuiControls/TJuiDialog.php

Braces +4 added lines, -4 removed lines

@@ -108,9 +108,10 @@ 
 		if(!isset($options['appendTo']))
 			$options['appendTo'] = 'form:first';
 
-		foreach($this->getControls() as $control)
-			if($control instanceof TJuiDialogButton)
+		foreach($this->getControls() as $control) {
+					if($control instanceof TJuiDialogButton)
 				$options['buttons'][] = $control->getPostBackOptions();
+		}
 
 		return $options;
 	}
@@ -193,8 +194,7 @@ 
 		{
 		  parent::renderContents($writer);
 			$this->getPage()->getCallbackClient()->replaceContent($this, $writer, false);
-		}
-		else
+		} else
 			parent::render($writer);
 	}
 }

framework/Web/UI/JuiControls/TJuiControlOptions.php

Braces +3 added lines, -2 removed lines

@@ -141,9 +141,10 @@
 	{
 		$ret = ($this->_options === null) ? [] : $this->_options;
 
-		foreach($this->_control->getValidEvents() as $event)
-			if($this->_control->hasEventHandler('on' . $event))
+		foreach($this->_control->getValidEvents() as $event) {
+					if($this->_control->hasEventHandler('on' . $event))
 				$ret[$event] = new TJavaScriptLiteral("function( event, ui ) { Prado.JuiCallback(" . TJavaScript::encode($this->_control->getUniqueID()) . ", " . TJavaScript::encode($event) . ", event, ui, this); }");
+		}
 
 		return $ret;
 	}

framework/Web/UI/TTemplate.php

Braces +36 added lines, -60 removed lines

@@ -233,8 +233,9 @@ 
 					$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);
 
@@ -244,8 +245,7 @@ 
 						$component->createdOnTemplate($parent);
 					if($component->getAllowChildControls())
 						$controls[$key] = $component;
-				}
-				elseif($component instanceof TComponent)
+				} elseif($component instanceof TComponent)
 				{
 					$controls[$key] = $component;
 					if(isset($properties['id']))
@@ -256,15 +256,15 @@ 
 						if(!$component->hasProperty('id'))
 							unset($properties['id']);
 					}
-					foreach($properties as $name => $value)
-						$this->configureComponent($component, $name, $value);
+					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)
 				{
@@ -275,8 +275,7 @@ 
 						$directChildren[] = $o;
 					else
 						$parent->addParsedObject($o);
-				}
-				else
+				} else
 				{
 					if($parent === $parentControl)
 						$directChildren[] = $object[1];
@@ -387,8 +386,7 @@ 
 					throw new TConfigurationException('template_tag_unexpected', $name, $value[1]);
 					break;
 			}
-		}
-		else
+		} else
 		{
 			if (substr($name, 0, 2) == 'js')
 				if ($value and !($value instanceof TJavaScriptLiteral))
@@ -436,8 +434,7 @@ 
 					throw new TConfigurationException('template_tag_unexpected', $name, $value[1]);
 					break;
 			}
-		}
-		else
+		} else
 			$component->setSubProperty($name, $value);
 	}
 
@@ -498,8 +495,7 @@ 
 						$stack[] = $type;
 						$container = $c - 1;
 					}
-				}
-				elseif(strpos($str, '</com:') === 0)	// closing component tag
+				} elseif(strpos($str, '</com:') === 0)	// closing component tag
 				{
 					if($expectPropEnd)
 						continue;
@@ -518,8 +514,7 @@ 
 						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;
@@ -529,8 +524,7 @@ 
 					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;
@@ -555,8 +549,7 @@ 
 						$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
 					{
@@ -568,8 +561,9 @@ 
 						$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)
@@ -578,8 +572,7 @@ 
 								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;
@@ -591,8 +584,7 @@ 
 							$expectPropEnd = true;
 						}
 					}
-				}
-				elseif(strpos($str, '</prop:') === 0)	// closing property
+				} elseif(strpos($str, '</prop:') === 0)	// closing property
 				{
 					$prop = strtolower($match[3][0]);
 					if(empty($stack))
@@ -619,23 +611,20 @@ 
 								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)];
 					$textStart = $matchEnd + 1;
-				}
-				else
+				} else
 					throw new TConfigurationException('template_matching_unexpected', $match);
 			}
 			if(!empty($stack))
@@ -646,8 +635,7 @@ 
 			}
 			if($textStart < strlen($input))
 				$tpl[$c++] = [$container,substr($input, $textStart)];
-		}
-		catch(\Exception $e)
+		} catch(\Exception $e)
 		{
 			if(($e instanceof TException) && ($e instanceof TTemplateException))
 				throw $e;
@@ -680,14 +668,12 @@ 
 				{
 					$parent = null;
 					$objects[$id] = $object;
-				}
-				else
+				} else
 				{
 					$parent = $object[0];
 					$merged = [$parent,[$object[1]]];
 				}
-			}
-			else
+			} else
 				$merged[1][] = $object[1];
 		}
 		if($parent !== null)
@@ -726,8 +712,7 @@ 
 					$attributes[$name] = $this->parseTemplateProperty(substr($value, 1, strlen($value) - 2), $match[2][1] + 1);
 				else
 					$attributes[$name] = $this->parseTemplateProperty($value, $match[2][1]);
-			}
-			else
+			} else
 			{
 				if($value[0] === '\'' || $value[0] === '"')
 					$attributes[$name] = $this->parseAttribute(substr($value, 1, strlen($value) - 2));
@@ -776,8 +761,7 @@ 
 				return [self::CONFIG_DATABIND,ltrim($expr, '.')];
 			else
 				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] === '~')
@@ -790,8 +774,7 @@ 
 				$literal = trim(substr($value, 3, strlen($value) - 5));
 				return [self::CONFIG_EXPRESSION,"rtrim(dirname(\$this->getApplication()->getRequest()->getApplicationUrl()), '\/').'/$literal'"];
 			}
-		}
-		else
+		} else
 			return $value;
 	}
 
@@ -813,16 +796,14 @@ 
 					$subname = substr($name, 0, $pos);
 					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))
 						throw new TConfigurationException('template_event_unknown', $type, $name);
 					elseif(!is_string($att))
 						throw new TConfigurationException('template_eventhandler_invalid', $type, $name);
-				}
-				else
+				} else
 				{
 					// a simple property
 					if (! ($class->hasMethod('set' . $name) || $class->hasMethod('setjs' . $name) || $this->isClassBehaviorMethod($class, 'set' . $name)))
@@ -831,8 +812,7 @@ 
 							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)
 							throw new TConfigurationException('template_controlid_invalid', $type);
@@ -841,8 +821,7 @@ 
 					}
 				}
 			}
-		}
-		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)
 			{
@@ -854,8 +833,7 @@ 
 					$subname = substr($name, 0, $pos);
 					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
 				{
@@ -869,8 +847,7 @@ 
 					}
 				}
 			}
-		}
-		else
+		} else
 			throw new TConfigurationException('template_component_required', $type);
 		return $class->getName();
 	}
@@ -906,8 +883,7 @@ 
 						$line = $line - $this->_includeAtLine[$i] + 1;
 						$srcFile = $this->_includedFiles[$i];
 						break;
-					}
-					else
+					} else
 						$line = $line - $this->_includeLines[$i] + 1;
 				}
 			}

framework/Web/UI/TPage.php

Braces +43 added lines, -38 removed lines

@@ -204,8 +204,7 @@ 
 				$this->processCallbackRequest($writer);
 			else
 				$this->processPostBackRequest($writer);
-		}
-		else
+		} else
 			$this->processNormalRequest($writer);
 
 		$this->_writer = null;
@@ -294,8 +293,9 @@ 
 	{
 		if(is_array($data))
 		{
-			foreach($data as $k => $v)
-				$data[$k] = self::decodeUTF8($v, $enc);
+			foreach($data as $k => $v) {
+							$data[$k] = self::decodeUTF8($v, $enc);
+			}
 			return $data;
 		} elseif(is_string($data)) {
 			return iconv('UTF-8', $enc . '//IGNORE', $data);
@@ -319,8 +319,9 @@ 
 		// Decode Callback postData from UTF-8 to current Charset
 		if (($g = $this->getApplication()->getGlobalization(false)) !== null &&
 			strtoupper($enc = $g->getCharset()) != 'UTF-8')
-				foreach ($this->_postData as $k => $v)
-					$this->_postData[$k] = self::decodeUTF8($v, $enc);
+				foreach ($this->_postData as $k => $v) {
+									$this->_postData[$k] = self::decodeUTF8($v, $enc);
+				}
 
 		Prado::trace("Page onPreInit()", 'Prado\Web\UI\TPage');
 		$this->onPreInit(null);
@@ -471,9 +472,10 @@ 
 		else
 		{
 			$list = new TList;
-			foreach($this->_validators as $validator)
-				if($validator->getValidationGroup() === $validationGroup)
+			foreach($this->_validators as $validator) {
+							if($validator->getValidationGroup() === $validationGroup)
 					$list->add($validator);
+			}
 			return $list;
 		}
 	}
@@ -492,10 +494,10 @@ 
 		{
 			if($validationGroup === null)
 			{
-				foreach($this->_validators as $validator)
-					$validator->validate();
-			}
-			else
+				foreach($this->_validators as $validator) {
+									$validator->validate();
+				}
+			} else
 			{
 				foreach($this->_validators as $validator)
 				{
@@ -518,13 +520,13 @@ 
 		{
 			if($this->_validators && $this->_validators->getCount())
 			{
-				foreach($this->_validators as $validator)
-					if(!$validator->getIsValid())
+				foreach($this->_validators as $validator) {
+									if(!$validator->getIsValid())
 						return false;
+				}
 			}
 			return true;
-		}
-		else
+		} else
 			throw new TInvalidOperationException('page_isvalid_unknown');
 	}
 
@@ -673,18 +675,22 @@ 
 		$theme = $this->getTheme();
 		if($theme instanceof ITheme)
 		{
-			foreach($theme->getStyleSheetFiles() as $url)
-				$cs->registerStyleSheetFile($url, $url, $this->getCssMediaType($url));
-			foreach($theme->getJavaScriptFiles() as $url)
-				$cs->registerHeadScriptFile($url, $url);
+			foreach($theme->getStyleSheetFiles() as $url) {
+							$cs->registerStyleSheetFile($url, $url, $this->getCssMediaType($url));
+			}
+			foreach($theme->getJavaScriptFiles() as $url) {
+							$cs->registerHeadScriptFile($url, $url);
+			}
 		}
 		$styleSheet = $this->getStyleSheetTheme();
 		if($styleSheet instanceof ITheme)
 		{
-			foreach($styleSheet->getStyleSheetFiles() as $url)
-				$cs->registerStyleSheetFile($url, $url, $this->getCssMediaType($url));
-			foreach($styleSheet->getJavaScriptFiles() as $url)
-				$cs->registerHeadScriptFile($url, $url);
+			foreach($styleSheet->getStyleSheetFiles() as $url) {
+							$cs->registerStyleSheetFile($url, $url, $this->getCssMediaType($url));
+			}
+			foreach($styleSheet->getJavaScriptFiles() as $url) {
+							$cs->registerHeadScriptFile($url, $url);
+			}
 		}
 
 		if($cs->getRequiresHead() && $this->getHead() === null)
@@ -810,8 +816,9 @@ 
 		$id = is_string($control)?$control:$control->getUniqueID();
 		$this->_controlsRegisteredForPostData[$id] = true;
 		$params = func_get_args();
-		foreach($this->getCachingStack() as $item)
-			$item->registerAction('Page', 'registerRequiresPostData', [$id]);
+		foreach($this->getCachingStack() as $item) {
+					$item->registerAction('Page', 'registerRequiresPostData', [$id]);
+		}
 	}
 
 	/**
@@ -878,15 +885,13 @@ 
 				{
 					if($control->loadPostData($key, $postData))
 						$this->_controlsPostDataChanged[] = $control;
-				}
-				elseif($control instanceof IPostBackEventHandler &&
+				} elseif($control instanceof IPostBackEventHandler &&
 					empty($this->_postData[self::FIELD_POSTBACK_TARGET]))
 				{
 					$this->_postData->add(self::FIELD_POSTBACK_TARGET, $key);  // not calling setPostBackEventTarget() because the control may be removed later
 				}
 				unset($this->_controlsRequiringPostData[$key]);
-			}
-			elseif($beforeLoad)
+			} elseif($beforeLoad)
 				$this->_restPostData->add($key, $value);
 		}
 
@@ -898,8 +903,7 @@ 
 				{
 					if($control->loadPostData($key, $this->_postData))
 						$this->_controlsPostDataChanged[] = $control;
-				}
-				else
+				} else
 					throw new TInvalidDataValueException('page_postbackcontrol_invalid', $key);
 				unset($this->_controlsRequiringPostData[$key]);
 			}
@@ -920,8 +924,9 @@ 
 	 */
 	protected function raiseChangedEvents()
 	{
-		foreach($this->_controlsPostDataChanged as $control)
-			$control->raisePostDataChangedEvent();
+		foreach($this->_controlsPostDataChanged as $control) {
+					$control->raisePostDataChangedEvent();
+		}
 	}
 
 	/**
@@ -978,8 +983,7 @@ 
 			else
 				$focus = $this->_focus;
 			$this->getClientScript()->registerFocusControl($focus);
-		}
-		elseif($this->_postData && ($lastFocus = $this->_postData->itemAt(self::FIELD_LASTFOCUS)) !== null)
+		} elseif($this->_postData && ($lastFocus = $this->_postData->itemAt(self::FIELD_LASTFOCUS)) !== null)
 			$this->getClientScript()->registerFocusControl($lastFocus);
 		$this->_inFormRender = false;
 	}
@@ -1196,8 +1200,9 @@ 
 	{
 		if($this->_cachingStack)
 		{
-			foreach($this->_cachingStack as $cache)
-				$cache->registerAction($context, $funcName, $funcParams);
+			foreach($this->_cachingStack as $cache) {
+							$cache->registerAction($context, $funcName, $funcParams);
+			}
 		}
 	}
 

framework/Web/UI/THtmlWriter.php

Braces +13 added lines, -10 removed lines

@@ -100,8 +100,9 @@ 
 	 */
 	public function addAttributes($attrs)
 	{
-		foreach($attrs as $name => $value)
-			$this->_attributes[THttpUtility::htmlStrip($name)] = THttpUtility::htmlEncode($value);
+		foreach($attrs as $name => $value) {
+					$this->_attributes[THttpUtility::htmlStrip($name)] = THttpUtility::htmlEncode($value);
+		}
 	}
 
 	/**
@@ -129,8 +130,9 @@ 
 	 */
 	public function addStyleAttributes($attrs)
 	{
-		foreach($attrs as $name => $value)
-			$this->_styles[THttpUtility::htmlStrip($name)] = THttpUtility::htmlEncode($value);
+		foreach($attrs as $name => $value) {
+					$this->_styles[THttpUtility::htmlStrip($name)] = THttpUtility::htmlEncode($value);
+		}
 	}
 
 	/**
@@ -195,21 +197,22 @@ 
 	public function renderBeginTag($tagName)
 	{
 		$str = '<' . $tagName;
-		foreach($this->_attributes as $name => $value)
-			$str .= ' ' . $name . '="' . $value . '"';
+		foreach($this->_attributes as $name => $value) {
+					$str .= ' ' . $name . '="' . $value . '"';
+		}
 		if(!empty($this->_styles))
 		{
 			$str .= ' style="';
-			foreach($this->_styles as $name => $value)
-				$str .= $name . ':' . $value . ';';
+			foreach($this->_styles as $name => $value) {
+							$str .= $name . ':' . $value . ';';
+			}
 			$str .= '"';
 		}
 		if(isset(self::$_simpleTags[$tagName]))
 		{
 			$str .= ' />';
 			$this->_openTags[] = '';
-		}
-		else
+		} else
 		{
 			$str .= '>';
 			$this->_openTags[] = $tagName;

framework/Web/UI/TControl.php

Braces +38 added lines, -40 removed lines

@@ -406,11 +406,9 @@ 
 					return $this->_id;
 				else
 					return ($this->_uid = $prefix . self::ID_SEPARATOR . $this->_id);
-			}
-			else	// no naming container
+			} else	// no naming container
 				return $this->_id;
-		}
-		else
+		} else
 			return $this->_uid;
 	}
 
@@ -560,12 +558,12 @@ 
 	{
 		if($checkParents)
 		{
-			for($control = $this;$control;$control = $control->_parent)
-				if(!$control->getVisible(false))
+			for($control = $this;$control;$control = $control->_parent) {
+							if(!$control->getVisible(false))
 					return false;
+			}
 			return true;
-		}
-		else
+		} else
 			return $this->getViewState('Visible', true);
 	}
 
@@ -591,12 +589,12 @@ 
 	{
 		if($checkParents)
 		{
-			for($control = $this;$control;$control = $control->_parent)
-				if(!$control->getViewState('Enabled', true))
+			for($control = $this;$control;$control = $control->_parent) {
+							if(!$control->getViewState('Enabled', true))
 					return false;
+			}
 			return true;
-		}
-		else
+		} else
 			return $this->getViewState('Enabled', true);
 	}
 
@@ -689,12 +687,12 @@ 
 	{
 		if($checkParents)
 		{
-			for($control = $this;$control !== null;$control = $control->getParent())
-				if($control->_flags & self::IS_DISABLE_VIEWSTATE)
+			for($control = $this;$control !== null;$control = $control->getParent()) {
+							if($control->_flags & self::IS_DISABLE_VIEWSTATE)
 					return false;
+			}
 			return true;
-		}
-		else
+		} else
 			return !($this->_flags & self::IS_DISABLE_VIEWSTATE);
 	}
 
@@ -779,8 +777,7 @@ 
 			if(is_object($this->_tempState[$key]) && $this->_trackViewState)
 				$this->_viewState[$key] = $this->_tempState[$key];
 			return $this->_tempState[$key];
-		}
-		else
+		} else
 			return $defaultValue;
 	}
 
@@ -800,8 +797,7 @@ 
 		{
 			unset($this->_tempState[$key]);
 			$this->_viewState[$key] = $value;
-		}
-		else
+		} else
 		{
 			unset($this->_viewState[$key]);
 			if($value === $defaultValue)
@@ -874,8 +870,9 @@ 
 		{
 			if(($context = $this->getTemplateControl()) === null)
 				$context = $this;
-			foreach($this->_rf[self::RF_DATA_BINDINGS] as $property => $expression)
-				$this->setSubProperty($property, $context->evaluateExpression($expression));
+			foreach($this->_rf[self::RF_DATA_BINDINGS] as $property => $expression) {
+							$this->setSubProperty($property, $context->evaluateExpression($expression));
+			}
 		}
 	}
 
@@ -888,8 +885,9 @@ 
 		{
 			if(($context = $this->getTemplateControl()) === null)
 				$context = $this;
-			foreach($this->_rf[self::RF_AUTO_BINDINGS] as $property => $expression)
-				$this->setSubProperty($property, $context->evaluateExpression($expression));
+			foreach($this->_rf[self::RF_AUTO_BINDINGS] as $property => $expression) {
+							$this->setSubProperty($property, $context->evaluateExpression($expression));
+			}
 		}
 	}
 
@@ -901,9 +899,10 @@ 
 		Prado::trace("dataBindChildren()", 'Prado\Web\UI\TControl');
 		if(isset($this->_rf[self::RF_CONTROLS]))
 		{
-			foreach($this->_rf[self::RF_CONTROLS] as $control)
-				if($control instanceof IBindable)
+			foreach($this->_rf[self::RF_CONTROLS] as $control) {
+							if($control instanceof IBindable)
 					$control->dataBind();
+			}
 		}
 	}
 
@@ -950,8 +949,7 @@ 
 					$this->createChildControls();
 				$this->_flags &= ~self::IS_CREATING_CHILD;
 				$this->_flags |= self::IS_CHILD_CREATED;
-			}
-			catch(Exception $e)
+			} catch(Exception $e)
 			{
 				$this->_flags &= ~self::IS_CREATING_CHILD;
 				$this->_flags |= self::IS_CHILD_CREATED;
@@ -1190,8 +1188,9 @@ 
 	final protected function isDescendentOf($ancestor)
 	{
 		$control = $this;
-		while($control !== $ancestor && $control->_parent)
-			$control = $control->_parent;
+		while($control !== $ancestor && $control->_parent) {
+					$control = $control->_parent;
+		}
 		return $control === $ancestor;
 	}
 
@@ -1227,8 +1226,7 @@ 
 				{
 					$state = $this->_rf[self::RF_CHILD_STATE][$control->_id];
 					unset($this->_rf[self::RF_CHILD_STATE][$control->_id]);
-				}
-				else
+				} else
 					$state = null;
 				$control->loadStateRecursive($state, !($this->_flags & self::IS_DISABLE_VIEWSTATE));
 				if($this->_stage >= self::CS_LOADED)
@@ -1367,9 +1365,10 @@ 
 			$this->_id = '';
 		if($this->getHasControls())
 		{
-			foreach($this->_rf[self::RF_CONTROLS] as $control)
-				if($control instanceof TControl)
+			foreach($this->_rf[self::RF_CONTROLS] as $control) {
+							if($control instanceof TControl)
 					$control->unloadRecursive();
+			}
 		}
 		if(isset($this->_rf[self::RF_ADAPTER]))
 			$this->_rf[self::RF_ADAPTER]->onUnload(null);
@@ -1633,8 +1632,7 @@ 
 			{
 				$this->_rf[self::RF_CONTROLSTATE] = &$state[1];
 				unset($state[1]);
-			}
-			else
+			} else
 				unset($this->_rf[self::RF_CONTROLSTATE]);
 			if($needViewState)
 			{
@@ -1711,8 +1709,7 @@ 
 		{
 			$page->applyControlStyleSheet($this);
 			$this->_flags |= self::IS_STYLESHEET_APPLIED;
-		}
-		elseif($this->_flags & self::IS_STYLESHEET_APPLIED)
+		} elseif($this->_flags & self::IS_STYLESHEET_APPLIED)
 			throw new TInvalidOperationException('control_stylesheet_applied', get_class($this));
 	}
 
@@ -1725,9 +1722,10 @@ 
 	{
 		if($recursive && $this->_uid !== null && isset($this->_rf[self::RF_CONTROLS]))
 		{
-			foreach($this->_rf[self::RF_CONTROLS] as $control)
-				if($control instanceof TControl)
+			foreach($this->_rf[self::RF_CONTROLS] as $control) {
+							if($control instanceof TControl)
 					$control->clearCachedUniqueID($recursive);
+			}
 		}
 		$this->_uid = null;
 	}

framework/Web/UI/WebControls/TRatingList.php

Braces +4 added lines, -4 removed lines

@@ -268,8 +268,7 @@ 
 			{
 				if($control->getVisible(true))
 					return $control->getClientID();
-			}
-			else
+			} else
 				return $id;
 		}
 		return '';
@@ -308,8 +307,9 @@ 
 	{
 		$types = ['blank', 'selected', 'half', 'combined'];
 		$files = [];
-		foreach($types as $type)
-			$files[$type] = $this->getAssetUrl("{$style}_{$type}{$fileExt}");
+		foreach($types as $type) {
+					$files[$type] = $this->getAssetUrl("{$style}_{$type}{$fileExt}");
+		}
 		return $files;
 	}
 

framework/Web/UI/WebControls/TBaseValidator.php

Braces +5 added lines, -6 removed lines

@@ -190,9 +190,10 @@ 
 	 */
 	private function getClientControlClass($control)
 	{
-		foreach(self::$_clientClass as $type)
-			if($control instanceof $type)
+		foreach(self::$_clientClass as $type) {
+					if($control instanceof $type)
 				return $type;
+		}
 		$reflectionClass = new \ReflectionClass($control);
 		return $reflectionClass->getShortName();
 	}
@@ -523,16 +524,14 @@ 
 				{
 					$this->setIsValid(true);
 					$this->onValidationSuccess();
-				}
-				else
+				} else
 				{
 					if($target)
 						$target->setIsValid(false);
 					$this->setIsValid(false);
 					$this->onValidationError();
 				}
-			}
-			else
+			} else
 			{
 				$this->evaluateIsValid();
 				$this->setIsValid(true);