pradosoft / prado

framework/Exceptions/TPhpErrorException.php

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
 			E_USER_NOTICE => "User Notice",
 			E_STRICT => "Runtime Notice"
 		];
-		$errorType = isset($errorTypes[$errno])?$errorTypes[$errno]:'Unknown Error';
+		$errorType = isset($errorTypes[$errno]) ? $errorTypes[$errno] : 'Unknown Error';
 		parent::__construct("[$errorType] $errstr (@line $errline in file $errfile).");
 	}
 

framework/Web/THttpUtility.php

Spacing +3 added lines, -3 removed lines

@@ -20,9 +20,9 @@
  */
 class THttpUtility
 {
-	private static $_encodeTable = ['<' => '&lt;','>' => '&gt;','"' => '&quot;'];
-	private static $_decodeTable = ['&lt;' => '<','&gt;' => '>','&quot;' => '"'];
-	private static $_stripTable = ['&lt;' => '','&gt;' => '','&quot;' => ''];
+	private static $_encodeTable = ['<' => '&lt;', '>' => '&gt;', '"' => '&quot;'];
+	private static $_decodeTable = ['&lt;' => '<', '&gt;' => '>', '&quot;' => '"'];
+	private static $_stripTable = ['&lt;' => '', '&gt;' => '', '&quot;' => ''];
 
 	/**
 	 * HTML-encodes a string.

framework/Web/UI/WebControls/TJavascriptLogger.php

Spacing +1 added lines, -1 removed lines

@@ -86,7 +86,7 @@
 		$code = strtoupper($this->getToggleKey());
 		$info = '(<a href="http://web.archive.org/web/20060512041505/gleepglop.com/javascripts/logger/" target="_blank">more info</a>).';
 		$link = '<a href="javascript:if(logConsole)logConsole.toggle()">toggle the javascript log console.</a>';
-		$usage = 'Press ALT-' . $code . ' (Or CTRL-' . $code . ' on OS X) to';
+		$usage = 'Press ALT-'.$code.' (Or CTRL-'.$code.' on OS X) to';
 		$writer->write("{$usage} {$link} {$info}");
 	}
 }

framework/IO/TTextWriter.php

Spacing +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
 	 */
 	public function writeLine($str = '')
 	{
-		$this->write($str . "\n");
+		$this->write($str."\n");
 	}
 }
 

framework/Data/SqlMap/Statements/TPostSelectBinding.php

Spacing +10 added lines, -10 removed lines

@@ -26,18 +26,18 @@
 	private $_keys;
 	private $_method = TMappedStatement::QUERY_FOR_LIST;
 
-	public function getStatement(){ return $this->_statement; }
-	public function setStatement($value){ $this->_statement = $value; }
+	public function getStatement() { return $this->_statement; }
+	public function setStatement($value) { $this->_statement = $value; }
 
-	public function getResultProperty(){ return $this->_property; }
-	public function setResultProperty($value){ $this->_property = $value; }
+	public function getResultProperty() { return $this->_property; }
+	public function setResultProperty($value) { $this->_property = $value; }
 
-	public function getResultObject(){ return $this->_resultObject; }
-	public function setResultObject($value){ $this->_resultObject = $value; }
+	public function getResultObject() { return $this->_resultObject; }
+	public function setResultObject($value) { $this->_resultObject = $value; }
 
-	public function getKeys(){ return $this->_keys; }
-	public function setKeys($value){ $this->_keys = $value; }
+	public function getKeys() { return $this->_keys; }
+	public function setKeys($value) { $this->_keys = $value; }
 
-	public function getMethod(){ return $this->_method; }
-	public function setMethod($value){ $this->_method = $value; }
+	public function getMethod() { return $this->_method; }
+	public function setMethod($value) { $this->_method = $value; }
 }
\ No newline at end of file

bin/prado-cli.php

Spacing +94 added lines, -94 removed lines

@@ -9,15 +9,15 @@ 
  * @license https://github.com/pradosoft/prado/blob/master/LICENSE
  */
 
-if(!isset($_SERVER['argv']) || php_sapi_name() !== 'cli')
+if (!isset($_SERVER['argv']) || php_sapi_name() !== 'cli')
 	die('Must be run from the command line');
 
 // Locate composer's autoloader
-if(file_exists($autoloader = realpath(__DIR__ . '/../vendor/autoload.php')))
+if (file_exists($autoloader = realpath(__DIR__.'/../vendor/autoload.php')))
 {
 	// if we are running inside a prado repo checkout, get out of bin/
 	include($autoloader);
-} elseif(file_exists($autoloader = realpath(__DIR__ . '/../../../autoload.php'))) {
+} elseif (file_exists($autoloader = realpath(__DIR__.'/../../../autoload.php'))) {
 	// if we are running from inside an application's vendor/ directory, get out of pradosoft/prado/bin/
 	include($autoloader);
 }
@@ -78,14 +78,14 @@ 
 	public static function getInstance()
 	{
 		static $instance;
-		if($instance === null)
+		if ($instance === null)
 			$instance = new self;
 		return $instance;
 	}
 
 	public static function printGreeting()
 	{
-		echo "Command line tools for Prado " . Prado::getVersion() . ".\n";
+		echo "Command line tools for Prado ".Prado::getVersion().".\n";
 	}
 
 	/**
@@ -94,12 +94,12 @@ 
 	 */
 	public function run($args)
 	{
-		if(count($args) > 1)
+		if (count($args) > 1)
 			array_shift($args);
 		$valid = false;
-		foreach($this->_actions as $class => $action)
+		foreach ($this->_actions as $class => $action)
 		{
-			if($action->isValidAction($args))
+			if ($action->isValidAction($args))
 			{
 				$valid |= $action->performAction($args);
 				break;
@@ -109,7 +109,7 @@ 
 				$valid = false;
 			}
 		}
-		if(!$valid)
+		if (!$valid)
 			$this->printHelp();
 	}
 
@@ -123,7 +123,7 @@ 
 		echo "usage: php prado-cli.php action <parameter> [optional]\n";
 		echo "example: php prado-cli.php -c mysite\n\n";
 		echo "actions:\n";
-		foreach($this->_actions as $action)
+		foreach ($this->_actions as $action)
 			echo $action->renderHelp();
 	}
 }
