adamjakab / SuiteCRM

tests/tests/include/utils/sugarFileUtilsTest.php

Spacing +28 added lines, -28 removed lines

@@ -17,13 +17,13 @@ 
 		$dir = "vfs://root";
 		
 		//non recursive
-		$result = sugar_mkdir($dir . "/mkdirTest");
-		$this->assertFileExists($dir . "/mkdirTest");
+		$result = sugar_mkdir($dir."/mkdirTest");
+		$this->assertFileExists($dir."/mkdirTest");
 		$this->assertTrue($result);
 		
 		//recursive
-		$result = sugar_mkdir($dir . "/mkdirTest/test",null,true);
-		$this->assertFileExists($dir . "/mkdirTest/test");
+		$result = sugar_mkdir($dir."/mkdirTest/test", null, true);
+		$this->assertFileExists($dir."/mkdirTest/test");
 		$this->assertTrue($result);
 	}
 	
@@ -40,9 +40,9 @@ 
 		//execute the method and test if it doesn't returns false and returns the number of bytes written
 
         $dir = "vfs://root";
-		$result = sugar_file_put_contents( $dir . '/testfile.txt', 'some test data');
+		$result = sugar_file_put_contents($dir.'/testfile.txt', 'some test data');
 		$this->assertNotFalse($result);
-		$this->assertEquals(14,$result);
+		$this->assertEquals(14, $result);
 	}
 
 	
@@ -52,7 +52,7 @@ 
         $this->markTestSkipped('Atomic file put cannot be tested with vfsStream');
 		//execute the method and test if it returns success(true)
         $dir = "vfs://root";
-		$result = sugar_file_put_contents_atomic( $dir . '/atomictestfile.txt', 'some test data');
+		$result = sugar_file_put_contents_atomic($dir.'/atomictestfile.txt', 'some test data');
 		$this->assertTrue($result);
 	}
 	
@@ -61,10 +61,10 @@ 
 	{
 		//execute the method and test if it doesn't returns false and returns the expected contents
         $dir = "vfs://root";
-		$result = file_get_contents( $dir . '/test.txt');
+		$result = file_get_contents($dir.'/test.txt');
 		
 		$this->assertNotFalse($result);
-		$this->assertEquals('Hello world!',$result);
+		$this->assertEquals('Hello world!', $result);
 	}
 	
 
@@ -73,19 +73,19 @@ 
 		//execute the method and test if it returns success(true)
 
         $dir = "vfs://root";
-		$test_dt = time() - 3600 ; 
-		$expected = date("m d Y H:i:s",  time() - 3600 );
+		$test_dt = time() - 3600; 
+		$expected = date("m d Y H:i:s", time() - 3600);
 		
 		//test wihout modified date param
-		$result = sugar_touch( $dir . '/testfiletouch.txt');
+		$result = sugar_touch($dir.'/testfiletouch.txt');
 		$this->assertTrue($result);		
 		
 		//test wih modified date param
-		$result = sugar_touch( $dir . '/testfiletouch.txt',$test_dt,$test_dt);
-		$file_dt = date ("m d Y H:i:s", filemtime($dir . '/testfiletouch.txt')) ;
+		$result = sugar_touch($dir.'/testfiletouch.txt', $test_dt, $test_dt);
+		$file_dt = date("m d Y H:i:s", filemtime($dir.'/testfiletouch.txt'));
 		
 		$this->assertTrue($result);
-		$this->assertSame($file_dt, $expected );
+		$this->assertSame($file_dt, $expected);
 	}
 	
 
@@ -94,7 +94,7 @@ 
         $this->markTestSkipped('Permissions cannot be tested with vfsStream');
 		//execute the method and test if it returns success(true)
         $dir = "vfs://test";
-		$result = sugar_chmod($dir . '/test.txt',0777);
+		$result = sugar_chmod($dir.'/test.txt', 0777);
 		$this->assertTrue($result);
 	}
 	
@@ -104,10 +104,10 @@ 
         $this->markTestSkipped('Permissions cannot be tested with vfsStream');
 		//execute the method and test if it returns success(true)
         $dir = "vfs://test";
-		$result = sugar_chown($dir . '/test.txt');
+		$result = sugar_chown($dir.'/test.txt');
 		$this->assertFalse($result);
 
-        $result = sugar_chown($dir . '/test.txt',org\bovigo\vfs\vfsStream::getCurrentUser());
+        $result = sugar_chown($dir.'/test.txt', org\bovigo\vfs\vfsStream::getCurrentUser());
         $this->assertTrue($result);
 
 	}
@@ -118,10 +118,10 @@ 
         $this->markTestSkipped('Permissions cannot be tested with vfsStream');
 		//execute the method and test if it returns success(true)
         $dir = "vfs://test";
-		$result = sugar_chgrp($dir . '/test.txt');
+		$result = sugar_chgrp($dir.'/test.txt');
 		$this->assertFalse($result);
 
-        $result = sugar_chgrp($dir . '/test.txt',org\bovigo\vfs\vfsStream::getCurrentGroup());
+        $result = sugar_chgrp($dir.'/test.txt', org\bovigo\vfs\vfsStream::getCurrentGroup());
         $this->assertFalse($result);
 	}
 	
@@ -129,11 +129,11 @@ 
 	public function testget_mode()
 	{
 		//test with all mods defined in config
-		$this->assertSame(1528,get_mode());
-		$this->assertSame(1528,get_mode('dir_mode', 10));
-		$this->assertSame(493,get_mode('file_mode', 10));
-		$this->assertSame('',get_mode('user', 10));
-		$this->assertSame('',get_mode('group', 10));
+		$this->assertSame(1528, get_mode());
+		$this->assertSame(1528, get_mode('dir_mode', 10));
+		$this->assertSame(493, get_mode('file_mode', 10));
+		$this->assertSame('', get_mode('user', 10));
+		$this->assertSame('', get_mode('group', 10));
 	}
 
 	public function testsugar_is_dir()
