pradosoft / prado

framework/Web/UI/ActiveControls/TStyleDiff.php

Braces +21 added lines, -16 removed lines

@@ -30,12 +30,14 @@ 
 	 */
 	protected function getCombinedStyle($obj)
 	{
-		if(!($obj instanceof TStyle))
-			return [];
+		if(!($obj instanceof TStyle)) {
+					return [];
+		}
 		$style = $obj->getStyleFields();
 		$style = array_merge($style, $this->getStyleFromString($obj->getCustomStyle()));
-		if($obj->hasFont())
-			$style = array_merge($style, $this->getStyleFromString($obj->getFont()->toString()));
+		if($obj->hasFont()) {
+					$style = array_merge($style, $this->getStyleFromString($obj->getFont()->toString()));
+		}
 		return $style;
 	}
 
@@ -46,13 +48,16 @@ 
 	protected function getStyleFromString($string)
 	{
 		$style = [];
-		if(!is_string($string)) return $style;
+		if(!is_string($string)) {
+			return $style;
+		}
 
 		foreach(explode(';', $string) as $sub)
 		{
 			$arr = explode(':', $sub);
-			if(isset($arr[1]) && trim($arr[0]) !== '')
-				$style[trim($arr[0])] = trim($arr[1]);
+			if(isset($arr[1]) && trim($arr[0]) !== '') {
+							$style[trim($arr[0])] = trim($arr[1]);
+			}
 		}
 		return $style;
 	}
@@ -66,8 +71,7 @@ 
 		{
 			return ($this->_new !== null) && $this->_new->hasCssClass()
 						? $this->_new->getCssClass() : null;
-		}
-		else
+		} else
 		{
 			return $this->_old->getCssClass() !== $this->_new->getCssClass() ?
 				$this->_new->getCssClass() : null;
@@ -90,16 +94,17 @@ 
 	 */
 	public function getDifference()
 	{
-		if($this->_new === null)
-			return $this->_null;
-		else
+		if($this->_new === null) {
+					return $this->_null;
+		} else
 		{
 			$css = $this->getCssClassDiff();
 			$style = $this->getStyleDiff();
-			if(($css !== null) || ($style !== null))
-				return ['CssClass' => $css, 'Style' => $style];
-			else
-				$this->_null;
+			if(($css !== null) || ($style !== null)) {
+							return ['CssClass' => $css, 'Style' => $style];
+			} else {
+							$this->_null;
+			}
 		}
 	}
 }

framework/Web/UI/ActiveControls/TActivePager.php

Braces +17 added lines, -17 removed lines

@@ -127,30 +127,30 @@ 
 	{
 		if($buttonType === TPagerButtonType::LinkButton)
 		{
-			if($enabled)
-				$button = new TActiveLinkButton;
-			else
+			if($enabled) {
+							$button = new TActiveLinkButton;
+			} else
 			{
 				$button = new TLabel;
 				$button->setText($text);
 				$button->setCssClass($this->getButtonCssClass());
 				return $button;
 			}
-		}
-		elseif($buttonType === TPagerButtonType::ImageButton)
+		} elseif($buttonType === TPagerButtonType::ImageButton)
 		{
 			$button = new TActiveImageButton;
 			$button->setImageUrl($this->getPageImageUrl($text, $commandName));
-			if($enabled)
-				$button->Visible = true;
-			else
-				$button->Visible = false;
-		}
-		else
+			if($enabled) {
+							$button->Visible = true;
+			} else {
+							$button->Visible = false;
+			}
+		} else
 		{
 			$button = new TActiveButton;
-			if(!$enabled)
-				$button->setEnabled(false);
+			if(!$enabled) {
+							$button->setEnabled(false);
+			}
 		}
 
 		if($buttonType === TPagerButtonType::ImageButton)
