pradosoft / prado

tests/simple_unit/Tickets/Ticket589Test.php

Braces +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
 		{
 			$manager->configureXml(dirname(__FILE__).'/sqlmap.xml');
 			$this->fail();
-		}catch(TSqlMapConfigurationException $e)
+		} catch(TSqlMapConfigurationException $e)
 		{
 			$expect = 'Invalid property \'parametrClass\' for class \'TSqlMapStatement\' for tag \'<statement id="findNotVisitedWatchedTopicList"';
 			$this->assertEqual(strpos($e->getMessage(),$expect),0);

tests/simple_unit/SqlMap/domain/AccountCollection.php

Braces +3 added lines, -2 removed lines

@@ -4,8 +4,9 @@
 {
 	public function addRange($accounts)
 	{
-		foreach($accounts as $account)
-			$this->add($account);
+		foreach($accounts as $account) {
+					$this->add($account);
+		}
 	}
 
 	public function copyTo(TList $array)

tests/simple_unit/SqlMap/BaseCase.php

Braces +46 added lines, -34 removed lines

@@ -38,14 +38,16 @@ 
 
 	public function __destruct()
 	{
-		if(!is_null($this->mapper))
-			$this->mapper->cacheConfiguration();
+		if(!is_null($this->mapper)) {
+					$this->mapper->cacheConfiguration();
+		}
 	}
 
 	function getConnection()
 	{
-		if(is_null($this->connection))
-			$this->connection = $this->config->getConnection();
+		if(is_null($this->connection)) {
+					$this->connection = $this->config->getConnection();
+		}
 		$this->connection->setActive(true);
 		return $this->connection;
 	}
@@ -112,10 +114,11 @@ 
 		$date = @mktime(8,15,0,2,15,2003);
 
 		$this->assertIdentical((int)$order->getID(), 1);
-		if($order->getDate() instanceof TDateTime)
-			$this->assertIdentical($order->getDate()->getTimestamp(), $date);
-		else
-			$this->fail();
+		if($order->getDate() instanceof TDateTime) {
+					$this->assertIdentical($order->getDate()->getTimestamp(), $date);
+		} else {
+					$this->fail();
+		}
 		$this->assertIdentical($order->getCardType(), 'VISA');
 		$this->assertIdentical($order->getCardNumber(), '999999999999');
 		$this->assertIdentical($order->getCardExpiry(), '05/03');
@@ -137,10 +140,11 @@ 
 		$date = @mktime(8,15,0,2,15,2003);
 
 		$this->assertIdentical(1, $order["Id"]);
-		if($order['Date'] instanceof TDateTime)
-			$this->assertIdentical($date, $order["Date"]->getTimestamp());
-		else
-			$this->fail();
+		if($order['Date'] instanceof TDateTime) {
+					$this->assertIdentical($date, $order["Date"]->getTimestamp());
+		} else {
+					$this->fail();
+		}
 		$this->assertIdentical("VISA", $order["CardType"]);
 		$this->assertIdentical("999999999999", $order["CardNumber"]);
 		$this->assertIdentical("05/03", $order["CardExpiry"]);
@@ -157,19 +161,22 @@ 
 	public function getResult($string)
 	{
 		$value = intval($string);
-		if($value == 100)
-			return true;
-		if($value == 200)
-			return false;
+		if($value == 100) {
+					return true;
+		}
+		if($value == 200) {
+					return false;
+		}
 		//throw new Exception('unexpected value '.$value);
 	}
 
 	public function getParameter($parameter)
 	{
-		if($parameter)
-			return 100;
-		else
-			return 200;
+		if($parameter) {
+					return 100;
+		} else {
+					return 200;
+		}
 	}
 
 	public function createNewInstance($data=null)
@@ -185,19 +192,22 @@ 
 
 	public function getResult($string)
 	{
-		if($string === self::YES)
-			return true;
-		if($string === self::NO)
-			return false;
+		if($string === self::YES) {
+					return true;
+		}
+		if($string === self::NO) {
+					return false;
+		}
 		//throw new Exception('unexpected value '.$string);
 	}
 
 	public function getParameter($parameter)
 	{
-		if($parameter)
-			return self::YES;
-		else
-			return self::NO;
+		if($parameter) {
+					return self::YES;
+		} else {
+					return self::NO;
+		}
 	}
 
 	public function createNewInstance($data=null)
@@ -221,10 +231,11 @@ 
 
 	public function getParameter($parameter)
 	{
-		if($parameter instanceof TDateTime)
-			return $parameter->getTimestamp();
-		else
-			return $parameter;
+		if($parameter instanceof TDateTime) {
+					return $parameter->getTimestamp();
+		} else {
+					return $parameter;
+		}
 	}
 
 	public function createNewInstance($data=null)
@@ -239,8 +250,9 @@ 
 
 	public function __construct($datetime=null)
 	{
-		if(!is_null($datetime))
-			$this->setDatetime($datetime);
+		if(!is_null($datetime)) {
+					$this->setDatetime($datetime);
+		}
 	}
 
 	public function getTimestamp()

tests/simple_unit/SqlMap/ActiveRecordSqlMapTest.php

Braces +3 added lines, -2 removed lines

@@ -79,7 +79,8 @@
 	{
 		$props = array('Account_Id', 'Account_FirstName', 'Account_LastName',
 						'Account_Email', 'Account_Banner_Option', 'Account_Cart_Option');
-		foreach($props as $prop)
-			$this->assertEqual($account1->{$prop}, $account2->{$prop});
+		foreach($props as $prop) {
+					$this->assertEqual($account1->{$prop}, $account2->{$prop});
+		}
 	}
 }

tests/simple_unit/SqlMap/GroupByTest.php

Braces +3 added lines, -2 removed lines

@@ -33,8 +33,9 @@
 		$this->initScript('account-init.sql');
 		$accounts = $this->sqlmap->queryForList("getAccountWithOrders");
 		$this->assertIdentical(5, count($accounts));
-		foreach($accounts as $account)
-			$this->assertIdentical(2, count($account->getOrders()));
+		foreach($accounts as $account) {
+					$this->assertIdentical(2, count($account->getOrders()));
+		}
 	}
 
 /**/

