pradosoft / prado

tests/test_tools/simpletest/options.php

Spacing +15 added lines, -15 removed lines

@@ -31,7 +31,7 @@ 
          *    @access public
          */
         static function ignore($class) {
-            $registry =SimpleTestOptions::_getRegistry();
+            $registry = SimpleTestOptions::_getRegistry();
             $registry['IgnoreList'][] = strtolower($class);
         }
 
@@ -44,7 +44,7 @@ 
          *    @static
          */
         static function isIgnored($class) {
-            $registry =SimpleTestOptions::_getRegistry();
+            $registry = SimpleTestOptions::_getRegistry();
             return in_array(strtolower($class), $registry['IgnoreList']);
         }
 
@@ -59,7 +59,7 @@ 
          *    @access public
          */
         static function setStubBaseClass($stub_base) {
-            $registry =SimpleTestOptions::_getRegistry();
+            $registry = SimpleTestOptions::_getRegistry();
             $registry['StubBaseClass'] = $stub_base;
         }
 
@@ -70,7 +70,7 @@ 
          *    @access public
          */
         static function getStubBaseClass() {
-            $registry =SimpleTestOptions::_getRegistry();
+            $registry = SimpleTestOptions::_getRegistry();
             return $registry['StubBaseClass'];
         }
 
@@ -85,7 +85,7 @@ 
          *    @access public
          */
         static function setMockBaseClass($mock_base) {
-            $registry =SimpleTestOptions::_getRegistry();
+            $registry = SimpleTestOptions::_getRegistry();
             $registry['MockBaseClass'] = $mock_base;
         }
 
@@ -96,7 +96,7 @@ 
          *    @access public
          */
         static function getMockBaseClass() {
-            $registry =SimpleTestOptions::_getRegistry();
+            $registry = SimpleTestOptions::_getRegistry();
             return $registry['MockBaseClass'];
         }
 
@@ -110,7 +110,7 @@ 
          *    @access public
          */
         static function addPartialMockCode($code = '') {
-            $registry =SimpleTestOptions::_getRegistry();
+            $registry = SimpleTestOptions::_getRegistry();
             $registry['AdditionalPartialMockCode'] = $code;
         }
 
@@ -120,7 +120,7 @@ 
          *    @access public
          */
         function getPartialMockCode() {
-            $registry =SimpleTestOptions::_getRegistry();
+            $registry = SimpleTestOptions::_getRegistry();
             return $registry['AdditionalPartialMockCode'];
         }
 