@@ -204,10 +204,10 @@ 
 		{
 			$this->setDisplay(($this->getPageCount() == 1)?TDisplayStyle::None:TDisplayStyle::Dynamic);
 			TWebControl::render($writer);
-			if($this->getActiveControl()->canUpdateClientSide())
-				$this->getPage()->getCallbackClient()->replaceContent($this, $writer);
-		}
-		else
+			if($this->getActiveControl()->canUpdateClientSide()) {
+							$this->getPage()->getCallbackClient()->replaceContent($this, $writer);
+			}
+		} else
 		{
 			$this->getPage()->getAdapter()->registerControlToRender($this, $writer);
 		}

framework/Web/UI/ActiveControls/TActivePageAdapter.php

Braces +31 added lines, -24 removed lines

@@ -120,8 +120,9 @@ 
 	public function registerControlToRender($control, $writer)
 	{
 		$id = $control->getUniqueID();
-		if(!isset($this->_controlsToRender[$id]))
-			$this->_controlsToRender[$id] = [$control,$writer];
+		if(!isset($this->_controlsToRender[$id])) {
+					$this->_controlsToRender[$id] = [$control,$writer];
+		}
 	}
 
 	/**
@@ -139,10 +140,11 @@ 
 	public function renderCallbackResponse($writer)
 	{
 		Prado::trace("ActivePage renderCallbackResponse()", 'Prado\Web\UI\ActiveControls\TActivePageAdapter');
-		if(($url = $this->getResponse()->getAdapter()->getRedirectedUrl()) === null)
-			$this->renderResponse($writer);
-		else
-			$this->redirect($url);
+		if(($url = $this->getResponse()->getAdapter()->getRedirectedUrl()) === null) {
+					$this->renderResponse($writer);
+		} else {
+					$this->redirect($url);
+		}
 	}
 
 	/**
@@ -164,8 +166,9 @@ 
 	{
 		Prado::trace("ActivePage renderResponse()", 'Prado\Web\UI\ActiveControls\TActivePageAdapter');
 		//renders all the defered render() calls.
-		foreach($this->_controlsToRender as $rid => $forRender)
-			$forRender[0]->render($forRender[1]);
+		foreach($this->_controlsToRender as $rid => $forRender) {
+					$forRender[0]->render($forRender[1]);
+		}
 
 		$response = $this->getResponse();
 
@@ -212,23 +215,27 @@ 
 
 		// collect all stylesheet file references
 		$stylesheets = $cs->getStyleSheetUrls();
-		if (count($stylesheets) > 0)
-		$this->appendContentPart($response, self::CALLBACK_STYLESHEETLIST_HEADER, TJavaScript::jsonEncode($stylesheets));
+		if (count($stylesheets) > 0) {
+				$this->appendContentPart($response, self::CALLBACK_STYLESHEETLIST_HEADER, TJavaScript::jsonEncode($stylesheets));
+		}
 
 		// collect all stylesheet snippets references
 		$stylesheets = $cs->getStyleSheetCodes();
-		if (count($stylesheets) > 0)
-		$this->appendContentPart($response, self::CALLBACK_STYLESHEET_HEADER, TJavaScript::jsonEncode($stylesheets));
+		if (count($stylesheets) > 0) {
+				$this->appendContentPart($response, self::CALLBACK_STYLESHEET_HEADER, TJavaScript::jsonEncode($stylesheets));
+		}
 
 		// collect all script file references
 		$scripts = $cs->getScriptUrls();
-		if (count($scripts) > 0)
-		$this->appendContentPart($response, self::CALLBACK_SCRIPTLIST_HEADER, TJavaScript::jsonEncode($scripts));
+		if (count($scripts) > 0) {
+				$this->appendContentPart($response, self::CALLBACK_SCRIPTLIST_HEADER, TJavaScript::jsonEncode($scripts));
+		}
 
 		// collect all hidden field references
 		$fields = $cs->getHiddenFields();
-		if (count($fields) > 0)
-		$this->appendContentPart($response, self::CALLBACK_HIDDENFIELDLIST_HEADER, TJavaScript::jsonEncode($fields));
+		if (count($fields) > 0) {
+				$this->appendContentPart($response, self::CALLBACK_HIDDENFIELDLIST_HEADER, TJavaScript::jsonEncode($fields));
+		}
 	}
 
 	/**
@@ -256,14 +263,12 @@ 
 				$param = $this->getCallbackEventParameter();
 				$result = new TCallbackEventParameter($this->getResponse(), $param);
 				$callbackHandler->raiseCallbackEvent($result);
-			}
-			else
+			} else
 			{
 				throw new TInvalidCallbackException(
 					'callback_invalid_handler', $callbackHandler->getUniqueID());
 			}
-		 }
-		 else
+		 } else
 		 {
 			$target = $this->getRequest()->itemAt(TPage::FIELD_CALLBACK_TARGET);
 			throw new TInvalidCallbackException('callback_invalid_target', $target);
@@ -279,8 +284,9 @@ 
 		if($this->_callbackEventTarget === null)
 		{
 			$eventTarget = $this->getRequest()->itemAt(TPage::FIELD_CALLBACK_TARGET);
-			if(!empty($eventTarget))
-				$this->_callbackEventTarget = $this->getPage()->findControl($eventTarget);
+			if(!empty($eventTarget)) {
+							$this->_callbackEventTarget = $this->getPage()->findControl($eventTarget);
+			}
 		}
 		return $this->_callbackEventTarget;
 	}
@@ -323,8 +329,9 @@ 
 	 */
 	public function getCallbackClientHandler()
 	{
-		if($this->_callbackClient === null)
-			$this->_callbackClient = new TCallbackClientScript;
+		if($this->_callbackClient === null) {
+					$this->_callbackClient = new TCallbackClientScript;
+		}
 		return $this->_callbackClient;
 	}
 }