@@ -145,18 +145,18 @@ 
 
 	protected function createDirectory($dir, $mask)
 	{
-		if(!is_dir($dir))
+		if (!is_dir($dir))
 		{
 			mkdir($dir);
 			echo "creating $dir\n";
 		}
-		if(is_dir($dir))
+		if (is_dir($dir))
 			chmod($dir, $mask);
 	}
 
 	protected function createFile($filename, $content)
 	{
-		if(!is_file($filename))
+		if (!is_file($filename))
 		{
 			file_put_contents($filename, $content);
 			echo "creating $filename\n";
@@ -172,16 +172,16 @@ 
 	public function renderHelp()
 	{
 		$params = [];
-		foreach($this->parameters as $v)
-			$params[] = '<' . $v . '>';
+		foreach ($this->parameters as $v)
+			$params[] = '<'.$v.'>';
 		$parameters = implode($params, ' ');
 		$options = [];
-		foreach($this->optional as $v)
-			$options[] = '[' . $v . ']';
-		$optional = (strlen($parameters) ? ' ' : '') . implode($options, ' ');
+		foreach ($this->optional as $v)
+			$options[] = '['.$v.']';
+		$optional = (strlen($parameters) ? ' ' : '').implode($options, ' ');
 		$description = '';
-		foreach(explode("\n", wordwrap($this->description, 65)) as $line)
-			$description .= '    ' . $line . "\n";
+		foreach (explode("\n", wordwrap($this->description, 65)) as $line)
+			$description .= '    '.$line."\n";
 		return <<<EOD
   {$this->action} {$parameters}{$optional}
 {$description}
@@ -191,17 +191,17 @@ 
 
 	protected function initializePradoApplication($directory)
 	{
-		$_SERVER['SCRIPT_FILENAME'] = $directory . '/index.php';
-		$app_dir = realpath($directory . '/protected/');
-		if($app_dir !== false && is_dir($app_dir))
+		$_SERVER['SCRIPT_FILENAME'] = $directory.'/index.php';
+		$app_dir = realpath($directory.'/protected/');
+		if ($app_dir !== false && is_dir($app_dir))
 		{
-			if(Prado::getApplication() === null)
+			if (Prado::getApplication() === null)
 			{
 				$app = new PradoShellApplication($app_dir);
 				$app->run();
 				$dir = substr(str_replace(realpath('./'), '', $app_dir), 1);
 				PradoCommandLineInterpreter::printGreeting();
-				echo '** Loaded PRADO appplication in directory "' . $dir . "\".\n";
+				echo '** Loaded PRADO appplication in directory "'.$dir."\".\n";
 			}
 
 			return Prado::getApplication();
@@ -209,9 +209,9 @@ 
 		else
 		{
 			PradoCommandLineInterpreter::printGreeting();
-			echo '+' . str_repeat('-', 77) . "+\n";
-			echo '** Unable to load PRADO application in directory "' . $directory . "\".\n";
-			echo '+' . str_repeat('-', 77) . "+\n";
+			echo '+'.str_repeat('-', 77)."+\n";
+			echo '** Unable to load PRADO application in directory "'.$directory."\".\n";
+			echo '+'.str_repeat('-', 77)."+\n";
 		}
 		return false;
 	}
@@ -243,25 +243,25 @@ 
 	 */
 	protected function createNewPradoProject($dir)
 	{
-		if(strlen(trim($dir)) == 0)
+		if (strlen(trim($dir)) == 0)
 			return;
 
 		$rootPath = realpath(dirname(trim($dir)));
 
-		if(basename($dir) !== '.')
-			$basePath = $rootPath . DIRECTORY_SEPARATOR . basename($dir);
+		if (basename($dir) !== '.')
+			$basePath = $rootPath.DIRECTORY_SEPARATOR.basename($dir);
 		else
 			$basePath = $rootPath;
 		$appName = basename($basePath);
-		$assetPath = $basePath . DIRECTORY_SEPARATOR . 'assets';
-		$protectedPath = $basePath . DIRECTORY_SEPARATOR . 'protected';
-		$runtimePath = $basePath . DIRECTORY_SEPARATOR . 'protected' . DIRECTORY_SEPARATOR . 'runtime';
-		$pagesPath = $protectedPath . DIRECTORY_SEPARATOR . 'pages';
+		$assetPath = $basePath.DIRECTORY_SEPARATOR.'assets';
+		$protectedPath = $basePath.DIRECTORY_SEPARATOR.'protected';
+		$runtimePath = $basePath.DIRECTORY_SEPARATOR.'protected'.DIRECTORY_SEPARATOR.'runtime';
+		$pagesPath = $protectedPath.DIRECTORY_SEPARATOR.'pages';
 
-		$indexFile = $basePath . DIRECTORY_SEPARATOR . 'index.php';
-		$htaccessFile = $protectedPath . DIRECTORY_SEPARATOR . '.htaccess';
-		$configFile = $protectedPath . DIRECTORY_SEPARATOR . 'application.xml';
-		$defaultPageFile = $pagesPath . DIRECTORY_SEPARATOR . 'Home.page';
+		$indexFile = $basePath.DIRECTORY_SEPARATOR.'index.php';
+		$htaccessFile = $protectedPath.DIRECTORY_SEPARATOR.'.htaccess';
+		$configFile = $protectedPath.DIRECTORY_SEPARATOR.'application.xml';
+		$defaultPageFile = $pagesPath.DIRECTORY_SEPARATOR.'Home.page';
 
 		$this->createDirectory($basePath, 0755);
 		$this->createDirectory($assetPath, 0777);
@@ -277,10 +277,10 @@ 
 
 	protected function renderIndexFile()
 	{
-		$framework = realpath(dirname(dirname(__FILE__))) . DIRECTORY_SEPARATOR . 'framework' . DIRECTORY_SEPARATOR . 'prado.php';
+		$framework = realpath(dirname(dirname(__FILE__))).DIRECTORY_SEPARATOR.'framework'.DIRECTORY_SEPARATOR.'prado.php';
 return '<?php
 
-$frameworkPath=\'' . $framework . '\';
+$frameworkPath=\'' . $framework.'\';
 
 // The following directory checks may be removed if performance is required
 $basePath=dirname(__FILE__);
@@ -390,22 +390,22 @@ 
 
 	protected function createTestFixtures($dir)
 	{
-		if(strlen(trim($dir)) == 0)
+		if (strlen(trim($dir)) == 0)
 			return;
 
 		$rootPath = realpath(dirname(trim($dir)));
-		$basePath = $rootPath . '/' . basename($dir);
+		$basePath = $rootPath.'/'.basename($dir);
 
-		$tests = $basePath . '/tests';
-		$unit_tests = $tests . '/unit';
-		$functional_tests = $tests . '/functional';
+		$tests = $basePath.'/tests';
+		$unit_tests = $tests.'/unit';
+		$functional_tests = $tests.'/functional';
 
 		$this->createDirectory($tests, 0755);
 		$this->createDirectory($unit_tests, 0755);
 		$this->createDirectory($functional_tests, 0755);
 
-		$unit_test_index = $tests . '/unit.php';
-		$functional_test_index = $tests . '/functional.php';
+		$unit_test_index = $tests.'/unit.php';
+		$functional_test_index = $tests.'/functional.php';
 
 		$this->createFile($unit_test_index, $this->renderUnitTestFixture());
 		$this->createFile($functional_test_index, $this->renderFunctionalTestFixture());
@@ -413,10 +413,10 @@ 
 
 	protected function renderUnitTestFixture()
 	{
-		$tester = realpath(dirname(dirname(__FILE__))) . '/tests/test_tools/unit_tests.php';
+		$tester = realpath(dirname(dirname(__FILE__))).'/tests/test_tools/unit_tests.php';
 return '<?php
 
-include_once \'' . $tester . '\';
+include_once \'' . $tester.'\';
 
 $app_directory = "../protected";
 $test_cases = dirname(__FILE__)."/unit";
@@ -428,10 +428,10 @@ 
 
 	protected function renderFunctionalTestFixture()
 	{
-		$tester = realpath(dirname(dirname(__FILE__))) . '/tests/test_tools/functional_tests.php';
+		$tester = realpath(dirname(dirname(__FILE__))).'/tests/test_tools/functional_tests.php';
 return '<?php
 
-include_once \'' . $tester . '\';
+include_once \'' . $tester.'\';
 
 $test_cases = dirname(__FILE__)."/functional";
 
@@ -457,7 +457,7 @@ 
 
 	public function performAction($args)
 	{
-		if(count($args) > 1)
+		if (count($args) > 1)
 			$this->initializePradoApplication($args[1]);
 
 		\Psy\debug([], Prado::getApplication());
@@ -483,13 +483,13 @@ 
 	{
 		$app_dir = count($args) > 3 ? $this->getAppDir($args[3]) : $this->getAppDir();
 		$this->_soap = count($args) > 4;
-		if($app_dir !== false)
+		if ($app_dir !== false)
 		{
 			$config = $this->getActiveRecordConfig($app_dir);
 			$output = $this->getOutputFile($app_dir, $args[2]);
-			if(is_file($output))
+			if (is_file($output))
 				echo "** File $output already exists, skiping. \n";
-			elseif($config !== false && $output !== false)
+			elseif ($config !== false && $output !== false)
 				$this->generateActiveRecord($config, $args[1], $output);
 		}
 		return true;
@@ -497,68 +497,68 @@ 
 
 	protected function getAppDir($dir = ".")
 	{
-		if(is_dir($dir))
+		if (is_dir($dir))
 			return realpath($dir);
-		if(false !== ($app_dir = realpath($dir . '/protected/')) && is_dir($app_dir))
+		if (false !== ($app_dir = realpath($dir.'/protected/')) && is_dir($app_dir))
 			return $app_dir;
-		echo '** Unable to find directory "' . $dir . "\".\n";
+		echo '** Unable to find directory "'.$dir."\".\n";
 		return false;
 	}
 
 	protected function getXmlFile($app_dir)
 	{
-		if(false !== ($xml = realpath($app_dir . '/application.xml')) && is_file($xml))
+		if (false !== ($xml = realpath($app_dir.'/application.xml')) && is_file($xml))
 			return $xml;
-		if(false !== ($xml = realpath($app_dir . '/protected/application.xml')) && is_file($xml))
+		if (false !== ($xml = realpath($app_dir.'/protected/application.xml')) && is_file($xml))
 			return $xml;
-		echo '** Unable to find application.xml in ' . $app_dir . "\n";
+		echo '** Unable to find application.xml in '.$app_dir."\n";
 		return false;
 	}
 
 	protected function getActiveRecordConfig($app_dir)
 	{
-		if(false === ($xml = $this->getXmlFile($app_dir)))
+		if (false === ($xml = $this->getXmlFile($app_dir)))
 			return false;
-		if(false !== ($app = $this->initializePradoApplication($app_dir)))
+		if (false !== ($app = $this->initializePradoApplication($app_dir)))
 		{
 			Prado::using('System.Data.ActiveRecord.TActiveRecordConfig');
-			foreach($app->getModules() as $module)
-				if($module instanceof TActiveRecordConfig)
+			foreach ($app->getModules() as $module)
+				if ($module instanceof TActiveRecordConfig)
 					return $module;
-			echo '** Unable to find TActiveRecordConfig module in ' . $xml . "\n";
+			echo '** Unable to find TActiveRecordConfig module in '.$xml."\n";
 		}
 		return false;
 	}
 
 	protected function getOutputFile($app_dir, $namespace)
 	{
-		if(is_file($namespace) && strpos($namespace, $app_dir) === 0)
+		if (is_file($namespace) && strpos($namespace, $app_dir) === 0)
 				return $namespace;
 		$file = Prado::getPathOfNamespace($namespace, ".php");
-		if($file !== null && false !== ($path = realpath(dirname($file))) && is_dir($path))
+		if ($file !== null && false !== ($path = realpath(dirname($file))) && is_dir($path))
 		{
-			if(strpos($path, $app_dir) === 0)
+			if (strpos($path, $app_dir) === 0)
 				return $file;
 		}
-		echo '** Output file ' . $file . ' must be within directory ' . $app_dir . "\n";
+		echo '** Output file '.$file.' must be within directory '.$app_dir."\n";
 		return false;
 	}
 
 	protected function generateActiveRecord($config, $tablename, $output)
 	{
 		$manager = TActiveRecordManager::getInstance();
-		if($manager->getDbConnection()) {
+		if ($manager->getDbConnection()) {
 			$gateway = $manager->getRecordGateway();
 			$tableInfo = $gateway->getTableInfo($manager->getDbConnection(), $tablename);
-			if(count($tableInfo->getColumns()) === 0)
+			if (count($tableInfo->getColumns()) === 0)
 			{
-				echo '** Unable to find table or view "' . $tablename . '" in "' . $manager->getDbConnection()->getConnectionString() . "\".\n";
+				echo '** Unable to find table or view "'.$tablename.'" in "'.$manager->getDbConnection()->getConnectionString()."\".\n";
 				return false;
 			}
 			else
 			{
 				$properties = [];
-				foreach($tableInfo->getColumns() as $field => $column)
+				foreach ($tableInfo->getColumns() as $field => $column)
 					$properties[] = $this->generateProperty($field, $column);
 			}
 
@@ -567,16 +567,16 @@ 
 			echo "  Writing class $classname to file $output\n";
 			file_put_contents($output, $class);
 		} else {
-			echo '** Unable to connect to database with ConnectionID=\'' . $config->getConnectionID() . "'. Please check your settings in application.xml and ensure your database connection is set up first.\n";
+			echo '** Unable to connect to database with ConnectionID=\''.$config->getConnectionID()."'. Please check your settings in application.xml and ensure your database connection is set up first.\n";
 		}
 	}
 
 	protected function generateProperty($field, $column)
 	{
 		$prop = '';
-		$name = '$' . $field;
+		$name = '$'.$field;
 		$type = $column->getPHPType();
-		if($this->_soap)
+		if ($this->_soap)
 		{
 			$prop .= <<<EOD
 
@@ -648,7 +648,7 @@ 
 			$con = $manager->getDbConnection();
 			$con->Active = true;
 
-			switch($con->getDriverName())
+			switch ($con->getDriverName())
 			   {
 				case 'mysqli':
 				case 'mysql':
@@ -665,7 +665,7 @@ 
 				case 'oci':
 //				case 'ibm':
 				default:
-					echo "\n    Sorry, generateAll is not implemented for " . $con->getDriverName() . "\n";
+					echo "\n    Sorry, generateAll is not implemented for ".$con->getDriverName()."\n";
 					
 			   }
 
@@ -677,9 +677,9 @@ 
 			}
 			$con->Active = false;
 			foreach ($tables as $key => $table) {
-				$output = $args[1] . "." . $this->_prefix . ucfirst($table) . $this->_postfix;
+				$output = $args[1].".".$this->_prefix.ucfirst($table).$this->_postfix;
 				if ($config !== false && $output !== false) {
-					$this->generate("generate " . $table . " " . $output . " " . $this->_soap . " " . $this->_overwrite);
+					$this->generate("generate ".$table." ".$output." ".$this->_soap." ".$this->_overwrite);
 				}
 			}
 		}
@@ -689,14 +689,14 @@ 
 	public function generate($l)
 	{
 		$input = explode(" ", trim($l));
-		if(count($input) > 2)
+		if (count($input) > 2)
 		{
 			$app_dir = '.';
-			if(Prado::getApplication() !== null)
+			if (Prado::getApplication() !== null)
 				$app_dir = dirname(Prado::getApplication()->getBasePath());
-			$args = [$input[0],$input[1], $input[2],$app_dir];
-			if(count($input) > 3)
-				$args = [$input[0],$input[1], $input[2],$app_dir,'soap'];
+			$args = [$input[0], $input[1], $input[2], $app_dir];
+			if (count($input) > 3)
+				$args = [$input[0], $input[1], $input[2], $app_dir, 'soap'];
 			$cmd = new PradoCommandLineActiveRecordGen;
 			$cmd->performAction($args);
 		}
@@ -709,18 +709,18 @@ 
 	protected function getAppDir($dir = ".") {
 		if (is_dir($dir))
 			return realpath($dir);
-		if (false !== ($app_dir = realpath($dir . '/protected/')) && is_dir($app_dir))
+		if (false !== ($app_dir = realpath($dir.'/protected/')) && is_dir($app_dir))
 			return $app_dir;
-		echo '** Unable to find directory "' . $dir . "\".\n";
+		echo '** Unable to find directory "'.$dir."\".\n";
 		return false;
 	}
 
 	protected function getXmlFile($app_dir) {
-		if (false !== ($xml = realpath($app_dir . '/application.xml')) && is_file($xml))
+		if (false !== ($xml = realpath($app_dir.'/application.xml')) && is_file($xml))
 			return $xml;
-		if (false !== ($xml = realpath($app_dir . '/protected/application.xml')) && is_file($xml))
+		if (false !== ($xml = realpath($app_dir.'/protected/application.xml')) && is_file($xml))
 			return $xml;
-		echo '** Unable to find application.xml in ' . $app_dir . "\n";
+		echo '** Unable to find application.xml in '.$app_dir."\n";
 		return false;
 	}
 
@@ -732,7 +732,7 @@ 
 			foreach ($app->getModules() as $module)
 				if ($module instanceof TActiveRecordConfig)
 					return $module;
-			echo '** Unable to find TActiveRecordConfig module in ' . $xml . "\n";
+			echo '** Unable to find TActiveRecordConfig module in '.$xml."\n";
 		}
 		return false;
 	}
@@ -745,7 +745,7 @@ 
 			if (strpos($path, $app_dir) === 0)
 				return $file;
 		}
-		echo '** Output file ' . $file . ' must be within directory ' . $app_dir . "\n";
+		echo '** Output file '.$file.' must be within directory '.$app_dir."\n";
 		return false;
 	}
 

framework/TApplication.php

Spacing +75 added lines, -75 removed lines

@@ -300,13 +300,13 @@ 
 		$this->setConfigurationType($configType);
 		$this->resolvePaths($basePath);
 
-		if($cacheConfig)
-			$this->_cacheFile = $this->_runtimePath . DIRECTORY_SEPARATOR . self::CONFIGCACHE_FILE;
+		if ($cacheConfig)
+			$this->_cacheFile = $this->_runtimePath.DIRECTORY_SEPARATOR.self::CONFIGCACHE_FILE;
 
 		// generates unique ID by hashing the runtime path
 		$this->_uniqueID = md5($this->_runtimePath);
 		$this->_parameters = new \Prado\Collections\TMap;
-		$this->_services = [$this->getPageServiceID() => ['TPageService',[],null]];
+		$this->_services = [$this->getPageServiceID() => ['TPageService', [], null]];
 
 		Prado::setPathOfAlias('Application', $this->_basePath);
 	}
@@ -324,11 +324,11 @@ 
 	protected function resolvePaths($basePath)
 	{
 		// determine configuration path and file
-		if(empty($basePath) || ($basePath = realpath($basePath)) === false)
+		if (empty($basePath) || ($basePath = realpath($basePath)) === false)
 			throw new TConfigurationException('application_basepath_invalid', $basePath);
-		if(is_dir($basePath) && is_file($basePath . DIRECTORY_SEPARATOR . $this->getConfigurationFileName()))
-			$configFile = $basePath . DIRECTORY_SEPARATOR . $this->getConfigurationFileName();
-		elseif(is_file($basePath))
+		if (is_dir($basePath) && is_file($basePath.DIRECTORY_SEPARATOR.$this->getConfigurationFileName()))
+			$configFile = $basePath.DIRECTORY_SEPARATOR.$this->getConfigurationFileName();
+		elseif (is_file($basePath))
 		{
 			$configFile = $basePath;
 			$basePath = dirname($configFile);
@@ -337,15 +337,15 @@ 
 			$configFile = null;
 
 		// determine runtime path
-		$runtimePath = $basePath . DIRECTORY_SEPARATOR . self::RUNTIME_PATH;
-		if(is_writable($runtimePath))
+		$runtimePath = $basePath.DIRECTORY_SEPARATOR.self::RUNTIME_PATH;
+		if (is_writable($runtimePath))
 		{
-			if($configFile !== null)
+			if ($configFile !== null)
 			{
-				$runtimePath .= DIRECTORY_SEPARATOR . basename($configFile) . '-' . Prado::getVersion();
-				if(!is_dir($runtimePath))
+				$runtimePath .= DIRECTORY_SEPARATOR.basename($configFile).'-'.Prado::getVersion();
+				if (!is_dir($runtimePath))
 				{
-					if(@mkdir($runtimePath) === false)
+					if (@mkdir($runtimePath) === false)
 						throw new TConfigurationException('application_runtimepath_failed', $runtimePath);
 					@chmod($runtimePath, PRADO_CHMOD); //make it deletable
 				}
@@ -372,11 +372,11 @@ 
 			$n = count(self::$_steps);
 			$this->_step = 0;
 			$this->_requestCompleted = false;
-			while($this->_step < $n)
+			while ($this->_step < $n)
 			{
-				if($this->_mode === TApplicationMode::Off)
+				if ($this->_mode === TApplicationMode::Off)
 					throw new THttpException(503, 'application_unavailable');
-				if($this->_requestCompleted)
+				if ($this->_requestCompleted)
 					break;
 				$method = self::$_steps[$this->_step];
 				Prado::trace("Executing $method()", 'Prado\TApplication');
@@ -384,7 +384,7 @@ 
 				$this->_step++;
 			}
 		}
-		catch(\Exception $e)
+		catch (\Exception $e)
 		{
 			$this->onError($e);
 		}
@@ -419,7 +419,7 @@ 
 	 */
 	public function getGlobalState($key, $defaultValue = null)
 	{
-		return isset($this->_globals[$key])?$this->_globals[$key]:$defaultValue;
+		return isset($this->_globals[$key]) ? $this->_globals[$key] : $defaultValue;
 	}
 
 	/**
@@ -435,11 +435,11 @@ 
 	public function setGlobalState($key, $value, $defaultValue = null, $forceSave = false)
 	{
 		$this->_stateChanged = true;
-		if($value === $defaultValue)
+		if ($value === $defaultValue)
 			unset($this->_globals[$key]);
 		else
 			$this->_globals[$key] = $value;
-		if($forceSave)
+		if ($forceSave)
 			$this->saveGlobals();
 	}
 
@@ -472,7 +472,7 @@ 
 	 */
 	protected function saveGlobals()
 	{
-		if($this->_stateChanged)
+		if ($this->_stateChanged)
 		{
 			$this->_stateChanged = false;
 			$this->getApplicationStatePersister()->save($this->_globals);
@@ -588,9 +588,9 @@ 
 	 */
 	public function getConfigurationFileExt()
 	{
-		if($this->_configFileExt === null)
+		if ($this->_configFileExt === null)
 		{
-			switch($this->_configType)
+			switch ($this->_configType)
 			{
 				case TApplication::CONFIG_TYPE_PHP:
 					$this->_configFileExt = TApplication::CONFIG_FILE_EXT_PHP;
@@ -608,9 +608,9 @@ 
 	public function getConfigurationFileName()
 	{
 		static $fileName;
-		if($fileName == null)
+		if ($fileName == null)
 		{
-			switch($this->_configType)
+			switch ($this->_configType)
 			{
 				case TApplication::CONFIG_TYPE_PHP:
 					$fileName = TApplication::CONFIG_FILE_PHP;
@@ -636,8 +636,8 @@ 
 	public function setRuntimePath($value)
 	{
 		$this->_runtimePath = $value;
-		if($this->_cacheFile)
-			$this->_cacheFile = $this->_runtimePath . DIRECTORY_SEPARATOR . self::CONFIGCACHE_FILE;
+		if ($this->_cacheFile)
+			$this->_cacheFile = $this->_runtimePath.DIRECTORY_SEPARATOR.self::CONFIGCACHE_FILE;
 		// generates unique ID by hashing the runtime path
 		$this->_uniqueID = md5($this->_runtimePath);
 	}
@@ -666,7 +666,7 @@ 
 	 */
 	public function setModule($id, IModule $module = null)
 	{
-		if(isset($this->_modules[$id]))
+		if (isset($this->_modules[$id]))
 			throw new TConfigurationException('application_moduleid_duplicated', $id);
 		else
 			$this->_modules[$id] = $module;
@@ -677,11 +677,11 @@ 
 	 */
 	public function getModule($id)
 	{
-		if(!array_key_exists($id, $this->_modules))
+		if (!array_key_exists($id, $this->_modules))
 			return null;
 
 		// force loading of a lazy module
-		if($this->_modules[$id] === null)
+		if ($this->_modules[$id] === null)
 		{
 			$module = $this->internalLoadModule($id, true);
 			$module[0]->init($module[1]);
@@ -716,7 +716,7 @@ 
 	 */
 	public function getRequest()
 	{
-		if(!$this->_request)
+		if (!$this->_request)
 		{
 			$this->_request = new \Prado\Web\THttpRequest;
 			$this->_request->init(null);
@@ -737,7 +737,7 @@ 
 	 */
 	public function getResponse()
 	{
-		if(!$this->_response)
+		if (!$this->_response)
 		{
 			$this->_response = new THttpResponse;
 			$this->_response->init(null);
@@ -758,7 +758,7 @@ 
 	 */
 	public function getSession()
 	{
-		if(!$this->_session)
+		if (!$this->_session)
 		{
 			$this->_session = new THttpSession;
 			$this->_session->init(null);
@@ -779,7 +779,7 @@ 
 	 */
 	public function getErrorHandler()
 	{
-		if(!$this->_errorHandler)
+		if (!$this->_errorHandler)
 		{
 			$this->_errorHandler = new TErrorHandler;
 			$this->_errorHandler->init(null);
@@ -800,7 +800,7 @@ 
 	 */
 	public function getSecurityManager()
 	{
-		if(!$this->_security)
+		if (!$this->_security)
 		{
 			$this->_security = new TSecurityManager;
 			$this->_security->init(null);
@@ -821,7 +821,7 @@ 
 	 */
 	public function getAssetManager()
 	{
-		if(!$this->_assetManager)
+		if (!$this->_assetManager)
 		{
 			$this->_assetManager = new TAssetManager;
 			$this->_assetManager->init(null);
@@ -842,7 +842,7 @@ 
 	 */
 	public function getApplicationStatePersister()
 	{
-		if(!$this->_statePersister)
+		if (!$this->_statePersister)
 		{
 			$this->_statePersister = new TApplicationStatePersister;
 			$this->_statePersister->init(null);
@@ -896,7 +896,7 @@ 
 	 */
 	public function getGlobalization($createIfNotExists = true)
 	{
-		if($this->_globalization === null && $createIfNotExists)
+		if ($this->_globalization === null && $createIfNotExists)
 		{
 			$this->_globalization = new TGlobalization;
 			$this->_globalization->init(null);
@@ -917,7 +917,7 @@ 
 	 */
 	public function getAuthorizationRules()
 	{
-		if($this->_authRules === null)
+		if ($this->_authRules === null)
 			$this->_authRules = new \Prado\Security\TAuthorizationRuleCollection;
 		return $this->_authRules;
 	}
@@ -930,7 +930,7 @@ 
 	protected function internalLoadModule($id, $force = false)
 	{
 		list($moduleClass, $initProperties, $configElement) = $this->_lazyModules[$id];
-		if(isset($initProperties['lazy']) && $initProperties['lazy'] && !$force)
+		if (isset($initProperties['lazy']) && $initProperties['lazy'] && !$force)
 		{
 			Prado::trace("Postponed loading of lazy module $id ({$moduleClass})", '\Prado\TApplication');
 			$this->setModule($id, null);
@@ -939,16 +939,16 @@ 
 
 		Prado::trace("Loading module $id ({$moduleClass})", '\Prado\TApplication');
 		$module = Prado::createComponent($moduleClass);
-		foreach($initProperties as $name => $value)
+		foreach ($initProperties as $name => $value)
 		{
-			if($name === 'lazy') continue;
+			if ($name === 'lazy') continue;
 			$module->setSubProperty($name, $value);
 		}
 		$this->setModule($id, $module);
 		// keep the key to avoid reuse of the old module id
 		$this->_lazyModules[$id] = null;
 
-		return [$module,$configElement];
+		return [$module, $configElement];
 	}
 	/**
 	 * Applies an application configuration.
@@ -957,32 +957,32 @@ 
 	 */
 	public function applyConfiguration($config, $withinService = false)
 	{
-		if($config->getIsEmpty())
+		if ($config->getIsEmpty())
 			return;
 
 		// set path aliases and using namespaces
-		foreach($config->getAliases() as $alias => $path)
+		foreach ($config->getAliases() as $alias => $path)
 			Prado::setPathOfAlias($alias, $path);
-		foreach($config->getUsings() as $using)
+		foreach ($config->getUsings() as $using)
 			Prado::using($using);
 
 		// set application properties
-		if(!$withinService)
+		if (!$withinService)
 		{
-			foreach($config->getProperties() as $name => $value)
+			foreach ($config->getProperties() as $name => $value)
 				$this->setSubProperty($name, $value);
 		}
 
-		if(empty($this->_services))
-			$this->_services = [$this->getPageServiceID() => ['Prado\Web\Services\TPageService',[],null]];
+		if (empty($this->_services))
+			$this->_services = [$this->getPageServiceID() => ['Prado\Web\Services\TPageService', [], null]];
 
 		// load parameters
-		foreach($config->getParameters() as $id => $parameter)
+		foreach ($config->getParameters() as $id => $parameter)
 		{
-			if(is_array($parameter))
+			if (is_array($parameter))
 			{
 				$component = Prado::createComponent($parameter[0]);
-				foreach($parameter[1] as $name => $value)
+				foreach ($parameter[1] as $name => $value)
 					$component->setSubProperty($name, $value);
 				$this->_parameters->add($id, $component);
 			}
@@ -992,29 +992,29 @@ 
 
 		// load and init modules specified in app config
 		$modules = [];
-		foreach($config->getModules() as $id => $moduleConfig)
+		foreach ($config->getModules() as $id => $moduleConfig)
 		{
-			if(!is_string($id))
-				$id = '_module' . count($this->_lazyModules);
+			if (!is_string($id))
+				$id = '_module'.count($this->_lazyModules);
 			$this->_lazyModules[$id] = $moduleConfig;
-			if($module = $this->internalLoadModule($id))
+			if ($module = $this->internalLoadModule($id))
 				$modules[] = $module;
 		}
-		foreach($modules as $module)
+		foreach ($modules as $module)
 			$module[0]->init($module[1]);
 
 		// load service
-		foreach($config->getServices() as $serviceID => $serviceConfig)
+		foreach ($config->getServices() as $serviceID => $serviceConfig)
 			$this->_services[$serviceID] = $serviceConfig;
 
 		// external configurations
-		foreach($config->getExternalConfigurations() as $filePath => $condition)
+		foreach ($config->getExternalConfigurations() as $filePath => $condition)
 		{
-			if($condition !== true)
+			if ($condition !== true)
 				$condition = $this->evaluateExpression($condition);
-			if($condition)
+			if ($condition)
 			{
-				if(($path = Prado::getPathOfNamespace($filePath, $this->getConfigurationFileExt())) === null || !is_file($path))
+				if (($path = Prado::getPathOfNamespace($filePath, $this->getConfigurationFileExt())) === null || !is_file($path))
 					throw new TConfigurationException('application_includefile_invalid', $filePath);
 				$cn = $this->getApplicationConfigurationClass();
 				$c = new $cn;
@@ -1037,13 +1037,13 @@ 
 	{
 		Prado::trace('Initializing application', 'Prado\TApplication');
 
-		if($this->_configFile !== null)
+		if ($this->_configFile !== null)
 		{
-			if($this->_cacheFile === null || @filemtime($this->_cacheFile) < filemtime($this->_configFile))
+			if ($this->_cacheFile === null || @filemtime($this->_cacheFile) < filemtime($this->_configFile))
 			{
 				$config = new TApplicationConfiguration;
 				$config->loadFromFile($this->_configFile);
-				if($this->_cacheFile !== null)
+				if ($this->_cacheFile !== null)
 					file_put_contents($this->_cacheFile, serialize($config), LOCK_EX);
 			}
 			else
@@ -1052,7 +1052,7 @@ 
 			$this->applyConfiguration($config, false);
 		}
 
-		if(($serviceID = $this->getRequest()->resolveRequest(array_keys($this->_services))) === null)
+		if (($serviceID = $this->getRequest()->resolveRequest(array_keys($this->_services))) === null)
 			$serviceID = $this->getPageServiceID();
 
 		$this->startService($serviceID);
@@ -1066,24 +1066,24 @@ 
 	 */
 	public function startService($serviceID)
 	{
-		if(isset($this->_services[$serviceID]))
+		if (isset($this->_services[$serviceID]))
 		{
 			list($serviceClass, $initProperties, $configElement) = $this->_services[$serviceID];
 			$service = Prado::createComponent($serviceClass);
-			if(!($service instanceof IService))
+			if (!($service instanceof IService))
 				throw new THttpException(500, 'application_service_invalid', $serviceClass);
-			if(!$service->getEnabled())
+			if (!$service->getEnabled())
 				throw new THttpException(500, 'application_service_unavailable', $serviceClass);
 			$service->setID($serviceID);
 			$this->setService($service);
 
-			foreach($initProperties as $name => $value)
+			foreach ($initProperties as $name => $value)
 				$service->setSubProperty($name, $value);
 
-			if($configElement !== null)
+			if ($configElement !== null)
 			{
 				$config = new TApplicationConfiguration;
-				if($this->getConfigurationType() == self::CONFIG_TYPE_PHP)
+				if ($this->getConfigurationType() == self::CONFIG_TYPE_PHP)
 					$config->loadFromPhp($configElement, $this->getBasePath());
 				else
 					$config->loadFromXml($configElement, $this->getBasePath());
@@ -1190,7 +1190,7 @@ 
 	 */
 	public function runService()
 	{
-		if($this->_service)
+		if ($this->_service)
 			$this->_service->run();
 	}
 
@@ -1238,7 +1238,7 @@ 
 	public function onEndRequest()
 	{
 		$this->flushOutput(false); // flush all remaining content in the buffer
-		$this->saveGlobals();  // save global state
+		$this->saveGlobals(); // save global state
 		$this->raiseEvent('OnEndRequest', $this, null);
 	}
 }

framework/Security/TUserManager.php

Spacing +36 added lines, -36 removed lines

@@ -111,9 +111,9 @@ 
 	public function init($config)
 	{
 		$this->loadUserData($config);
-		if($this->_userFile !== null)
+		if ($this->_userFile !== null)
 		{
-			if($this->getApplication()->getConfigurationType() == TApplication::CONFIG_TYPE_PHP)
+			if ($this->getApplication()->getConfigurationType() == TApplication::CONFIG_TYPE_PHP)
 			{
 				$userFile = include $this->_userFile;
 				$this->loadUserDataFromPhp($userFile);
@@ -134,7 +134,7 @@ 
 	 */
 	private function loadUserData($config)
 	{
-		if($this->getApplication()->getConfigurationType() == TApplication::CONFIG_TYPE_PHP)
+		if ($this->getApplication()->getConfigurationType() == TApplication::CONFIG_TYPE_PHP)
 			$this->loadUserDataFromPhp($config);
 		else
 			$this->loadUserDataFromXml($config);
@@ -146,33 +146,33 @@ 
 	 */
 	private function loadUserDataFromPhp($config)
 	{
-		if(isset($config['users']) && is_array($config['users']))
+		if (isset($config['users']) && is_array($config['users']))
 		{
-			foreach($config['users'] as $user)
+			foreach ($config['users'] as $user)
 			{
-				$name = trim(strtolower(isset($user['name'])?$user['name']:''));
-				$password = isset($user['password'])?$user['password']:'';
+				$name = trim(strtolower(isset($user['name']) ? $user['name'] : ''));
+				$password = isset($user['password']) ? $user['password'] : '';
 				$this->_users[$name] = $password;
-				$roles = isset($user['roles'])?$user['roles']:'';
-				if($roles !== '')
+				$roles = isset($user['roles']) ? $user['roles'] : '';
+				if ($roles !== '')
 				{
-					foreach(explode(',', $roles) as $role)
+					foreach (explode(',', $roles) as $role)
 					{
-						if(($role = trim($role)) !== '')
+						if (($role = trim($role)) !== '')
 							$this->_roles[$name][] = $role;
 					}
 				}
 			}
 		}
-		if(isset($config['roles']) && is_array($config['roles']))
+		if (isset($config['roles']) && is_array($config['roles']))
 		{
-			foreach($config['roles'] as $role)
+			foreach ($config['roles'] as $role)
 			{
-				$name = isset($role['name'])?$role['name']:'';
-				$users = isset($role['users'])?$role['users']:'';
-				foreach(explode(',', $users) as $user)
+				$name = isset($role['name']) ? $role['name'] : '';
+				$users = isset($role['users']) ? $role['users'] : '';
+				foreach (explode(',', $users) as $user)
 				{
-					if(($user = trim($user)) !== '')
+					if (($user = trim($user)) !== '')
 						$this->_roles[strtolower($user)][] = $name;
 				}
 			}
@@ -185,24 +185,24 @@ 
 	 */
 	private function loadUserDataFromXml($xmlNode)
 	{
-		foreach($xmlNode->getElementsByTagName('user') as $node)
+		foreach ($xmlNode->getElementsByTagName('user') as $node)
 		{
 			$name = trim(strtolower($node->getAttribute('name')));
 			$this->_users[$name] = $node->getAttribute('password');
-			if(($roles = trim($node->getAttribute('roles'))) !== '')
+			if (($roles = trim($node->getAttribute('roles'))) !== '')
 			{
-				foreach(explode(',', $roles) as $role)
+				foreach (explode(',', $roles) as $role)
 				{
-					if(($role = trim($role)) !== '')
+					if (($role = trim($role)) !== '')
 						$this->_roles[$name][] = $role;
 				}
 			}
 		}
-		foreach($xmlNode->getElementsByTagName('role') as $node)
+		foreach ($xmlNode->getElementsByTagName('role') as $node)
 		{
-			foreach(explode(',', $node->getAttribute('users')) as $user)
+			foreach (explode(',', $node->getAttribute('users')) as $user)
 			{
-				if(($user = trim($user)) !== '')
+				if (($user = trim($user)) !== '')
 					$this->_roles[strtolower($user)][] = $node->getAttribute('name');
 			}
 		}
@@ -248,9 +248,9 @@ 
 	 */
 	public function setUserFile($value)
 	{
-		if($this->_initialized)
+		if ($this->_initialized)
 			throw new TInvalidOperationException('usermanager_userfile_unchangeable');
-		elseif(($this->_userFile = Prado::getPathOfNamespace($value, self::USER_FILE_EXT)) === null || !is_file($this->_userFile))
+		elseif (($this->_userFile = Prado::getPathOfNamespace($value, self::USER_FILE_EXT)) === null || !is_file($this->_userFile))
 			throw new TConfigurationException('usermanager_userfile_invalid', $value);
 	}
 
@@ -294,9 +294,9 @@ 
 	 */
 	public function validateUser($username, $password)
 	{
-		if($this->_passwordMode === TUserManagerPasswordMode::MD5)
+		if ($this->_passwordMode === TUserManagerPasswordMode::MD5)
 			$password = md5($password);
-		elseif($this->_passwordMode === TUserManagerPasswordMode::SHA1)
+		elseif ($this->_passwordMode === TUserManagerPasswordMode::SHA1)
 			$password = sha1($password);
 		$username = strtolower($username);
 		return (isset($this->_users[$username]) && $this->_users[$username] === $password);
@@ -309,7 +309,7 @@ 
 	 */
 	public function getUser($username = null)
 	{
-		if($username === null)
+		if ($username === null)
 		{
 			$user = new TUser($this);
 			$user->setIsGuest(true);
@@ -318,12 +318,12 @@ 
 		else
 		{
 			$username = strtolower($username);
-			if(isset($this->_users[$username]))
+			if (isset($this->_users[$username]))
 			{
 				$user = new TUser($this);
 				$user->setName($username);
 				$user->setIsGuest(false);
-				if(isset($this->_roles[$username]))
+				if (isset($this->_roles[$username]))
 					$user->setRoles($this->_roles[$username]);
 				return $user;
 			}
@@ -340,13 +340,13 @@ 
 	 */
 	public function getUserFromCookie($cookie)
 	{
-		if(($data = $cookie->getValue()) !== '')
+		if (($data = $cookie->getValue()) !== '')
 		{
 			$data = unserialize($data);
-			if(is_array($data) && count($data) === 2)
+			if (is_array($data) && count($data) === 2)
 			{
 				list($username, $token) = $data;
-				if(isset($this->_users[$username]) && $token === md5($username . $this->_users[$username]))
+				if (isset($this->_users[$username]) && $token === md5($username.$this->_users[$username]))
 					return $this->getUser($username);
 			}
 		}
@@ -362,9 +362,9 @@ 
 	{
 		$user = $this->getApplication()->getUser();
 		$username = strtolower($user->getName());
-		if(isset($this->_users[$username]))
+		if (isset($this->_users[$username]))
 		{
-			$data = [$username,md5($username . $this->_users[$username])];
+			$data = [$username, md5($username.$this->_users[$username])];
 			$cookie->setValue(serialize($data));
 		}
 	}

framework/Security/TAuthManager.php

Spacing +34 added lines, -34 removed lines

@@ -89,22 +89,22 @@ 
 	 */
 	public function init($config)
 	{
-		if($this->_userManager === null)
+		if ($this->_userManager === null)
 			throw new TConfigurationException('authmanager_usermanager_required');
-		if($this->_returnUrlVarName === null)
-			$this->_returnUrlVarName = $this->getApplication()->getID() . ':' . self::RETURN_URL_VAR;
+		if ($this->_returnUrlVarName === null)
+			$this->_returnUrlVarName = $this->getApplication()->getID().':'.self::RETURN_URL_VAR;
 		$application = $this->getApplication();
-		if(is_string($this->_userManager))
+		if (is_string($this->_userManager))
 		{
-			if(($users = $application->getModule($this->_userManager)) === null)
+			if (($users = $application->getModule($this->_userManager)) === null)
 				throw new TConfigurationException('authmanager_usermanager_inexistent', $this->_userManager);
-			if(!($users instanceof IUserManager))
+			if (!($users instanceof IUserManager))
 				throw new TConfigurationException('authmanager_usermanager_invalid', $this->_userManager);
 			$this->_userManager = $users;
 		}
-		$application->attachEventHandler('OnAuthentication', [$this,'doAuthentication']);
-		$application->attachEventHandler('OnEndRequest', [$this,'leave']);
-		$application->attachEventHandler('OnAuthorization', [$this,'doAuthorization']);
+		$application->attachEventHandler('OnAuthentication', [$this, 'doAuthentication']);
+		$application->attachEventHandler('OnEndRequest', [$this, 'leave']);
+		$application->attachEventHandler('OnAuthorization', [$this, 'doAuthorization']);
 		$this->_initialized = true;
 	}
 
@@ -122,9 +122,9 @@ 
 	 */
 	public function setUserManager($provider)
 	{
-		if($this->_initialized)
+		if ($this->_initialized)
 			throw new TInvalidOperationException('authmanager_usermanager_unchangeable');
-		if(!is_string($provider) && !($provider instanceof IUserManager))
+		if (!is_string($provider) && !($provider instanceof IUserManager))
 			throw new TConfigurationException('authmanager_usermanager_invalid', $this->_userManager);
 		$this->_userManager = $provider;
 	}
@@ -160,7 +160,7 @@ 
 		$this->onAuthenticate($param);
 
 		$service = $this->getService();
-		if(($service instanceof TPageService) && $service->getRequestedPagePath() === $this->getLoginPage())
+		if (($service instanceof TPageService) && $service->getRequestedPagePath() === $this->getLoginPage())
 			$this->_skipAuthorization = true;
 	}
 
@@ -173,7 +173,7 @@ 
 	 */
 	public function doAuthorization($sender, $param)
 	{
-		if(!$this->_skipAuthorization)
+		if (!$this->_skipAuthorization)
 		{
 			$this->onAuthorize($param);
 		}
@@ -189,10 +189,10 @@ 
 	public function leave($sender, $param)
 	{
 		$application = $this->getApplication();
-		if($application->getResponse()->getStatusCode() === 401)
+		if ($application->getResponse()->getStatusCode() === 401)
 		{
 			$service = $application->getService();
-			if($service instanceof TPageService)
+			if ($service instanceof TPageService)
 			{
 				$returnUrl = $application->getRequest()->getRequestUri();
 				$this->setReturnUrl($returnUrl);
@@ -284,7 +284,7 @@ 
 		$application = $this->getApplication();
 
 		// restoring user info from session
-		if(($session = $application->getSession()) === null)
+		if (($session = $application->getSession()) === null)
 			throw new TConfigurationException('authmanager_session_required');
 		$session->open();
 		$sessionInfo = $session->itemAt($this->getUserKey());
@@ -295,12 +295,12 @@ 
 		($expiretime = $session->itemAt('AuthExpireTime')) && $expiretime < time();
 
 		// try authenticating through cookie if possible
-		if($this->getAllowAutoLogin() && ($user->getIsGuest() || $isAuthExpired))
+		if ($this->getAllowAutoLogin() && ($user->getIsGuest() || $isAuthExpired))
 		{
 			$cookie = $this->getRequest()->getCookies()->itemAt($this->getUserKey());
-			if($cookie instanceof THttpCookie)
+			if ($cookie instanceof THttpCookie)
 			{
-				if(($user2 = $this->_userManager->getUserFromCookie($cookie)) !== null)
+				if (($user2 = $this->_userManager->getUserFromCookie($cookie)) !== null)
 				{
 					$user = $user2;
 					$this->updateSessionUser($user);
@@ -313,13 +313,13 @@ 
 		$application->setUser($user);
 
 		// handle authentication expiration or update expiration time
-		if($isAuthExpired)
+		if ($isAuthExpired)
 			$this->onAuthExpire($param);
 		else
 			$session->add('AuthExpireTime', time() + $this->_authExpire);
 
 		// event handler gets a chance to do further auth work
-		if($this->hasEventHandler('OnAuthenticate'))
+		if ($this->hasEventHandler('OnAuthenticate'))
 			$this->raiseEvent('OnAuthenticate', $this, $application);
 	}
 
@@ -331,7 +331,7 @@ 
 	public function onAuthExpire($param)
 	{
 		$this->logout();
-		if($this->hasEventHandler('OnAuthExpire'))
+		if ($this->hasEventHandler('OnAuthExpire'))
 			$this->raiseEvent('OnAuthExpire', $this, $param);
 	}
 
@@ -345,9 +345,9 @@ 
 	public function onAuthorize($param)
 	{
 		$application = $this->getApplication();
-		if($this->hasEventHandler('OnAuthorize'))
+		if ($this->hasEventHandler('OnAuthorize'))
 			$this->raiseEvent('OnAuthorize', $this, $application);
-		if(!$application->getAuthorizationRules()->isUserAllowed($application->getUser(), $application->getRequest()->getRequestType(), $application->getRequest()->getUserHostAddress()))
+		if (!$application->getAuthorizationRules()->isUserAllowed($application->getUser(), $application->getRequest()->getRequestType(), $application->getRequest()->getUserHostAddress()))
 		{
 			$application->getResponse()->setStatusCode(401);
 			$application->completeRequest();
@@ -360,7 +360,7 @@ 
 	 */
 	public function getUserKey()
 	{
-		if($this->_userKey === null)
+		if ($this->_userKey === null)
 			$this->_userKey = $this->generateUserKey();
 		return $this->_userKey;
 	}
@@ -371,7 +371,7 @@ 
 	 */
 	protected function generateUserKey()
 	{
-		return md5($this->getApplication()->getUniqueID() . 'prado:user');
+		return md5($this->getApplication()->getUniqueID().'prado:user');
 	}
 
 	/**
@@ -381,9 +381,9 @@ 
 	 */
 	public function updateSessionUser($user)
 	{
-		if(!$user->getIsGuest())
+		if (!$user->getIsGuest())
 		{
-			if(($session = $this->getSession()) === null)
+			if (($session = $this->getSession()) === null)
 				throw new TConfigurationException('authmanager_session_required');
 			else
 				$session->add($this->getUserKey(), $user->saveToString());
@@ -398,7 +398,7 @@ 
 	 */
 	public function switchUser($username)
 	{
-		if(($user = $this->_userManager->getUser($username)) === null)
+		if (($user = $this->_userManager->getUser($username)) === null)
 			return false;
 		$this->updateSessionUser($user);
 		$this->getApplication()->setUser($user);
@@ -416,14 +416,14 @@ 
 	 */
 	public function login($username, $password, $expire = 0)
 	{
-		if($this->_userManager->validateUser($username, $password))
+		if ($this->_userManager->validateUser($username, $password))
 		{
-			if(($user = $this->_userManager->getUser($username)) === null)
+			if (($user = $this->_userManager->getUser($username)) === null)
 				return false;
 			$this->updateSessionUser($user);
 			$this->getApplication()->setUser($user);
 
-			if($expire > 0)
+			if ($expire > 0)
 			{
 				$cookie = new THttpCookie($this->getUserKey(), '');
 				$cookie->setExpire(time() + $expire);
@@ -443,11 +443,11 @@ 
 	 */
 	public function logout()
 	{
-		if(($session = $this->getSession()) === null)
+		if (($session = $this->getSession()) === null)
 			throw new TConfigurationException('authmanager_session_required');
 		$this->getApplication()->getUser()->setIsGuest(true);
 		$session->destroy();
-		if($this->getAllowAutoLogin())
+		if ($this->getAllowAutoLogin())
 		{
 			$cookie = new THttpCookie($this->getUserKey(), '');
 			$this->getResponse()->getCookies()->add($cookie);