@@ -158,9 +158,9 @@ 
 	{
 		$cache_dir = rtrim($GLOBALS['sugar_config']['cache_dir'], '/\\');
 		
-		$this->assertSame($cache_dir . '/', sugar_cached('')); //invalid file
-		$this->assertSame($cache_dir . '/config', sugar_cached('config')); //valid file		
-		$this->assertSame($cache_dir . '/modules' , sugar_cached('modules')); //valid file
+		$this->assertSame($cache_dir.'/', sugar_cached('')); //invalid file
+		$this->assertSame($cache_dir.'/config', sugar_cached('config')); //valid file		
+		$this->assertSame($cache_dir.'/modules', sugar_cached('modules')); //valid file
 		
 	}
 

tests/tests/include/utils/logicUtilsTest.php

Spacing +20 added lines, -20 removed lines

@@ -11,37 +11,37 @@ 
 	public function testget_hook_array()
 	{
 		//test with a vaid module. it will return an array
-		$AccountsHooks =  get_hook_array('Accounts');
+		$AccountsHooks = get_hook_array('Accounts');
 		$this->assertTrue(is_array($AccountsHooks));
 		
 		//test with an invalid array. it will throw an file include exception.
 		$BugsHooks = "";
-		try{
-			$BugsHooks =  get_hook_array('Bugs');
-		}catch(Exception $e){}
+		try {
+			$BugsHooks = get_hook_array('Bugs');
+		}catch (Exception $e) {}
 		
 		$this->assertFalse(is_array($BugsHooks));
 		
 	}
 	
 	
-	private function getTestHook(){
+	private function getTestHook() {
 		
 		//array containing hooks array to test
 		
         $hook_array = Array();
         $hook_array['after_ui_footer'] = Array();
-        $hook_array['after_ui_footer'][] = Array(10, 'popup_onload', 'modules/SecurityGroups/AssignGroups.php','AssignGroups', 'popup_onload');
+        $hook_array['after_ui_footer'][] = Array(10, 'popup_onload', 'modules/SecurityGroups/AssignGroups.php', 'AssignGroups', 'popup_onload');
         $hook_array['after_ui_frame'] = Array();
-        $hook_array['after_ui_frame'][] = Array(20, 'mass_assign', 'modules/SecurityGroups/AssignGroups.php','AssignGroups', 'mass_assign');
-        $hook_array['after_ui_frame'][] = Array(1, 'Load Social JS', 'custom/include/social/hooks.php','hooks', 'load_js');
+        $hook_array['after_ui_frame'][] = Array(20, 'mass_assign', 'modules/SecurityGroups/AssignGroups.php', 'AssignGroups', 'mass_assign');
+        $hook_array['after_ui_frame'][] = Array(1, 'Load Social JS', 'custom/include/social/hooks.php', 'hooks', 'load_js');
         $hook_array['after_save'] = Array();
-        $hook_array['after_save'][] = Array(30, 'popup_select', 'modules/SecurityGroups/AssignGroups.php','AssignGroups', 'popup_select');
-        $hook_array['after_save'][] = Array(1, 'AOD Index Changes', 'modules/AOD_Index/AOD_LogicHooks.php','AOD_LogicHooks', 'saveModuleChanges');
+        $hook_array['after_save'][] = Array(30, 'popup_select', 'modules/SecurityGroups/AssignGroups.php', 'AssignGroups', 'popup_select');
+        $hook_array['after_save'][] = Array(1, 'AOD Index Changes', 'modules/AOD_Index/AOD_LogicHooks.php', 'AOD_LogicHooks', 'saveModuleChanges');
         $hook_array['after_delete'] = array();
-        $hook_array['after_delete'][] = Array(1, 'AOD Index changes', 'modules/AOD_Index/AOD_LogicHooks.php','AOD_LogicHooks', 'saveModuleDelete');
+        $hook_array['after_delete'][] = Array(1, 'AOD Index changes', 'modules/AOD_Index/AOD_LogicHooks.php', 'AOD_LogicHooks', 'saveModuleDelete');
         $hook_array['after_restore'] = array();
-        $hook_array['after_restore'][] = Array(1, 'AOD Index changes', 'modules/AOD_Index/AOD_LogicHooks.php','AOD_LogicHooks', 'saveModuleRestore');
+        $hook_array['after_restore'][] = Array(1, 'AOD Index changes', 'modules/AOD_Index/AOD_LogicHooks.php', 'AOD_LogicHooks', 'saveModuleRestore');
         
 		return $hook_array;
 	}
@@ -52,9 +52,9 @@ 
 		
 		$hook_array = $this->getTestHook();
 		return array(
-				array($hook_array,'after_save',array(0,'popup_select'), true),
-				array($hook_array,'after_save',array(0,'foo'), false),
-				array($hook_array,'foo',array(0,'popup_select'), false),
+				array($hook_array, 'after_save', array(0, 'popup_select'), true),
+				array($hook_array, 'after_save', array(0, 'foo'), false),
+				array($hook_array, 'foo', array(0, 'popup_select'), false),
 		);	
 	}
 	
@@ -64,7 +64,7 @@ 
 	public function testcheck_existing_element($hook_array, $event, $action_array, $expected)
 	{
 		//test with dataset containing valid and invalid cases
-		$this->assertSame( check_existing_element($hook_array,$event, $action_array) , $expected);
+		$this->assertSame(check_existing_element($hook_array, $event, $action_array), $expected);
 		
 	}
 	
