pradosoft / prado

framework/Web/UI/ActiveControls/TScalarDiff.php

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
 	 */
 	public function getDifference()
 	{
-		if(gettype($this->_new) === gettype($this->_old)
+		if (gettype($this->_new) === gettype($this->_old)
 			&& $this->_new === $this->_old)
 			return $this->_null;
 		else

Braces +5 added lines, -4 removed lines

@@ -28,9 +28,10 @@
 	public function getDifference()
 	{
 		if(gettype($this->_new) === gettype($this->_old)
-			&& $this->_new === $this->_old)
-			return $this->_null;
-		else
-			return $this->_new;
+			&& $this->_new === $this->_old) {
+					return $this->_null;
+		} else {
+					return $this->_new;
+		}
 	}
 }

framework/Web/UI/ActiveControls/TActiveFileUpload.php

Spacing +37 added lines, -37 removed lines

@@ -89,7 +89,7 @@ 
 	 * TActiveControlAdapter. If you override this class, be sure to set the
 	 * adapter appropriately by, for example, by calling this constructor.
 	 */
-	public function __construct(){
+	public function __construct() {
 		parent::__construct();
 		$this->setAdapter(new TActiveControlAdapter($this));
 	}
@@ -102,7 +102,7 @@ 
 	protected function getAssetUrl($file = '')
 	{
 		$base = $this->getPage()->getClientScript()->getPradoScriptAssetUrl();
-		return $base . '/' . self::SCRIPT_PATH . '/' . $file;
+		return $base.'/'.self::SCRIPT_PATH.'/'.$file;
 	}
 
 
@@ -114,7 +114,7 @@ 
 	 */
 	public function onFileUpload($param)
 	{
-		if ($this->_flag->getValue() && $this->getPage()->getIsPostBack() && $param == $this->_target->getUniqueID()){
+		if ($this->_flag->getValue() && $this->getPage()->getIsPostBack() && $param == $this->_target->getUniqueID()) {
 		  $params = new TActiveFileUploadCallbackParams;
 		  // save the files so that they will persist past the end of this return.
 		  foreach ($this->getFiles() as $file) {
@@ -129,12 +129,12 @@ 
           	 Options = new Object();
           	 Options.clientID = '{$this->getClientID()}';
           	 Options.targetID = '{$this->_target->getUniqueID()}';
-          	 Options.errorCode = '" . (int)!$this->getHasAllFiles() . "';
+          	 Options.errorCode = '".(int) !$this->getHasAllFiles()."';
           	 Options.callbackToken = '{$this->pushParamsAndGetToken($params)}';
-          	 Options.fileName = '" . TJavaScript::jsonEncode($this->getMultiple() ? array_column($params->files, 'fileName') : $this->getFileName(), JSON_HEX_APOS) . "';
-             Options.fileSize = '" . TJavaScript::jsonEncode($this->getMultiple() ? array_column($params->files, 'fileSize') : $this->getFileSize(), JSON_HEX_APOS) . "';
-             Options.fileType = '" . TJavaScript::jsonEncode($this->getMultiple() ? array_column($params->files, 'fileType') : $this->getFileType(), JSON_HEX_APOS) . "';
-             Options.errorCode = '" . TJavaScript::jsonEncode($this->getMultiple() ? array_column($params->files, 'errorCode') : $this->getErrorCode(), JSON_HEX_APOS) . "';
+          	 Options.fileName = '".TJavaScript::jsonEncode($this->getMultiple() ? array_column($params->files, 'fileName') : $this->getFileName(), JSON_HEX_APOS)."';
+             Options.fileSize = '" . TJavaScript::jsonEncode($this->getMultiple() ? array_column($params->files, 'fileSize') : $this->getFileSize(), JSON_HEX_APOS)."';
+             Options.fileType = '" . TJavaScript::jsonEncode($this->getMultiple() ? array_column($params->files, 'fileType') : $this->getFileType(), JSON_HEX_APOS)."';
+             Options.errorCode = '" . TJavaScript::jsonEncode($this->getMultiple() ? array_column($params->files, 'errorCode') : $this->getErrorCode(), JSON_HEX_APOS)."';
           	 parent.Prado.WebUI.TActiveFileUpload.onFileUpload(Options);
            </script>";
 
@@ -146,7 +146,7 @@ 
 	 * @return string the path where the uploaded file will be stored temporarily, in namespace format
 	 * default "Application.runtime.*"
 	 */
-	public function getTempPath(){
+	public function getTempPath() {
 		return $this->getViewState('TempPath', 'Application.runtime.*');
 	}
 
@@ -154,7 +154,7 @@ 
 	 * @param string the path where the uploaded file will be stored temporarily in namespace format
 	 * default "Application.runtime.*"
 	 */
-	public function setTempPath($value){
+	public function setTempPath($value) {
 		$this->setViewState('TempPath', $value, 'Application.runtime.*');
 	}
 
@@ -162,7 +162,7 @@ 
 	 * @return boolean a value indicating whether an automatic callback to the server will occur whenever the user modifies the text in the TTextBox control and then tabs out of the component. Defaults to true.
 	 * Note: When set to false, you will need to trigger the callback yourself.
 	 */
-	public function getAutoPostBack(){
+	public function getAutoPostBack() {
 		return $this->getViewState('AutoPostBack', true);
 	}
 
@@ -170,28 +170,28 @@ 
 	 * @param boolean a value indicating whether an automatic callback to the server will occur whenever the user modifies the text in the TTextBox control and then tabs out of the component. Defaults to true.
 	 * Note: When set to false, you will need to trigger the callback yourself.
 	 */
-	public function setAutoPostBack($value){
+	public function setAutoPostBack($value) {
 		$this->setViewState('AutoPostBack', TPropertyValue::ensureBoolean($value), true);
 	}
 
 	/**
 	 * @return string A chuck of javascript that will need to be called if {{@link getAutoPostBack AutoPostBack} is set to false}
 	 */
-	public function getCallbackJavascript(){
+	public function getCallbackJavascript() {
 		return "Prado.WebUI.TActiveFileUpload.fileChanged(\"{$this->getClientID()}\")";
 	}
 
 	/**
 	 * @throws TInvalidDataValueException if the {@link getTempPath TempPath} is not writable.
 	 */
-	public function onInit($sender){
+	public function onInit($sender) {
 		parent::onInit($sender);
 
 		if (!Prado::getApplication()->getCache())
 		  if (!Prado::getApplication()->getSecurityManager())
 			throw new Exception('TActiveFileUpload needs either an application level cache or a security manager to work securely');
 
-		if (!is_writable(Prado::getPathOfNamespace($this->getTempPath()))){
+		if (!is_writable(Prado::getPathOfNamespace($this->getTempPath()))) {
 			throw new TInvalidDataValueException("activefileupload_temppath_invalid", $this->getTempPath());
 		}
 	}
@@ -203,12 +203,12 @@ 
 	 * This method is mainly used by framework and control developers.
 	 * @param TCallbackEventParameter the event parameter
 	 */
-	public function raiseCallbackEvent($param){
+	public function raiseCallbackEvent($param) {
 		$cp = $param->getCallbackParameter();
-		if ($key = $cp->targetID == $this->_target->getUniqueID()){
+		if ($key = $cp->targetID == $this->_target->getUniqueID()) {
 
 			$params = $this->popParamsByToken($cp->callbackToken);
-	  foreach($params->files as $index => $file)
+	  foreach ($params->files as $index => $file)
 	  {
 			$_FILES[$key]['name'][$index] = stripslashes($file['fileName']);
 			$_FILES[$key]['size'][$index] = intval($file['fileSize']);
@@ -237,7 +237,7 @@ 
 		if ($cache = Prado::getApplication()->getCache())
 			{
 				// this is the most secure method, file info can't be forged from client side, no matter what
-				$token = md5('TActiveFileUpload::Params::' . $this->ClientID . '::' . rand(1000 * 1000, 9999 * 1000));
+				$token = md5('TActiveFileUpload::Params::'.$this->ClientID.'::'.rand(1000 * 1000, 9999 * 1000));
 				$cache->set($token, serialize($params), 5 * 60); // expire in 5 minutes - the callback should arrive back in seconds, actually
 			}
 		elseif ($mgr = Prado::getApplication()->getSecurityManager())
@@ -280,7 +280,7 @@ 
 	{
 		parent::onPreRender($param);
 
-		if(!$this->getPage()->getIsPostBack() && isset($_GET['TActiveFileUpload_InputId']) && isset($_GET['TActiveFileUpload_TargetId']) && $_GET['TActiveFileUpload_InputId'] == $this->getClientID())
+		if (!$this->getPage()->getIsPostBack() && isset($_GET['TActiveFileUpload_InputId']) && isset($_GET['TActiveFileUpload_TargetId']) && $_GET['TActiveFileUpload_InputId'] == $this->getClientID())
 		{
 		  $params = new TActiveFileUploadCallbackParams;
 		  foreach ($this->getFiles() as $file) {
@@ -295,12 +295,12 @@ 
           	 Options = new Object();
           	 Options.clientID = '{$_GET['TActiveFileUpload_InputId']}';
           	 Options.targetID = '{$_GET['TActiveFileUpload_TargetId']}';
-			       Options.errorCode = '" . (int)!$this->getHasAllFiles() . "';
+			       Options.errorCode = '".(int) !$this->getHasAllFiles()."';
           	 Options.callbackToken = '{$this->pushParamsAndGetToken($params)}';
-          	 Options.fileName = '" . TJavaScript::jsonEncode($this->getMultiple() ? array_column($params->files, 'fileName') : $this->getFileName(), JSON_HEX_APOS) . "';
-             Options.fileSize = '" . TJavaScript::jsonEncode($this->getMultiple() ? array_column($params->files, 'fileSize') : $this->getFileSize(), JSON_HEX_APOS) . "';
-             Options.fileType = '" . TJavaScript::jsonEncode($this->getMultiple() ? array_column($params->files, 'fileType') : $this->getFileType(), JSON_HEX_APOS) . "';
-             Options.errorCode = '" . TJavaScript::jsonEncode($this->getMultiple() ? array_column($params->files, 'errorCode') : $this->getErrorCode(), JSON_HEX_APOS) . "';
+          	 Options.fileName = '".TJavaScript::jsonEncode($this->getMultiple() ? array_column($params->files, 'fileName') : $this->getFileName(), JSON_HEX_APOS)."';
+             Options.fileSize = '" . TJavaScript::jsonEncode($this->getMultiple() ? array_column($params->files, 'fileSize') : $this->getFileSize(), JSON_HEX_APOS)."';
+             Options.fileType = '" . TJavaScript::jsonEncode($this->getMultiple() ? array_column($params->files, 'fileType') : $this->getFileType(), JSON_HEX_APOS)."';
+             Options.errorCode = '" . TJavaScript::jsonEncode($this->getMultiple() ? array_column($params->files, 'errorCode') : $this->getErrorCode(), JSON_HEX_APOS)."';
            	 parent.Prado.WebUI.TActiveFileUpload.onFileUpload(Options);
            </script>";
 		}
@@ -343,11 +343,11 @@ 
 	/**
 	 * Removes localfile on ending of the callback.
 	 */
-	public function onUnload($param){
+	public function onUnload($param) {
 		if ($this->getPage()->getIsCallback())
 		{
-		  foreach($this->getFiles() as $file)
-			if($file->getHasFile() && file_exists($file->getLocalName())){
+		  foreach ($this->getFiles() as $file)
+			if ($file->getHasFile() && file_exists($file->getLocalName())) {
 			  unlink($file->getLocalName());
 		  }
 		}
@@ -357,7 +357,7 @@ 
 	/**
 	 * @return TBaseActiveCallbackControl standard callback control options.
 	 */
-	public function getActiveControl(){
+	public function getActiveControl() {
 		return $this->getAdapter()->getBaseActiveControl();
 	}
 
@@ -373,7 +373,7 @@ 
 	 * Adds ID attribute, and renders the javascript for active component.
 	 * @param THtmlWriter the writer used for the rendering purpose
 	 */
-	public function addAttributesToRender($writer){
+	public function addAttributesToRender($writer) {
 		parent::addAttributesToRender($writer);
 		$writer->addAttribute('id', $this->getClientID());
 
@@ -383,7 +383,7 @@ 
 	/**
 	 * @return string corresponding javascript class name for this control.
 	 */
-	protected function getClientClassName(){
+	protected function getClientClassName() {
 		return 'Prado.WebUI.TActiveFileUpload';
 	}
 
@@ -397,7 +397,7 @@ 
 	 * 					completeID => complete client ID,
 	 * 					errorID => error client ID)
 	 */
-	protected function getClientOptions(){
+	protected function getClientOptions() {
 		$options['ID'] = $this->getClientID();
 		$options['EventTarget'] = $this->getUniqueID();
 
@@ -419,8 +419,8 @@ 
 	 * If true, you will not be able to save the uploaded file again.
 	 * @return boolean true if the file saving is successful
 	 */
-	public function saveAs($fileName, $deleteTempFile = true, $index = 0){
-		if (($this->getErrorCode($index) === UPLOAD_ERR_OK) && (file_exists($this->getLocalName($index)))){
+	public function saveAs($fileName, $deleteTempFile = true, $index = 0) {
+		if (($this->getErrorCode($index) === UPLOAD_ERR_OK) && (file_exists($this->getLocalName($index)))) {
 			if ($deleteTempFile)
 				return rename($this->getLocalName($index), $fileName);
 			else
@@ -433,7 +433,7 @@ 
 	 * @return TImage the image displayed when an upload
 	 * 		completes successfully.
 	 */
-	public function getSuccessImage(){
+	public function getSuccessImage() {
 		$this->ensureChildControls();
 		return $this->_success;
 	}
@@ -442,7 +442,7 @@ 
 	 * @return TImage the image displayed when an upload
 	 * 		does not complete successfully.
 	 */
-	public function getErrorImage(){
+	public function getErrorImage() {
 		$this->ensureChildControls();
 		return $this->_error;
 	}
@@ -451,7 +451,7 @@ 
 	 * @return TImage the image displayed when an upload
 	 * 		is in progress.
 	 */
-	public function getBusyImage(){
+	public function getBusyImage() {
 		$this->ensureChildControls();
 		return $this->_busy;
 	}

Braces +22 added lines, -20 removed lines

@@ -187,9 +187,10 @@ 
 	public function onInit($sender){
 		parent::onInit($sender);
 
-		if (!Prado::getApplication()->getCache())
-		  if (!Prado::getApplication()->getSecurityManager())
+		if (!Prado::getApplication()->getCache()) {
+				  if (!Prado::getApplication()->getSecurityManager())
 			throw new Exception('TActiveFileUpload needs either an application level cache or a security manager to work securely');
+		}
 
 		if (!is_writable(Prado::getPathOfNamespace($this->getTempPath()))){
 			throw new TInvalidDataValueException("activefileupload_temppath_invalid", $this->getTempPath());
@@ -239,14 +240,13 @@ 
 				// this is the most secure method, file info can't be forged from client side, no matter what
 				$token = md5('TActiveFileUpload::Params::' . $this->ClientID . '::' . rand(1000 * 1000, 9999 * 1000));
 				$cache->set($token, serialize($params), 5 * 60); // expire in 5 minutes - the callback should arrive back in seconds, actually
-			}
-		elseif ($mgr = Prado::getApplication()->getSecurityManager())
+			} elseif ($mgr = Prado::getApplication()->getSecurityManager())
 			{
 				// this is a less secure method, file info can be still forged from client side, but only if attacker knows the secret application key
 				$token = urlencode(base64_encode($mgr->encrypt(serialize($params))));
-			}
-		else
-			throw new Exception('TActiveFileUpload needs either an application level cache or a security manager to work securely');
+			} else {
+					throw new Exception('TActiveFileUpload needs either an application level cache or a security manager to work securely');
+		}
 
 		return $token;
 	}
@@ -259,14 +259,13 @@ 
 				assert($v != '');
 				$cache->delete($token); // remove it from cache so it can't be used again and won't take up space either
 				$params = unserialize($v);
-			}
-		elseif ($mgr = Prado::getApplication()->getSecurityManager())
+			} elseif ($mgr = Prado::getApplication()->getSecurityManager())
 			{
 				$v = $mgr->decrypt(base64_decode(urldecode($token)));
 				$params = unserialize($v);
-			}
-		else
-			throw new Exception('TActiveFileUpload needs either an application level cache or a security manager to work securely');
+			} else {
+					throw new Exception('TActiveFileUpload needs either an application level cache or a security manager to work securely');
+		}
 
 		assert($params instanceof TActiveFileUploadCallbackParams);
 
@@ -346,10 +345,11 @@ 
 	public function onUnload($param){
 		if ($this->getPage()->getIsCallback())
 		{
-		  foreach($this->getFiles() as $file)
-			if($file->getHasFile() && file_exists($file->getLocalName())){
+		  foreach($this->getFiles() as $file) {
+		  			if($file->getHasFile() && file_exists($file->getLocalName())){
 			  unlink($file->getLocalName());
 		  }
+		  }
 		}
 		parent::onUnload($param);
 	}
@@ -421,12 +421,14 @@ 
 	 */
 	public function saveAs($fileName, $deleteTempFile = true, $index = 0){
 		if (($this->getErrorCode($index) === UPLOAD_ERR_OK) && (file_exists($this->getLocalName($index)))){
-			if ($deleteTempFile)
-				return rename($this->getLocalName($index), $fileName);
-			else
-				return copy($this->getLocalName($index), $fileName);
-		} else
-			return false;
+			if ($deleteTempFile) {
+							return rename($this->getLocalName($index), $fileName);
+			} else {
+							return copy($this->getLocalName($index), $fileName);
+			}
+		} else {
+					return false;
+		}
 	}
 
 	/**

framework/Web/UI/ActiveControls/TActiveLiteralColumn.php

Spacing +10 added lines, -10 removed lines

@@ -42,10 +42,10 @@ 
 	protected function initializeHeaderCell($cell, $columnIndex) {
 		$text = $this->getHeaderText();
 
-		if(($classPath = $this->getHeaderRenderer()) !== '') {
+		if (($classPath = $this->getHeaderRenderer()) !== '') {
 			$control = Prado::createComponent($classPath);
-			if($control instanceof \Prado\IDataRenderer) {
-				if($control instanceof IItemDataRenderer) {
+			if ($control instanceof \Prado\IDataRenderer) {
+				if ($control instanceof IItemDataRenderer) {
 					$item = $cell->getParent();
 					$control->setItemIndex($item->getItemIndex());
 					$control->setItemType($item->getItemType());
@@ -54,21 +54,21 @@ 
 			}
 			$cell->getControls()->add($control);
 		}
-		elseif($this->getAllowSorting()) {
+		elseif ($this->getAllowSorting()) {
 				$sortExpression = $this->getSortExpression();
-				if(($url = $this->getHeaderImageUrl()) !== '') {
+				if (($url = $this->getHeaderImageUrl()) !== '') {
 					$button = new TActiveImageButton;
 					$button->setImageUrl($url);
 					$button->setCommandName(TDataGrid::CMD_SORT);
 					$button->setCommandParameter($sortExpression);
-					if($text !== '') {
+					if ($text !== '') {
 						$button->setAlternateText($text);
 						$button->setToolTip($text);
 					}
 					$button->setCausesValidation(false);
 					$cell->getControls()->add($button);
 				}
-				elseif($text !== '') {
+				elseif ($text !== '') {
 						$button = new TActiveLinkButton;
 						$button->setText($text);
 						$button->setCommandName(TDataGrid::CMD_SORT);
@@ -80,16 +80,16 @@ 
 						$cell->setText(' ');
 			}
 			else {
-				if(($url = $this->getHeaderImageUrl()) !== '') {
+				if (($url = $this->getHeaderImageUrl()) !== '') {
 					$image = new TActiveImage;
 					$image->setImageUrl($url);
-					if($text !== '') {
+					if ($text !== '') {
 						$image->setAlternateText($text);
 						$image->setToolTip($text);
 					}
 					$cell->getControls()->add($image);
 				}
-				elseif($text !== '')
+				elseif ($text !== '')
 						$cell->setText($text);
 					else
 						$cell->setText(' ');

Braces +10 added lines, -13 removed lines

@@ -53,8 +53,7 @@ 
 				$control->setData($text);
 			}
 			$cell->getControls()->add($control);
-		}
-		elseif($this->getAllowSorting()) {
+		} elseif($this->getAllowSorting()) {
 				$sortExpression = $this->getSortExpression();
 				if(($url = $this->getHeaderImageUrl()) !== '') {
 					$button = new TActiveImageButton;
@@ -67,19 +66,17 @@ 
 					}
 					$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);
@@ -88,11 +85,11 @@ 
 						$image->setToolTip($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/TActiveListItemCollection.php

Spacing +2 added lines, -2 removed lines

@@ -94,7 +94,7 @@ 
 	public function insertAt($index, $value)
 	{
 		parent::insertAt($index, $value);
-		if($this->canUpdateClientSide())
+		if ($this->canUpdateClientSide())
 			$this->_hasChanged = true;
 	}
 
@@ -105,7 +105,7 @@ 
 	public function removeAt($index)
 	{
 		parent::removeAt($index);
-		if($this->canUpdateClientSide())
+		if ($this->canUpdateClientSide())
 			$this->_hasChanged = true;
 	}
 }
\ No newline at end of file

Braces +6 added lines, -4 removed lines

@@ -94,8 +94,9 @@ 
 	public function insertAt($index, $value)
 	{
 		parent::insertAt($index, $value);
-		if($this->canUpdateClientSide())
-			$this->_hasChanged = true;
+		if($this->canUpdateClientSide()) {
+					$this->_hasChanged = true;
+		}
 	}
 
 	/**
@@ -105,7 +106,8 @@ 
 	public function removeAt($index)
 	{
 		parent::removeAt($index);
-		if($this->canUpdateClientSide())
-			$this->_hasChanged = true;
+		if($this->canUpdateClientSide()) {
+					$this->_hasChanged = true;
+		}
 	}
 }
\ No newline at end of file

framework/Web/UI/ActiveControls/TActiveListControlAdapter.php

Spacing +18 added lines, -18 removed lines

@@ -44,18 +44,18 @@ 
 	 */
 	public function setSelectedIndex($index)
 	{
-		if($this->canUpdateClientSide())
+		if ($this->canUpdateClientSide())
 		{
 			$this->updateListItems();
 			// if a prompt is set, we mimic the postback behaviour of not counting it
 			// in the index. We assume the prompt is _always_ the first item (Issue #368)
 			$promptValue = $this->getControl()->getPromptValue();
-			if($promptValue === '')
+			if ($promptValue === '')
 				$promptValue = $this->getControl()->getPromptText();
-			if($promptValue !== '')
+			if ($promptValue !== '')
 				$index++;
 
-			if($index >= 0 && $index <= $this->getControl()->getItemCount())
+			if ($index >= 0 && $index <= $this->getControl()->getItemCount())
 			$this->getPage()->getCallbackClient()->select(
 					$this->getControl(), 'Index', $index);
 		}
@@ -67,25 +67,25 @@ 
 	 */
 	public function setSelectedIndices($indices)
 	{
-		if($this->canUpdateClientSide())
+		if ($this->canUpdateClientSide())
 		{
 			$this->updateListItems();
 			$n = $this->getControl()->getItemCount();
 
 			$promptValue = $this->getControl()->getPromptValue();
-			if($promptValue === '')
+			if ($promptValue === '')
 				$promptValue = $this->getControl()->getPromptText();
 
 			$list = [];
-			foreach($indices as $index)
+			foreach ($indices as $index)
 			{
 				$index = intval($index);
-				if($promptValue !== '')
+				if ($promptValue !== '')
 					$index++;
-				if($index >= 0 && $index <= $n)
+				if ($index >= 0 && $index <= $n)
 					$list[] = $index;
 			}
-			if(count($list) > 0)
+			if (count($list) > 0)
 				$this->getPage()->getCallbackClient()->select(
 					$this->getControl(), 'Indices', $list);
 		}
@@ -97,7 +97,7 @@ 
 	 */
 	public function setSelectedValue($value)
 	{
-		if($this->canUpdateClientSide())
+		if ($this->canUpdateClientSide())
 		{
 			$this->updateListItems();
 			$this->getPage()->getCallbackClient()->select(
@@ -111,13 +111,13 @@ 
 	 */
 	public function setSelectedValues($values)
 	{
-		if($this->canUpdateClientSide())
+		if ($this->canUpdateClientSide())
 		{
 			$this->updateListItems();
 			$list = [];
-			foreach($values as $value)
+			foreach ($values as $value)
 				$list[] = $value;
-			if(count($list) > 0)
+			if (count($list) > 0)
 				$this->getPage()->getCallbackClient()->select(
 					$this->getControl(), 'Values', $list);
 		}
@@ -128,10 +128,10 @@ 
 	 */
 	public function clearSelection()
 	{
-		if($this->canUpdateClientSide())
+		if ($this->canUpdateClientSide())
 		{
 			$this->updateListItems();
-			if($this->getControl() instanceof TActiveDropDownList)
+			if ($this->getControl() instanceof TActiveDropDownList)
 			{
 				// clearing a TActiveDropDownList's selection actually doesn't select the first item;
 				// we mimic the postback behaviour selecting it (Issue #368)
@@ -147,10 +147,10 @@ 
 	 */
 	public function updateListItems()
 	{
-		if($this->canUpdateClientSide())
+		if ($this->canUpdateClientSide())
 		{
 			$items = $this->getControl()->getItems();
-			if($items instanceof TActiveListItemCollection
+			if ($items instanceof TActiveListItemCollection
 				&& $items->getListHasChanged())
 			{
 				$items->updateClientSide();

Braces +27 added lines, -18 removed lines

@@ -50,14 +50,17 @@ 
 			// if a prompt is set, we mimic the postback behaviour of not counting it
 			// in the index. We assume the prompt is _always_ the first item (Issue #368)
 			$promptValue = $this->getControl()->getPromptValue();
-			if($promptValue === '')
-				$promptValue = $this->getControl()->getPromptText();
-			if($promptValue !== '')
-				$index++;
+			if($promptValue === '') {
+							$promptValue = $this->getControl()->getPromptText();
+			}
+			if($promptValue !== '') {
+							$index++;
+			}
 
-			if($index >= 0 && $index <= $this->getControl()->getItemCount())
-			$this->getPage()->getCallbackClient()->select(
+			if($index >= 0 && $index <= $this->getControl()->getItemCount()) {
+						$this->getPage()->getCallbackClient()->select(
 					$this->getControl(), 'Index', $index);
+			}
 		}
 	}
 
@@ -73,21 +76,25 @@ 
 			$n = $this->getControl()->getItemCount();
 
 			$promptValue = $this->getControl()->getPromptValue();
-			if($promptValue === '')
-				$promptValue = $this->getControl()->getPromptText();
+			if($promptValue === '') {
+							$promptValue = $this->getControl()->getPromptText();
+			}
 
 			$list = [];
 			foreach($indices as $index)
 			{
 				$index = intval($index);
-				if($promptValue !== '')
-					$index++;
-				if($index >= 0 && $index <= $n)
-					$list[] = $index;
+				if($promptValue !== '') {
+									$index++;
+				}
+				if($index >= 0 && $index <= $n) {
+									$list[] = $index;
+				}
 			}
-			if(count($list) > 0)
-				$this->getPage()->getCallbackClient()->select(
+			if(count($list) > 0) {
+							$this->getPage()->getCallbackClient()->select(
 					$this->getControl(), 'Indices', $list);
+			}
 		}
 	}
 
@@ -115,11 +122,13 @@ 
 		{
 			$this->updateListItems();
 			$list = [];
-			foreach($values as $value)
-				$list[] = $value;
-			if(count($list) > 0)
-				$this->getPage()->getCallbackClient()->select(
+			foreach($values as $value) {
+							$list[] = $value;
+			}
+			if(count($list) > 0) {
+							$this->getPage()->getCallbackClient()->select(
 					$this->getControl(), 'Values', $list);
+			}
 		}
 	}
 

framework/Web/UI/ActiveControls/TActiveDataList.php

Spacing +7 added lines, -7 removed lines

@@ -57,7 +57,7 @@ 
   public function setDataSource($value)
   {
 	parent::setDataSource($value);
-	if($this->getActiveControl()->canUpdateClientSide()) {
+	if ($this->getActiveControl()->canUpdateClientSide()) {
 	  $this->renderPager();
 	  $this->getPage()->getAdapter()->registerControlToRender($this, $this->getResponse()->createHtmlWriter());
 	}
@@ -69,7 +69,7 @@ 
    */
   protected function getContainerID()
   {
-	return $this->ClientID . '_Container';
+	return $this->ClientID.'_Container';
   }
 
   /**
@@ -80,9 +80,9 @@ 
    */
   public function render($writer)
   {
-	if($this->getHasPreRendered()) {
+	if ($this->getHasPreRendered()) {
 	  $this->renderDataList($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);
@@ -97,9 +97,9 @@ 
   private function renderPager()
   {
 	$pager = $this->getPage()->findControlsByType('Prado\Web\UI\ActiveControls\TActivePager', false);
-	foreach($pager as $item)
+	foreach ($pager as $item)
 	{
-	  if($item->ControlToPaginate == $this->ID) {
+	  if ($item->ControlToPaginate == $this->ID) {
 		$writer = $this->getResponse()->createHtmlWriter();
 		$this->getPage()->getAdapter()->registerControlToRender($item, $writer);
 	  }
@@ -113,7 +113,7 @@ 
    */
   private function renderDataList($writer)
   {
-	$writer->write('<span id="' . $this->getContainerID() . '">');
+	$writer->write('<span id="'.$this->getContainerID().'">');
 	parent::render($writer);
 	$writer->write('</span>');
   }

Braces +4 added lines, -3 removed lines

@@ -82,9 +82,10 @@
   {
 	if($this->getHasPreRendered()) {
 	  $this->renderDataList($writer);
-	  if($this->getActiveControl()->canUpdateClientSide()) $this->getPage()->getCallbackClient()->replaceContent($this->getContainerID(), $writer);
-	}
-	else {
+	  if($this->getActiveControl()->canUpdateClientSide()) {
+	  	$this->getPage()->getCallbackClient()->replaceContent($this->getContainerID(), $writer);
+	  }
+	} else {
 	  $this->getPage()->getAdapter()->registerControlToRender($this, $writer);
 	}
   }

framework/Web/UI/ActiveControls/TActiveHtmlArea.php

Spacing +2 added lines, -2 removed lines

@@ -64,9 +64,9 @@
    */
   public function setText($value) {
 	parent::setText($value);
-	if($this->getActiveControl()->canUpdateClientSide() && $this->getHasLoadedPostData())
+	if ($this->getActiveControl()->canUpdateClientSide() && $this->getHasLoadedPostData())
 	{
-	  if($this->getEnableVisualEdit())
+	  if ($this->getEnableVisualEdit())
 	  {
 		$value = str_ireplace(["\r\n", "\n"], "", $value);
 		$command = "tinyMCE.getInstanceById('{$this->getClientID()}').execCommand('mceSetContent',false,'{$value}')";

Braces +2 added lines, -2 removed lines

@@ -71,9 +71,9 @@
 		$value = str_ireplace(["\r\n", "\n"], "", $value);
 		$command = "tinyMCE.getInstanceById('{$this->getClientID()}').execCommand('mceSetContent',false,'{$value}')";
 		$this->getPage()->getCallbackClient()->evaluateScript($command);
+	  } else {
+	  		$this->getPage()->getCallbackClient()->setValue($this, $value);
 	  }
-	  else
-		$this->getPage()->getCallbackClient()->setValue($this, $value);
 	}
   }
 

framework/Web/UI/ActiveControls/TInPlaceTextBox.php

Spacing +14 added lines, -14 removed lines

@@ -82,7 +82,7 @@ 
 	{
 		$value = TPropertyValue::ensureBoolean($value);
 		$this->setViewState('DisplayTextBox', $value, false);
-		if($this->getActiveControl()->canUpdateClientSide())
+		if ($this->getActiveControl()->canUpdateClientSide())
 			$this->callClientFunction('setDisplayTextBox', $value);
 	}
 
@@ -102,8 +102,8 @@ 
 	protected function callClientFunction($func, $value)
 	{
 		$client = $this->getPage()->getCallbackClient();
-		$code = $this->getClientClassName() . '.' . $func;
-		$client->callClientFunction($code, [$this,$value]);
+		$code = $this->getClientClassName().'.'.$func;
+		$client->callClientFunction($code, [$this, $value]);
 	}
 
 	/**
@@ -128,8 +128,8 @@ 
 	protected function getExternalControlID()
 	{
 		$extID = $this->getEditTriggerControlID();
-		if($extID === null) return '';
-		if(($control = $this->findControl($extID)) !== null)
+		if ($extID === null) return '';
+		if (($control = $this->findControl($extID)) !== null)
 			return $control->getClientID();
 		return $extID;
 	}
@@ -141,11 +141,11 @@ 
 	 */
 	public function setText($value)
 	{
-		if(TTextBox::getText() === $value)
+		if (TTextBox::getText() === $value)
 			return;
 
 		TTextBox::setText($value);
-		if($this->getActiveControl()->canUpdateClientSide())
+		if ($this->getActiveControl()->canUpdateClientSide())
 		{
 			$client = $this->getPage()->getCallbackClient();
 			$client->update($this->getLabelClientID(), $value);
@@ -158,10 +158,10 @@ 
 	 * @param boolean value
 	 * @since 3.1.2
 	 */
-	public function setReadOnly ($value)
+	public function setReadOnly($value)
 	{
 		$value = TPropertyValue::ensureBoolean($value);
-		if(TTextBox::getReadOnly() === $value)
+		if (TTextBox::getReadOnly() === $value)
 			return;
 
 		TTextBox::setReadOnly($value);
@@ -185,7 +185,7 @@ 
 	 */
 	public function renderContents($writer)
 	{
-		if(($text = $this->getText()) === '')
+		if (($text = $this->getText()) === '')
 			parent::renderContents($writer);
 		else
 			$writer->write($text);
@@ -196,7 +196,7 @@ 
 	 */
 	protected function getLabelClientID()
 	{
-		return $this->getClientID() . '__label';
+		return $this->getClientID().'__label';
 	}
 
 	/**
@@ -209,7 +209,7 @@ 
 	public function onCallback($param)
 	{
 		$action = $param->getCallbackParameter();
-		if(is_array($action) && $action[0] === '__InlineEditor_loadExternalText__')
+		if (is_array($action) && $action[0] === '__InlineEditor_loadExternalText__')
 		{
 			$parameter = new TCallbackEventParameter($this->getResponse(), $action[1]);
 			$this->onLoadingText($parameter);
@@ -229,7 +229,7 @@ 
 		$options['AutoHide'] = $this->getAutoHideTextBox() == false ? '' : true;
 		$options['AutoPostBack'] = $this->getAutoPostBack() == false ? '' : true;
 		$options['Columns'] = $this->getColumns();
-		if($this->getTextMode() === 'MultiLine')
+		if ($this->getTextMode() === 'MultiLine')
 		{
 			$options['Rows'] = $this->getRows();
 			$options['Wrap'] = $this->getWrap() == false ? '' : true;
@@ -240,7 +240,7 @@ 
 			$options['MaxLength'] = $length > 0 ? $length : '';
 		}
 
-		if($this->hasEventHandler('OnLoadingText'))
+		if ($this->hasEventHandler('OnLoadingText'))
 			$options['LoadTextOnEdit'] = true;
 
 		$options['ReadOnly'] = $this->getReadOnly();

Braces +24 added lines, -17 removed lines

@@ -82,8 +82,9 @@ 
 	{
 		$value = TPropertyValue::ensureBoolean($value);
 		$this->setViewState('DisplayTextBox', $value, false);
-		if($this->getActiveControl()->canUpdateClientSide())
-			$this->callClientFunction('setDisplayTextBox', $value);
+		if($this->getActiveControl()->canUpdateClientSide()) {
+					$this->callClientFunction('setDisplayTextBox', $value);
+		}
 	}
 
 	/**
@@ -128,9 +129,12 @@ 
 	protected function getExternalControlID()
 	{
 		$extID = $this->getEditTriggerControlID();
-		if($extID === null) return '';
-		if(($control = $this->findControl($extID)) !== null)
-			return $control->getClientID();
+		if($extID === null) {
+			return '';
+		}
+		if(($control = $this->findControl($extID)) !== null) {
+					return $control->getClientID();
+		}
 		return $extID;
 	}
 
@@ -141,8 +145,9 @@ 
 	 */
 	public function setText($value)
 	{
-		if(TTextBox::getText() === $value)
-			return;
+		if(TTextBox::getText() === $value) {
+					return;
+		}
 
 		TTextBox::setText($value);
 		if($this->getActiveControl()->canUpdateClientSide())
@@ -161,8 +166,9 @@ 
 	public function setReadOnly ($value)
 	{
 		$value = TPropertyValue::ensureBoolean($value);
-		if(TTextBox::getReadOnly() === $value)
-			return;
+		if(TTextBox::getReadOnly() === $value) {
+					return;
+		}
 
 		TTextBox::setReadOnly($value);
 		if ($this->getActiveControl()->canUpdateClientSide())
@@ -185,10 +191,11 @@ 
 	 */
 	public function renderContents($writer)
 	{
-		if(($text = $this->getText()) === '')
-			parent::renderContents($writer);
-		else
-			$writer->write($text);
+		if(($text = $this->getText()) === '') {
+					parent::renderContents($writer);
+		} else {
+					$writer->write($text);
+		}
 	}
 
 	/**
@@ -233,15 +240,15 @@ 
 		{
 			$options['Rows'] = $this->getRows();
 			$options['Wrap'] = $this->getWrap() == false ? '' : true;
-		}
-		else
+		} else
 		{
 			$length = $this->getMaxLength();
 			$options['MaxLength'] = $length > 0 ? $length : '';
 		}
 
-		if($this->hasEventHandler('OnLoadingText'))
-			$options['LoadTextOnEdit'] = true;
+		if($this->hasEventHandler('OnLoadingText')) {
+					$options['LoadTextOnEdit'] = true;
+		}
 
 		$options['ReadOnly'] = $this->getReadOnly();
 		return $options;

framework/Web/UI/ActiveControls/TActiveRepeater.php

Spacing +6 added lines, -6 removed lines

@@ -61,7 +61,7 @@ 
 	 */
 	public function setDataSource($value) {
 		parent::setDataSource($value);
-		if($this->getActiveControl()->canUpdateClientSide()) {
+		if ($this->getActiveControl()->canUpdateClientSide()) {
 			$this->renderPager();
 			$this->getPage()->getAdapter()->registerControlToRender($this, $this->getResponse()->createHtmlWriter());
 		}
@@ -88,7 +88,7 @@ 
 	 * @return string container id
 	 */
 	public function getSurroundingTagID() {
-		return $this->getClientID() . '_Container';
+		return $this->getClientID().'_Container';
 	}
 
 	/**
@@ -98,9 +98,9 @@ 
 	 * @param THtmlWriter writer for the rendering purpose
 	 */
 	public function render($writer) {
-		if($this->getHasPreRendered()) {
+		if ($this->getHasPreRendered()) {
 			$this->renderRepeater($writer);
-			if($this->getActiveControl()->canUpdateClientSide()) $this->getPage()->getCallbackClient()->replaceContent($this->getSurroundingTagId(), $writer);
+			if ($this->getActiveControl()->canUpdateClientSide()) $this->getPage()->getCallbackClient()->replaceContent($this->getSurroundingTagId(), $writer);
 		}
 		else {
 			$this->getPage()->getAdapter()->registerControlToRender($this, $writer);
@@ -114,8 +114,8 @@ 
 	 */
 	private function renderPager() {
 		$pager = $this->getPage()->findControlsByType('Prado\Web\UI\ActiveControls\TActivePager', false);
-		foreach($pager as $item) {
-			if($item->ControlToPaginate == $this->ID) {
+		foreach ($pager as $item) {
+			if ($item->ControlToPaginate == $this->ID) {
 				$writer = $this->getResponse()->createHtmlWriter();
 				$this->getPage()->getAdapter()->registerControlToRender($item, $writer);
 			}

Braces +4 added lines, -3 removed lines

@@ -100,9 +100,10 @@
 	public function render($writer) {
 		if($this->getHasPreRendered()) {
 			$this->renderRepeater($writer);
-			if($this->getActiveControl()->canUpdateClientSide()) $this->getPage()->getCallbackClient()->replaceContent($this->getSurroundingTagId(), $writer);
-		}
-		else {
+			if($this->getActiveControl()->canUpdateClientSide()) {
+				$this->getPage()->getCallbackClient()->replaceContent($this->getSurroundingTagId(), $writer);
+			}
+		} else {
 			$this->getPage()->getAdapter()->registerControlToRender($this, $writer);
 		}
 	}