pradosoft / prado

framework/Web/UI/ActiveControls/TActivePanel.php

Braces +4 added lines, -4 removed lines

@@ -80,10 +80,10 @@
 		if($this->getHasPreRendered())
 		{
 			parent::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);
 			if ($this->getHasControls())

framework/Web/UI/ActiveControls/TActiveDataGridPager.php

Braces +3 added lines, -2 removed lines

@@ -31,8 +31,9 @@
 	 */
 	public function getClientSide()
 	{
-		if($this->_callbackoptions === null)
-			$this->_callbackoptions = new TCallbackOptions;
+		if($this->_callbackoptions === null) {
+					$this->_callbackoptions = new TCallbackOptions;
+		}
 		return $this->_callbackoptions->getClientSide();
 	}
 }
\ No newline at end of file

framework/Web/UI/ActiveControls/TTriggeredCallback.php

Braces +3 added lines, -2 removed lines

@@ -50,8 +50,9 @@
 	protected function getTargetControl()
 	{
 		$id = $this->getControlID();
-		if(($control = $this->findControl($id)) instanceof TControl)
-			return $control->getClientID();
+		if(($control = $this->findControl($id)) instanceof TControl) {
+					return $control->getClientID();
+		}
 		if($id === '')
 		{
 			throw new TConfigurationException(

framework/Web/UI/ActiveControls/TActiveHyperLinkColumn.php

Braces +15 added lines, -16 removed lines

@@ -51,8 +51,7 @@ 
 				$control->setData($text);
 			}
 			$cell->getControls()->add($control);
-		}
-		elseif($this->getAllowSorting())
+		} elseif($this->getAllowSorting())
 		{
 			$sortExpression = $this->getSortExpression();
 			if(($url = $this->getHeaderImageUrl()) !== '')
@@ -61,12 +60,12 @@ 
 				$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);
@@ -74,24 +73,24 @@ 
 				$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/TActiveTableCell.php

Braces +18 added lines, -12 removed lines

@@ -129,8 +129,9 @@ 
 	{
 		parent::addAttributesToRender($writer);
 		$writer->addAttribute('id', $this->getClientID());
-		if ($this->hasEventHandler('OnCellSelected'))
-			$this->getActiveControl()->registerCallbackClientScript($this->getClientClassName(), $this->getPostBackOptions());
+		if ($this->hasEventHandler('OnCellSelected')) {
+					$this->getActiveControl()->registerCallbackClientScript($this->getClientClassName(), $this->getPostBackOptions());
+		}
 	}
 
 	/**
@@ -145,17 +146,18 @@ 
 		if ($this->getHasPreRendered())
 		{
 			parent::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);
 			// If we update a TActiveTableCell on callback, we shouldn't update all childs,
 			// because the whole content will be replaced by the parent.
 			if ($this->getHasControls())
 			{
-				foreach ($this->findControlsByType('Prado\Web\UI\ActiveControls\IActiveControl', false) as $control)
-					$control->getActiveControl()->setEnableUpdate(false);
+				foreach ($this->findControlsByType('Prado\Web\UI\ActiveControls\IActiveControl', false) as $control) {
+									$control->getActiveControl()->setEnableUpdate(false);
+				}
 			}
 		}
 	}
@@ -180,8 +182,9 @@ 
 	 */
 	public function getCellIndex()
 	{
-		foreach ($this->getRow()->getCells() as $key => $row)
-			if ($row == $this) return $key;
+		foreach ($this->getRow()->getCells() as $key => $row) {
+					if ($row == $this) return $key;
+		}
 		throw new TConfigurationException('tactivetablecell_control_notincollection', get_class($this), $this->getUniqueID());
 	}
 
@@ -199,8 +202,11 @@ 
 			{
 				$row = $row->getParent();
 			}
-			if ($row instanceof TTableRow) $this->_row = $row;
-			else throw new TConfigurationException('tactivetablecell_control_outoftable', get_class($this), $this->getUniqueID());
+			if ($row instanceof TTableRow) {
+				$this->_row = $row;
+			} else {
+				throw new TConfigurationException('tactivetablecell_control_outoftable', get_class($this), $this->getUniqueID());
+			}
 		}
 		return $this->_row;
 	}

framework/Web/UI/ActiveControls/TActiveClientScript.php

Braces +3 added lines, -2 removed lines

@@ -51,8 +51,9 @@
 			{
 				$cs = $this->getPage()->getClientScript();
 				$uniqueid = $this->ClientID . '_custom';
-				if(!$cs->isScriptFileRegistered($uniqueid))
-					$cs->registerScriptFile($uniqueid, $scriptUrl);
+				if(!$cs->isScriptFileRegistered($uniqueid)) {
+									$cs->registerScriptFile($uniqueid, $scriptUrl);
+				}
 			} else {
 				$writer->write("<script type=\"text/javascript\" src=\"$scriptUrl\"></script>\n");
 			}

framework/Web/UI/ActiveControls/TActiveMultiView.php

Braces +11 added lines, -8 removed lines