@@ -74,7 +74,7 @@ 
 		
 		$hook_array = $this->getTestHook();
 		$expected = "<?php\n// Do not store anything in this file that is not part of the array or the hook version.  This file will	\n// be automatically rebuilt in the future. \n \$hook_version = 1; \n\$hook_array = Array(); \n// position, file, function \n\$hook_array['after_ui_footer'] = Array(); \n\$hook_array['after_ui_footer'][] = Array(10, 'popup_onload', 'modules/SecurityGroups/AssignGroups.php','AssignGroups', 'popup_onload'); \n\$hook_array['after_ui_frame'] = Array(); \n\$hook_array['after_ui_frame'][] = Array(20, 'mass_assign', 'modules/SecurityGroups/AssignGroups.php','AssignGroups', 'mass_assign'); \n\$hook_array['after_ui_frame'][] = Array(1, 'Load Social JS', 'custom/include/social/hooks.php','hooks', 'load_js'); \n\$hook_array['after_save'] = Array(); \n\$hook_array['after_save'][] = Array(30, 'popup_select', 'modules/SecurityGroups/AssignGroups.php','AssignGroups', 'popup_select'); \n\$hook_array['after_save'][] = Array(1, 'AOD Index Changes', 'modules/AOD_Index/AOD_LogicHooks.php','AOD_LogicHooks', 'saveModuleChanges'); \n\$hook_array['after_delete'] = Array(); \n\$hook_array['after_delete'][] = Array(1, 'AOD Index changes', 'modules/AOD_Index/AOD_LogicHooks.php','AOD_LogicHooks', 'saveModuleDelete'); \n\$hook_array['after_restore'] = Array(); \n\$hook_array['after_restore'][] = Array(1, 'AOD Index changes', 'modules/AOD_Index/AOD_LogicHooks.php','AOD_LogicHooks', 'saveModuleRestore'); \n\n\n\n?>";
-		$this->assertEquals($expected,replace_or_add_logic_type($hook_array));
+		$this->assertEquals($expected, replace_or_add_logic_type($hook_array));
 		
 	}
 	
@@ -85,13 +85,13 @@ 
 	
 		$vfs = vfsStream::setup('custom/modules/TEST_Test');
 	
-		if ( $vfs->hasChild('logic_hooks.php')  == true) {
+		if ($vfs->hasChild('logic_hooks.php') == true) {
 			unlink('custom/modules/TEST_Test/logic_hooks.php');
 			rmdir('custom/modules/TEST_Test');
 		}
 		
 		$expectedContents = "<?php\n// Do not store anything in this file that is not part of the array or the hook version.  This file will	\n// be automatically rebuilt in the future. \n \$hook_version = 1; \n\$hook_array = Array(); \n// position, file, function \n\$hook_array['after_ui_footer'] = Array(); \n\$hook_array['after_ui_footer'][] = Array(10, 'popup_onload', 'modules/SecurityGroups/AssignGroups.php','AssignGroups', 'popup_onload'); \n\$hook_array['after_ui_frame'] = Array(); \n\$hook_array['after_ui_frame'][] = Array(20, 'mass_assign', 'modules/SecurityGroups/AssignGroups.php','AssignGroups', 'mass_assign'); \n\$hook_array['after_ui_frame'][] = Array(1, 'Load Social JS', 'custom/include/social/hooks.php','hooks', 'load_js'); \n\$hook_array['after_save'] = Array(); \n\$hook_array['after_save'][] = Array(30, 'popup_select', 'modules/SecurityGroups/AssignGroups.php','AssignGroups', 'popup_select'); \n\$hook_array['after_save'][] = Array(1, 'AOD Index Changes', 'modules/AOD_Index/AOD_LogicHooks.php','AOD_LogicHooks', 'saveModuleChanges'); \n\$hook_array['after_delete'] = Array(); \n\$hook_array['after_delete'][] = Array(1, 'AOD Index changes', 'modules/AOD_Index/AOD_LogicHooks.php','AOD_LogicHooks', 'saveModuleDelete'); \n\$hook_array['after_restore'] = Array(); \n\$hook_array['after_restore'][] = Array(1, 'AOD Index changes', 'modules/AOD_Index/AOD_LogicHooks.php','AOD_LogicHooks', 'saveModuleRestore'); \n\n\n\n?>";
-		write_logic_file('TEST_Test',$expectedContents);
+		write_logic_file('TEST_Test', $expectedContents);
 		
 		//Check file created
 		$this->assertFileExists('custom/modules/TEST_Test/logic_hooks.php');
@@ -115,7 +115,7 @@ 
 		
 		$hook_array = $this->getTestHook();
 		$expected = "// Do not store anything in this file that is not part of the array or the hook version.  This file will	\n// be automatically rebuilt in the future. \n \$hook_version = 1; \n\$hook_array = Array(); \n// position, file, function \n\$hook_array['after_ui_footer'] = Array(); \n\$hook_array['after_ui_footer'][] = Array(10, 'popup_onload', 'modules/SecurityGroups/AssignGroups.php','AssignGroups', 'popup_onload'); \n\$hook_array['after_ui_frame'] = Array(); \n\$hook_array['after_ui_frame'][] = Array(20, 'mass_assign', 'modules/SecurityGroups/AssignGroups.php','AssignGroups', 'mass_assign'); \n\$hook_array['after_ui_frame'][] = Array(1, 'Load Social JS', 'custom/include/social/hooks.php','hooks', 'load_js'); \n\$hook_array['after_save'] = Array(); \n\$hook_array['after_save'][] = Array(30, 'popup_select', 'modules/SecurityGroups/AssignGroups.php','AssignGroups', 'popup_select'); \n\$hook_array['after_save'][] = Array(1, 'AOD Index Changes', 'modules/AOD_Index/AOD_LogicHooks.php','AOD_LogicHooks', 'saveModuleChanges'); \n\$hook_array['after_delete'] = Array(); \n\$hook_array['after_delete'][] = Array(1, 'AOD Index changes', 'modules/AOD_Index/AOD_LogicHooks.php','AOD_LogicHooks', 'saveModuleDelete'); \n\$hook_array['after_restore'] = Array(); \n\$hook_array['after_restore'][] = Array(1, 'AOD Index changes', 'modules/AOD_Index/AOD_LogicHooks.php','AOD_LogicHooks', 'saveModuleRestore'); \n\n\n";
-		$this->assertEquals($expected,build_logic_file($hook_array));	
+		$this->assertEquals($expected, build_logic_file($hook_array));	
 	}
 	
 }

