pradosoft / prado

framework/Web/UI/TControlCollection.php

Spacing +4 added lines, -4 removed lines

@@ -60,12 +60,12 @@ 
 	 */
 	public function insertAt($index, $item)
 	{
-		if($item instanceof TControl)
+		if ($item instanceof TControl)
 		{
 			parent::insertAt($index, $item);
 			$this->_o->addedControl($item);
 		}
-		elseif(is_string($item) || ($item instanceof IRenderable))
+		elseif (is_string($item) || ($item instanceof IRenderable))
 			parent::insertAt($index, $item);
 		else
 			throw new TInvalidDataTypeException('controlcollection_control_required');
@@ -81,7 +81,7 @@ 
 	public function removeAt($index)
 	{
 		$item = parent::removeAt($index);
-		if($item instanceof TControl)
+		if ($item instanceof TControl)
 			$this->_o->removedControl($item);
 		return $item;
 	}
@@ -92,7 +92,7 @@ 
 	public function clear()
 	{
 		parent::clear();
-		if($this->_o instanceof INamingContainer)
+		if ($this->_o instanceof INamingContainer)
 			$this->_o->clearNamingContainer();
 	}
 }
\ No newline at end of file

framework/Web/UI/TTheme.php

Spacing +41 added lines, -41 removed lines

@@ -89,33 +89,33 @@ 
 		$this->_name = basename($themePath);
 		$cacheValid = false;
 		// TODO: the following needs to be cleaned up (Qiang)
-		if(($cache = $this->getApplication()->getCache()) !== null)
+		if (($cache = $this->getApplication()->getCache()) !== null)
 		{
-			$array = $cache->get(self::THEME_CACHE_PREFIX . $themePath);
-			if(is_array($array))
+			$array = $cache->get(self::THEME_CACHE_PREFIX.$themePath);
+			if (is_array($array))
 			{
 				list($skins, $cssFiles, $jsFiles, $timestamp) = $array;
-				if($this->getApplication()->getMode() !== TApplicationMode::Performance)
+				if ($this->getApplication()->getMode() !== TApplicationMode::Performance)
 				{
-					if(($dir = opendir($themePath)) === false)
+					if (($dir = opendir($themePath)) === false)
 						throw new TIOException('theme_path_inexistent', $themePath);
 					$cacheValid = true;
-					while(($file = readdir($dir)) !== false)
+					while (($file = readdir($dir)) !== false)
 					{
-						if($file === '.' || $file === '..')
+						if ($file === '.' || $file === '..')
 							continue;
-						elseif(basename($file, '.css') !== $file)
-							$this->_cssFiles[] = $themeUrl . '/' . $file;
-						elseif(basename($file, '.js') !== $file)
-							$this->_jsFiles[] = $themeUrl . '/' . $file;
-						elseif(basename($file, self::SKIN_FILE_EXT) !== $file && filemtime($themePath . DIRECTORY_SEPARATOR . $file) > $timestamp)
+						elseif (basename($file, '.css') !== $file)
+							$this->_cssFiles[] = $themeUrl.'/'.$file;
+						elseif (basename($file, '.js') !== $file)
+							$this->_jsFiles[] = $themeUrl.'/'.$file;
+						elseif (basename($file, self::SKIN_FILE_EXT) !== $file && filemtime($themePath.DIRECTORY_SEPARATOR.$file) > $timestamp)
 						{
 							$cacheValid = false;
 							break;
 						}
 					}
 					closedir($dir);
-					if($cacheValid)
+					if ($cacheValid)
 						$this->_skins = $skins;
 				}
 				else
@@ -127,34 +127,34 @@ 
 				}
 			}
 		}