@@ -70,11 +70,12 @@ 
 	{
 		if($this->getHasPreRendered()) {
 			$this->renderMultiView($writer);
-			if($this->getActiveControl()->canUpdateClientSide())
-				$this->getPage()->getCallbackClient()->replaceContent($this->getContainerID(), $writer);
+			if($this->getActiveControl()->canUpdateClientSide()) {
+							$this->getPage()->getCallbackClient()->replaceContent($this->getContainerID(), $writer);
+			}
+		} else {
+					$this->getPage()->getAdapter()->registerControlToRender($this, $writer);
 		}
-		else
-			$this->getPage()->getAdapter()->registerControlToRender($this, $writer);
 	}
 
 	/**
@@ -97,8 +98,9 @@ 
 	public function setActiveViewIndex($value)
 	{
 		parent::setActiveViewIndex($value);
-		if($this->getActiveControl()->canUpdateClientSide())
-			$this->getPage()->getAdapter()->registerControlToRender($this, $this->getResponse()->createHtmlWriter());
+		if($this->getActiveControl()->canUpdateClientSide()) {
+					$this->getPage()->getAdapter()->registerControlToRender($this, $this->getResponse()->createHtmlWriter());
+		}
 	}
 
 	/**
@@ -108,7 +110,8 @@ 
 	public function setActiveView($value)
 	{
 		parent::setActiveView($value);
-		if($this->getActiveControl()->canUpdateClientSide())
-			$this->getPage()->getAdapter()->registerControlToRender($this, $this->getResponse()->createHtmlWriter());
+		if($this->getActiveControl()->canUpdateClientSide()) {
+					$this->getPage()->getAdapter()->registerControlToRender($this, $this->getResponse()->createHtmlWriter());
+		}
 	}
 }

framework/Web/UI/ActiveControls/TBaseActiveCallbackControl.php

Braces +27 added lines, -22 removed lines

@@ -63,11 +63,12 @@ 
 	 */
 	public function setClientSide($client)
 	{
-		if($this->getOption('ClientSide') === null)
-			$this->setOption('ClientSide', $client);
-		else
-			throw new TConfigurationException(
+		if($this->getOption('ClientSide') === null) {
+					$this->setOption('ClientSide', $client);
+		} else {
+					throw new TConfigurationException(
 				'active_controls_client_side_exists', $this->getControl()->getID());
+		}
 	}
 
 	/**
@@ -114,20 +115,21 @@ 
 			{
 				$control = $this->getControl()->getSubProperty(substr($id, 0, $pos));
 				$newid = substr($id, $pos + 1);
-				if ($control !== null)
-					$control = $control->$newid;
-			}
-			else
+				if ($control !== null) {
+									$control = $control->$newid;
+				}
+			} else
 			{
 				// TCheckBoxList overrides findControl() with a fake implementation
 				// but accepts a second parameter to use the standard one
 				$control = $this->getControl()->findControl($id, true);
 			}
 
-			if($control instanceof TCallbackOptions)
-				return $control->getClientSide()->getOptions()->toArray();
-			else
-				throw new TConfigurationException('callback_invalid_callback_options', $this->getControl()->getID(), $id);
+			if($control instanceof TCallbackOptions) {
+							return $control->getClientSide()->getOptions()->toArray();
+			} else {
+							throw new TConfigurationException('callback_invalid_callback_options', $this->getControl()->getID(), $id);
+			}
 		}
 
 		return [];
@@ -179,9 +181,9 @@ 
 		{
 			$group = $this->getValidationGroup();
 			return $this->getPage()->getValidators($group)->getCount() > 0;
+		} else {
+					return false;
 		}
-		else
-			return false;
 	}
 
 	/**
@@ -213,8 +215,9 @@ 
 		$options['ValidationGroup'] = $this->getValidationGroup();
 		$options['CallbackParameter'] = $this->getCallbackParameter();
 		// needed for TCallback
-		if(!isset($options['EventTarget']))
-			$options['EventTarget'] = $this->getControl()->getUniqueID();
+		if(!isset($options['EventTarget'])) {
+					$options['EventTarget'] = $this->getControl()->getUniqueID();
+		}
 		return $options;
 	}
 
@@ -230,14 +233,16 @@ 
 	public function registerCallbackClientScript($class, $options = null)
 	{
 		$cs = $this->getPage()->getClientScript();
-		if(is_array($options))
-			$options = array_merge($this->getClientSideOptions(), $options);
-		else
-			$options = $this->getClientSideOptions();
+		if(is_array($options)) {
+					$options = array_merge($this->getClientSideOptions(), $options);
+		} else {
+					$options = $this->getClientSideOptions();
+		}
 
 		//remove true as default to save bytes
-		if($options['CausesValidation'] === true)
-			$options['CausesValidation'] = '';
+		if($options['CausesValidation'] === true) {
+					$options['CausesValidation'] = '';
+		}
 		$cs->registerCallbackControl($class, $options);
 	}
 

framework/Web/UI/ActiveControls/TCallbackResponseWriter.php

Braces +3 added lines, -2 removed lines

@@ -68,8 +68,9 @@
 	public function flush()
 	{
 		$content = parent::flush();
-		if(empty($content))
-			return "";
+		if(empty($content)) {
+					return "";
+		}
 		return '<!--' . $this->getBoundary() . '-->' . $content . '<!--//' . $this->getBoundary() . '-->';
 	}
 }