tests/tests/include/utils/dbUtilsTest.php

Spacing +36 added lines, -36 removed lines

@@ -10,22 +10,22 @@ 
 	{
 		//array containing all possible types supported by db_convert
 		return array(
-				array(gmdate('Y-m-d H:i:s'),'today',array(),'CURDATE()'),
-				array('text','left',array(2),'LEFT(text,2)'),
-				array('2015-11-16 19:10:52','date_format',array(),'DATE_FORMAT(2015-11-16 19:10:52,\'%Y-%m-%d\')'),
-				array('2015-11-16 19:10:52','time_format',array(), '2015-11-16 19:10:52' ),
-				array('2015-11-16','date',array(), '2015-11-16' ),
-				array('19:10:52','time',array(), '19:10:52' ),
-				array('2015-11-16 19:10:52','datetime',array(), '2015-11-16 19:10:52'),
-				array(Null,'ifnull',array(0), 'IFNULL(0)' ),
-				array('value1 ','concat',array('value2'), 'CONCAT(value1 ,value2)' ), 
-				array('2015-11-16 19:10:52','quarter',array(), 'QUARTER(2015-11-16 19:10:52)' ),
-				array('value1','length',array(), 'LENGTH(value1)' ),
-				array('2015-11-16 19:32:29', 'month',array(), 'MONTH(2015-11-16 19:32:29)' ),
-				array('2015-11-16','add_date',array('1','DAY'), 'DATE_ADD(2015-11-16, INTERVAL 1 DAY)' ),
-				array('19:10:52','add_time',array('1', 'HOUR'), 'DATE_ADD(19:10:52, INTERVAL + CONCAT(1, \':\', HOUR) HOUR_MINUTE)' ),
-				array('col','avg',array(2), 'avg(col)' ),
-				array('2015-11-16 19:32:29','add_tz_offset',array(), '2015-11-16 19:32:29 + INTERVAL 0 MINUTE' ),
+				array(gmdate('Y-m-d H:i:s'), 'today', array(), 'CURDATE()'),
+				array('text', 'left', array(2), 'LEFT(text,2)'),
+				array('2015-11-16 19:10:52', 'date_format', array(), 'DATE_FORMAT(2015-11-16 19:10:52,\'%Y-%m-%d\')'),
+				array('2015-11-16 19:10:52', 'time_format', array(), '2015-11-16 19:10:52'),
+				array('2015-11-16', 'date', array(), '2015-11-16'),
+				array('19:10:52', 'time', array(), '19:10:52'),
+				array('2015-11-16 19:10:52', 'datetime', array(), '2015-11-16 19:10:52'),
+				array(Null, 'ifnull', array(0), 'IFNULL(0)'),
+				array('value1 ', 'concat', array('value2'), 'CONCAT(value1 ,value2)'), 
+				array('2015-11-16 19:10:52', 'quarter', array(), 'QUARTER(2015-11-16 19:10:52)'),
+				array('value1', 'length', array(), 'LENGTH(value1)'),
+				array('2015-11-16 19:32:29', 'month', array(), 'MONTH(2015-11-16 19:32:29)'),
+				array('2015-11-16', 'add_date', array('1', 'DAY'), 'DATE_ADD(2015-11-16, INTERVAL 1 DAY)'),
+				array('19:10:52', 'add_time', array('1', 'HOUR'), 'DATE_ADD(19:10:52, INTERVAL + CONCAT(1, \':\', HOUR) HOUR_MINUTE)'),
+				array('col', 'avg', array(2), 'avg(col)'),
+				array('2015-11-16 19:32:29', 'add_tz_offset', array(), '2015-11-16 19:32:29 + INTERVAL 0 MINUTE'),
 				
 		);
 	}
@@ -36,8 +36,8 @@ 
 	public function testdb_convert($string, $type, $params, $expected)
 	{
 		//execute the method and test if it returns expected values for all types
-		$actual = db_convert($string, $type,$params);
-		$this->assertSame($expected,$actual);
+		$actual = db_convert($string, $type, $params);
+		$this->assertSame($expected, $actual);
 	}
 	
 	
@@ -48,10 +48,10 @@ 
 		//execute the method and test if it returns expected values
 		
 		$table = "Table1"; 
-		$fields = Array('Col1','Col2','Col3');
+		$fields = Array('Col1', 'Col2', 'Col3');
 		$expected = "LTRIM(RTRIM(CONCAT(IFNULL(Table1.Col1,''),'',IFNULL(Table1.Col2,''),'',IFNULL(Table1.Col3,''))))";
 		$actual = db_concat($table, $fields);
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 	}
 	
 	
@@ -59,11 +59,11 @@ 
 	{
 		//execute the method and test if it returns expected values
 		
-		$this->assertSame("2015-11-16 19:32:29", from_db_convert("2015-11-16 19:32:29","date")); 
-		$this->assertSame("19:32:29",from_db_convert("19:32:29","time"));
-		$this->assertSame("2015-11-16 19:32:29",from_db_convert("2015-11-16 19:32:29","datetime"));
-		$this->assertSame("2015-11-16 19:32:29",from_db_convert("2015-11-16 19:32:29","datetimecombo"));
-		$this->assertSame("2015-11-16 19:32:29",from_db_convert("2015-11-16 19:32:29","timestamp"));
+		$this->assertSame("2015-11-16 19:32:29", from_db_convert("2015-11-16 19:32:29", "date")); 
+		$this->assertSame("19:32:29", from_db_convert("19:32:29", "time"));
+		$this->assertSame("2015-11-16 19:32:29", from_db_convert("2015-11-16 19:32:29", "datetime"));
+		$this->assertSame("2015-11-16 19:32:29", from_db_convert("2015-11-16 19:32:29", "datetimecombo"));
+		$this->assertSame("2015-11-16 19:32:29", from_db_convert("2015-11-16 19:32:29", "timestamp"));
 		
 	}
 	
