pradosoft / prado

tests/simple_unit/ActiveRecord/FindByPksTestCase.php

Doc Comments +4 added lines

@@ -53,6 +53,10 @@
 		$this->assertIsDepSection($ds[1], 2, 5);
 	}
 
+	/**
+	 * @param integer $dep_id
+	 * @param integer $sec_id
+	 */
 	function assertIsDepSection($dep, $dep_id, $sec_id)
 	{
 		$this->assertTrue($dep instanceof DepSections);

Spacing +6 added lines, -6 removed lines

@@ -1,13 +1,13 @@ 
 <?php
 Prado::using('System.Data.ActiveRecord.TActiveRecord');
-require_once(dirname(__FILE__).'/records/DepartmentRecord.php');
-require_once(dirname(__FILE__).'/records/DepSections.php');
+require_once(dirname(__FILE__) . '/records/DepartmentRecord.php');
+require_once(dirname(__FILE__) . '/records/DepSections.php');
 
 class FindByPksTestCase extends UnitTestCase
 {
 	function setup()
 	{
-		$conn = new TDbConnection('pgsql:host=localhost;dbname=test', 'test','test');
+		$conn = new TDbConnection('pgsql:host=localhost;dbname=test', 'test', 'test');
 		TActiveRecordManager::getInstance()->setDbConnection($conn);
 	}
 
@@ -27,7 +27,7 @@ 
 
 	function test_find_by_pks()
 	{
-		$deps = DepartmentRecord::finder()->findAllByPks(1,2,4);
+		$deps = DepartmentRecord::finder()->findAllByPks(1, 2, 4);
 		$this->assertEqual(count($deps), 3);
 
 		$this->assertEqual($deps[0]->department_id, 1);
@@ -37,7 +37,7 @@ 
 
 	function test_find_by_pks_with_invalid()
 	{
-		$deps = DepartmentRecord::finder()->findAllByPks(4,2,14);
+		$deps = DepartmentRecord::finder()->findAllByPks(4, 2, 14);
 		$this->assertEqual(count($deps), 2);
 
 		$this->assertEqual($deps[0]->department_id, 2);
@@ -46,7 +46,7 @@ 
 
 	function test_find_by_composite_pks()
 	{
-		$ds = DepSections::finder()->findAllByPks(array(1,1), array(2,5));
+		$ds = DepSections::finder()->findAllByPks(array(1, 1), array(2, 5));
 		$this->assertEqual(count($ds), 2);
 
 		$this->assertIsDepSection($ds[0], 1, 1);

tests/simple_unit/ActiveRecord/UserRecordTestCase.php

Doc Comments +4 added lines

@@ -53,6 +53,10 @@
 		$this->assertTrue($user->delete());
 	}
 
+	/**
+	 * @param UserRecord $user
+	 * @param null|TActiveRecord $check
+	 */
 	function assertSameUser($user,$check)
 	{
 		$props = array('username', 'password', 'email', 'first_name', 'last_name', 'job_title',

Spacing +6 added lines, -6 removed lines

@@ -1,12 +1,12 @@ 
 <?php
 Prado::using('System.Data.ActiveRecord.TActiveRecord');
-require_once(dirname(__FILE__).'/records/UserRecord.php');
+require_once(dirname(__FILE__) . '/records/UserRecord.php');
 
 class UserRecordTestCase extends UnitTestCase
 {
 	function setup()
 	{
-		$conn = new TDbConnection('pgsql:host=localhost;dbname=test', 'test','test');
+		$conn = new TDbConnection('pgsql:host=localhost;dbname=test', 'test', 'test');
 		TActiveRecordManager::getInstance()->setDbConnection($conn);
 	}
 
@@ -22,7 +22,7 @@ 
 		$this->assertNotNull($user1);
 
 		$user2 = UserRecord::finder()->findByPk('admin');
-		$this->assertFalse($user1===$user2);
+		$this->assertFalse($user1 === $user2);
 	}
 
 	function testFindByPk_returns_null()
@@ -53,12 +53,12 @@ 
 		$this->assertTrue($user->delete());
 	}
 
-	function assertSameUser($user,$check)
+	function assertSameUser($user, $check)
 	{
 		$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);
 	}
 }

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/DbCommon/PgsqlColumnTest.php

Doc Comments +3 added lines

@@ -119,6 +119,9 @@
 		$this->assertEqual(array('id'), $table->getPrimaryKeys());
 	}
 
+	/**
+	 * @param TDbTableInfo $table
+	 */
 	function assertColumn($columns, $table)
 	{
 		foreach($columns as $id=>$asserts)

Spacing +4 added lines, -4 removed lines

@@ -6,7 +6,7 @@ 
 {
 	function create_meta_data()
 	{
-		$conn = new TDbConnection('pgsql:host=localhost;dbname=test', 'test','test');
+		$conn = new TDbConnection('pgsql:host=localhost;dbname=test', 'test', 'test');
 		return new TPgsqlMetaData($conn);
 	}
 
@@ -121,12 +121,12 @@ 
 
 	function assertColumn($columns, $table)
 	{
-		foreach($columns as $id=>$asserts)
+		foreach ($columns as $id=>$asserts)
 		{
 			$column = $table->Columns[$id];
-			foreach($asserts as $property=>$assert)
+			foreach ($asserts as $property=>$assert)
 			{
-				$ofAssert= var_export($assert,true);
+				$ofAssert = var_export($assert, true);
 				$value = $column->{$property};
 				$ofValue = var_export($value, true);
 				$this->assertEqual($value, $assert,

tests/simple_unit/SqlMap/BaseCase.php

Doc Comments +7 added lines

@@ -31,6 +31,9 @@ 
 		$this->ScriptDirectory = $this->config->getScriptDir();
 	}
 
+	/**
+	 * @param string $feature
+	 */
 	public function hasSupportFor($feature)
 	{
 		return $this->config->hasFeature($feature);
@@ -63,6 +66,7 @@ 
 
 	/**
 	 * Run a sql batch for the datasource.
+	 * @param string $script
 	 */
 	protected function initScript($script)
 	{
@@ -248,6 +252,9 @@ 
 		return strtotime($this->getDatetime());
 	}
 
+	/**
+	 * @return string
+	 */
 	public function getDateTime()
 	{
 		return $this->_datetime;

Spacing +22 added lines, -22 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-require_once(dirname(__FILE__).'/common.php');
+require_once(dirname(__FILE__) . '/common.php');
 Prado::using('System.Data.SqlMap.TSqlMapManager');
 
 /**
@@ -38,13 +38,13 @@ 
 
 	public function __destruct()
 	{
-		if(!is_null($this->mapper))
+		if (!is_null($this->mapper))
 			$this->mapper->cacheConfiguration();
 	}
 
 	function getConnection()
 	{
-		if(is_null($this->connection))
+		if (is_null($this->connection))
 			$this->connection = $this->config->getConnection();
 		$this->connection->setActive(true);
 		return $this->connection;
@@ -67,7 +67,7 @@ 
 	protected function initScript($script)
 	{
 		$runner = $this->config->getScriptRunner();
-		$runner->runScript($this->getConnection(), $this->ScriptDirectory.$script);
+		$runner->runScript($this->getConnection(), $this->ScriptDirectory . $script);
 	}
 
 	/**
@@ -109,10 +109,10 @@ 
 	 */
 	protected function assertOrder1(Order $order)
 	{
-		$date = @mktime(8,15,0,2,15,2003);
+		$date = @mktime(8, 15, 0, 2, 15, 2003);
 
-		$this->assertIdentical((int)$order->getID(), 1);
-		if($order->getDate() instanceof TDateTime)
+		$this->assertIdentical((int) $order->getID(), 1);
+		if ($order->getDate() instanceof TDateTime)
 			$this->assertIdentical($order->getDate()->getTimestamp(), $date);
 		else
 			$this->fail();
@@ -126,7 +126,7 @@ 
 
 	function assertAccount1AsHashArray($account)
 	{
-		$this->assertIdentical(1, (int)$account["Id"]);
+		$this->assertIdentical(1, (int) $account["Id"]);
 		$this->assertIdentical("Joe", $account["FirstName"]);
 		$this->assertIdentical("Dalton", $account["LastName"]);
 		$this->assertIdentical("Joe.Dalton@somewhere.com", $account["EmailAddress"]);
@@ -134,10 +134,10 @@ 
 
 	function AssertOrder1AsHashArray($order)
 	{
-		$date = @mktime(8,15,0,2,15,2003);
+		$date = @mktime(8, 15, 0, 2, 15, 2003);
 
 		$this->assertIdentical(1, $order["Id"]);
-		if($order['Date'] instanceof TDateTime)
+		if ($order['Date'] instanceof TDateTime)
 			$this->assertIdentical($date, $order["Date"]->getTimestamp());
 		else
 			$this->fail();
@@ -157,22 +157,22 @@ 
 	public function getResult($string)
 	{
 		$value = intval($string);
-		if($value == 100)
+		if ($value == 100)
 			return true;
-		if($value == 200)
+		if ($value == 200)
 			return false;
 		//throw new Exception('unexpected value '.$value);
 	}
 
 	public function getParameter($parameter)
 	{
-		if($parameter)
+		if ($parameter)
 			return 100;
 		else
 			return 200;
 	}
 
-	public function createNewInstance($data=null)
+	public function createNewInstance($data = null)
 	{
 		throw new TDataMapperException('can not create');
 	}
@@ -185,22 +185,22 @@ 
 
 	public function getResult($string)
 	{
-		if($string === self::YES)
+		if ($string === self::YES)
 			return true;
-		if($string === self::NO)
+		if ($string === self::NO)
 			return false;
 		//throw new Exception('unexpected value '.$string);
 	}
 
 	public function getParameter($parameter)
 	{
-		if($parameter)
+		if ($parameter)
 			return self::YES;
 		else
 			return self::NO;
 	}
 
-	public function createNewInstance($data=null)
+	public function createNewInstance($data = null)
 	{
 		throw new TDataMapperException('can not create');
 	}
@@ -221,13 +221,13 @@ 
 
 	public function getParameter($parameter)
 	{
-		if($parameter instanceof TDateTime)
+		if ($parameter instanceof TDateTime)
 			return $parameter->getTimestamp();
 		else
 			return $parameter;
 	}
 
-	public function createNewInstance($data=null)
+	public function createNewInstance($data = null)
 	{
 		return new TDateTime;
 	}
@@ -237,9 +237,9 @@ 
 {
 	private $_datetime;
 
-	public function __construct($datetime=null)
+	public function __construct($datetime = null)
 	{
-		if(!is_null($datetime))
+		if (!is_null($datetime))
 			$this->setDatetime($datetime);
 	}
 

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/common.php

Doc Comments +7 added lines

@@ -51,12 +51,19 @@
 	protected $baseFile;
 	protected $targetFile;
 
+	/**
+	 * @param string $base
+	 * @param string $target
+	 */
 	public function __construct($base, $target)
 	{
 		$this->baseFile = $base;
 		$this->targetFile = $target;
 	}
 
+	/**
+	 * @param string $script
+	 */
 	function runScript($connection, $script)
 	{
 		copy($this->baseFile, $this->targetFile);

Spacing +34 added lines, -34 removed lines

@@ -2,33 +2,33 @@ 
 
 Prado::using('System.Data.TDbConnection');
 
-if(!defined('SQLMAP_TESTS'))
+if (!defined('SQLMAP_TESTS'))
 	define('SQLMAP_TESTS', realpath(dirname(__FILE__)));
 
-if(!class_exists('Account', false))
+if (!class_exists('Account', false))
 {
-	include(SQLMAP_TESTS.'/domain/A.php');
-	include(SQLMAP_TESTS.'/domain/Account.php');
-	include(SQLMAP_TESTS.'/domain/AccountBis.php');
-	include(SQLMAP_TESTS.'/domain/AccountCollection.php');
-	include(SQLMAP_TESTS.'/domain/B.php');
-	include(SQLMAP_TESTS.'/domain/Document.php');
-	include(SQLMAP_TESTS.'/domain/Book.php');
-	include(SQLMAP_TESTS.'/domain/C.php');
-	include(SQLMAP_TESTS.'/domain/Category.php');
-	include(SQLMAP_TESTS.'/domain/Complex.php');
-	include(SQLMAP_TESTS.'/domain/D.php');
-	include(SQLMAP_TESTS.'/domain/DocumentCollection.php');
-	include(SQLMAP_TESTS.'/domain/E.php');
-	include(SQLMAP_TESTS.'/domain/F.php');
-	include(SQLMAP_TESTS.'/domain/LineItem.php');
-	include(SQLMAP_TESTS.'/domain/LineItemCollection.php');
-	include(SQLMAP_TESTS.'/domain/Newspaper.php');
-	include(SQLMAP_TESTS.'/domain/Order.php');
-	include(SQLMAP_TESTS.'/domain/Other.php');
-	include(SQLMAP_TESTS.'/domain/Sample.php');
-	include(SQLMAP_TESTS.'/domain/Search.php');
-	include(SQLMAP_TESTS.'/domain/User.php');
+	include(SQLMAP_TESTS . '/domain/A.php');
+	include(SQLMAP_TESTS . '/domain/Account.php');
+	include(SQLMAP_TESTS . '/domain/AccountBis.php');
+	include(SQLMAP_TESTS . '/domain/AccountCollection.php');
+	include(SQLMAP_TESTS . '/domain/B.php');
+	include(SQLMAP_TESTS . '/domain/Document.php');
+	include(SQLMAP_TESTS . '/domain/Book.php');
+	include(SQLMAP_TESTS . '/domain/C.php');
+	include(SQLMAP_TESTS . '/domain/Category.php');
+	include(SQLMAP_TESTS . '/domain/Complex.php');
+	include(SQLMAP_TESTS . '/domain/D.php');
+	include(SQLMAP_TESTS . '/domain/DocumentCollection.php');
+	include(SQLMAP_TESTS . '/domain/E.php');
+	include(SQLMAP_TESTS . '/domain/F.php');
+	include(SQLMAP_TESTS . '/domain/LineItem.php');
+	include(SQLMAP_TESTS . '/domain/LineItemCollection.php');
+	include(SQLMAP_TESTS . '/domain/Newspaper.php');
+	include(SQLMAP_TESTS . '/domain/Order.php');
+	include(SQLMAP_TESTS . '/domain/Other.php');
+	include(SQLMAP_TESTS . '/domain/Sample.php');
+	include(SQLMAP_TESTS . '/domain/Search.php');
+	include(SQLMAP_TESTS . '/domain/User.php');
 }
 
 class DefaultScriptRunner
@@ -37,10 +37,10 @@ 
 	{
 		$sql = file_get_contents($script);
 		$lines = explode(';', $sql);
-		foreach($lines as $line)
+		foreach ($lines as $line)
 		{
 			$line = trim($line);
-			if(strlen($line) > 0)
+			if (strlen($line) > 0)
 				$connection->createCommand($line)->execute();
 		}
 	}
@@ -70,11 +70,11 @@ 
 
 	public function __construct()
 	{
-		$this->_sqlmapConfigFile = SQLMAP_TESTS.'/sqlite.xml';
-		$this->_scriptDir = SQLMAP_TESTS.'/scripts/sqlite/';
+		$this->_sqlmapConfigFile = SQLMAP_TESTS . '/sqlite.xml';
+		$this->_scriptDir = SQLMAP_TESTS . '/scripts/sqlite/';
 
-		$this->targetFile = realpath(SQLMAP_TESTS.'/sqlite/tests.db');
-		$this->baseFile = realpath(SQLMAP_TESTS.'/sqlite/backup.db');
+		$this->targetFile = realpath(SQLMAP_TESTS . '/sqlite/tests.db');
+		$this->baseFile = realpath(SQLMAP_TESTS . '/sqlite/backup.db');
 		$file = realpath($this->targetFile);
 		$this->_connection = new TDbConnection("sqlite:{$file}");
 	}
@@ -89,8 +89,8 @@ 
 {
 	public function __construct()
 	{
-		$this->_sqlmapConfigFile = SQLMAP_TESTS.'/mysql.xml';
-		$this->_scriptDir = SQLMAP_TESTS.'/scripts/mysql/';
+		$this->_sqlmapConfigFile = SQLMAP_TESTS . '/mysql.xml';
+		$this->_scriptDir = SQLMAP_TESTS . '/scripts/mysql/';
 		$this->_features = array('insert_id');
 		$dsn = 'mysql:host=localhost;dbname=sqlmap_test;port=3307';
 		$this->_connection = new TDbConnection($dsn, 'test5', 'test5');
@@ -101,9 +101,9 @@ 
 {
 	public function __construct()
 	{
-		$this->_sqlmap = SQLMAP_TESTS.'/mssql.xml';
+		$this->_sqlmap = SQLMAP_TESTS . '/mssql.xml';
 		$this->_connectionString = 'odbc_mssql://sqlmap_tests';
-		$this->_scriptDir = SQLMAP_TESTS.'/scripts/mssql/';
+		$this->_scriptDir = SQLMAP_TESTS . '/scripts/mssql/';
 		$this->_features = array('insert_id');
 	}
 }

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/SqlMap/domain/Account.php

Doc Comments +16 added lines

@@ -11,15 +11,31 @@
 	private $_CartOptions=0;
 
 	public function getID(){ return $this->_ID; }
+
+	/**
+	 * @param integer $value
+	 */
 	public function setID($value){ $this->_ID = intval($value); }
 
 	public function getFirstName(){ return $this->_FirstName; }
+
+	/**
+	 * @param string $value
+	 */
 	public function setFirstName($value){ $this->_FirstName = $value; }
 
 	public function getLastName(){ return $this->_LastName; }
+
+	/**
+	 * @param string $value
+	 */
 	public function setLastName($value){ $this->_LastName = $value; }
 
 	public function getEmailAddress(){ return $this->_EmailAddress; }
+
+	/**
+	 * @param string|null $value
+	 */
 	public function setEmailAddress($value){ $this->_EmailAddress = $value; }
 
 	public function getIDS(){ return $this->_IDS; }

Spacing +21 added lines, -21 removed lines

@@ -2,33 +2,33 @@
 
 class Account
 {
-	private $_ID=0;
-	private $_FirstName='';
-	private $_LastName='';
-	private $_EmailAddress=null;
-	private $_IDS='';
-	private $_BannerOptions=0;
-	private $_CartOptions=0;
+	private $_ID = 0;
+	private $_FirstName = '';
+	private $_LastName = '';
+	private $_EmailAddress = null;
+	private $_IDS = '';
+	private $_BannerOptions = 0;
+	private $_CartOptions = 0;
 
-	public function getID(){ return $this->_ID; }
-	public function setID($value){ $this->_ID = intval($value); }
+	public function getID() { return $this->_ID; }
+	public function setID($value) { $this->_ID = intval($value); }
 
-	public function getFirstName(){ return $this->_FirstName; }
-	public function setFirstName($value){ $this->_FirstName = $value; }
+	public function getFirstName() { return $this->_FirstName; }
+	public function setFirstName($value) { $this->_FirstName = $value; }
 
-	public function getLastName(){ return $this->_LastName; }
-	public function setLastName($value){ $this->_LastName = $value; }
+	public function getLastName() { return $this->_LastName; }
+	public function setLastName($value) { $this->_LastName = $value; }
 
-	public function getEmailAddress(){ return $this->_EmailAddress; }
-	public function setEmailAddress($value){ $this->_EmailAddress = $value; }
+	public function getEmailAddress() { return $this->_EmailAddress; }
+	public function setEmailAddress($value) { $this->_EmailAddress = $value; }
 
-	public function getIDS(){ return $this->_IDS; }
-	public function setIDS($value){ $this->_IDS = $value; }
+	public function getIDS() { return $this->_IDS; }
+	public function setIDS($value) { $this->_IDS = $value; }
 
-	public function getBannerOptions(){ return $this->_BannerOptions; }
-	public function setBannerOptions($value){ $this->_BannerOptions = $value; }
+	public function getBannerOptions() { return $this->_BannerOptions; }
+	public function setBannerOptions($value) { $this->_BannerOptions = $value; }
 
-	public function getCartOptions(){ return $this->_CartOptions; }
-	public function setCartOptions($value){ $this->_CartOptions = $value; }
+	public function getCartOptions() { return $this->_CartOptions; }
+	public function setCartOptions($value) { $this->_CartOptions = $value; }
 
 }

tests/simple_unit/SqlMap/domain/LineItem.php

Doc Comments +20 added lines

@@ -10,18 +10,38 @@
 	private $_PictureData='';
 
 	public function getID(){ return $this->_ID; }
+
+	/**
+	 * @param integer $value
+	 */
 	public function setID($value){ $this->_ID = $value; }
 
 	public function getOrder(){ return $this->_Order; }
+
+	/**
+	 * @param Order $value
+	 */
 	public function setOrder($value){ $this->_Order = $value; }
 
 	public function getCode(){ return $this->_Code; }
+
+	/**
+	 * @param string $value
+	 */
 	public function setCode($value){ $this->_Code = $value; }
 
 	public function getQuantity(){ return $this->_Quantity; }
+
+	/**
+	 * @param integer $value
+	 */
 	public function setQuantity($value){ $this->_Quantity = $value; }
 
 	public function getPrice(){ return $this->_Price; }
+
+	/**
+	 * @param double $value
+	 */
 	public function setPrice($value){ $this->_Price = $value; }
 
 	public function getPictureData(){ return $this->_PictureData; }

Spacing +18 added lines, -18 removed lines

@@ -2,29 +2,29 @@
 
 class LineItem
 {
-	private $_ID=-1;
-	private $_Order='';
-	private $_Code='';
-	private $_Quantity=-1;
-	private $_Price=0.0;
-	private $_PictureData='';
+	private $_ID = -1;
+	private $_Order = '';
+	private $_Code = '';
+	private $_Quantity = -1;
+	private $_Price = 0.0;
+	private $_PictureData = '';
 
-	public function getID(){ return $this->_ID; }
-	public function setID($value){ $this->_ID = $value; }
+	public function getID() { return $this->_ID; }
+	public function setID($value) { $this->_ID = $value; }
 
-	public function getOrder(){ return $this->_Order; }
-	public function setOrder($value){ $this->_Order = $value; }
+	public function getOrder() { return $this->_Order; }
+	public function setOrder($value) { $this->_Order = $value; }
 
-	public function getCode(){ return $this->_Code; }
-	public function setCode($value){ $this->_Code = $value; }
+	public function getCode() { return $this->_Code; }
+	public function setCode($value) { $this->_Code = $value; }
 
-	public function getQuantity(){ return $this->_Quantity; }
-	public function setQuantity($value){ $this->_Quantity = $value; }
+	public function getQuantity() { return $this->_Quantity; }
+	public function setQuantity($value) { $this->_Quantity = $value; }
 
-	public function getPrice(){ return $this->_Price; }
-	public function setPrice($value){ $this->_Price = $value; }
+	public function getPrice() { return $this->_Price; }
+	public function setPrice($value) { $this->_Price = $value; }
 
-	public function getPictureData(){ return $this->_PictureData; }
-	public function setPictureData($value){ $this->_PictureData = $value; }
+	public function getPictureData() { return $this->_PictureData; }
+	public function setPictureData($value) { $this->_PictureData = $value; }
 
 }

tests/simple_unit/SqlMap/domain/Other.php

Doc Comments +16 added lines

@@ -8,14 +8,30 @@
 	private $_Bool2=false;
 
 	public function getBool2(){ return $this->_Bool2; }
+
+	/**
+	 * @param boolean $value
+	 */
 	public function setBool2($value){ $this->_Bool2 = $value; }
 
 	public function getBool(){ return $this->_Bool; }
+
+	/**
+	 * @param boolean $value
+	 */
 	public function setBool($value){ $this->_Bool = $value; }
 
 	public function getInt(){ return $this->_Int; }
+
+	/**
+	 * @param integer $value
+	 */
 	public function setInt($value){ $this->_Int = $value; }
 
 	public function getLong(){ return $this->_Long; }
+
+	/**
+	 * @param integer $value
+	 */
 	public function setLong($value){ $this->_Long = $value; }
 }

Spacing +12 added lines, -12 removed lines

@@ -2,20 +2,20 @@
 
 class Other
 {
-	private $_Int=-1;
-	private $_Long=-1;
-	private $_Bool=false;
-	private $_Bool2=false;
+	private $_Int = -1;
+	private $_Long = -1;
+	private $_Bool = false;
+	private $_Bool2 = false;
 
-	public function getBool2(){ return $this->_Bool2; }
-	public function setBool2($value){ $this->_Bool2 = $value; }
+	public function getBool2() { return $this->_Bool2; }
+	public function setBool2($value) { $this->_Bool2 = $value; }
 
-	public function getBool(){ return $this->_Bool; }
-	public function setBool($value){ $this->_Bool = $value; }
+	public function getBool() { return $this->_Bool; }
+	public function setBool($value) { $this->_Bool = $value; }
 
-	public function getInt(){ return $this->_Int; }
-	public function setInt($value){ $this->_Int = $value; }
+	public function getInt() { return $this->_Int; }
+	public function setInt($value) { $this->_Int = $value; }
 
-	public function getLong(){ return $this->_Long; }
-	public function setLong($value){ $this->_Long = $value; }
+	public function getLong() { return $this->_Long; }
+	public function setLong($value) { $this->_Long = $value; }
 }

tests/simple_unit/SqlMap/InheritanceTest.php

Doc Comments +14 added lines

@@ -95,12 +95,21 @@ 
 		$this->assertNewspaper($news, 4, "Le Canard enchaine", "Paris");
 	}
 
+	/**
+	 * @param integer $id
+	 * @param string $title
+	 */
 	function AssertDocument(Document $document, $id, $title)
 	{
 		$this->assertEqual($id, $document->getID());
 		$this->assertEqual($title, $document->getTitle());
 	}
 
+	/**
+	 * @param integer $id
+	 * @param string $title
+	 * @param integer $pageNumber
+	 */
 	function AssertBook(Book $book, $id, $title, $pageNumber)
 	{
 		$this->assertEqual($id, $book->getId());
@@ -108,6 +117,11 @@ 
 		$this->assertEqual($pageNumber, (int)$book->getPageNumber());
 	}
 
+	/**
+	 * @param integer $id
+	 * @param string $title
+	 * @param string $city
+	 */
 	function AssertNewspaper(Newspaper $news, $id, $title, $city)
 	{
 		$this->assertEqual($id, $news->getId());

Spacing +3 added lines, -3 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-require_once(dirname(__FILE__).'/BaseCase.php');
+require_once(dirname(__FILE__) . '/BaseCase.php');
 
 /**
  * @package System.DataAccess.SQLMap
@@ -105,7 +105,7 @@ 
 	{
 		$this->assertEqual($id, $book->getId());
 		$this->assertEqual($title, $book->getTitle());
-		$this->assertEqual($pageNumber, (int)$book->getPageNumber());
+		$this->assertEqual($pageNumber, (int) $book->getPageNumber());
 	}
 
 	function AssertNewspaper(Newspaper $news, $id, $title, $city)
@@ -137,7 +137,7 @@ 
 		throw new TDataMapperException('not implemented');
 	}
 
-	public function createNewInstance($data=null)
+	public function createNewInstance($data = null)
 	{
 		throw new TDataMapperException('can not create');
 	}