@@ -135,7 +135,7 @@ 
          *    @access public
          */
         static function useProxy($proxy, $username = false, $password = false) {
-            $registry =SimpleTestOptions::_getRegistry();
+            $registry = SimpleTestOptions::_getRegistry();
             $registry['DefaultProxy'] = $proxy;
             $registry['DefaultProxyUsername'] = $username;
             $registry['DefaultProxyPassword'] = $password;
@@ -147,7 +147,7 @@ 
          *    @access public
          */
         function getDefaultProxy() {
-            $registry =SimpleTestOptions::_getRegistry();
+            $registry = SimpleTestOptions::_getRegistry();
             return $registry['DefaultProxy'];
         }
 
@@ -157,7 +157,7 @@ 
          *    @access public
          */
         function getDefaultProxyUsername() {
-            $registry =SimpleTestOptions::_getRegistry();
+            $registry = SimpleTestOptions::_getRegistry();
             return $registry['DefaultProxyUsername'];
         }
 
@@ -167,7 +167,7 @@ 
          *    @access public
          */
         function getDefaultProxyPassword() {
-            $registry =SimpleTestOptions::_getRegistry();
+            $registry = SimpleTestOptions::_getRegistry();
             return $registry['DefaultProxyPassword'];
         }
 
@@ -179,7 +179,7 @@ 
          */
         static function _getRegistry() {
             static $registry = false;
-            if (! $registry) {
+            if (!$registry) {
                 $registry = SimpleTestOptions::_getDefaults();
             }
             return $registry;
@@ -269,7 +269,7 @@ 
                 $is_identical = SimpleTestCompatibility::_isIdenticalType(
                         $first[$key],
                         $second[$key]);
-                if (! $is_identical) {
+                if (!$is_identical) {
                     return false;
                 }
             }
@@ -305,7 +305,7 @@ 
          */
         static function isA($object, $class) {
             if (version_compare(phpversion(), '5') >= 0) {
-                if (! class_exists($class, false)) {
+                if (!class_exists($class, false)) {
                     return false;
                 }
                 eval("\$is_a = \$object instanceof $class;");

tests/test_tools/simpletest/parser.php

Spacing +23 added lines, -23 removed lines

@@ -9,19 +9,19 @@ 
     /**#@+
      * Lexer mode stack constants
      */
-    if (! defined('LEXER_ENTER')) {
+    if (!defined('LEXER_ENTER')) {
         define('LEXER_ENTER', 1);
     }
-    if (! defined('LEXER_MATCHED')) {
+    if (!defined('LEXER_MATCHED')) {
         define('LEXER_MATCHED', 2);
     }
-    if (! defined('LEXER_UNMATCHED')) {
+    if (!defined('LEXER_UNMATCHED')) {
         define('LEXER_UNMATCHED', 3);
     }
-    if (! defined('LEXER_EXIT')) {
+    if (!defined('LEXER_EXIT')) {
         define('LEXER_EXIT', 4);
     }
-    if (! defined('LEXER_SPECIAL')) {
+    if (!defined('LEXER_SPECIAL')) {
         define('LEXER_SPECIAL', 5);
     }
     /**#@-*/
@@ -80,7 +80,7 @@ 
             if (count($this->_patterns) == 0) {
                 return false;
             }
-            if (! preg_match($this->_getCompoundedRegex(), $subject, $matches)) {
+            if (!preg_match($this->_getCompoundedRegex(), $subject, $matches)) {
                 $match = '';
                 return false;
             }
@@ -221,11 +221,11 @@ 
          *    @access public
          */
         function addPattern($pattern, $mode = "accept") {
-            if (! isset($this->_regexes[$mode])) {
+            if (!isset($this->_regexes[$mode])) {
                 $this->_regexes[$mode] = new ParallelRegex($this->_case);
             }
             $this->_regexes[$mode]->addPattern($pattern);
-            if (! isset($this->_mode_handlers[$mode])) {
+            if (!isset($this->_mode_handlers[$mode])) {
                 $this->_mode_handlers[$mode] = $mode;
             }
         }
@@ -244,11 +244,11 @@ 
          *    @access public
          */
         function addEntryPattern($pattern, $mode, $new_mode) {
-            if (! isset($this->_regexes[$mode])) {
+            if (!isset($this->_regexes[$mode])) {
                 $this->_regexes[$mode] = new ParallelRegex($this->_case);
             }
             $this->_regexes[$mode]->addPattern($pattern, $new_mode);
-            if (! isset($this->_mode_handlers[$new_mode])) {
+            if (!isset($this->_mode_handlers[$new_mode])) {
                 $this->_mode_handlers[$new_mode] = $new_mode;
             }
         }
@@ -262,11 +262,11 @@ 
          *    @access public
          */
         function addExitPattern($pattern, $mode) {
-            if (! isset($this->_regexes[$mode])) {
+            if (!isset($this->_regexes[$mode])) {
                 $this->_regexes[$mode] = new ParallelRegex($this->_case);
             }
             $this->_regexes[$mode]->addPattern($pattern, "__exit");
-            if (! isset($this->_mode_handlers[$mode])) {
+            if (!isset($this->_mode_handlers[$mode])) {
                 $this->_mode_handlers[$mode] = $mode;
             }
         }
@@ -284,11 +284,11 @@ 
          *    @access public
          */
         function addSpecialPattern($pattern, $mode, $special) {
-            if (! isset($this->_regexes[$mode])) {
+            if (!isset($this->_regexes[$mode])) {
                 $this->_regexes[$mode] = new ParallelRegex($this->_case);
             }
             $this->_regexes[$mode]->addPattern($pattern, "_$special");
-            if (! isset($this->_mode_handlers[$special])) {
+            if (!isset($this->_mode_handlers[$special])) {
                 $this->_mode_handlers[$special] = $special;
             }
         }
@@ -314,13 +314,13 @@ 
          *    @access public
          */
         function parse($raw) {
-            if (! isset($this->_parser)) {
+            if (!isset($this->_parser)) {
                 return false;
             }
             $length = strlen($raw);
             while (is_array($parsed = $this->_reduce($raw))) {
                 list($raw, $unmatched, $matched, $mode) = $parsed;
-                if (! $this->_dispatchTokens($unmatched, $matched, $mode)) {
+                if (!$this->_dispatchTokens($unmatched, $matched, $mode)) {
                     return false;
                 }
                 if ($raw === '') {
@@ -331,7 +331,7 @@ 
                 }
                 $length = strlen($raw);
             }
-            if (! $parsed) {
+            if (!$parsed) {
                 return false;
             }
             return $this->_invokeParser($raw, LEXER_UNMATCHED);
@@ -350,21 +350,21 @@ 
          *    @access private
          */
         function _dispatchTokens($unmatched, $matched, $mode = false) {
-            if (! $this->_invokeParser($unmatched, LEXER_UNMATCHED)) {
+            if (!$this->_invokeParser($unmatched, LEXER_UNMATCHED)) {
                 return false;
             }
             if (is_bool($mode)) {
                 return $this->_invokeParser($matched, LEXER_MATCHED);
             }
             if ($this->_isModeEnd($mode)) {
-                if (! $this->_invokeParser($matched, LEXER_EXIT)) {
+                if (!$this->_invokeParser($matched, LEXER_EXIT)) {
                     return false;
                 }
                 return $this->_mode->leave();
             }
             if ($this->_isSpecialMode($mode)) {
                 $this->_mode->enter($this->_decodeSpecial($mode));
-                if (! $this->_invokeParser($matched, LEXER_SPECIAL)) {
+                if (!$this->_invokeParser($matched, LEXER_SPECIAL)) {
                     return false;
                 }
                 return $this->_mode->leave();
@@ -631,7 +631,7 @@ 
          *    @access public
          */
         function acceptEndToken($token, $event) {
-            if (! preg_match('/<\/(.*)>/', $token, $matches)) {
+            if (!preg_match('/<\/(.*)>/', $token, $matches)) {
                 return false;
             }
             return $this->_listener->endElement(strtolower($matches[1]));
@@ -651,7 +651,7 @@ 
             }
             if ($event == LEXER_SPECIAL) {
                 $this->_attributes[$this->_current_attribute] .=
-                        preg_replace('/^=\s*/' , '', SimpleHtmlSaxParser::decodeHtml($token));
+                        preg_replace('/^=\s*/', '', SimpleHtmlSaxParser::decodeHtml($token));
             }
             return true;
         }
@@ -698,7 +698,7 @@ 
          */
         static function decodeHtml($html) {
             static $translations;
-            if (! isset($translations)) {
+            if (!isset($translations)) {
                 $translations = array_flip(get_html_translation_table(HTML_ENTITIES));
             }
             return strtr($html, $translations);

tests/test_tools/simpletest/invoker.php

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
     require_once(dirname(__FILE__) . '/scorer.php');
     require_once(dirname(__FILE__) . '/expectation.php');
     require_once(dirname(__FILE__) . '/dumper.php');
-    if (! defined('SIMPLE_TEST')) {
+    if (!defined('SIMPLE_TEST')) {
         define('SIMPLE_TEST', dirname(__FILE__) . '/');
     }
     /**#@-*/

tests/test_tools/simpletest/browser.php

Spacing +26 added lines, -26 removed lines

@@ -55,7 +55,7 @@ 
          *    @access private
          */
         function _atBeginning() {
-            return ($this->_position == 0) && ! $this->_isEmpty();
+            return ($this->_position == 0) && !$this->_isEmpty();
         }
 
         /**
@@ -64,7 +64,7 @@ 
          *    @access private
          */
         function _atEnd() {
-            return ($this->_position + 1 >= count($this->_sequence)) && ! $this->_isEmpty();
+            return ($this->_position + 1 >= count($this->_sequence)) && !$this->_isEmpty();
         }
 
         /**
@@ -144,7 +144,7 @@ 
             if ($this->_isEmpty()) {
                 return;
             }
-            while (! $this->_atEnd()) {
+            while (!$this->_atEnd()) {
                 array_pop($this->_sequence);
             }
         }
@@ -247,7 +247,7 @@ 
          */
         function &_parse($response, $depth = 0) {
             $page = $this->_buildPage($response);
-            if ($this->_ignore_frames || ! $page->hasFrames() || ($depth > $this->_maximum_nested_frames)) {
+            if ($this->_ignore_frames || !$page->hasFrames() || ($depth > $this->_maximum_nested_frames)) {
                 return $page;
             }
             $frameset = new SimpleFrameset($page);
@@ -303,7 +303,7 @@ 
          */
         function _load($url, $parameters) {
             $frame = $url->getTarget();
-            if (! $frame || ! $this->_page->hasFrames() || (strtolower($frame) == '_top')) {
+            if (!$frame || !$this->_page->hasFrames() || (strtolower($frame) == '_top')) {
                 return $this->_loadPage($url, $parameters);
             }
             return $this->_loadFrame(array($frame), $url, $parameters);
@@ -460,14 +460,14 @@ 
          *    @access public
          */
         function head($url, $parameters = false) {
-            if (! is_object($url)) {
+            if (!is_object($url)) {
                 $url = new SimpleUrl($url);
             }
             if ($this->getUrl()) {
                 $url = $url->makeAbsolute($this->getUrl());
             }
             $response = $this->_user_agent->fetchResponse($url, new SimpleHeadEncoding($parameters));
-            return ! $response->isError();
+            return !$response->isError();
         }
 
         /**
@@ -479,7 +479,7 @@ 
          *    @access public
          */
         function get($url, $parameters = false) {
-            if (! is_object($url)) {
+            if (!is_object($url)) {
                 $url = new SimpleUrl($url);
             }
             if ($this->getUrl()) {
@@ -496,7 +496,7 @@ 
          *    @access public
          */
         function post($url, $parameters = false) {
-            if (! is_object($url)) {
+            if (!is_object($url)) {
                 $url = new SimpleUrl($url);
             }
             if ($this->getUrl()) {
@@ -539,11 +539,11 @@ 
          *    @access public
          */
         function back() {
-            if (! $this->_history->back()) {
+            if (!$this->_history->back()) {
                 return false;
             }
             $content = $this->retry();
-            if (! $content) {
+            if (!$content) {
                 $this->_history->forward();
             }
             return $content;
@@ -559,11 +559,11 @@ 
          *    @access public
          */
         function forward() {
-            if (! $this->_history->forward()) {
+            if (!$this->_history->forward()) {
                 return false;
             }
             $content = $this->retry();
-            if (! $content) {
+            if (!$content) {
                 $this->_history->back();
             }
             return $content;
@@ -580,11 +580,11 @@ 
          *    @access public
          */
         function authenticate($username, $password) {
-            if (! $this->_page->getRealm()) {
+            if (!$this->_page->getRealm()) {
                 return false;
             }
             $url = $this->_page->getUrl();
-            if (! $url) {
+            if (!$url) {
                 return false;
             }
             $this->_user_agent->setIdentity(
@@ -850,7 +850,7 @@ 
          *    @access public
          */
         function clickSubmit($label = 'Submit', $additional = false) {
-            if (! ($form = $this->_page->getFormBySubmit(new SimpleByLabel($label)))) {
+            if (!($form = $this->_page->getFormBySubmit(new SimpleByLabel($label)))) {
                 return false;
             }
             $success = $this->_load(
@@ -868,7 +868,7 @@ 
          *    @access public
          */
         function clickSubmitByName($name, $additional = false) {
-            if (! ($form = $this->_page->getFormBySubmit(new SimpleByName($name)))) {
+            if (!($form = $this->_page->getFormBySubmit(new SimpleByName($name)))) {
                 return false;
             }
             $success = $this->_load(
@@ -886,7 +886,7 @@ 
          *    @access public
          */
         function clickSubmitById($id, $additional = false) {
-            if (! ($form = $this->_page->getFormBySubmit(new SimpleById($id)))) {
+            if (!($form = $this->_page->getFormBySubmit(new SimpleById($id)))) {
                 return false;
             }
             $success = $this->_load(
@@ -909,7 +909,7 @@ 
          *    @access public
          */
         function clickImage($label, $x = 1, $y = 1, $additional = false) {
-            if (! ($form = $this->_page->getFormByImage(new SimpleByLabel($label)))) {
+            if (!($form = $this->_page->getFormByImage(new SimpleByLabel($label)))) {
                 return false;
             }
             $success = $this->_load(
@@ -932,7 +932,7 @@ 
          *    @access public
          */
         function clickImageByName($name, $x = 1, $y = 1, $additional = false) {
-            if (! ($form = $this->_page->getFormByImage(new SimpleByName($name)))) {
+            if (!($form = $this->_page->getFormByImage(new SimpleByName($name)))) {
                 return false;
             }
             $success = $this->_load(
@@ -954,7 +954,7 @@ 
          *    @access public
          */
         function clickImageById($id, $x = 1, $y = 1, $additional = false) {
-            if (! ($form = $this->_page->getFormByImage(new SimpleById($id)))) {
+            if (!($form = $this->_page->getFormByImage(new SimpleById($id)))) {
                 return false;
             }
             $success = $this->_load(
@@ -971,7 +971,7 @@ 
          *    @access public
          */
         function submitFormById($id) {
-            if (! ($form = $this->_page->getFormById($id))) {
+            if (!($form = $this->_page->getFormById($id))) {
                 return false;
             }
             $success = $this->_load(
@@ -1019,7 +1019,7 @@ 
          *    @access public
          */
         function clickLinkById($id) {
-            if (! ($url = $this->_page->getUrlById($id))) {
+            if (!($url = $this->_page->getUrlById($id))) {
                 return false;
             }
             $this->_load($url, new SimpleGetEncoding());
@@ -1033,7 +1033,7 @@ 
          *    @access public
          */
         function isLinkById($id) {
-            return (boolean)$this->_page->getUrlById($id);
+            return (boolean) $this->_page->getUrlById($id);
         }
 
         /**
@@ -1045,10 +1045,10 @@ 
          */
         function click($label) {
             $raw = $this->clickSubmit($label);
-            if (! $raw) {
+            if (!$raw) {
                 $raw = $this->clickLink($label);
             }
-            if (! $raw) {
+            if (!$raw) {
                 $raw = $this->clickImage($label);
             }
             return $raw;

tests/test_tools/simpletest/selector.php

Spacing +1 added lines, -1 removed lines

@@ -64,7 +64,7 @@
          *    @access public
          */
         function isMatch($widget) {
-            if (! method_exists($widget, 'isLabel')) {
+            if (!method_exists($widget, 'isLabel')) {
                 return false;
             }
             return $widget->isLabel($this->_label);

tests/test_tools/PradoGenericSelenium2Test.php

Spacing +29 added lines, -29 removed lines

@@ -45,9 +45,9 @@ 
 */
 	);
 
-	static $baseurl='http://127.0.0.1/prado-master/tests/FunctionalTests/';
+	static $baseurl = 'http://127.0.0.1/prado-master/tests/FunctionalTests/';
 
-	static $timeout=5; //seconds
+	static $timeout = 5; //seconds
 
 	protected function setUp()
 	{
@@ -61,11 +61,11 @@ 
 		list($id, $attr) = explode('@', $idattr);
 
 		$element = $this->getElement($id);
-		$value=$element->attribute($attr);
+		$value = $element->attribute($attr);
 
-		if(strpos($txt, 'regexp:')===0)
+		if (strpos($txt, 'regexp:') === 0)
 		{
-			$this->assertRegExp('/'.substr($txt, 7).'/', $value);
+			$this->assertRegExp('/' . substr($txt, 7) . '/', $value);
 		} else {
 			$this->assertEquals($txt, $value);
 		}
@@ -73,13 +73,13 @@ 
 
 	protected function getElement($id)
 	{
-		if(strpos($id, 'id=')===0) {
+		if (strpos($id, 'id=') === 0) {
 			return $this->byId(substr($id, 3));
-		} elseif(strpos($id, 'name=')===0) {
+		} elseif (strpos($id, 'name=') === 0) {
 			return $this->byName(substr($id, 5));
-		} elseif(strpos($id, '//')===0) {
+		} elseif (strpos($id, '//') === 0) {
 			return $this->byXPath($id);
-		} elseif(strpos($id, '$')!==false) {
+		} elseif (strpos($id, '$') !== false) {
 			return $this->byName($id);
 		} else {
 			return $this->byId($id);
@@ -93,7 +93,7 @@ 
 
 	protected function assertValue($id, $txt)
 	{
-		try{
+		try {
 			$this->assertEquals($txt, $this->getElement($id)->value());
 		} catch (PHPUnit_Extensions_Selenium2TestCase_WebDriverException $e) {
 			//stale element reference. try second time.
@@ -104,7 +104,7 @@ 
 
 	protected function assertVisible($id)
 	{
-		try{
+		try {
 			$this->assertTrue($this->getElement($id)->displayed());
 		} catch (PHPUnit_Extensions_Selenium2TestCase_WebDriverException $e) {
 			//stale element reference. try second time.
@@ -115,7 +115,7 @@ 
 
 	protected function assertNotVisible($id)
 	{
-		try{
+		try {
 			$this->assertFalse($this->getElement($id)->displayed());
 		} catch (PHPUnit_Extensions_Selenium2TestCase_WebDriverException $e) {
 			//stale element reference. try second time.
@@ -126,7 +126,7 @@ 
 
 	protected function assertElementPresent($id)
 	{
-		$this->assertTrue($this->getElement($id)!==null);
+		$this->assertTrue($this->getElement($id) !== null);
 	}
 
 	protected function assertElementNotPresent($id)
@@ -137,10 +137,10 @@ 
 			$this->assertEquals(PHPUnit_Extensions_Selenium2TestCase_WebDriverException::NoSuchElement, $e->getCode());
 			return;
 		}
-		$this->fail('The element '.$id.' shouldn\'t exist.');
+		$this->fail('The element ' . $id . ' shouldn\'t exist.');
 	}
 
-	protected function type($id, $txt='')
+	protected function type($id, $txt = '')
 	{
 		$element = $this->getElement($id);
 		$element->clear();
@@ -149,7 +149,7 @@ 
 		$this->byCssSelector('body')->click();
 	}
 
-	protected function typeSpecial($id, $txt='')
+	protected function typeSpecial($id, $txt = '')
 	{
 		$element = $this->getElement($id);
 		// clear the textbox without using clear() that triggers onchange()
@@ -159,16 +159,16 @@ 
 		// sequences of end+backspace and start+delete
 
 		$element->click();
-		while(strlen($element->value())>0)
+		while (strlen($element->value()) > 0)
 		{
 			$this->keys(PHPUnit_Extensions_Selenium2TestCase_Keys::END);
 			// the number 100 is purely empiric
-			for($i=0;$i<100;$i++)
+			for ($i = 0; $i < 100; $i++)
 				$this->keys(PHPUnit_Extensions_Selenium2TestCase_Keys::BACKSPACE);
 
 			$this->keys(PHPUnit_Extensions_Selenium2TestCase_Keys::HOME);
 			// the number 100 is purely empiric
-			for($i=0;$i<100;$i++)
+			for ($i = 0; $i < 100; $i++)
 				$this->keys(PHPUnit_Extensions_Selenium2TestCase_Keys::DELETE);
 		}
 
@@ -207,20 +207,20 @@ 
 
 	protected function assertSelectedIndex($id, $value)
 	{
-		$options=parent::select($this->getElement($id))->selectOptionValues();
-		$curval=parent::select($this->getElement($id))->selectedValue();
+		$options = parent::select($this->getElement($id))->selectOptionValues();
+		$curval = parent::select($this->getElement($id))->selectedValue();
 
-		$i=0;
-		foreach($options as $option)
+		$i = 0;
+		foreach ($options as $option)
 		{
-			if($option==$curval)
+			if ($option == $curval)
 			{
 				$this->assertEquals($i, $value);
 				return;
 			}
 			$i++;
 		}
-		$this->fail('Current value '.$curval.' not found in: '.implode(',', $options));
+		$this->fail('Current value ' . $curval . ' not found in: ' . implode(',', $options));
 	}
 
 	protected function assertSelected($id, $label)
@@ -241,7 +241,7 @@ 
 	protected function assertAlertNotPresent()
 	{
 		try {
-			$foo=$this->alertText();
+			$foo = $this->alertText();
 		} catch (PHPUnit_Extensions_Selenium2TestCase_WebDriverException $e) {
 			$this->assertEquals(PHPUnit_Extensions_Selenium2TestCase_WebDriverException::NoAlertOpenError, $e->getCode());
 			return;
@@ -251,13 +251,13 @@ 
 
 	protected function pause($msec)
 	{
-		usleep($msec*1000);
+		usleep($msec * 1000);
 	}
 
 	public function assertSourceContains($text)
 	{
 		$found = strpos($this->source(), $text) !== false;
-		for($i=0;$i<10 && ! $found; $i++) {
+		for ($i = 0; $i < 10 && !$found; $i++) {
 			$this->pause(20);
 			$found = strpos($this->source(), $text) !== false;
 		}
@@ -267,7 +267,7 @@ 
 	public function assertSourceNotContains($text)
 	{
 		$found = strpos($this->source(), $text) !== false;
-		for($i=0;$i<10 && $found; $i++) {
+		for ($i = 0; $i < 10 && $found; $i++) {
 			$this->pause(20);
 			$found = strpos($this->source(), $text) !== false;
 		}

tests/unit/TComponentTest.php

Spacing +188 added lines, -188 removed lines

@@ -20,7 +20,7 @@ 
   }
 
   public function setText($value) {
-    $this->_text=$value;
+    $this->_text = $value;
   }
 
   public function getReadOnlyProperty() {
@@ -32,22 +32,22 @@ 
   }
 
   public function getObject() {
-    if(!$this->_object) {
-      $this->_object=new NewComponent;
-      $this->_object->_text='object text';
+    if (!$this->_object) {
+      $this->_object = new NewComponent;
+      $this->_object->_text = 'object text';
     }
     return $this->_object;
   }
 
   public function onMyEvent($param) {
-    $this->raiseEvent('OnMyEvent',$this,$param);
+    $this->raiseEvent('OnMyEvent', $this, $param);
   }
 
-  public function myEventHandler($sender,$param) {
-    $this->_eventHandled=true;
+  public function myEventHandler($sender, $param) {
+    $this->_eventHandled = true;
   }
 
-  public function eventReturnValue($sender,$param) {
+  public function eventReturnValue($sender, $param) {
     return $param->Return;
   }
 
@@ -95,22 +95,22 @@ 
 		return $this->_callorder;
 	}
 	public function __dycall($method, $args) {
-		if(strncasecmp($method,'fx',2)!==0) return;
+		if (strncasecmp($method, 'fx', 2) !== 0) return;
 		$this->_callorder[] = 'fxcall';
 	}
-	public function fxGlobalListener($sender,$param,$name) {
+	public function fxGlobalListener($sender, $param, $name) {
 		$this->_callorder[] = 'fxGL';
 	}
-	public function fxPrimaryGlobalEvent($sender,$param,$name) {
+	public function fxPrimaryGlobalEvent($sender, $param, $name) {
 		$this->_callorder[] = 'primary';
 	}
-	public function commonRaiseEventListener($sender,$param,$name) {
+	public function commonRaiseEventListener($sender, $param, $name) {
 		$this->_callorder[] = 'com';
 	}
-	public function postglobalRaiseEventListener($sender,$param,$name) {
+	public function postglobalRaiseEventListener($sender, $param, $name) {
 		$this->_callorder[] = 'postgl';
 	}
-	public function preglobalRaiseEventListener($sender,$param,$name) {
+	public function preglobalRaiseEventListener($sender, $param, $name) {
 		$this->_callorder[] = 'pregl';
 	}
 }
@@ -123,7 +123,7 @@ 
 		$this->_baseObject = $object;
 		return $laa * $sol;
 	}
-	public function getLastClassObject() {return $this->_baseObject;}
+	public function getLastClassObject() {return $this->_baseObject; }
 }
 
 class FooFooClassBehavior extends FooClassBehavior {
@@ -164,10 +164,10 @@ 
 	private $_instanceReturn = null;
 
 	public function moreFunction($laa, $sol) {
-		return pow($laa+$sol+1, 2);
+		return pow($laa + $sol + 1, 2);
 	}
 
-	public function isinstanceof($class, $instance=null) {
+	public function isinstanceof($class, $instance = null) {
 		return $this->_instanceReturn;
 	}
 	public function setInstanceReturn($value) {
@@ -176,14 +176,14 @@ 
 }
 class DynamicCallComponent extends NewComponent implements IDynamicMethods {
 	public function __dycall($method, $args) {
-		if($method === 'dyPowerFunction')
+		if ($method === 'dyPowerFunction')
 			return pow($args[0], $args[1]);
-		if($method === 'dyDivisionFunction')
+		if ($method === 'dyDivisionFunction')
 			return $args[0] / $args[1];
-		if($method === 'fxPowerFunction')
-			return 2*pow($args[0], $args[1]);
-		if($method === 'fxDivisionFunction')
-			return 2*$args[0] / $args[1];
+		if ($method === 'fxPowerFunction')
+			return 2 * pow($args[0], $args[1]);
+		if ($method === 'fxDivisionFunction')
+			return 2 * $args[0] / $args[1];
 	}
 }
 
@@ -201,8 +201,8 @@ 
 		return true;
 	}
 
-	public function onBehaviorEvent($sender, $param,$responsetype=null,$postfunction=null) {
-    	return $this->getOwner()->raiseEvent('onBehaviorEvent',$sender,$param,$responsetype,$postfunction);
+	public function onBehaviorEvent($sender, $param, $responsetype = null, $postfunction = null) {
+    	return $this->getOwner()->raiseEvent('onBehaviorEvent', $sender, $param, $responsetype, $postfunction);
 	}
 	public function fxGlobalBehaviorEvent($sender, $param) {
 	}
@@ -294,22 +294,22 @@ 
 
 		return $chain->dyUnlisten($fx);
 	}
-	public function dyPreRaiseEvent($name,$sender,$param,$responsetype,$postfunction, $chain) {
+	public function dyPreRaiseEvent($name, $sender, $param, $responsetype, $postfunction, $chain) {
 		$this->lastCall = 3;
 		$this->arglist = func_get_args();
 
-		return $chain->dyPreRaiseEvent($name);// Calls the next event, within a chain, if parameters are left off, they are filled in with
+		return $chain->dyPreRaiseEvent($name); // Calls the next event, within a chain, if parameters are left off, they are filled in with
 		//	 the original parameters passed to the dynamic event. Parameters can be passed if they are changed.
 	}
-	public function dyIntraRaiseEventTestHandler($handler,$sender,$param,$name, $chain) {
+	public function dyIntraRaiseEventTestHandler($handler, $sender, $param, $name, $chain) {
 		$this->lastCall += 4;
 		$this->arglist = func_get_args();
 	}
-	public function dyIntraRaiseEventPostHandler($name,$sender,$param,$handler, $chain) {
+	public function dyIntraRaiseEventPostHandler($name, $sender, $param, $handler, $chain) {
 		$this->lastCall += 5;
 		$this->arglist = func_get_args();
 	}
-	public function dyPostRaiseEvent($responses,$name,$sender,$param,$responsetype,$postfunction, $chain) {
+	public function dyPostRaiseEvent($responses, $name, $sender, $param, $responsetype, $postfunction, $chain) {
 		$this->lastCall += 6;
 		$this->arglist = func_get_args();
 	}
@@ -332,11 +332,11 @@ 
 		$this->lastCall = 10;
 		$this->arglist = func_get_args();
 	}
-	public function dyAttachBehavior($name,$behavior, $chain) {
+	public function dyAttachBehavior($name, $behavior, $chain) {
 		$this->lastCall = 11;
 		$this->arglist = func_get_args();
 	}
-	public function dyDetachBehavior($name,$behavior, $chain) {
+	public function dyDetachBehavior($name, $behavior, $chain) {
 		$this->lastCall = 12;
 		$this->arglist = func_get_args();
 	}
@@ -348,11 +348,11 @@ 
 		$this->lastCall = 14;
 		$this->arglist = func_get_args();
 	}
-	public function dyEnableBehavior($name,$behavior, $chain) {
+	public function dyEnableBehavior($name, $behavior, $chain) {
 		$this->lastCall = 15;
 		$this->arglist = func_get_args();
 	}
-	public function dyDisableBehavior($name,$behavior, $chain) {
+	public function dyDisableBehavior($name, $behavior, $chain) {
 		$this->lastCall = 16;
 		$this->arglist = func_get_args();
 	}
@@ -370,11 +370,11 @@ 
 	}
 	public function __dycall($method, $args) {
 		$this->_dyMethod = $method;
-		if($method == 'dyTestDynamicBehaviorMethod')
+		if ($method == 'dyTestDynamicBehaviorMethod')
 			return $args[0] / $args[1];
 	}
 	public function dyTestIntraEvent($param1, $param2, $chain) {
-		return $chain->dyTestIntraEvent($param1*2*$param2, $param2);
+		return $chain->dyTestIntraEvent($param1 * 2 * $param2, $param2);
 	}
 	public function TestBehaviorMethod($param1, $param2) {
 		return $param1 * $param2;
@@ -391,11 +391,11 @@ 
 	public function __dycall($method, $args) {
 		$this->_dyMethod = $method;
 		$object = array_shift($args);
-		if($method == 'dyTestDynamicClassBehaviorMethod')
+		if ($method == 'dyTestDynamicClassBehaviorMethod')
 			return $args[0] / $args[1];
 	}
 	public function dyTestIntraEvent($object, $param1, $param2, $chain) {
-		return $chain->dyTestIntraEvent($param1*2*$param2, $param2);
+		return $chain->dyTestIntraEvent($param1 * 2 * $param2, $param2);
 	}
 	public function TestBehaviorMethod($object, $param1, $param2) {
 		return $param1 * $param2;
@@ -420,7 +420,7 @@ 
   public function tearDown() {
   	// PHP version 5.3.6 doesn't call the __destruct method when unsetting variables;
   	//	Thus any object that listens must be explicitly call unlisten in this version of PHP.
-  	if($this->component)
+  	if ($this->component)
 	    $this->component->unlisten();
     $this->component = null;
   }
@@ -461,7 +461,7 @@ 
 	$this->assertEquals(25, $component->Bar->moreFunction(2, 2));
 	$this->assertEquals(8, $component->FooBar->moreFunction(2, 2));
 
-  	$component->unlisten();// unwind object and class behaviors
+  	$component->unlisten(); // unwind object and class behaviors
   	$this->component->detachClassBehavior('FooBar', 'NewComponent');
   	$this->component->detachClassBehavior('Bar', 'NewComponentNoListen');
 
@@ -545,7 +545,7 @@ 
     try {
       $this->component->attachClassBehavior('FooClassBehavior', new BarClassBehavior);
       $this->fail('TInvalidOperationException not raised when overwriting an existing behavior');
-    } catch(TInvalidOperationException $e) {
+    } catch (TInvalidOperationException $e) {
     }
 
 
@@ -553,7 +553,7 @@ 
     try {
   	  $this->component->attachClassBehavior('FooBarBehavior', 'FooBarBehavior', 'TComponent');
       $this->fail('TInvalidOperationException not raised when trying to place a behavior on the root object TComponent');
-    } catch(TInvalidOperationException $e) {
+    } catch (TInvalidOperationException $e) {
     }
 
 
@@ -815,7 +815,7 @@ 
     try {
 	  $this->component->faaEverMore(true, true);
       $this->fail('TApplicationException not raised trying to execute a undefined class method');
-    } catch(TApplicationException $e) {}
+    } catch (TApplicationException $e) {}
 
     $this->assertNull($this->component->asa('FooBehavior'));
     $this->assertFalse($this->component->isa('FooBehavior'));
@@ -825,7 +825,7 @@ 
     try {
       $this->component->attachBehavior('FooBehavior', new TComponent);
       $this->fail('TApplicationException not raised trying to execute a undefined class method');
-    } catch(TInvalidDataTypeException $e) {}
+    } catch (TInvalidDataTypeException $e) {}
 
     $this->component->attachBehavior('FooBehavior', new FooBehavior);
 
@@ -836,14 +836,14 @@ 
 
     try {
 	    $this->assertTrue($this->component->faaEverMore(true, true));
-    } catch(TApplicationException $e) {
+    } catch (TApplicationException $e) {
       $this->fail('TApplicationException raised while trying to execute a behavior class method');
     }
 
     try {
 	  $this->component->noMethodHere(true);
       $this->fail('TApplicationException not raised trying to execute a undefined class method');
-    } catch(TApplicationException $e) {}
+    } catch (TApplicationException $e) {}
 
     $this->assertTrue($this->component->disableBehavior('FooBehavior'));
 
@@ -853,7 +853,7 @@ 
     try {
 	  $this->component->faaEverMore(true, true);
       $this->fail('TApplicationException not raised trying to execute a undefined class method');
-    } catch(TApplicationException $e) {}
+    } catch (TApplicationException $e) {}
 
     $this->assertTrue($this->component->enableBehavior('FooBehavior'));
 
@@ -862,7 +862,7 @@ 
 
     try {
 	    $this->assertTrue($this->component->faaEverMore(true, true));
-    } catch(TApplicationException $e) {
+    } catch (TApplicationException $e) {
       $this->fail('TApplicationException raised while trying to execute a behavior class method');
     }
 
@@ -962,14 +962,14 @@ 
     try {
 	  $this->component->faaEverMore(true, true);
       $this->fail('TApplicationException not raised trying to execute a undefined class method');
-    } catch(TApplicationException $e) {}
+    } catch (TApplicationException $e) {}
 
     $this->component->attachBehavior('FooBehavior', new FooBehavior);
 
     $this->assertTrue($this->component->isa('FooBehavior'));
     try {
 	    $this->assertTrue($this->component->faaEverMore(true, true));
-    } catch(TApplicationException $e) {
+    } catch (TApplicationException $e) {
       $this->fail('TApplicationException raised while trying to execute a behavior class method');
     }
 
@@ -980,14 +980,14 @@ 
     try {
 	  $this->component->faaEverMore(true, true);
       $this->fail('TApplicationException not raised trying to execute a undefined class method');
-    } catch(TApplicationException $e) {}
+    } catch (TApplicationException $e) {}
 
     $this->assertTrue($this->component->enableBehavior('FooBehavior'));
 
     $this->assertTrue($this->component->isa('FooBehavior'));
     try {
 	    $this->assertTrue($this->component->faaEverMore(true, true));
-    } catch(TApplicationException $e) {
+    } catch (TApplicationException $e) {
       $this->fail('TApplicationException raised while trying to execute a behavior class method');
     }
 
@@ -999,7 +999,7 @@ 
     try {
 	  $this->component->moreFunction(true, true);
       $this->fail('TApplicationException not raised trying to execute an undefined class method');
-    } catch(TApplicationException $e) {}
+    } catch (TApplicationException $e) {}
 
     $this->component->attachClassBehavior('BarClassBehavior', new BarClassBehavior);
 
@@ -1008,7 +1008,7 @@ 
 
     try {
 	    $this->assertTrue($this->component->moreFunction(true, true));
-    } catch(TApplicationException $e) {
+    } catch (TApplicationException $e) {
       $this->fail('TApplicationException raised while trying to execute a behavior class method');
     }
 
@@ -1035,7 +1035,7 @@ 
     try {
 	    $this->assertNull($this->component->moreFunction(3, 4));
       	$this->fail('TApplicationException not raised trying to execute a disabled behavior');
-    } catch(TApplicationException $e) {}
+    } catch (TApplicationException $e) {}
     $this->assertTrue($this->component->enableBehavior('FooBarBehavior'));
 
 	// Test the global event space, this should work and return false because no function implements these methods
@@ -1061,7 +1061,7 @@ 
     try {
     	$this->component->faafaaEverMore(3, 4);
       	$this->fail('TApplicationException not raised trying to execute a disabled behavior');
-    } catch(TApplicationException $e) {}
+    } catch (TApplicationException $e) {}
 
 
 
@@ -1174,18 +1174,18 @@ 
   }
 
   public function testGetProperty() {
-    $this->assertTrue('default'===$this->component->Text);
+    $this->assertTrue('default' === $this->component->Text);
     try {
-      $value2=$this->component->Caption;
+      $value2 = $this->component->Caption;
       $this->fail('exception not raised when getting undefined property');
-    } catch(TInvalidOperationException $e) {
+    } catch (TInvalidOperationException $e) {
     }
 
     $this->assertTrue($this->component->OnMyEvent instanceof TPriorityList);
     try {
-      $value2=$this->component->onUndefinedEvent;
+      $value2 = $this->component->onUndefinedEvent;
       $this->fail('exception not raised when getting undefined property');
-    } catch(TInvalidOperationException $e) {
+    } catch (TInvalidOperationException $e) {
     }
 
     //Without the function parenthesis, the function is _not_ called but the __get
@@ -1201,9 +1201,9 @@ 
     $this->component->enableBehaviors();
 
     try {
-      $value2=$this->component->Excitement;
+      $value2 = $this->component->Excitement;
       $this->fail('exception not raised when getting undefined property');
-    } catch(TInvalidOperationException $e) {
+    } catch (TInvalidOperationException $e) {
     }
 
     $this->component->attachBehavior('BehaviorTestBehavior', $behavior = new BehaviorTestBehavior);
@@ -1214,7 +1214,7 @@ 
     try {
       $this->assertEquals('faa', $this->component->Excitement);
       $this->fail('exception not raised when getting undefined property');
-    } catch(TInvalidOperationException $e) {
+    } catch (TInvalidOperationException $e) {
     }
 
     $this->component->enableBehaviors();
@@ -1226,13 +1226,13 @@ 
     try {
 	      $behavior = $this->component->BehaviorTestBehavior2;
 	      $this->fail('exception not raised when getting undefined property');
-	    } catch(TInvalidOperationException $e) {
+	    } catch (TInvalidOperationException $e) {
     }
 
     try {
       $this->assertEquals('faa', $this->component->Excitement);
       $this->fail('exception not raised when getting undefined property');
-    } catch(TInvalidOperationException $e) {
+    } catch (TInvalidOperationException $e) {
     }
     $this->component->enableBehavior('BehaviorTestBehavior');
     $this->assertEquals('faa', $this->component->getExcitement());
@@ -1244,39 +1244,39 @@ 
   }
 
   public function testSetProperty() {
-    $value='new value';
-    $this->component->Text=$value;
-    $text=$this->component->Text;
-    $this->assertTrue($value===$this->component->Text);
+    $value = 'new value';
+    $this->component->Text = $value;
+    $text = $this->component->Text;
+    $this->assertTrue($value === $this->component->Text);
     try {
-      $this->component->NewMember=$value;
+      $this->component->NewMember = $value;
       $this->fail('exception not raised when setting undefined property');
-    } catch(TInvalidOperationException $e) {
+    } catch (TInvalidOperationException $e) {
     }
 
     // Test get only properties is a set function
     try {
       $this->component->ReadOnlyProperty = 'setting read only';
       $this->fail('a property without a set function was set to a new value without error');
-    } catch(TInvalidOperationException $e) {
+    } catch (TInvalidOperationException $e) {
     }
 
     try {
       $this->component->ReadOnlyJsProperty = 'jssetting read only';
       $this->fail('a js property without a set function was set to a new value without error');
-    } catch(TInvalidOperationException $e) {
+    } catch (TInvalidOperationException $e) {
     }
 
     try {
       $this->component->JsReadOnlyJsProperty = 'jssetting read only';
       $this->fail('a js property without a set function was set to a new value without error');
-    } catch(TInvalidOperationException $e) {
+    } catch (TInvalidOperationException $e) {
     }
 
     $this->assertEquals(0, $this->component->getEventHandlers('onMyEvent')->getCount());
-    $this->component->onMyEvent = array($this->component,'myEventHandler');
+    $this->component->onMyEvent = array($this->component, 'myEventHandler');
     $this->assertEquals(1, $this->component->getEventHandlers('onMyEvent')->getCount());
-    $this->component->onMyEvent[] = array($this->component,'Object.myEventHandler');
+    $this->component->onMyEvent[] = array($this->component, 'Object.myEventHandler');
     $this->assertEquals(2, $this->component->getEventHandlers('onMyEvent')->getCount());
 
     $this->component->getEventHandlers('onMyEvent')->clear();
@@ -1287,7 +1287,7 @@ 
     try {
       $this->component->Excitement = 'laa';
       $this->fail('exception not raised when getting undefined property');
-    } catch(TInvalidOperationException $e) {
+    } catch (TInvalidOperationException $e) {
     }
 
     $this->component->attachBehavior('BehaviorTestBehavior', $behavior1 = new BehaviorTestBehavior);
@@ -1302,7 +1302,7 @@ 
       $this->component->Excitement = false;
       $this->assertEquals(false, $this->component->Excitement);
       $this->fail('exception not raised when getting undefined property');
-    } catch(TInvalidOperationException $e) {
+    } catch (TInvalidOperationException $e) {
     }
 
     $this->component->enableBehaviors();
@@ -1315,7 +1315,7 @@ 
       $this->component->Excitement = false;
       $this->assertEquals(false, $this->component->Excitement);
       $this->fail('exception not raised when getting undefined property');
-    } catch(TInvalidOperationException $e) {
+    } catch (TInvalidOperationException $e) {
     }
     $this->component->enableBehavior('BehaviorTestBehavior');
     $this->component->Excitement = 'sol';
@@ -1341,9 +1341,9 @@ 
     $this->assertTrue($this->component->BehaviorTestBehavior->onBehaviorEvent instanceof TPriorityList);
 
     $this->assertEquals(0, $this->component->BehaviorTestBehavior->getEventHandlers('onBehaviorEvent')->getCount());
-    $this->component->onBehaviorEvent = array($this->component,'myEventHandler');
+    $this->component->onBehaviorEvent = array($this->component, 'myEventHandler');
     $this->assertEquals(1, $this->component->BehaviorTestBehavior->getEventHandlers('onBehaviorEvent')->getCount());
-    $this->component->onBehaviorEvent[] = array($this->component,'Object.myEventHandler');
+    $this->component->onBehaviorEvent[] = array($this->component, 'Object.myEventHandler');
     $this->assertEquals(2, $this->component->BehaviorTestBehavior->getEventHandlers('onBehaviorEvent')->getCount());
 
     $this->component->BehaviorTestBehavior->getEventHandlers('onBehaviorEvent')->clear();
@@ -1366,10 +1366,10 @@ 
     $this->assertTrue(isset($this->component->BehaviorTestBehavior));
     $this->assertFalse(isset($this->component->onBehaviorEvent));
 
-    $this->component->attachEventHandler('onBehaviorEvent','foo');
+    $this->component->attachEventHandler('onBehaviorEvent', 'foo');
     $this->assertTrue(isset($this->component->onBehaviorEvent));
 
-    $this->component->attachEventHandler('onMyEvent','foo');
+    $this->component->attachEventHandler('onMyEvent', 'foo');
     $this->assertTrue(isset($this->component->onMyEvent));
 
     $this->assertTrue(isset($this->component->Excitement));
@@ -1391,7 +1391,7 @@ 
 
  	// object events
  	$this->assertEquals(0, $this->component->onMyEvent->Count);
-    $this->component->attachEventHandler('onMyEvent','foo');
+    $this->component->attachEventHandler('onMyEvent', 'foo');
  	$this->assertEquals(1, $this->component->onMyEvent->Count);
  	unset($this->component->onMyEvent);
  	$this->assertEquals(0, $this->component->onMyEvent->Count);
@@ -1408,7 +1408,7 @@ 
   	try {
 	  	unset($this->component->Object);
     	$this->fail('TInvalidOperationException not raised when unsetting get only property');
-  	} catch(TInvalidOperationException $e) {}
+  	} catch (TInvalidOperationException $e) {}
 
     $this->component->attachBehavior('BehaviorTestBehavior', new BehaviorTestBehavior);
  	$this->assertTrue($this->component->asa('BehaviorTestBehavior') instanceof BehaviorTestBehavior);
@@ -1437,7 +1437,7 @@ 
   	try {
 	  	unset($this->component->ReadOnly);
     	$this->fail('TInvalidOperationException not raised when unsetting get only property');
-  	} catch(TInvalidOperationException $e) {}
+  	} catch (TInvalidOperationException $e) {}
 
   	$this->component->onBehaviorEvent = 'foo';
   	$this->assertEquals(1, count($this->component->onBehaviorEvent));
@@ -1453,12 +1453,12 @@ 
   }
 
   public function testGetSubProperty() {
-    $this->assertTrue('object text'===$this->component->getSubProperty('Object.Text'));
+    $this->assertTrue('object text' === $this->component->getSubProperty('Object.Text'));
   }
 
   public function testSetSubProperty() {
-    $this->component->setSubProperty('Object.Text','new object text');
-    $this->assertEquals('new object text',$this->component->getSubProperty('Object.Text'));
+    $this->component->setSubProperty('Object.Text', 'new object text');
+    $this->assertEquals('new object text', $this->component->getSubProperty('Object.Text'));
   }
 
   public function testHasEvent() {
@@ -1492,11 +1492,11 @@ 
 
   public function testHasEventHandler() {
     $this->assertFalse($this->component->hasEventHandler('OnMyEvent'));
-    $this->component->attachEventHandler('OnMyEvent','foo');
+    $this->component->attachEventHandler('OnMyEvent', 'foo');
     $this->assertTrue($this->component->hasEventHandler('OnMyEvent'));
 
     $this->assertFalse($this->component->hasEventHandler('fxNonExistantGlobalEvent'));
-    $this->component->attachEventHandler('fxNonExistantGlobalEvent','foo');
+    $this->component->attachEventHandler('fxNonExistantGlobalEvent', 'foo');
     $this->assertTrue($this->component->hasEventHandler('fxNonExistantGlobalEvent'));
 
     //Test behavior events
@@ -1505,7 +1505,7 @@ 
     $this->assertFalse($this->component->hasEventHandler('onBehaviorEvent'));
     $this->assertFalse($this->component->BehaviorTestBehavior->hasEventHandler('onBehaviorEvent'));
 
-    $this->component->attachEventHandler('onBehaviorEvent','foo');
+    $this->component->attachEventHandler('onBehaviorEvent', 'foo');
     $this->assertTrue($this->component->hasEventHandler('onBehaviorEvent'));
 
     $this->component->disableBehavior('BehaviorTestBehavior');
@@ -1517,23 +1517,23 @@ 
   }
 
   public function testGetEventHandlers() {
-    $list=$this->component->getEventHandlers('OnMyEvent');
-    $this->assertTrue(($list instanceof TPriorityList) && ($list->getCount()===0));
-    $this->component->attachEventHandler('OnMyEvent','foo');
-    $this->assertTrue(($list instanceof TPriorityList) && ($list->getCount()===1));
+    $list = $this->component->getEventHandlers('OnMyEvent');
+    $this->assertTrue(($list instanceof TPriorityList) && ($list->getCount() === 0));
+    $this->component->attachEventHandler('OnMyEvent', 'foo');
+    $this->assertTrue(($list instanceof TPriorityList) && ($list->getCount() === 1));
     try {
-      $list=$this->component->getEventHandlers('YourEvent');
+      $list = $this->component->getEventHandlers('YourEvent');
       $this->fail('exception not raised when getting event handlers for undefined event');
-    } catch(TInvalidOperationException $e) {
+    } catch (TInvalidOperationException $e) {
     }
 
-    $list=$this->component->getEventHandlers('fxRandomEvent');
-    $this->assertTrue(($list instanceof TPriorityList) && ($list->getCount()===0));
-    $this->component->attachEventHandler('fxRandomEvent','foo');
-    $this->assertTrue(($list instanceof TPriorityList) && ($list->getCount()===1));
+    $list = $this->component->getEventHandlers('fxRandomEvent');
+    $this->assertTrue(($list instanceof TPriorityList) && ($list->getCount() === 0));
+    $this->component->attachEventHandler('fxRandomEvent', 'foo');
+    $this->assertTrue(($list instanceof TPriorityList) && ($list->getCount() === 1));
     try {
-      $list=$this->component->getEventHandlers('fxSomeUndefinedGlobalEvent');
-    } catch(TInvalidOperationException $e) {
+      $list = $this->component->getEventHandlers('fxSomeUndefinedGlobalEvent');
+    } catch (TInvalidOperationException $e) {
       $this->fail('exception raised when getting event handlers for universal global event');
     }
 
@@ -1541,43 +1541,43 @@ 
 
     //Test behavior events
     try {
-      $list=$this->component->getEventHandlers('onBehaviorEvent');
+      $list = $this->component->getEventHandlers('onBehaviorEvent');
       $this->fail('exception not raised when getting event handlers for undefined event');
-    } catch(TInvalidOperationException $e) {
+    } catch (TInvalidOperationException $e) {
     }
     $this->assertFalse($this->component->hasEventHandler('onBehaviorEvent'));
 
     $this->component->attachBehavior('BehaviorTestBehavior', new BehaviorTestBehavior);
-    $list=$this->component->getEventHandlers('onBehaviorEvent');
-    $this->assertTrue(($list instanceof TPriorityList) && ($list->getCount()===0));
-    $this->component->attachEventHandler('onBehaviorEvent','foo');
-    $this->assertTrue(($list instanceof TPriorityList) && ($list->getCount()===1));
+    $list = $this->component->getEventHandlers('onBehaviorEvent');
+    $this->assertTrue(($list instanceof TPriorityList) && ($list->getCount() === 0));
+    $this->component->attachEventHandler('onBehaviorEvent', 'foo');
+    $this->assertTrue(($list instanceof TPriorityList) && ($list->getCount() === 1));
 
     $this->component->disableBehavior('BehaviorTestBehavior');
     try {
-      $list=$this->component->getEventHandlers('onBehaviorEvent');
+      $list = $this->component->getEventHandlers('onBehaviorEvent');
       $this->fail('exception not raised when getting event handlers for undefined event');
-    } catch(TInvalidOperationException $e) {
+    } catch (TInvalidOperationException $e) {
     }
     $this->component->enableBehavior('BehaviorTestBehavior');
-    $this->assertTrue(($this->component->getEventHandlers('onBehaviorEvent') instanceof TPriorityList) && ($list->getCount()===1));
+    $this->assertTrue(($this->component->getEventHandlers('onBehaviorEvent') instanceof TPriorityList) && ($list->getCount() === 1));
 
   }
 
   public function testAttachEventHandler() {
 
-    $this->component->attachEventHandler('OnMyEvent','foo');
+    $this->component->attachEventHandler('OnMyEvent', 'foo');
     $this->assertEquals(1, $this->component->getEventHandlers('OnMyEvent')->getCount());
     try {
-      $this->component->attachEventHandler('YourEvent','foo');
+      $this->component->attachEventHandler('YourEvent', 'foo');
       $this->fail('exception not raised when attaching event handlers for undefined event');
-    } catch(TInvalidOperationException $e) {
+    } catch (TInvalidOperationException $e) {
     }
 
     //Testing the priorities of attaching events
-    $this->component->attachEventHandler('OnMyEvent','foopre', 5);
-    $this->component->attachEventHandler('OnMyEvent','foopost', 15);
-    $this->component->attachEventHandler('OnMyEvent','foobar', 10);
+    $this->component->attachEventHandler('OnMyEvent', 'foopre', 5);
+    $this->component->attachEventHandler('OnMyEvent', 'foopost', 15);
+    $this->component->attachEventHandler('OnMyEvent', 'foobar', 10);
     $this->assertEquals(4, $this->component->getEventHandlers('OnMyEvent')->getCount());
     $list = $this->component->getEventHandlers('OnMyEvent');
     $this->assertEquals('foopre', $list[0]);
@@ -1588,19 +1588,19 @@ 
 
     //Test attaching behavior events
     try {
-      $this->component->attachEventHandler('onBehaviorEvent','foo');
+      $this->component->attachEventHandler('onBehaviorEvent', 'foo');
       $this->fail('exception not raised when getting event handlers for undefined event');
-    } catch(TInvalidOperationException $e) {
+    } catch (TInvalidOperationException $e) {
     }
     $this->component->attachBehavior('BehaviorTestBehavior', new BehaviorTestBehavior);
 
-    $this->component->attachEventHandler('onBehaviorEvent','foo');
+    $this->component->attachEventHandler('onBehaviorEvent', 'foo');
 
     //Testing the priorities of attaching behavior events
-    $this->component->attachEventHandler('onBehaviorEvent','foopre', 5);
-    $this->component->attachEventHandler('onBehaviorEvent','foopost', 15);
-    $this->component->attachEventHandler('onBehaviorEvent','foobar', 10);
-    $this->component->attachEventHandler('onBehaviorEvent','foobarfoobar', 10);
+    $this->component->attachEventHandler('onBehaviorEvent', 'foopre', 5);
+    $this->component->attachEventHandler('onBehaviorEvent', 'foopost', 15);
+    $this->component->attachEventHandler('onBehaviorEvent', 'foobar', 10);
+    $this->component->attachEventHandler('onBehaviorEvent', 'foobarfoobar', 10);
     $this->assertEquals(5, $this->component->getEventHandlers('onBehaviorEvent')->getCount());
     $list = $this->component->getEventHandlers('onBehaviorEvent');
     $this->assertEquals('foopre', $list[0]);
@@ -1611,9 +1611,9 @@ 
 
     $this->component->disableBehavior('BehaviorTestBehavior');
     try {
-      $this->component->attachEventHandler('onBehaviorEvent','bar');
+      $this->component->attachEventHandler('onBehaviorEvent', 'bar');
       $this->fail('exception not raised when getting event handlers for undefined event');
-    } catch(TInvalidOperationException $e) {
+    } catch (TInvalidOperationException $e) {
     }
     $this->component->enableBehavior('BehaviorTestBehavior');
 
@@ -1621,17 +1621,17 @@ 
 
   public function testDetachEventHandler() {
 
-    $this->component->attachEventHandler('OnMyEvent','foo');
+    $this->component->attachEventHandler('OnMyEvent', 'foo');
     $this->assertEquals(1, $this->component->getEventHandlers('OnMyEvent')->getCount());
 
-    $this->component->attachEventHandler('OnMyEvent','foopre', 5);
-    $this->component->attachEventHandler('OnMyEvent','foopost', 15);
-    $this->component->attachEventHandler('OnMyEvent','foobar', 10);
-    $this->component->attachEventHandler('OnMyEvent','foobarfoobar', 10);
+    $this->component->attachEventHandler('OnMyEvent', 'foopre', 5);
+    $this->component->attachEventHandler('OnMyEvent', 'foopost', 15);
+    $this->component->attachEventHandler('OnMyEvent', 'foobar', 10);
+    $this->component->attachEventHandler('OnMyEvent', 'foobarfoobar', 10);
 
 
 
-    $this->component->detachEventHandler('OnMyEvent','foo');
+    $this->component->detachEventHandler('OnMyEvent', 'foo');
     $list = $this->component->getEventHandlers('OnMyEvent');
     $this->assertEquals(4, $list->getCount());
 
@@ -1640,32 +1640,32 @@ 
     $this->assertEquals('foobarfoobar', $list[2]);
     $this->assertEquals('foopost', $list[3]);
 
-    $this->component->detachEventHandler('OnMyEvent','foopre', null);
+    $this->component->detachEventHandler('OnMyEvent', 'foopre', null);
     $this->assertEquals(4, $list->getCount());
 
-    $this->component->detachEventHandler('OnMyEvent','foopre', 5);
+    $this->component->detachEventHandler('OnMyEvent', 'foopre', 5);
     $this->assertEquals(3, $list->getCount());
 
 
     // Now do detaching of behavior on events
     try {
-      $this->component->attachEventHandler('onBehaviorEvent','foo');
+      $this->component->attachEventHandler('onBehaviorEvent', 'foo');
       $this->fail('exception not raised when getting event handlers for undefined event');
-    } catch(TInvalidOperationException $e) {
+    } catch (TInvalidOperationException $e) {
     }
     $this->component->attachBehavior('BehaviorTestBehavior', new BehaviorTestBehavior);
 
-    $this->component->attachEventHandler('onBehaviorEvent','foo');
+    $this->component->attachEventHandler('onBehaviorEvent', 'foo');
     $this->assertEquals(1, $this->component->getEventHandlers('onBehaviorEvent')->getCount());
 
-    $this->component->attachEventHandler('onBehaviorEvent','foopre', 5);
-    $this->component->attachEventHandler('onBehaviorEvent','foopost', 15);
-    $this->component->attachEventHandler('onBehaviorEvent','foobar', 10);
-    $this->component->attachEventHandler('onBehaviorEvent','foobarfoobar', 10);
+    $this->component->attachEventHandler('onBehaviorEvent', 'foopre', 5);
+    $this->component->attachEventHandler('onBehaviorEvent', 'foopost', 15);
+    $this->component->attachEventHandler('onBehaviorEvent', 'foobar', 10);
+    $this->component->attachEventHandler('onBehaviorEvent', 'foobarfoobar', 10);
 
 
 
-    $this->component->detachEventHandler('onBehaviorEvent','foo');
+    $this->component->detachEventHandler('onBehaviorEvent', 'foo');
     $list = $this->component->getEventHandlers('onBehaviorEvent');
     $this->assertEquals(4, $list->getCount());
 
@@ -1674,10 +1674,10 @@ 
     $this->assertEquals('foobarfoobar', $list[2]);
     $this->assertEquals('foopost', $list[3]);
 
-    $this->component->detachEventHandler('onBehaviorEvent','foopre', null);
+    $this->component->detachEventHandler('onBehaviorEvent', 'foopre', null);
     $this->assertEquals(4, $list->getCount());
 
-    $this->component->detachEventHandler('onBehaviorEvent','foopre', 5);
+    $this->component->detachEventHandler('onBehaviorEvent', 'foopre', 5);
     $this->assertEquals(3, $list->getCount());
   }
 
@@ -1685,13 +1685,13 @@ 
 
 
   public function testRaiseEvent() {
-    $this->component->attachEventHandler('OnMyEvent',array($this->component,'myEventHandler'));
+    $this->component->attachEventHandler('OnMyEvent', array($this->component, 'myEventHandler'));
     $this->assertFalse($this->component->isEventHandled());
-    $this->component->raiseEvent('OnMyEvent',$this,null);
+    $this->component->raiseEvent('OnMyEvent', $this, null);
     $this->assertTrue($this->component->isEventHandled());
-    $this->component->attachEventHandler('OnMyEvent',array($this->component,'Object.myEventHandler'));
+    $this->component->attachEventHandler('OnMyEvent', array($this->component, 'Object.myEventHandler'));
     $this->assertFalse($this->component->Object->isEventHandled());
-    $this->component->raiseEvent('OnMyEvent',$this,null);
+    $this->component->raiseEvent('OnMyEvent', $this, null);
     $this->assertTrue($this->component->Object->isEventHandled());
 
     $this->component->resetEventHandled();
@@ -1701,13 +1701,13 @@ 
     // Test a behavior on event
     $this->component->attachBehavior('test', new BehaviorTestBehavior);
 
-    $this->component->attachEventHandler('onBehaviorEvent',array($this->component,'myEventHandler'));
+    $this->component->attachEventHandler('onBehaviorEvent', array($this->component, 'myEventHandler'));
     $this->assertFalse($this->component->isEventHandled());
-    $this->component->raiseEvent('onBehaviorEvent',$this,null);
+    $this->component->raiseEvent('onBehaviorEvent', $this, null);
     $this->assertTrue($this->component->isEventHandled());
-    $this->component->attachEventHandler('onBehaviorEvent',array($this->component,'Object.myEventHandler'));
+    $this->component->attachEventHandler('onBehaviorEvent', array($this->component, 'Object.myEventHandler'));
     $this->assertFalse($this->component->Object->isEventHandled());
-    $this->component->raiseEvent('onBehaviorEvent',$this,null);
+    $this->component->raiseEvent('onBehaviorEvent', $this, null);
     $this->assertTrue($this->component->Object->isEventHandled());
 
     //test behavior enabled/disabled events
@@ -1717,14 +1717,14 @@ 
     $this->component->Object->resetEventHandled();
 
     try {
-    	$this->component->attachEventHandler('onBehaviorEvent',array($this->component,'myEventHandler'));
+    	$this->component->attachEventHandler('onBehaviorEvent', array($this->component, 'myEventHandler'));
       $this->fail('exception not raised when getting event handlers for undefined event');
-    } catch(TInvalidOperationException $e) {}
+    } catch (TInvalidOperationException $e) {}
     $this->assertFalse($this->component->isEventHandled());
     try {
-      $this->component->raiseEvent('onBehaviorEvent',$this,null);
+      $this->component->raiseEvent('onBehaviorEvent', $this, null);
       $this->fail('exception not raised when getting event handlers for undefined event');
-    } catch(TInvalidOperationException $e) {}
+    } catch (TInvalidOperationException $e) {}
     $this->assertFalse($this->component->isEventHandled());
 
     $this->component->enableBehavior('test');
@@ -1735,12 +1735,12 @@ 
 
     $this->assertFalse($this->component->isEventHandled());
     $this->assertFalse($this->component->Object->isEventHandled());
-    $this->assertEquals(array(), $this->component->onBehaviorEvent($this,$this->component));
+    $this->assertEquals(array(), $this->component->onBehaviorEvent($this, $this->component));
     $this->assertTrue($this->component->isEventHandled());
     $this->assertTrue($this->component->Object->isEventHandled());
 
     // This accumulates all the responses from each of the events
-    $arr=$this->component->onBehaviorEvent($this, $this->component, TEventResults::EVENT_RESULT_ALL);
+    $arr = $this->component->onBehaviorEvent($this, $this->component, TEventResults::EVENT_RESULT_ALL);
     $this->assertEquals($this, $arr[0]['sender']);
     $this->assertEquals($this->component, $arr[0]['param']);
     $this->assertTrue(null === $arr[0]['response']);
@@ -1752,7 +1752,7 @@ 
     $this->assertEquals(2, count($arr));
 
     // This tests without the default filtering-out of null
-    $arr=$this->component->onBehaviorEvent($this, $this->component, false);
+    $arr = $this->component->onBehaviorEvent($this, $this->component, false);
     $this->assertEquals(array(null, null), $arr);
 
 
@@ -1763,9 +1763,9 @@ 
     $this->component->onBehaviorEvent = array($this, 'returnValue1');
 
     // Test the per event post processing function
-    $arr=$this->component->onBehaviorEvent($this, $this->component, array($this, 'postEventFunction'));
+    $arr = $this->component->onBehaviorEvent($this, $this->component, array($this, 'postEventFunction'));
     $this->assertEquals(array(exp(4), exp(1)), $arr);
-    $arr=$this->component->onBehaviorEvent($this, $this->component, array($this, 'postEventFunction2'));
+    $arr = $this->component->onBehaviorEvent($this, $this->component, array($this, 'postEventFunction2'));
     $this->assertEquals(array(sin(4), sin(1)), $arr);
 
 
@@ -1774,7 +1774,7 @@ 
 
     $this->component->onBehaviorEvent = array($this, 'ffValue4');
     $this->component->onBehaviorEvent = array($this, 'ffValue2');
-    $arr=$this->component->onBehaviorEvent($this, 5, TEventResults::EVENT_RESULT_FEED_FORWARD);
+    $arr = $this->component->onBehaviorEvent($this, 5, TEventResults::EVENT_RESULT_FEED_FORWARD);
     $this->assertEquals(array(20, 40), $arr);
 
 
@@ -1783,16 +1783,16 @@ 
     //Order of these events affects the response order in feed forward
     $this->component->onBehaviorEvent = array($this, 'ffValue2');
     $this->component->onBehaviorEvent = array($this, 'ffValue4');
-    $arr=$this->component->onBehaviorEvent($this, 5, TEventResults::EVENT_RESULT_FEED_FORWARD);
+    $arr = $this->component->onBehaviorEvent($this, 5, TEventResults::EVENT_RESULT_FEED_FORWARD);
     $this->assertEquals(array(10, 40), $arr);
   }
 
-  public function returnValue1(){return 1;}
-  public function returnValue4(){return 4;}
-  public function postEventFunction($sender, $param, $caller, $response){return exp($response);}
-  public function postEventFunction2($sender, $param, $caller, $response){return sin($response);}
-  public function ffValue2($sender, $param){return $param*2;}
-  public function ffValue4($sender, $param){return $param*4;}
+  public function returnValue1() {return 1; }
+  public function returnValue4() {return 4; }
+  public function postEventFunction($sender, $param, $caller, $response) {return exp($response); }
+  public function postEventFunction2($sender, $param, $caller, $response) {return sin($response); }
+  public function ffValue2($sender, $param) {return $param * 2; }
+  public function ffValue4($sender, $param) {return $param * 4; }
 
 
   public function testGlobalEventListenerInRaiseEvent() {
@@ -1816,7 +1816,7 @@ 
     try {
 	    $this->component->objectAndBehaviorUndefinedMethod();
 		$this->fail('exception not raised when evaluating an undefined method by the object and behavior');
-    } catch(TApplicationException $e) {
+    } catch (TApplicationException $e) {
     }
 
     // calling undefined dynamic method, caught by the __dycall method in the behavior and implemented
@@ -1851,7 +1851,7 @@ 
     try {
 	    $this->component->objectAndBehaviorUndefinedMethod();
 		$this->fail('exception not raised when evaluating an undefined method by the object and behavior');
-    } catch(TApplicationException $e) {
+    } catch (TApplicationException $e) {
     }
 
     // calling undefined dynamic method, caught by the __dycall method in the behavior and implemented
@@ -1899,12 +1899,12 @@ 
 
 
   public function testEvaluateExpression() {
-    $expression="1+2";
-    $this->assertTrue(3===$this->component->evaluateExpression($expression));
+    $expression = "1+2";
+    $this->assertTrue(3 === $this->component->evaluateExpression($expression));
     try {
-      $button=$this->component->evaluateExpression('$this->button');
+      $button = $this->component->evaluateExpression('$this->button');
       $this->fail('exception not raised when evaluating an invalid exception');
-    } catch(Exception $e) {
+    } catch (Exception $e) {
     }
   }
 
@@ -1912,13 +1912,13 @@ 
 
 
   public function testEvaluateStatements() {
-    $statements='$a="test string"; echo $a;';
-    $this->assertEquals('test string',$this->component->evaluateStatements($statements));
+    $statements = '$a="test string"; echo $a;';
+    $this->assertEquals('test string', $this->component->evaluateStatements($statements));
     try {
-      $statements='$a=new NewComponent; echo $a->button;';
-      $button=$this->component->evaluateStatements($statements);
+      $statements = '$a=new NewComponent; echo $a->button;';
+      $button = $this->component->evaluateStatements($statements);
       $this->fail('exception not raised when evaluating an invalid statement');
-    } catch(Exception $e) {
+    } catch (Exception $e) {
     }
   }
 
@@ -1944,7 +1944,7 @@ 
 
     $this->assertEquals(' aa bb cc __ .. ++ || !! ?? ', $this->component->dyUndefinedEvent(' aa bb cc __ .. ++ || !! ?? '));
 
-    $this->assertEquals(0.25, $this->component->dyPowerFunction(2,2));
+    $this->assertEquals(0.25, $this->component->dyPowerFunction(2, 2));
 
 
     $this->component->detachBehavior('dy1');
@@ -1972,7 +1972,7 @@ 
 
     $this->assertEquals(' aa bb cc __ .. ++ || !! ?? ', $this->component->dyUndefinedEvent(' aa bb cc __ .. ++ || !! ?? '));
 
-    $this->assertEquals(0.25, $this->component->dyPowerFunction(2,2));
+    $this->assertEquals(0.25, $this->component->dyPowerFunction(2, 2));
 
 
   }
@@ -2001,7 +2001,7 @@ 
     $this->assertEquals(3, $this->component->evaluateExpression('1+2'));
     $this->assertEquals(7, $this->component->IntraEvents->LastCall);
 
-    $statements='$a="test string"; echo $a;';
+    $statements = '$a="test string"; echo $a;';
     $this->assertEquals('test string', $this->component->evaluateStatements($statements));
     $this->assertEquals(8, $this->component->IntraEvents->LastCall);
 
@@ -2032,8 +2032,8 @@ 
     $this->assertEquals(12, $this->component->IntraEvents->LastCall);
 
 
-    $this->component->attachEventHandler('OnMyEvent',array($this->component,'myEventHandler'));
-    $this->component->raiseEvent('OnMyEvent',$this,null);
+    $this->component->attachEventHandler('OnMyEvent', array($this->component, 'myEventHandler'));
+    $this->component->raiseEvent('OnMyEvent', $this, null);
 
     //3 + 4 + 5 + 6 = 18 (the behavior adds these together when each raiseEvent dynamic intra event is called)
     $this->assertEquals(18, $this->component->IntraEvents->LastCall);

tests/unit/Collections/TPriorityListTest.php

Spacing +44 added lines, -44 removed lines

@@ -75,7 +75,7 @@ 
 	public function testConstructTList()
 	{
 		$a    = array(
-			1,2,3
+			1, 2, 3
 		);
 		$list = new TPriorityList($a);
 		$this->assertEquals(3, $list->getCount());
@@ -241,7 +241,7 @@ 
 	public function testCanNotRemoveWhenReadOnlyTList()
 	{
 		$list = new TPriorityList(array(
-			1,2,3
+			1, 2, 3
 		), true);
 		try {
 			$list->remove(2);
@@ -251,7 +251,7 @@ 
 		}
 		
 		$list = new TPriorityList(array(
-			1,2,3
+			1, 2, 3
 		), true);
 		try {
 			$list->remove(10);
@@ -279,7 +279,7 @@ 
 	public function testCanNotRemoveAtWhenReadOnlyTList()
 	{
 		$list = new TPriorityList(array(
-			1,2,3
+			1, 2, 3
 		), true);
 		try {
 			$list->removeAt(2);
@@ -289,7 +289,7 @@ 
 		}
 		
 		$list = new TPriorityList(array(
-			1,2,3
+			1, 2, 3
 		), true);
 		try {
 			$list->removeAt(10);
@@ -310,7 +310,7 @@ 
 	public function testCanNotClearWhenReadOnlyTList()
 	{
 		$list = new TPriorityList(array(
-			1,2,3
+			1, 2, 3
 		), true);
 		try {
 			$list->clear();
@@ -338,7 +338,7 @@ 
 	public function testCopyFromTList()
 	{
 		$array = array(
-			$this->item3,$this->item1
+			$this->item3, $this->item1
 		);
 		$this->list->copyFrom($array);
 		$this->assertTrue(count($array) == 2 && $this->list[0] === $this->item3 && $this->list[1] === $this->item1);
@@ -353,7 +353,7 @@ 
 	public function testMergeWithTList()
 	{
 		$array = array(
-			$this->item3,$this->item1
+			$this->item3, $this->item1
 		);
 		$this->list->mergeWith($array);
 		$this->assertTrue($this->list->getCount() == 4 && $this->list[0] === $this->item1 && $this->list[3] === $this->item1);
@@ -408,33 +408,33 @@ 
 	public function testOffsetSetAddTList()
 	{
 		$list = new TPriorityList(array(
-			1,2,3
+			1, 2, 3
 		));
 		$list->offsetSet(null, 4);
 		self::assertEquals(array(
-			1,2,3,4
+			1, 2, 3, 4
 		), $list->toArray());
 	}
 	
 	public function testOffsetSetReplaceTList()
 	{
 		$list = new TPriorityList(array(
-			1,2,3
+			1, 2, 3
 		));
 		$list->offsetSet(1, 4);
 		self::assertEquals(array(
-			1,4,3
+			1, 4, 3
 		), $list->toArray());
 	}
 	
 	public function testOffsetUnsetTList()
 	{
 		$list = new TPriorityList(array(
-			1,2,3
+			1, 2, 3
 		));
 		$list->offsetUnset(1);
 		self::assertEquals(array(
-			1,3
+			1, 3
 		), $list->toArray());
 	}
 	
@@ -449,7 +449,7 @@ 
 	public function testConstructTPriorityList()
 	{
 		$a = array(
-			'a' => 1,'0.5' => 2,9 => 8
+			'a' => 1, '0.5' => 2, 9 => 8
 		);
 		
 		$list = new TPriorityList($a);
@@ -646,38 +646,38 @@ 
 		
 		$plist->insertAtIndexInPriority(4, false);
 		$this->assertEquals(array(
-			3,4
+			3, 4
 		), $plist->toArray());
 		$plist->insertAtIndexInPriority(5, 2);
 		$this->assertEquals(array(
-			3,4,5
+			3, 4, 5
 		), $plist->toArray());
 		
 		$plist->insertAtIndexInPriority(6, false, null);
 		$this->assertEquals(array(
-			3,4,5,6
+			3, 4, 5, 6
 		), $plist->toArray());
 		$plist->insertAtIndexInPriority(7, 5, null);
 		$this->assertEquals(array(
-			3,4,5,6,7
+			3, 4, 5, 6, 7
 		), $plist->toArray());
 		
 		$plist->insertAtIndexInPriority(8, false, 10);
 		$this->assertEquals(array(
-			3,4,5,6,7,8
+			3, 4, 5, 6, 7, 8
 		), $plist->toArray());
 		$plist->insertAtIndexInPriority(9, 7, 10);
 		$this->assertEquals(array(
-			3,4,5,6,7,8,9
+			3, 4, 5, 6, 7, 8, 9
 		), $plist->toArray());
 		
 		$plist->insertAtIndexInPriority(10, false, 100);
 		$this->assertEquals(array(
-			3,4,5,6,7,8,9,10
+			3, 4, 5, 6, 7, 8, 9, 10
 		), $plist->toArray());
 		$plist->insertAtIndexInPriority(11, 1, 100);
 		$this->assertEquals(array(
-			3,4,5,6,7,8,9,10,11
+			3, 4, 5, 6, 7, 8, 9, 10, 11
 		), $plist->toArray());
 		
 		$plist = new TPriorityList();
@@ -689,38 +689,38 @@ 
 		
 		$plist->insertAtIndexInPriority(4, false, null, true);
 		$this->assertEquals(array(
-			3,4
+			3, 4
 		), $plist->toArray());
 		$plist->insertAtIndexInPriority(5, 2, null, true);
 		$this->assertEquals(array(
-			3,4,5
+			3, 4, 5
 		), $plist->toArray());
 		
 		$plist->insertAtIndexInPriority(6, false, null, true);
 		$this->assertEquals(array(
-			3,4,5,6
+			3, 4, 5, 6
 		), $plist->toArray());
 		$plist->insertAtIndexInPriority(7, 5, null, true);
 		$this->assertEquals(array(
-			3,4,5,6,7
+			3, 4, 5, 6, 7
 		), $plist->toArray());
 		
 		$plist->insertAtIndexInPriority(8, false, 10, true);
 		$this->assertEquals(array(
-			3,4,5,6,7,8
+			3, 4, 5, 6, 7, 8
 		), $plist->toArray());
 		$plist->insertAtIndexInPriority(9, 7, 10, true);
 		$this->assertEquals(array(
-			3,4,5,6,7,8,9
+			3, 4, 5, 6, 7, 8, 9
 		), $plist->toArray());
 		
 		$plist->insertAtIndexInPriority(10, false, 100, true);
 		$this->assertEquals(array(
-			3,4,5,6,7,8,9,10
+			3, 4, 5, 6, 7, 8, 9, 10
 		), $plist->toArray());
 		$plist->insertAtIndexInPriority(11, 1, 100, true);
 		$this->assertEquals(array(
-			3,4,5,6,7,8,9,10,11
+			3, 4, 5, 6, 7, 8, 9, 10, 11
 		), $plist->toArray());
 		
 		$plist = new TPriorityList();
@@ -732,38 +732,38 @@ 
 		
 		$plist->insertAtIndexInPriority(4, false, null, false);
 		$this->assertEquals(array(
-			3,4
+			3, 4
 		), $plist->toArray());
 		$plist->insertAtIndexInPriority(5, 2, null, false);
 		$this->assertEquals(array(
-			3,4,5
+			3, 4, 5
 		), $plist->toArray());
 		
 		$plist->insertAtIndexInPriority(6, false, null, false);
 		$this->assertEquals(array(
-			3,4,5,6
+			3, 4, 5, 6
 		), $plist->toArray());
 		$plist->insertAtIndexInPriority(7, 5, null, false);
 		$this->assertEquals(array(
-			3,4,5,6,7
+			3, 4, 5, 6, 7
 		), $plist->toArray());
 		
 		$plist->insertAtIndexInPriority(8, false, 10, false);
 		$this->assertEquals(array(
-			3,4,5,6,7,8
+			3, 4, 5, 6, 7, 8
 		), $plist->toArray());
 		$plist->insertAtIndexInPriority(9, 7, 10, false);
 		$this->assertEquals(array(
-			3,4,5,6,7,8,9
+			3, 4, 5, 6, 7, 8, 9
 		), $plist->toArray());
 		
 		$plist->insertAtIndexInPriority(10, false, 100, false);
 		$this->assertEquals(array(
-			3,4,5,6,7,8,9,10
+			3, 4, 5, 6, 7, 8, 9, 10
 		), $plist->toArray());
 		$plist->insertAtIndexInPriority(11, 1, 100, false);
 		$this->assertEquals(array(
-			3,4,5,6,7,8,9,10,11
+			3, 4, 5, 6, 7, 8, 9, 10, 11
 		), $plist->toArray());
 	}
 	
@@ -921,7 +921,7 @@ 
 	public function testMergeWithTPriorityList()
 	{
 		$plist = new TPriorityList(array(
-			$this->item3,$this->item1
+			$this->item3, $this->item1
 		));
 		$plist->mergeWith($this->plist);
 		$this->assertEquals(6, $plist->getCount());
@@ -1087,7 +1087,7 @@ 
 		$list->add(3, 15);
 		$list->offsetSet(null, 4); // Appending like this, items get the default priority; not linear behavior
 		self::assertEquals(array(
-			1,2,4,3
+			1, 2, 4, 3
 		), $list->toArray());
 	}
 	
@@ -1099,7 +1099,7 @@ 
 		$list->add(3, 15);
 		$list->offsetSet(1, 4);
 		self::assertEquals(array(
-			1,4,3
+			1, 4, 3
 		), $list->toArray());
 	}
 	
@@ -1111,7 +1111,7 @@ 
 		$list->add(3, 15);
 		$list->offsetSet(3, 4);
 		self::assertEquals(array(
-			1,2,3,4
+			1, 2, 3, 4
 		), $list->toArray());
 	}
 	
@@ -1123,7 +1123,7 @@ 
 		$list->add(3, 15);
 		$list->offsetUnset(1);
 		self::assertEquals(array(
-			1,3
+			1, 3
 		), $list->toArray());
 	}
 	

tests/unit/Collections/TPagedListTest.php

Spacing +6 added lines, -6 removed lines

@@ -63,7 +63,7 @@ 
 		$list = new TPagedList();
 		try {
 			$list->PageSize = 0;
-		} catch(TInvalidDataValueException $e) {
+		} catch (TInvalidDataValueException $e) {
 			return;
 		}
 		self::fail('An expected TInvalidDataValueException was not raised');
@@ -96,7 +96,7 @@ 
 		$myList = new MyPagedList();
 		$list->attachEventHandler('OnFetchData', array($myList, 'fetchData'));
 		self::assertEquals(false, $myList->hasFetchedData());
-		$list->onFetchData(new TPagedListFetchDataEventParameter($list->CurrentPageIndex, $list->PageSize*$list->CurrentPageIndex, $list->PageSize));
+		$list->onFetchData(new TPagedListFetchDataEventParameter($list->CurrentPageIndex, $list->PageSize * $list->CurrentPageIndex, $list->PageSize));
 		self::assertEquals(true, $myList->hasFetchedData());
 	}
 
@@ -174,13 +174,13 @@ 
 		self::assertInstanceOf('ArrayIterator', $list->getIterator());
 		$n = 0;
 		$found = 0;
-		foreach($list as $index => $item) {
-			foreach($list as $a => $b); // test of iterator
+		foreach ($list as $index => $item) {
+			foreach ($list as $a => $b); // test of iterator
 			$n++;
-			if($index === 0 && $item === 1) {
+			if ($index === 0 && $item === 1) {
 				$found++;
 			}
-			if($index === 1 && $item === 2) {
+			if ($index === 1 && $item === 2) {
 				$found++;
 			}
 		}