@@ -75,12 +75,12 @@ 
 		$string = "";
 		$expected = "";
 		$actual = to_html($string);
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 		
 		$string = "'test'&trial<\">";
 		$expected = "&#039;test&#039;&trial&lt;&quot;&gt;";
 		$actual = to_html($string);
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 		
 	}
 	
@@ -92,26 +92,26 @@ 
 		$string = "";
 		$expected = "";
 		$actual = from_html($string);
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 		
 		$string = "&#039;test&#039;&trial&lt;&quot;&gt;";
 		$expected = "'test'&trial<\">"; 
 		$actual = from_html($string);
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 		
 	}
 	
 	
 	public function testgetValidDBName()
 	{
-		$expected ="";
+		$expected = "";
 		$actual = getValidDBName("");
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 		
 		
 		$expected = "col";
 		$actual = getValidDBName("Col");
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 		
 	}
 	
@@ -120,13 +120,13 @@ 
 	{
 		//valid value
 		$expected = true;
-		$actual = isValidDBName("suitecrmtest","mysql");
-		$this->assertSame($expected,$actual);
+		$actual = isValidDBName("suitecrmtest", "mysql");
+		$this->assertSame($expected, $actual);
 
 		//invalid value
 		$expected = false;
-		$actual = isValidDBName("suite/crm.test","mysql");
-		$this->assertSame($expected,$actual);
+		$actual = isValidDBName("suite/crm.test", "mysql");
+		$this->assertSame($expected, $actual);
 		
 		
 	}

tests/tests/include/utils/layoutUtilsTest.php

Spacing +33 added lines, -33 removed lines

@@ -14,19 +14,19 @@ 
 		
 		//help param true
 		$html1 = get_form_header("test Header", "test subheader", true);
-		$this->assertGreaterThan(0,strlen($html1));
-		$this->assertNotFalse(strpos($html1,"test Header"));
-		$this->assertNotFalse(strpos($html1,"test subheader"));
+		$this->assertGreaterThan(0, strlen($html1));
+		$this->assertNotFalse(strpos($html1, "test Header"));
+		$this->assertNotFalse(strpos($html1, "test subheader"));
 		
 		
 		// help param false
 		$html2 = get_form_header("new test Header", "new test subheader", false);
-		$this->assertGreaterThan(0,strlen($html2));
-		$this->assertNotFalse(strpos($html2,"new test Header"));
-		$this->assertNotFalse(strpos($html2,"new test subheader"));
+		$this->assertGreaterThan(0, strlen($html2));
+		$this->assertNotFalse(strpos($html2, "new test Header"));
+		$this->assertNotFalse(strpos($html2, "new test subheader"));
 		
 		
-		$this->assertGreaterThan(strlen($html2),strlen($html1));
+		$this->assertGreaterThan(strlen($html2), strlen($html1));
 		
 	}
 	
@@ -36,25 +36,25 @@ 
 		//execute the method and test if it returns html and contains the values provided in parameters
 		
 		//with show_create true, generates more html
-		$html1 = get_module_title("Users", "Users Home", true );
-		$this->assertGreaterThan(0,strlen($html1));
-		$this->assertNotFalse(strpos($html1,"Users"));
-		$this->assertNotFalse(strpos($html1,"Users Home"));
+		$html1 = get_module_title("Users", "Users Home", true);
+		$this->assertGreaterThan(0, strlen($html1));
+		$this->assertNotFalse(strpos($html1, "Users"));
+		$this->assertNotFalse(strpos($html1, "Users Home"));
 		
 		//with show_create false, generates less html
-		$html2 = get_module_title("Users", "Users Home", false );
-		$this->assertGreaterThan(0,strlen($html2));
-		$this->assertNotFalse(strpos($html2,"Users"));
-		$this->assertNotFalse(strpos($html2,"Users Home"));
-		$this->assertGreaterThan(strlen($html2),strlen($html1));
+		$html2 = get_module_title("Users", "Users Home", false);
+		$this->assertGreaterThan(0, strlen($html2));
+		$this->assertNotFalse(strpos($html2, "Users"));
+		$this->assertNotFalse(strpos($html2, "Users Home"));
+		$this->assertGreaterThan(strlen($html2), strlen($html1));
 
 		//with show_create flase and count > 1, generates more html compared to count =0
-		$html3 = get_module_title("Users", "Users Home", false, 2 );
-		$this->assertGreaterThan(0,strlen($html3));
-		$this->assertNotFalse(strpos($html3,"Users"));
-		$this->assertNotFalse(strpos($html3,"Users Home"));
-		$this->assertGreaterThan(strlen($html2),strlen($html3));
-		$this->assertGreaterThan(strlen($html3),strlen($html1));
+		$html3 = get_module_title("Users", "Users Home", false, 2);
+		$this->assertGreaterThan(0, strlen($html3));
+		$this->assertNotFalse(strpos($html3, "Users"));
+		$this->assertNotFalse(strpos($html3, "Users Home"));
+		$this->assertGreaterThan(strlen($html2), strlen($html3));
+		$this->assertGreaterThan(strlen($html3), strlen($html1));
 		
 	}
 	
@@ -65,15 +65,15 @@ 
 		
 		//with show_create false, generates less html
 		$html1 = getClassicModuleTitle("users", Array('Users Home'));
-		$this->assertGreaterThan(0,strlen($html1));
-		$this->assertNotFalse(strpos($html1,"Users Home"));
+		$this->assertGreaterThan(0, strlen($html1));
+		$this->assertNotFalse(strpos($html1, "Users Home"));
 		
 
 		//with show_create true, generates more html