tests/simple_unit/SqlMap/common.php

Braces +5 added lines, -3 removed lines

@@ -2,8 +2,9 @@ 
 
 Prado::using('System.Data.TDbConnection');
 
-if(!defined('SQLMAP_TESTS'))
+if(!defined('SQLMAP_TESTS')) {
 	define('SQLMAP_TESTS', realpath(dirname(__FILE__)));
+}
 
 if(!class_exists('Account', false))
 {
@@ -40,8 +41,9 @@ 
 		foreach($lines as $line)
 		{
 			$line = trim($line);
-			if(strlen($line) > 0)
-				$connection->createCommand($line)->execute();
+			if(strlen($line) > 0) {
+							$connection->createCommand($line)->execute();
+			}
 		}
 	}
 }

tests/simple_unit/ActiveRecord/UserRecordTestCase.php

Braces +3 added lines, -2 removed lines

@@ -58,7 +58,8 @@
 		$props = array('username', 'password', 'email', 'first_name', 'last_name', 'job_title',
 						'work_phone', 'work_fax', 'active', 'department_id', 'salutation',
 						'hint_question', 'hint_answer');
-		foreach($props as $prop)
-			$this->assertEqual($user->$prop,$check->$prop);
+		foreach($props as $prop) {
+					$this->assertEqual($user->$prop,$check->$prop);
+		}
 	}
 }

tests/simple_unit/ActiveRecord/ActiveRecordRegistryTestCase.php

Braces +3 added lines, -6 removed lines

@@ -57,8 +57,7 @@ 
 		{
 			$registry->registerRemoved($obj);
 			$this->fail();
-		}
-		catch(TActiveRecordException $e)
+		} catch(TActiveRecordException $e)
 		{
 			$this->pass();
 		}
@@ -91,8 +90,7 @@ 
 		{
 			$registry->registerClean($obj);
 			$this->fail();
-		}
-		catch(TActiveRecordException $e)
+		} catch(TActiveRecordException $e)
 		{
 			$this->pass();
 		}
@@ -128,8 +126,7 @@ 
 		{
 			$registry->registerClean($obj);
 			$this->fail();
-		}
-		catch(TActiveRecordException $e)
+		} catch(TActiveRecordException $e)
 		{
 			$this->pass();
 		}

tests/simple_unit/ActiveRecord/ForeignKeyTestCase.php

Braces +3 added lines, -2 removed lines

@@ -9,8 +9,9 @@
 
 	public function getDbConnection()
 	{
-		if(self::$conn===null)
-			self::$conn = new TDbConnection('sqlite:'.dirname(__FILE__).'/fk_tests.db');
+		if(self::$conn===null) {
+					self::$conn = new TDbConnection('sqlite:'.dirname(__FILE__).'/fk_tests.db');
+		}
 		return self::$conn;
 	}
 }