\ No newline at end of file

framework/Web/UI/ActiveControls/TCallbackPageStateTracker.php

Braces +20 added lines, -14 removed lines

@@ -109,8 +109,9 @@ 
 			if($new !== $old)
 			{
 				$diff = new $details[0]($new, $old, $this->_nullObject);
-				if(($change = $diff->getDifference()) !== $this->_nullObject)
-					$changes[] = [$details[1],[$change]];
+				if(($change = $diff->getDifference()) !== $this->_nullObject) {
+									$changes[] = [$details[1],[$change]];
+				}
 			}
 		}
 		return $changes;
@@ -121,8 +122,9 @@ 
 	 */
 	public function respondToChanges()
 	{
-		foreach($this->getChanges() as $change)
-			call_user_func_array($change[0], $change[1]);
+		foreach($this->getChanges() as $change) {
+					call_user_func_array($change[0], $change[1]);
+		}
 	}
 
 	/**
@@ -167,10 +169,11 @@ 
 	 */
 	protected function updateVisible($visible)
 	{
-		if($visible === false)
-			$this->client()->replaceContent($this->_control, "<span id=\"" . $this->_control->getClientID() . "\" style=\"display:none\" ></span>");
-		else
-			$this->client()->replaceContent($this->_control, $this->_control);
+		if($visible === false) {
+					$this->client()->replaceContent($this->_control, "<span id=\"" . $this->_control->getClientID() . "\" style=\"display:none\" ></span>");
+		} else {
+					$this->client()->replaceContent($this->_control, $this->_control);
+		}
 	}
 
 	/**
@@ -188,10 +191,12 @@ 
 	 */
 	protected function updateStyle($style)
 	{
-		if($style['CssClass'] !== null)
-			$this->client()->setAttribute($this->_control, 'class', $style['CssClass']);
-		if(is_array($style['Style']) && count($style['Style']) > 0)
-			$this->client()->setStyle($this->_control, $style['Style']);
+		if($style['CssClass'] !== null) {
+					$this->client()->setAttribute($this->_control, 'class', $style['CssClass']);
+		}
+		if(is_array($style['Style']) && count($style['Style']) > 0) {
+					$this->client()->setStyle($this->_control, $style['Style']);
+		}
 	}
 
 	/**
@@ -200,7 +205,8 @@ 
 	 */
 	protected function updateAttributes($attributes)
 	{
-		foreach($attributes as $name => $value)
-			$this->client()->setAttribute($this->_control, $name, $value);
+		foreach($attributes as $name => $value) {
+					$this->client()->setAttribute($this->_control, $name, $value);
+		}
 	}
 }