-		$html2 = getClassicModuleTitle("users", Array('Users Home'),true);
-		$this->assertGreaterThan(0,strlen($html2));
-		$this->assertNotFalse(strpos($html2,"Users Home"));
-		$this->assertGreaterThan(strlen($html1),strlen($html2));
+		$html2 = getClassicModuleTitle("users", Array('Users Home'), true);
+		$this->assertGreaterThan(0, strlen($html2));
+		$this->assertNotFalse(strpos($html2, "Users Home"));
+		$this->assertGreaterThan(strlen($html1), strlen($html2));
 		
 	}
 	
@@ -87,17 +87,17 @@ 
 		insert_popup_header();
 		$renderedContent1 = ob_get_contents();
 		ob_end_clean();
-		$this->assertGreaterThan(0,strlen($renderedContent1));
+		$this->assertGreaterThan(0, strlen($renderedContent1));
 		
 		
 		//with includeJS false, generates less html
 		ob_start();
-		insert_popup_header('',false);
+		insert_popup_header('', false);
 		$renderedContent2 = ob_get_contents();
 		ob_end_clean();
-		$this->assertGreaterThan(0,strlen($renderedContent2));		
+		$this->assertGreaterThan(0, strlen($renderedContent2));		
 		
-		$this->assertGreaterThan(strlen($renderedContent2),strlen($renderedContent1));
+		$this->assertGreaterThan(strlen($renderedContent2), strlen($renderedContent1));
 		
 	}
 	
@@ -110,7 +110,7 @@ 
 		insert_popup_footer();
 		$renderedContent = ob_get_contents();
 		ob_end_clean();
-		$this->assertGreaterThan(0,strlen($renderedContent));
+		$this->assertGreaterThan(0, strlen($renderedContent));
 		
 	}
 

tests/tests/include/utils/autoloaderTest.php

Spacing +3 added lines, -3 removed lines

@@ -2,7 +2,7 @@ 
 
 
 