-		if(!$cacheValid)
+		if (!$cacheValid)
 		{
 			$this->_cssFiles = [];
 			$this->_jsFiles = [];
 			$this->_skins = [];
-			if(($dir = opendir($themePath)) === false)
+			if (($dir = opendir($themePath)) === false)
 				throw new TIOException('theme_path_inexistent', $themePath);
-			while(($file = readdir($dir)) !== false)
+			while (($file = readdir($dir)) !== false)
 			{
-				if($file === '.' || $file === '..')
+				if ($file === '.' || $file === '..')
 					continue;
-				elseif(basename($file, '.css') !== $file)
-					$this->_cssFiles[] = $themeUrl . '/' . $file;
-				elseif(basename($file, '.js') !== $file)
-					$this->_jsFiles[] = $themeUrl . '/' . $file;
-				elseif(basename($file, self::SKIN_FILE_EXT) !== $file)
+				elseif (basename($file, '.css') !== $file)
+					$this->_cssFiles[] = $themeUrl.'/'.$file;
+				elseif (basename($file, '.js') !== $file)
+					$this->_jsFiles[] = $themeUrl.'/'.$file;
+				elseif (basename($file, self::SKIN_FILE_EXT) !== $file)
 				{
-					$template = new TTemplate(file_get_contents($themePath . '/' . $file), $themePath, $themePath . '/' . $file);
-					foreach($template->getItems() as $skin)
+					$template = new TTemplate(file_get_contents($themePath.'/'.$file), $themePath, $themePath.'/'.$file);
+					foreach ($template->getItems() as $skin)
 					{
-						if(!isset($skin[2]))  // a text string, ignored
+						if (!isset($skin[2]))  // a text string, ignored
 							continue;
-						elseif($skin[0] !== -1)
+						elseif ($skin[0] !== -1)
 							throw new TConfigurationException('theme_control_nested', $skin[1], dirname($themePath));
 						$type = $skin[1];
-						$id = isset($skin[2]['skinid'])?$skin[2]['skinid']:0;
+						$id = isset($skin[2]['skinid']) ? $skin[2]['skinid'] : 0;
 						unset($skin[2]['skinid']);
-						if(isset($this->_skins[$type][$id]))
+						if (isset($this->_skins[$type][$id]))
 							throw new TConfigurationException('theme_skinid_duplicated', $type, $id, dirname($themePath));
 						/*
 						foreach($skin[2] as $name=>$value)
@@ -170,8 +170,8 @@ 
 			closedir($dir);
 			sort($this->_cssFiles);
 			sort($this->_jsFiles);
-			if($cache !== null)
-				$cache->set(self::THEME_CACHE_PREFIX . $themePath, [$this->_skins,$this->_cssFiles,$this->_jsFiles,time()]);
+			if ($cache !== null)
+				$cache->set(self::THEME_CACHE_PREFIX.$themePath, [$this->_skins, $this->_cssFiles, $this->_jsFiles, time()]);
 		}
 	}
 
@@ -251,22 +251,22 @@ 
 	public function applySkin($control)
 	{
 		$type = get_class($control);
-		if(($id = $control->getSkinID()) === '')
+		if (($id = $control->getSkinID()) === '')
 			$id = 0;
-		if(isset($this->_skins[$type][$id]))
+		if (isset($this->_skins[$type][$id]))
 		{
-			foreach($this->_skins[$type][$id] as $name => $value)
+			foreach ($this->_skins[$type][$id] as $name => $value)
 			{
 				Prado::trace("Applying skin $name to $type", 'Prado\Web\UI\TThemeManager');
-				if(is_array($value))
+				if (is_array($value))
 				{
-					switch($value[0])
+					switch ($value[0])
 					{
 						case TTemplate::CONFIG_EXPRESSION:
 							$value = $this->evaluateExpression($value[1]);
 							break;
 						case TTemplate::CONFIG_ASSET:
-							$value = $this->_themeUrl . '/' . ltrim($value[1], '/');
+							$value = $this->_themeUrl.'/'.ltrim($value[1], '/');
 							break;
 						case TTemplate::CONFIG_DATABIND:
 							$control->bindProperty($name, $value[1]);
@@ -285,15 +285,15 @@ 
 							break;
 					}
 				}
-				if(!is_array($value))
+				if (!is_array($value))
 				{
-					if(strpos($name, '.') === false)	// is simple property or custom attribute
+					if (strpos($name, '.') === false)	// is simple property or custom attribute
 					{
-						if($control->hasProperty($name))
+						if ($control->hasProperty($name))
 						{
-							if($control->canSetProperty($name))
+							if ($control->canSetProperty($name))
 							{
-								$setter = 'set' . $name;
+								$setter = 'set'.$name;
 								$control->$setter($value);
 							}
 							else

framework/Web/UI/ActiveControls/TActiveEditCommandColumn.php

Spacing +4 added lines, -4 removed lines

@@ -36,17 +36,17 @@
  */
 class TActiveEditCommandColumn extends TEditCommandColumn {
 	protected function createButton($commandName, $text, $causesValidation, $validationGroup) {
-		if($this->getButtonType() === TButtonColumnType::LinkButton)
+		if ($this->getButtonType() === TButtonColumnType::LinkButton)
 			$button = new TActiveLinkButton;
-		elseif($this->getButtonType() === TButtonColumnType::PushButton)
+		elseif ($this->getButtonType() === TButtonColumnType::PushButton)
 				$button = new TActiveButton;
 			else  // image buttons
 			{
 				$button = new TActiveImageButton;
 				$button->setToolTip($text);
-				if(strcasecmp($commandName, 'Update') === 0)
+				if (strcasecmp($commandName, 'Update') === 0)
 					$url = $this->getUpdateImageUrl();
-				elseif(strcasecmp($commandName, 'Cancel') === 0)
+				elseif (strcasecmp($commandName, 'Cancel') === 0)
 						$url = $this->getCancelImageUrl();
 					else
 						$url = $this->getEditImageUrl();

framework/Web/UI/ActiveControls/TCallbackResponseAdapter.php

Spacing +3 added lines, -3 removed lines

@@ -59,7 +59,7 @@ 
 	 */
 	public function flushContent()
 	{
-		foreach($this->_writers as $writer)
+		foreach ($this->_writers as $writer)
 			echo $writer->flush();
 		parent::flushContent();
 	}
@@ -86,8 +86,8 @@ 
 	 */
 	public function httpRedirect($url)
 	{
-		if($url[0] === '/')
-			$url = $this->getRequest()->getBaseUrl() . $url;
+		if ($url[0] === '/')
+			$url = $this->getRequest()->getBaseUrl().$url;
 		$this->_redirectUrl = str_replace('&', '&', $url);
 	}
 

framework/Web/UI/ActiveControls/TActiveLabel.php

Spacing +4 added lines, -4 removed lines

@@ -59,11 +59,11 @@ 
 	 */
 	public function setText($value)
 	{
-		if(parent::getText() === $value)
+		if (parent::getText() === $value)
 			return;
 
 		parent::setText($value);
-		if($this->getActiveControl()->canUpdateClientSide())
+		if ($this->getActiveControl()->canUpdateClientSide())
 			$this->getPage()->getCallbackClient()->update($this, $value);
 	}
 
@@ -75,11 +75,11 @@ 
 	 */
 	public function setForControl($value)
 	{
-		if(parent::getForControl() === $value)
+		if (parent::getForControl() === $value)
 			return;
 
 		parent::setForControl($value);
-		if($this->getActiveControl()->canUpdateClientSide())
+		if ($this->getActiveControl()->canUpdateClientSide())
 		{
 			$id = $this->findControl($value)->getClientID();
 			$this->getPage()->getCallbackClient()->setAttribute($this, 'for', $id);

framework/Web/UI/ActiveControls/TActiveControlAdapter.php

Spacing +9 added lines, -9 removed lines

@@ -70,9 +70,9 @@ 
 	 */
 	protected function setBaseControlClass($type)
 	{
-		if($type === null)
+		if ($type === null)
 		{
-			if($this->getControl() instanceof ICallbackEventHandler)
+			if ($this->getControl() instanceof ICallbackEventHandler)
 				$this->_activeControlType = 'Prado\\Web\UI\\ActiveControls\\TBaseActiveCallbackControl';
 			else
 				$this->_activeControlType = 'Prado\\Web\UI\\ActiveControls\\TBaseActiveControl';
@@ -95,11 +95,11 @@ 
 	public function render($writer)
 	{
 		$this->getPage()->getClientScript()->registerPradoScript('ajax');
-		if($this->_control->getVisible(false))
+		if ($this->_control->getVisible(false))
 		{
 			parent::render($writer);
 		} else {
-			$writer->write("<span id=\"" . $this->_control->getClientID() . "\" style=\"display:none\"></span>");
+			$writer->write("<span id=\"".$this->_control->getClientID()."\" style=\"display:none\"></span>");
 		}
 	}
 
@@ -116,7 +116,7 @@ 
 	 */
 	public function getBaseActiveControl()
 	{
-		if($this->_baseActiveControl === null)
+		if ($this->_baseActiveControl === null)
 		{
 			$type = $this->_activeControlType;
 			$this->_baseActiveControl = new $type($this->getControl());
@@ -129,10 +129,10 @@ 
 	 */
 	protected function getIsTrackingPageState()
 	{
-		if($this->getPage()->getIsCallback())
+		if ($this->getPage()->getIsCallback())
 		{
 			$target = $this->getPage()->getCallbackEventTarget();
-			if($target instanceof ICallbackEventHandler)
+			if ($target instanceof ICallbackEventHandler)
 			{
 				$client = $target->getActiveControl()->getClientSide();
 				return $client->getEnablePageStateUpdate();
@@ -146,7 +146,7 @@ 
 	 */
 	public function onLoad($param)
 	{
-		if($this->getIsTrackingPageState())
+		if ($this->getIsTrackingPageState())
 		{
 		  $stateTrackerClass = $this->_stateTrackerClass;
 			$this->_stateTracker = new $stateTrackerClass($this->getControl());
@@ -161,7 +161,7 @@ 
 	 */
 	public function saveState()
 	{
-		if(($this->_stateTracker !== null)
+		if (($this->_stateTracker !== null)
 			&& $this->getControl()->getActiveControl()->canUpdateClientSide(true))
 		{
 			$this->_stateTracker->respondToChanges();

framework/Web/UI/ActiveControls/TCallbackErrorHandler.php

Spacing +5 added lines, -5 removed lines

@@ -40,7 +40,7 @@ 
 	 */
 	protected function displayException($exception)
 	{
-		if($this->getApplication()->getMode() === TApplicationMode::Debug)
+		if ($this->getApplication()->getMode() === TApplicationMode::Debug)
 		{
 			$response = $this->getApplication()->getResponse();
 			$trace = $this->getExceptionStackTrace($exception);
@@ -63,7 +63,7 @@ 
 		}
 		else
 		{
-			error_log("Error happened while processing an existing error:\n" . $exception->__toString());
+			error_log("Error happened while processing an existing error:\n".$exception->__toString());
 			header('HTTP/1.0 500 Internal Server Error', true, 500);
 		}
 		$this->getApplication()->getResponse()->flush();
@@ -79,11 +79,11 @@ 
 		$data['file'] = $exception->getFile();
 		$data['line'] = $exception->getLine();
 		$data['trace'] = $exception->getTrace();
-		if($exception instanceof TPhpErrorException)
+		if ($exception instanceof TPhpErrorException)
 		{
 			// if PHP exception, we want to show the 2nd stack level context
 			// because the 1st stack level is of little use (it's in error handler)
-			if(isset($trace[0]) && isset($trace[0]['file']) && isset($trace[0]['line']))
+			if (isset($trace[0]) && isset($trace[0]['file']) && isset($trace[0]['line']))
 			{
 				$data['file'] = $trace[0]['file'];
 				$data['line'] = $trace[0]['line'];
@@ -91,7 +91,7 @@ 
 		}
 		$data['type'] = get_class($exception);
 		$data['message'] = $exception->getMessage();
-		$data['version'] = $_SERVER['SERVER_SOFTWARE'] . ' ' . Prado::getVersion();
+		$data['version'] = $_SERVER['SERVER_SOFTWARE'].' '.Prado::getVersion();
 		$data['time'] = @strftime('%Y-%m-%d %H:%M', time());
 		return $data;
 	}

framework/Web/UI/ActiveControls/TActiveListBox.php

Spacing +3 added lines, -3 removed lines

@@ -89,13 +89,13 @@
 	 */
 	public function setSelectionMode($value)
 	{
-		if(parent::getSelectionMode() === $value)
+		if (parent::getSelectionMode() === $value)
 			return;
 
 		parent::setSelectionMode($value);
 		$multiple = $this->getIsMultiSelect();
-		$id = $this->getUniqueID(); $multi_id = $id . '[]';
-		if($this->getActiveControl()->canUpdateClientSide())
+		$id = $this->getUniqueID(); $multi_id = $id.'[]';
+		if ($this->getActiveControl()->canUpdateClientSide())
 		{
 			$client = $this->getPage()->getCallbackClient();
 			$client->setAttribute($this, 'multiple', $multiple ? 'multiple' : false);

framework/Web/UI/ActiveControls/TCallbackClientSide.php

Spacing +1 added lines, -1 removed lines

@@ -273,7 +273,7 @@
 	 */
 	public function setPostBackTarget($value)
 	{
-		if($value instanceof TControl)
+		if ($value instanceof TControl)
 			$value = $value->getUniqueID();
 		$this->setOption('EventTarget', $value);
 	}