framework/Web/UI/ActiveControls/TCallbackOptions.php

Braces +3 added lines, -2 removed lines

@@ -40,8 +40,9 @@
 	 */
 	public function getClientSide()
 	{
-		if($this->_clientSide === null)
-			$this->_clientSide = $this->createClientSide();
+		if($this->_clientSide === null) {
+					$this->_clientSide = $this->createClientSide();
+		}
 		return $this->_clientSide;
 	}
 

framework/Web/UI/ActiveControls/TActiveCheckBox.php

Braces +20 added lines, -14 removed lines

@@ -93,13 +93,15 @@ 
 	 */
 	public function setText($value)
 	{
-		if(parent::getText() === $value)
-			return;
+		if(parent::getText() === $value) {
+					return;
+		}
 
 		parent::setText($value);
-		if($this->getActiveControl()->canUpdateClientSide())
-			$this->getPage()->getCallbackClient()->update(
+		if($this->getActiveControl()->canUpdateClientSide()) {
+					$this->getPage()->getCallbackClient()->update(
 				$this->getDefaultLabelID(), $value);
+		}
 	}
 
 	/**
@@ -111,12 +113,14 @@ 
 	public function setChecked($value)
 	{
 		$value = TPropertyValue::ensureBoolean($value);
-		if(parent::getChecked() === $value)
-			return;
+		if(parent::getChecked() === $value) {
+					return;
+		}
 
 		parent::setChecked($value);
-		if($this->getActiveControl()->canUpdateClientSide())
-			$this->getPage()->getCallbackClient()->check($this, $value);
+		if($this->getActiveControl()->canUpdateClientSide()) {
+					$this->getPage()->getCallbackClient()->check($this, $value);
+		}
 	}
 
 	/**
@@ -140,9 +144,10 @@ 
 	protected function renderInputTag($writer, $clientID, $onclick)
 	{
 		parent::renderInputTag($writer, $clientID, $onclick);
-		if ($this->getAutoPostBack())
-			$this->getActiveControl()->registerCallbackClientScript(
+		if ($this->getAutoPostBack()) {
+					$this->getActiveControl()->registerCallbackClientScript(
 				$this->getClientClassName(), $this->getPostBackOptions());
+		}
 	}
 
 	/**
@@ -181,10 +186,11 @@ 
 	 */
 	protected function getDefaultLabelID()
 	{
-		if($attributes = $this->getViewState('LabelAttributes', null))
-			return TCheckBox::getLabelAttributes()->itemAt('id');
-		else
-			return $this->getClientID() . '_label';
+		if($attributes = $this->getViewState('LabelAttributes', null)) {
+					return TCheckBox::getLabelAttributes()->itemAt('id');
+		} else {
+					return $this->getClientID() . '_label';
+		}
 	}
 }
 

framework/Web/UI/ActiveControls/TBaseActiveControl.php

Braces +3 added lines, -2 removed lines