-class SugarAutoLoaderTest extends PHPUnit_Framework_TestCase{
+class SugarAutoLoaderTest extends PHPUnit_Framework_TestCase {
 
 
     public function testautoload()
@@ -24,14 +24,14 @@ 
 	}
 
 
-	public function testloadAll(){
+	public function testloadAll() {
 		
 		//execute the method and check if it works and doesn't throws an exception
 		//this method only includes file so there is no output to test.
 		try {
 			SugarAutoLoader::loadAll();
 			$this->assertTrue(TRUE);
-		} catch (Exception $e) {
+		}catch (Exception $e) {
 			$this->fail();
 		}
 		 

tests/tests/include/utils/zipUtilsTest.php

Spacing +3 added lines, -3 removed lines

@@ -5,15 +5,15 @@
 {
 
 	
-	public function testunzip(){
+	public function testunzip() {
 		$this->markTestIncomplete('Can Not be implemented');
 	}
 	
-	public function testunzip_file(){
+	public function testunzip_file() {
 		$this->markTestIncomplete('Can Not be implemented');
 	}
 	
-	public function testzip_dir(){
+	public function testzip_dir() {
 		$this->markTestIncomplete('Can Not be implemented');
 	}
 	

tests/tests/include/utils/activityUtilsTest.php

Spacing +2 added lines, -2 removed lines

@@ -13,12 +13,12 @@
 		//with rel_users_table manually set
 		$bean = new User();
 		$bean->rel_users_table = "users_signatures";
-		$list = build_related_list_by_user_id($bean,'1','');
+		$list = build_related_list_by_user_id($bean, '1', '');
 		$this->assertTrue(is_array($list));
 
 		//with rel_users_table set by default 
 		$bean = new Meeting();
-		$list = build_related_list_by_user_id($bean,'1','');
+		$list = build_related_list_by_user_id($bean, '1', '');
 		$this->assertTrue(is_array($list));
 		
 	

tests/tests/include/utils/progressBarUtilsTest.php

Spacing +3 added lines, -3 removed lines

@@ -24,7 +24,7 @@ 
 		//execute the method and test if it doesn't throw an exception.
 		//this method uses flush so we cannot get and verify content printed
 		try {
-			display_flow_bar("test",0);
+			display_flow_bar("test", 0);
 			$this->assertTrue(true);
 		}
 		catch (Exception $e) {
@@ -67,7 +67,7 @@ 
 		//execute the method and test if it doesn't throw an exception.
 		//this method uses flush so we cannot get and verify content printed
 		try {
-			display_progress_bar("test",80, 100);
+			display_progress_bar("test", 80, 100);
 			$this->assertTrue(true);
 		}
 		catch (Exception $e) {
@@ -81,7 +81,7 @@ 
 		//execute the method and test if it doesn't throw an exception.
 		//this method uses flush so we cannot get and verify content printed
 		try {
-			update_progress_bar("test",80, 100);
+			update_progress_bar("test", 80, 100);
 			$this->assertTrue(true);
 		}
 		catch (Exception $e) {

tests/tests/include/utils/LogicHookTest.php

Spacing +71 added lines, -71 removed lines

@@ -22,7 +22,7 @@ 
 	{
 		//execute the method and test if it returns correct class instances
 		$LogicHook = LogicHook::initialize();
-		$this->assertInstanceOf('LogicHook',$LogicHook);
+		$this->assertInstanceOf('LogicHook', $LogicHook);
 		
 	}
 
@@ -32,8 +32,8 @@ 
 		
 		$LogicHook = new LogicHook();
 		$result = $LogicHook->setBean(new User());
-		$this->assertInstanceOf('LogicHook',$result);
-		$this->assertInstanceOf('User',$result->bean);		
+		$this->assertInstanceOf('LogicHook', $result);
+		$this->assertInstanceOf('User', $result->bean);		
 	}
 
 
@@ -63,48 +63,48 @@ 
     	    	
     	//execute the method and test if it returns expected contents
     	
-    	$expected_hook_map = array (
+    	$expected_hook_map = array(
     			'before_save' =>
-    			array (
-    					array ( 'file' => 'custom/modules/Accounts/logic_hooks.php', 'index' => 0,),
+    			array(
+    					array('file' => 'custom/modules/Accounts/logic_hooks.php', 'index' => 0,),
     			),
     			'after_save' =>
-    			array (
-    					array ('file' => 'custom/modules/Accounts/logic_hooks.php', 'index' => 0,),
-    					array ('file' => 'custom/modules/Accounts/logic_hooks.php', 'index' => 1,),
-    					array ('file' => 'custom/modules/Accounts/logic_hooks.php','index' => 2,),
-  						array ('file' => 'custom/modules/Accounts/logic_hooks.php','index' => 3,),
+    			array(
+    					array('file' => 'custom/modules/Accounts/logic_hooks.php', 'index' => 0,),
+    					array('file' => 'custom/modules/Accounts/logic_hooks.php', 'index' => 1,),
+    					array('file' => 'custom/modules/Accounts/logic_hooks.php', 'index' => 2,),
+  						array('file' => 'custom/modules/Accounts/logic_hooks.php', 'index' => 3,),
     			),
     			'after_relationship_add' =>
-    			array (
-    					array ('file' => 'custom/modules/Accounts/logic_hooks.php', 'index' => 0,),
+    			array(
+    					array('file' => 'custom/modules/Accounts/logic_hooks.php', 'index' => 0,),
     			),
     			'after_relationship_delete' =>
-    			array (
-						array ('file' => 'custom/modules/Accounts/logic_hooks.php','index' => 0,),
+    			array(
+						array('file' => 'custom/modules/Accounts/logic_hooks.php', 'index' => 0,),
     			),
     	);
     	
  
-    	$expected_hookscan = array (
+    	$expected_hookscan = array(
     			'before_save' =>
-    			array (
-    					array (77, 'updateGeocodeInfo', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'updateGeocodeInfo',),
+    			array(
+    					array(77, 'updateGeocodeInfo', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'updateGeocodeInfo',),
     			),
     			'after_save' =>
-    			array (
-    				    array (77, 'updateRelatedMeetingsGeocodeInfo', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'updateRelatedMeetingsGeocodeInfo',),
-    					array (78, 'updateRelatedProjectGeocodeInfo', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'updateRelatedProjectGeocodeInfo', ),
-    					array (79, 'updateRelatedOpportunitiesGeocodeInfo', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'updateRelatedOpportunitiesGeocodeInfo',),
-    					array (80, 'updateRelatedCasesGeocodeInfo', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'updateRelatedCasesGeocodeInfo',),
+    			array(
+    				    array(77, 'updateRelatedMeetingsGeocodeInfo', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'updateRelatedMeetingsGeocodeInfo',),
+    					array(78, 'updateRelatedProjectGeocodeInfo', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'updateRelatedProjectGeocodeInfo',),
+    					array(79, 'updateRelatedOpportunitiesGeocodeInfo', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'updateRelatedOpportunitiesGeocodeInfo',),
+    					array(80, 'updateRelatedCasesGeocodeInfo', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'updateRelatedCasesGeocodeInfo',),
     			),
     			'after_relationship_add' =>
-    			array (
-    					array ( 77, 'addRelationship', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'addRelationship',),
+    			array(
+    					array(77, 'addRelationship', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'addRelationship',),
     			),
     			'after_relationship_delete' =>
-    			array (
-    					array ( 77, 'deleteRelationship', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'deleteRelationship',),
+    			array(
+    					array(77, 'deleteRelationship', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'deleteRelationship',),
     			),
     	);
     	
@@ -114,8 +114,8 @@ 
     	$hookscan = $LogicHook->getHooksList();
     	
     	
-    	$this->assertSame($expected_hook_map,$hook_map);
-    	$this->assertSame($expected_hookscan,$hookscan);
+    	$this->assertSame($expected_hook_map, $hook_map);
+    	$this->assertSame($expected_hookscan, $hookscan);
     	
     	
     }
@@ -139,52 +139,52 @@ 
 	{
 		//execute the method and test if it returns expected contents
 		
-		$expected_accounts = array (
-				'after_ui_frame' => Array (),
+		$expected_accounts = array(
+				'after_ui_frame' => Array(),
 				'before_save' =>
-				array (
-						array (77, 'updateGeocodeInfo', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'updateGeocodeInfo',),
+				array(
+						array(77, 'updateGeocodeInfo', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'updateGeocodeInfo',),
 				),
 				'after_save' =>
-				array (
-						array (77, 'updateRelatedMeetingsGeocodeInfo', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'updateRelatedMeetingsGeocodeInfo',),
-						array (78, 'updateRelatedProjectGeocodeInfo', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'updateRelatedProjectGeocodeInfo', ),
-						array (79, 'updateRelatedOpportunitiesGeocodeInfo', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'updateRelatedOpportunitiesGeocodeInfo',),
-						array (80, 'updateRelatedCasesGeocodeInfo', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'updateRelatedCasesGeocodeInfo',),
+				array(
+						array(77, 'updateRelatedMeetingsGeocodeInfo', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'updateRelatedMeetingsGeocodeInfo',),
+						array(78, 'updateRelatedProjectGeocodeInfo', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'updateRelatedProjectGeocodeInfo',),
+						array(79, 'updateRelatedOpportunitiesGeocodeInfo', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'updateRelatedOpportunitiesGeocodeInfo',),
+						array(80, 'updateRelatedCasesGeocodeInfo', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'updateRelatedCasesGeocodeInfo',),
 				),
 				'after_relationship_add' =>
-				array (
-						array ( 77, 'addRelationship', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'addRelationship',),
+				array(
+						array(77, 'addRelationship', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'addRelationship',),
 				),
 				'after_relationship_delete' =>
-				array (
-						array ( 77, 'deleteRelationship', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'deleteRelationship',),
+				array(
+						array(77, 'deleteRelationship', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'deleteRelationship',),
 				),
 		);
 		
-		$expected_default = array (
+		$expected_default = array(
 					'after_ui_footer' =>
-					array (
-							array (10,'popup_onload','modules/SecurityGroups/AssignGroups.php','AssignGroups','popup_onload',),
+					array(
+							array(10, 'popup_onload', 'modules/SecurityGroups/AssignGroups.php', 'AssignGroups', 'popup_onload',),
 					),
 					'after_ui_frame' =>
-					array (
-							array (20, 'mass_assign', 'modules/SecurityGroups/AssignGroups.php', 'AssignGroups', 'mass_assign',),
-							array ( 1, 'Load Social JS', 'include/social/hooks.php', 'hooks', 'load_js',),
+					array(
+							array(20, 'mass_assign', 'modules/SecurityGroups/AssignGroups.php', 'AssignGroups', 'mass_assign',),
+							array(1, 'Load Social JS', 'include/social/hooks.php', 'hooks', 'load_js',),
 					),
 					'after_save' =>
-					array (
-							array ( 30,'popup_select', 'modules/SecurityGroups/AssignGroups.php','AssignGroups','popup_select',),
-							array ( 1, 'AOD Index Changes', 'modules/AOD_Index/AOD_LogicHooks.php', 'AOD_LogicHooks', 'saveModuleChanges',),
-							array ( 99, 'AOW_Workflow', 'modules/AOW_WorkFlow/AOW_WorkFlow.php', 'AOW_WorkFlow','run_bean_flows',),
+					array(
+							array(30, 'popup_select', 'modules/SecurityGroups/AssignGroups.php', 'AssignGroups', 'popup_select',),
+							array(1, 'AOD Index Changes', 'modules/AOD_Index/AOD_LogicHooks.php', 'AOD_LogicHooks', 'saveModuleChanges',),
+							array(99, 'AOW_Workflow', 'modules/AOW_WorkFlow/AOW_WorkFlow.php', 'AOW_WorkFlow', 'run_bean_flows',),
 					),
 					'after_delete' =>
-					array (
-							array ( 1, 'AOD Index changes', 'modules/AOD_Index/AOD_LogicHooks.php', 'AOD_LogicHooks','saveModuleDelete',),
+					array(
+							array(1, 'AOD Index changes', 'modules/AOD_Index/AOD_LogicHooks.php', 'AOD_LogicHooks', 'saveModuleDelete',),
 					),
 					'after_restore' =>
-					array (
-							array ( 1, 'AOD Index changes', 'modules/AOD_Index/AOD_LogicHooks.php', 'AOD_LogicHooks', 'saveModuleRestore',),
+					array(
+							array(1, 'AOD Index changes', 'modules/AOD_Index/AOD_LogicHooks.php', 'AOD_LogicHooks', 'saveModuleRestore',),
 					),
 				);
 		
@@ -206,26 +206,26 @@ 
 	{
 		//execute the method and test if it returns expected contents
 		
-		$expected = array (
-				'after_ui_frame' => Array (),
+		$expected = array(
+				'after_ui_frame' => Array(),
 				'before_save' =>
-				array (
-						array (77, 'updateGeocodeInfo', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'updateGeocodeInfo',),
+				array(
+						array(77, 'updateGeocodeInfo', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'updateGeocodeInfo',),
 				),
 				'after_save' =>
-				array (
-						array (77, 'updateRelatedMeetingsGeocodeInfo', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'updateRelatedMeetingsGeocodeInfo',),
-						array (78, 'updateRelatedProjectGeocodeInfo', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'updateRelatedProjectGeocodeInfo', ),
-						array (79, 'updateRelatedOpportunitiesGeocodeInfo', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'updateRelatedOpportunitiesGeocodeInfo',),
-						array (80, 'updateRelatedCasesGeocodeInfo', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'updateRelatedCasesGeocodeInfo',),
+				array(
+						array(77, 'updateRelatedMeetingsGeocodeInfo', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'updateRelatedMeetingsGeocodeInfo',),
+						array(78, 'updateRelatedProjectGeocodeInfo', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'updateRelatedProjectGeocodeInfo',),
+						array(79, 'updateRelatedOpportunitiesGeocodeInfo', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'updateRelatedOpportunitiesGeocodeInfo',),
+						array(80, 'updateRelatedCasesGeocodeInfo', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'updateRelatedCasesGeocodeInfo',),
 				),
 				'after_relationship_add' =>
-				array (
-						array ( 77, 'addRelationship', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'addRelationship',),
+				array(
+						array(77, 'addRelationship', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'addRelationship',),
 				),
 				'after_relationship_delete' =>
-				array (
-						array ( 77, 'deleteRelationship', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'deleteRelationship',),
+				array(
+						array(77, 'deleteRelationship', 'modules/Accounts/AccountsJjwg_MapsLogicHook.php', 'AccountsJjwg_MapsLogicHook', 'deleteRelationship',),
 				),
 		);
 		
@@ -236,7 +236,7 @@ 
 		$this->assertSame($expected, $hooks);
 		
 		//test wit hrefresh true
-		$hooks = $LogicHook->getHooks('Accounts',true);
+		$hooks = $LogicHook->getHooks('Accounts', true);
 		$this->assertSame($expected, $hooks);
 		
 		
@@ -271,7 +271,7 @@ 
 		$hooks = $LogicHook->loadHooks('');
 		
 		try {
-			$LogicHook->process_hooks($hooks, 'after_ui_footer',array() );
+			$LogicHook->process_hooks($hooks, 'after_ui_footer', array());
 			$this->assertTrue(true);
 		}
 		catch (Exception $e) {