@@ -63,8 +63,9 @@
 	protected function setOption($name, $value, $default = null)
 	{
 		$value = ($value === null) ? $default : $value;
-		if($value !== null)
-			$this->_options->add($name, $value);
+		if($value !== null) {
+					$this->_options->add($name, $value);
+		}
 	}
 
 	/**

framework/Web/UI/ActiveControls/TActiveTemplateColumn.php

Braces +16 added lines, -17 removed lines

@@ -49,42 +49,41 @@
 				$control->setData($text);
 			}
 			$cell->getControls()->add($control);
-		}
-		elseif($this->getAllowSorting()) {
+		} elseif($this->getAllowSorting()) {
 				$sortExpression = $this->getSortExpression();
 				if(($url = $this->getHeaderImageUrl()) !== '') {
 					$button = new TActiveImageButton;
 					$button->setImageUrl($url);
 					$button->setCommandName(TDataGrid::CMD_SORT);
 					$button->setCommandParameter($sortExpression);
-					if($text !== '')
-						$button->setAlternateText($text);
+					if($text !== '') {
+											$button->setAlternateText($text);
+					}
 					$button->setCausesValidation(false);
 					$cell->getControls()->add($button);
-				}
-				elseif($text !== '') {
+				} elseif($text !== '') {
 						$button = new TActiveLinkButton;
 						$button->setText($text);
 						$button->setCommandName(TDataGrid::CMD_SORT);
 						$button->setCommandParameter($sortExpression);
 						$button->setCausesValidation(false);
 						$cell->getControls()->add($button);
+					} else {
+											$cell->setText(' ');
 					}
-					else
-						$cell->setText(' ');
-			}
-			else {
+			} else {
 				if(($url = $this->getHeaderImageUrl()) !== '') {
 					$image = new TActiveImage;
 					$image->setImageUrl($url);
-					if($text !== '')
-						$image->setAlternateText($text);
+					if($text !== '') {
+											$image->setAlternateText($text);
+					}
 					$cell->getControls()->add($image);
-				}
-				elseif($text !== '')
-						$cell->setText($text);
-					else
-						$cell->setText(' ');
+				} elseif($text !== '') {
+										$cell->setText($text);
+				} else {
+											$cell->setText(' ');
+					}
 			}
 	}
 }
\ No newline at end of file

framework/Web/UI/ActiveControls/TActiveButtonColumn.php

Braces +10 added lines, -9 removed lines

@@ -34,11 +34,11 @@ 
 	public function initializeCell($cell, $columnIndex, $itemType) {
 		if($itemType === TListItemType::Item || $itemType === TListItemType::AlternatingItem || $itemType === TListItemType::SelectedItem || $itemType === TListItemType::EditItem) {
 			$buttonType = $this->getButtonType();
-			if($buttonType === TButtonColumnType::LinkButton)
-				$button = new TActiveLinkButton;
-			elseif($buttonType === TButtonColumnType::PushButton)
-					$button = new TActiveButton;
-				else // image button
+			if($buttonType === TButtonColumnType::LinkButton) {
+							$button = new TActiveLinkButton;
+			} elseif($buttonType === TButtonColumnType::PushButton) {
+								$button = new TActiveButton;
+			} else // image button
 				{
 					$button = new TActiveImageButton;
 					$button->setImageUrl($this->getImageUrl());
@@ -48,12 +48,13 @@ 
 			$button->setCommandName($this->getCommandName());
 			$button->setCausesValidation($this->getCausesValidation());
 			$button->setValidationGroup($this->getValidationGroup());
-			if($this->getDataTextField() !== '' || ($buttonType === TButtonColumnType::ImageButton && $this->getDataImageUrlField() !== ''))
-				$button->attachEventHandler('OnDataBinding', [$this,'dataBindColumn']);
+			if($this->getDataTextField() !== '' || ($buttonType === TButtonColumnType::ImageButton && $this->getDataImageUrlField() !== '')) {
+							$button->attachEventHandler('OnDataBinding', [$this,'dataBindColumn']);
+			}
 			$cell->getControls()->add($button);
 			$cell->registerObject('Button', $button);
+		} else {
+					parent::initializeCell($cell, $columnIndex, $itemType);
 		}
-		else
-			parent::initializeCell($cell, $columnIndex, $itemType);
 	}
 }
\ No newline at end of file