adamjakab / SuiteCRM

tests/tests/include/MVC/View/views/view.noaccessTest.php

Indentation +12 added lines, -12 removed lines

@@ -3,22 +3,22 @@
 class ViewNoaccessTest extends PHPUnit_Framework_TestCase
 {
 
-	public function testdisplay()
-	{
-		//executet the method and check for default attributes and check that it returns some html.
-		$view = new ViewNoaccess();
+    public function testdisplay()
+    {
+        //executet the method and check for default attributes and check that it returns some html.
+        $view = new ViewNoaccess();
 	
-		ob_start();
+        ob_start();
 		
-		$view->display();
+        $view->display();
 		
-		$renderedContent = ob_get_contents();
-		ob_end_clean();
+        $renderedContent = ob_get_contents();
+        ob_end_clean();
 		
-		$this->assertAttributeEquals('noaccess','type', $view);
-		$this->assertGreaterThan(0,strlen($renderedContent));
+        $this->assertAttributeEquals('noaccess','type', $view);
+        $this->assertGreaterThan(0,strlen($renderedContent));
 		
-		$this->assertEquals(False,json_decode($renderedContent)); //check that it doesn't return json.
+        $this->assertEquals(False,json_decode($renderedContent)); //check that it doesn't return json.
 
- 	}
+        }
 }

Spacing +3 added lines, -3 removed lines

@@ -15,10 +15,10 @@
 		$renderedContent = ob_get_contents();
 		ob_end_clean();
 		
-		$this->assertAttributeEquals('noaccess','type', $view);
-		$this->assertGreaterThan(0,strlen($renderedContent));
+		$this->assertAttributeEquals('noaccess', 'type', $view);
+		$this->assertGreaterThan(0, strlen($renderedContent));
 		
-		$this->assertEquals(False,json_decode($renderedContent)); //check that it doesn't return json.
+		$this->assertEquals(False, json_decode($renderedContent)); //check that it doesn't return json.
 
  	}
 }

tests/tests/include/MVC/View/views/view.modulelistmenuTest.php

Indentation +20 added lines, -20 removed lines

@@ -2,31 +2,31 @@
 
 class ViewModulelistmenuTest extends PHPUnit_Framework_TestCase
 {
- 	public function test__construct()
- 	{
+        public function test__construct()
+        {
  
- 		//execute the contructor and check for the Object type and options attribute
- 		$view = new ViewModulelistmenu();
+            //execute the contructor and check for the Object type and options attribute
+            $view = new ViewModulelistmenu();
  			
- 		$this->assertInstanceOf('ViewModulelistmenu',$view);
- 		$this->assertInstanceOf('SugarView',$view);
- 		$this->assertTrue(is_array($view->options)); 		
+            $this->assertInstanceOf('ViewModulelistmenu',$view);
+            $this->assertInstanceOf('SugarView',$view);
+            $this->assertTrue(is_array($view->options)); 		
  		
- 	}	
+        }	
  	
- 	public function testdisplay()
- 	{
- 		//execute the method with required child objects preset. it should return some html. 
- 		$view = new ViewModulelistmenu();
- 		$view->ss = new Sugar_Smarty();
+        public function testdisplay()
+        {
+            //execute the method with required child objects preset. it should return some html. 
+            $view = new ViewModulelistmenu();
+            $view->ss = new Sugar_Smarty();
  		
- 		ob_start();
- 		$view->display();
- 		$renderedContent = ob_get_contents();
- 		ob_end_clean();
+            ob_start();
+            $view->display();
+            $renderedContent = ob_get_contents();
+            ob_end_clean();
 
- 		$this->assertGreaterThan(0,strlen($renderedContent));
- 		$this->assertEquals(false,is_array($renderedContent));
- 	}
+            $this->assertGreaterThan(0,strlen($renderedContent));
+            $this->assertEquals(false,is_array($renderedContent));
+        }
 }
 ?>

Spacing +4 added lines, -4 removed lines

@@ -8,8 +8,8 @@ 
  		//execute the contructor and check for the Object type and options attribute
  		$view = new ViewModulelistmenu();
  			
- 		$this->assertInstanceOf('ViewModulelistmenu',$view);
- 		$this->assertInstanceOf('SugarView',$view);
+ 		$this->assertInstanceOf('ViewModulelistmenu', $view);
+ 		$this->assertInstanceOf('SugarView', $view);
  		$this->assertTrue(is_array($view->options)); 		
  		
  	}	
@@ -25,8 +25,8 @@ 
  		$renderedContent = ob_get_contents();
  		ob_end_clean();
 
- 		$this->assertGreaterThan(0,strlen($renderedContent));
- 		$this->assertEquals(false,is_array($renderedContent));
+ 		$this->assertGreaterThan(0, strlen($renderedContent));
+ 		$this->assertEquals(false, is_array($renderedContent));
  	}
 }
 ?>

tests/tests/include/MVC/View/views/view.vcardTest.php

Indentation +15 added lines, -15 removed lines

@@ -4,25 +4,25 @@
 class ViewVcardTest extends PHPUnit_Framework_TestCase
 {
 
-	public function testdisplay()
+    public function testdisplay()
     {
-    	//execute the method with required child objects preset and check for the Object type and type attribute
+        //execute the method with required child objects preset and check for the Object type and type attribute
   
-    	$view = new ViewVcard();
-		$view->module = "Contacts";
-		$view->bean = new Contact();
+        $view = new ViewVcard();
+        $view->module = "Contacts";
+        $view->bean = new Contact();
 		
-    	//execute the method and test if it works and does not throws an exception other than headers output exception.
-    	try {
-    		$view->display();
+        //execute the method and test if it works and does not throws an exception other than headers output exception.
+        try {
+            $view->display();
     			
-    	} catch (Exception $e) {
-    		$this->assertStringStartsWith('Cannot modify header information',$e->getMessage());
-    	}
+        } catch (Exception $e) {
+            $this->assertStringStartsWith('Cannot modify header information',$e->getMessage());
+        }
     	
-    	$this->assertInstanceOf('ViewVcard',$view);
-    	$this->assertInstanceOf('SugarView',$view);
-    	$this->assertAttributeEquals('detail','type', $view);
- 	}
+        $this->assertInstanceOf('ViewVcard',$view);
+        $this->assertInstanceOf('SugarView',$view);
+        $this->assertAttributeEquals('detail','type', $view);
+        }
 }
 ?>

Spacing +5 added lines, -5 removed lines

@@ -16,13 +16,13 @@
     	try {
     		$view->display();
     			
-    	} catch (Exception $e) {
-    		$this->assertStringStartsWith('Cannot modify header information',$e->getMessage());
+    	}catch (Exception $e) {
+    		$this->assertStringStartsWith('Cannot modify header information', $e->getMessage());
     	}
     	
-    	$this->assertInstanceOf('ViewVcard',$view);
-    	$this->assertInstanceOf('SugarView',$view);
-    	$this->assertAttributeEquals('detail','type', $view);
+    	$this->assertInstanceOf('ViewVcard', $view);
+    	$this->assertInstanceOf('SugarView', $view);
+    	$this->assertAttributeEquals('detail', 'type', $view);
  	}
 }
 ?>

tests/tests/include/MVC/View/views/view.classicTest.php

Indentation +40 added lines, -40 removed lines

@@ -5,64 +5,64 @@
 
     public function test__construct( )
     {
-    	//execute the contructor and check for the Object type and type attribute
+        //execute the contructor and check for the Object type and type attribute
     	
-    	//test with no paramerters
-    	$view = new ViewClassic();
-    	$this->assertInstanceOf('ViewClassic',$view);
-    	$this->assertInstanceOf('SugarView',$view);
-    	$this->assertAttributeEquals('','type', $view);
+        //test with no paramerters
+        $view = new ViewClassic();
+        $this->assertInstanceOf('ViewClassic',$view);
+        $this->assertInstanceOf('SugarView',$view);
+        $this->assertAttributeEquals('','type', $view);
 
     	
-    	//test with bean parameter;
-    	$bean = new User();
-    	$view = new ViewClassic($bean);
-    	$this->assertInstanceOf('ViewClassic',$view);
-    	$this->assertInstanceOf('SugarView',$view);
-    	$this->assertAttributeEquals('','type', $view);
+        //test with bean parameter;
+        $bean = new User();
+        $view = new ViewClassic($bean);
+        $this->assertInstanceOf('ViewClassic',$view);
+        $this->assertInstanceOf('SugarView',$view);
+        $this->assertAttributeEquals('','type', $view);
     		
- 	}
+        }
  	
 
     public function testdisplay()
     {
-    	error_reporting(E_ERROR | E_PARSE);
+        error_reporting(E_ERROR | E_PARSE);
 
     	
-    	//test with a valid module but invalid action. it should return false.
-    	$view = new ViewClassic();
-    	$view->module = "Home";
-    	$view->action = "";	 
-    	$ret = $view->display();
-    	$this->assertFalse($ret);
+        //test with a valid module but invalid action. it should return false.
+        $view = new ViewClassic();
+        $view->module = "Home";
+        $view->action = "";	 
+        $ret = $view->display();
+        $this->assertFalse($ret);
     	
 
     	
-    	//test with a valid module and uncustomized action. it should return true
-    	$view = new ViewClassic();
-    	$view->module = "Home";
-    	$view->action = "About";
+        //test with a valid module and uncustomized action. it should return true
+        $view = new ViewClassic();
+        $view->module = "Home";
+        $view->action = "About";
     	
-    	ob_start();
-    	$ret = $view->display();
-    	$renderedContent = ob_get_contents();
-    	ob_end_clean();
-    	$this->assertGreaterThan(0,strlen($renderedContent));
-    	$this->assertTrue($ret);
+        ob_start();
+        $ret = $view->display();
+        $renderedContent = ob_get_contents();
+        ob_end_clean();
+        $this->assertGreaterThan(0,strlen($renderedContent));
+        $this->assertTrue($ret);
     
     	
     	
-    	//test with a valid module and customized action. it should return true
-    	$view = new ViewClassic();
-    	$view->module = "Home";
-    	$view->action = "index";
+        //test with a valid module and customized action. it should return true
+        $view = new ViewClassic();
+        $view->module = "Home";
+        $view->action = "index";
 
-    	ob_start();
-    	$ret = $view->display();
-    	$renderedContent = ob_get_contents();
-    	ob_end_clean();
-    	$this->assertGreaterThan(0,strlen($renderedContent));
-    	$this->assertTrue($ret);
+        ob_start();
+        $ret = $view->display();
+        $renderedContent = ob_get_contents();
+        ob_end_clean();
+        $this->assertGreaterThan(0,strlen($renderedContent));
+        $this->assertTrue($ret);
     	
     	
     }

Spacing +8 added lines, -8 removed lines

@@ -9,17 +9,17 @@ 
     	
     	//test with no paramerters
     	$view = new ViewClassic();
-    	$this->assertInstanceOf('ViewClassic',$view);
-    	$this->assertInstanceOf('SugarView',$view);
-    	$this->assertAttributeEquals('','type', $view);
+    	$this->assertInstanceOf('ViewClassic', $view);
+    	$this->assertInstanceOf('SugarView', $view);
+    	$this->assertAttributeEquals('', 'type', $view);
 
     	
     	//test with bean parameter;
     	$bean = new User();
     	$view = new ViewClassic($bean);
-    	$this->assertInstanceOf('ViewClassic',$view);
-    	$this->assertInstanceOf('SugarView',$view);
-    	$this->assertAttributeEquals('','type', $view);
+    	$this->assertInstanceOf('ViewClassic', $view);
+    	$this->assertInstanceOf('SugarView', $view);
+    	$this->assertAttributeEquals('', 'type', $view);
     		
  	}
  	
@@ -47,7 +47,7 @@ 
     	$ret = $view->display();
     	$renderedContent = ob_get_contents();
     	ob_end_clean();
-    	$this->assertGreaterThan(0,strlen($renderedContent));
+    	$this->assertGreaterThan(0, strlen($renderedContent));
     	$this->assertTrue($ret);
     
     	
@@ -61,7 +61,7 @@ 
     	$ret = $view->display();
     	$renderedContent = ob_get_contents();
     	ob_end_clean();
-    	$this->assertGreaterThan(0,strlen($renderedContent));
+    	$this->assertGreaterThan(0, strlen($renderedContent));
     	$this->assertTrue($ret);
     	
     	

tests/tests/include/MVC/View/views/view.quickeditTest.php

Indentation +38 added lines, -38 removed lines

@@ -5,61 +5,61 @@
   
     public function testpreDisplay()
     {
-    	//check without setting any values, it should execute without any issues.
-    	$view = new ViewQuickedit();
-    	$view->preDisplay();
-    	$this->assertEquals(0,count($_REQUEST));
+        //check without setting any values, it should execute without any issues.
+        $view = new ViewQuickedit();
+        $view->preDisplay();
+        $this->assertEquals(0,count($_REQUEST));
     	
-    	//check with values preset but without a valid bean id, it sould not change Request parameters
-    	$_REQUEST['source_module'] = "Users";
-    	$_REQUEST['module']= 'Users';
-    	$_REQUEST['record'] = '';
-    	$request = $_REQUEST;
+        //check with values preset but without a valid bean id, it sould not change Request parameters
+        $_REQUEST['source_module'] = "Users";
+        $_REQUEST['module']= 'Users';
+        $_REQUEST['record'] = '';
+        $request = $_REQUEST;
     	
-    	$view->preDisplay();
-    	$this->assertSame($request,$_REQUEST);
+        $view->preDisplay();
+        $this->assertSame($request,$_REQUEST);
     
-    	//check with values preset, it sould set some addiiotnal Request parameters
-    	$_REQUEST['record'] = 1;
-    	$view->preDisplay();
-    	$this->assertNotSame($request,$_REQUEST);
+        //check with values preset, it sould set some addiiotnal Request parameters
+        $_REQUEST['record'] = 1;
+        $view->preDisplay();
+        $this->assertNotSame($request,$_REQUEST);
        	
     }
 
  
     public function testdisplay()
     {	    
-    	error_reporting(E_ALL);
-    	//error_reporting(E_ERROR | E_PARSE |E_NOTICE);
+        error_reporting(E_ALL);
+        //error_reporting(E_ERROR | E_PARSE |E_NOTICE);
     	
-    	//execute the method with required child objects and paramerers preset. it will rteturn some html.
-    	$view = new ViewQuickedit();
-    	$_REQUEST['module'] = 'Accounts';
-    	$_REQUEST['action'] = 'SubpanelCreates';
+        //execute the method with required child objects and paramerers preset. it will rteturn some html.
+        $view = new ViewQuickedit();
+        $_REQUEST['module'] = 'Accounts';
+        $_REQUEST['action'] = 'SubpanelCreates';
     			
-    	try{
-    		$view->bean = new Account();
-    	}
-    	Catch(Exception $e){
-    		$this->assertStringStartsWith('mysqli_query()',$e->getMessage());
-    	}
+        try{
+            $view->bean = new Account();
+        }
+        Catch(Exception $e){
+            $this->assertStringStartsWith('mysqli_query()',$e->getMessage());
+        }
 
     	
-    	try{
-    		ob_start();
+        try{
+            ob_start();
     		
-    		$view->display();
+            $view->display();
 
-    		$renderedContent = ob_get_contents();
-    		ob_end_clean();
+            $renderedContent = ob_get_contents();
+            ob_end_clean();
     		 
-    		$this->assertGreaterThan(0,strlen($renderedContent));
-    		$this->assertNotEquals(False,json_decode($renderedContent));
+            $this->assertGreaterThan(0,strlen($renderedContent));
+            $this->assertNotEquals(False,json_decode($renderedContent));
     	
-    	}
-    	Catch(Exception $e){
-    		$this->assertStringStartsWith('mysqli_query()',$e->getMessage());
-    	}   	
+        }
+        Catch(Exception $e){
+            $this->assertStringStartsWith('mysqli_query()',$e->getMessage());
+        }   	
     		
 
     	

Spacing +12 added lines, -12 removed lines

@@ -8,21 +8,21 @@ 
     	//check without setting any values, it should execute without any issues.
     	$view = new ViewQuickedit();
     	$view->preDisplay();
-    	$this->assertEquals(0,count($_REQUEST));
+    	$this->assertEquals(0, count($_REQUEST));
     	
     	//check with values preset but without a valid bean id, it sould not change Request parameters
     	$_REQUEST['source_module'] = "Users";
-    	$_REQUEST['module']= 'Users';
+    	$_REQUEST['module'] = 'Users';
     	$_REQUEST['record'] = '';
     	$request = $_REQUEST;
     	
     	$view->preDisplay();
-    	$this->assertSame($request,$_REQUEST);
+    	$this->assertSame($request, $_REQUEST);
     
     	//check with values preset, it sould set some addiiotnal Request parameters
     	$_REQUEST['record'] = 1;
     	$view->preDisplay();
-    	$this->assertNotSame($request,$_REQUEST);
+    	$this->assertNotSame($request, $_REQUEST);
        	
     }
 
@@ -37,15 +37,15 @@ 
     	$_REQUEST['module'] = 'Accounts';
     	$_REQUEST['action'] = 'SubpanelCreates';
     			
-    	try{
+    	try {
     		$view->bean = new Account();
     	}
-    	Catch(Exception $e){
-    		$this->assertStringStartsWith('mysqli_query()',$e->getMessage());
+    	Catch (Exception $e) {
+    		$this->assertStringStartsWith('mysqli_query()', $e->getMessage());
     	}
 
     	
-    	try{
+    	try {
     		ob_start();
     		
     		$view->display();
@@ -53,12 +53,12 @@ 
     		$renderedContent = ob_get_contents();
     		ob_end_clean();
     		 
-    		$this->assertGreaterThan(0,strlen($renderedContent));
-    		$this->assertNotEquals(False,json_decode($renderedContent));
+    		$this->assertGreaterThan(0, strlen($renderedContent));
+    		$this->assertNotEquals(False, json_decode($renderedContent));
     	
     	}
-    	Catch(Exception $e){
-    		$this->assertStringStartsWith('mysqli_query()',$e->getMessage());
+    	Catch (Exception $e) {
+    		$this->assertStringStartsWith('mysqli_query()', $e->getMessage());
     	}   	
     		
 

Braces +2 added lines, -4 removed lines

@@ -39,8 +39,7 @@ 
     			
     	try{
     		$view->bean = new Account();
-    	}
-    	Catch(Exception $e){
+    	} Catch(Exception $e){
     		$this->assertStringStartsWith('mysqli_query()',$e->getMessage());
     	}
 
@@ -56,8 +55,7 @@ 
     		$this->assertGreaterThan(0,strlen($renderedContent));
     		$this->assertNotEquals(False,json_decode($renderedContent));
     	
-    	}
-    	Catch(Exception $e){
+    	} Catch(Exception $e){
     		$this->assertStringStartsWith('mysqli_query()',$e->getMessage());
     	}   	
     		

tests/tests/include/MVC/View/views/view.ajaxTest.php

Indentation +8 added lines, -8 removed lines

@@ -3,13 +3,13 @@
 class ViewAjaxTest extends PHPUnit_Framework_TestCase
 {
 	
- 	public function testViewAjax()
- 	{
- 		//execute the contructor and check for the Object type and attributes 		
- 		$view = new ViewAjax();
- 		$this->assertInstanceOf('ViewAjax',$view);
- 		$this->assertInstanceOf('SugarView',$view);
- 		$this->assertTrue( is_array($view->options));
+        public function testViewAjax()
+        {
+            //execute the contructor and check for the Object type and attributes 		
+            $view = new ViewAjax();
+            $this->assertInstanceOf('ViewAjax',$view);
+            $this->assertInstanceOf('SugarView',$view);
+            $this->assertTrue( is_array($view->options));
  			
- 	}
+        }
 }

Spacing +3 added lines, -3 removed lines

@@ -7,9 +7,9 @@
  	{
  		//execute the contructor and check for the Object type and attributes 		
  		$view = new ViewAjax();
- 		$this->assertInstanceOf('ViewAjax',$view);
- 		$this->assertInstanceOf('SugarView',$view);
- 		$this->assertTrue( is_array($view->options));
+ 		$this->assertInstanceOf('ViewAjax', $view);
+ 		$this->assertInstanceOf('SugarView', $view);
+ 		$this->assertTrue(is_array($view->options));
  			
  	}
 }

tests/tests/include/MVC/View/views/view.xmlTest.php

Indentation +21 added lines, -21 removed lines

@@ -3,35 +3,35 @@
 
 class ViewXMLTest extends PHPUnit_Framework_TestCase {
 	
-	function testViewXML(){
-		
-		//execute the contructor and check for the Object type and type attribute
- 		$view = new ViewXML();
- 		$this->assertInstanceOf('ViewXML',$view);
- 		$this->assertInstanceOf('SugarView',$view);
- 		$this->assertAttributeEquals('detail','type', $view);
- 	}
+    function testViewXML(){
+		
+        //execute the contructor and check for the Object type and type attribute
+            $view = new ViewXML();
+            $this->assertInstanceOf('ViewXML',$view);
+            $this->assertInstanceOf('SugarView',$view);
+            $this->assertAttributeEquals('detail','type', $view);
+        }
  	
-	function testdisplay(){
+    function testdisplay(){
 		
-		//execute the method and check for rexcetions. it should return some html. 
-		$view = new ViewXML();
+        //execute the method and check for rexcetions. it should return some html. 
+        $view = new ViewXML();
 		
-		try {
+        try {
 		
-			ob_start();
+            ob_start();
 				
-			$view->display();
+            $view->display();
 		
-			$renderedContent = ob_get_contents();
-			ob_end_clean();
+            $renderedContent = ob_get_contents();
+            ob_end_clean();
 		
-			$this->assertGreaterThan(0,strlen($renderedContent));
+            $this->assertGreaterThan(0,strlen($renderedContent));
 		
-		} catch (Exception $e) {
-			$this->fail();
-		}
+        } catch (Exception $e) {
+            $this->fail();
+        }
 		
- 	}
+        }
 }
 ?>

Spacing +7 added lines, -7 removed lines

@@ -3,16 +3,16 @@ 
 
 class ViewXMLTest extends PHPUnit_Framework_TestCase {
 	
-	function testViewXML(){
+	function testViewXML() {
 		
 		//execute the contructor and check for the Object type and type attribute
  		$view = new ViewXML();
- 		$this->assertInstanceOf('ViewXML',$view);
- 		$this->assertInstanceOf('SugarView',$view);
- 		$this->assertAttributeEquals('detail','type', $view);
+ 		$this->assertInstanceOf('ViewXML', $view);
+ 		$this->assertInstanceOf('SugarView', $view);
+ 		$this->assertAttributeEquals('detail', 'type', $view);
  	}
  	
-	function testdisplay(){
+	function testdisplay() {
 		
 		//execute the method and check for rexcetions. it should return some html. 
 		$view = new ViewXML();
@@ -26,9 +26,9 @@ 
 			$renderedContent = ob_get_contents();
 			ob_end_clean();
 		
-			$this->assertGreaterThan(0,strlen($renderedContent));
+			$this->assertGreaterThan(0, strlen($renderedContent));
 		
-		} catch (Exception $e) {
+		}catch (Exception $e) {
 			$this->fail();
 		}
 		

tests/tests/include/MVC/View/views/view.quickTest.php

Indentation +24 added lines, -24 removed lines

@@ -2,39 +2,39 @@
 
 class ViewQuickTest extends PHPUnit_Framework_TestCase{
 	
- 	public function testViewQuick(){
+        public function testViewQuick(){
  		
- 		//execute the contructor and check for the Object type and type attribute
+            //execute the contructor and check for the Object type and type attribute
  		
- 		$view = new ViewQuick();
+            $view = new ViewQuick();
  		
- 		$this->assertInstanceOf('ViewQuick',$view);
- 		$this->assertInstanceOf('ViewDetail',$view);
- 		$this->assertAttributeEquals('detail','type', $view);
- 		$this->assertTrue(is_array($view->options));
- 	}
+            $this->assertInstanceOf('ViewQuick',$view);
+            $this->assertInstanceOf('ViewDetail',$view);
+            $this->assertAttributeEquals('detail','type', $view);
+            $this->assertTrue(is_array($view->options));
+        }
  	
- 	public function testdisplay(){
+        public function testdisplay(){
 
- 		$view = new ViewQuick();
+            $view = new ViewQuick();
  		
- 		//execute the method with required child objects preset. it will return some html.
- 		$view->dv = new DetailView2();
- 		$view->dv->ss = new Sugar_Smarty();
- 		$view->dv->module = "Users";
- 		$view->bean= new User();
- 		$view->bean->id =1;
- 		$view->dv->setup("Users",$view->bean);
+            //execute the method with required child objects preset. it will return some html.
+            $view->dv = new DetailView2();
+            $view->dv->ss = new Sugar_Smarty();
+            $view->dv->module = "Users";
+            $view->bean= new User();
+            $view->bean->id =1;
+            $view->dv->setup("Users",$view->bean);
  		
- 		ob_start();
+            ob_start();
  		
- 		$view->display();
+            $view->display();
  		
- 		$renderedContent = ob_get_contents();
- 		ob_end_clean();
+            $renderedContent = ob_get_contents();
+            ob_end_clean();
  		
- 		$this->assertGreaterThan(0,strlen($renderedContent));
- 		$this->assertNotEquals(False,json_decode($renderedContent));
- 	}
+            $this->assertGreaterThan(0,strlen($renderedContent));
+            $this->assertNotEquals(False,json_decode($renderedContent));
+        }
  	
 }

Spacing +11 added lines, -11 removed lines

@@ -1,20 +1,20 @@ 
 <?php
 
-class ViewQuickTest extends PHPUnit_Framework_TestCase{
+class ViewQuickTest extends PHPUnit_Framework_TestCase {
 	
- 	public function testViewQuick(){
+ 	public function testViewQuick() {
  		
  		//execute the contructor and check for the Object type and type attribute
  		
  		$view = new ViewQuick();
  		
- 		$this->assertInstanceOf('ViewQuick',$view);
- 		$this->assertInstanceOf('ViewDetail',$view);
- 		$this->assertAttributeEquals('detail','type', $view);
+ 		$this->assertInstanceOf('ViewQuick', $view);
+ 		$this->assertInstanceOf('ViewDetail', $view);
+ 		$this->assertAttributeEquals('detail', 'type', $view);
  		$this->assertTrue(is_array($view->options));
  	}
  	
- 	public function testdisplay(){
+ 	public function testdisplay() {
 
  		$view = new ViewQuick();
  		
@@ -22,9 +22,9 @@ 
  		$view->dv = new DetailView2();
  		$view->dv->ss = new Sugar_Smarty();
  		$view->dv->module = "Users";
- 		$view->bean= new User();
- 		$view->bean->id =1;
- 		$view->dv->setup("Users",$view->bean);
+ 		$view->bean = new User();
+ 		$view->bean->id = 1;
+ 		$view->dv->setup("Users", $view->bean);
  		
  		ob_start();
  		
@@ -33,8 +33,8 @@ 
  		$renderedContent = ob_get_contents();
  		ob_end_clean();
  		
- 		$this->assertGreaterThan(0,strlen($renderedContent));
- 		$this->assertNotEquals(False,json_decode($renderedContent));
+ 		$this->assertGreaterThan(0, strlen($renderedContent));
+ 		$this->assertNotEquals(False, json_decode($renderedContent));
  	}
  	
 }

tests/tests/include/MVC/View/views/view.multieditTest.php

Indentation +31 added lines, -31 removed lines

@@ -1,45 +1,45 @@ 
 <?php
 
 
- class ViewMultieditTest  extends PHPUnit_Framework_TestCase{
+    class ViewMultieditTest  extends PHPUnit_Framework_TestCase{
 
- 	function testViewMultiedit(){
+        function testViewMultiedit(){
  		
- 		//execute the contructor and check for the Object type and type attribute
- 		$view = new ViewMultiedit();
- 		$this->assertInstanceOf('ViewMultiedit',$view);
- 		$this->assertInstanceOf('SugarView',$view);
- 		$this->assertAttributeEquals('edit','type', $view);
- 	}
+            //execute the contructor and check for the Object type and type attribute
+            $view = new ViewMultiedit();
+            $this->assertInstanceOf('ViewMultiedit',$view);
+            $this->assertInstanceOf('SugarView',$view);
+            $this->assertAttributeEquals('edit','type', $view);
+        }
  	
- 	function testdisplay(){
+        function testdisplay(){
 
 
- 		//test without action value and modules list in REQUEST object
- 		$view = new ViewMultiedit();
- 		ob_start();
- 		$view->display();
- 		$renderedContent = ob_get_contents();
- 		ob_end_clean();
- 		$this->assertEquals(0,strlen($renderedContent));
+            //test without action value and modules list in REQUEST object
+            $view = new ViewMultiedit();
+            ob_start();
+            $view->display();
+            $renderedContent = ob_get_contents();
+            ob_end_clean();
+            $this->assertEquals(0,strlen($renderedContent));
  		
 
  		
- 		//test with valid action value to get link in return
- 		$view = new ViewMultiedit();
- 		$view->action = 'AjaxFormSave';
- 		$view->module = 'Users';
- 		$view->bean = new User();
- 		$view->bean->id =1;
- 		ob_start(); 		
- 		$view->display(); 		
- 		$renderedContent = ob_get_contents();
- 		ob_end_clean(); 		
- 		$this->assertGreaterThan(0,strlen($renderedContent));
+            //test with valid action value to get link in return
+            $view = new ViewMultiedit();
+            $view->action = 'AjaxFormSave';
+            $view->module = 'Users';
+            $view->bean = new User();
+            $view->bean->id =1;
+            ob_start(); 		
+            $view->display(); 		
+            $renderedContent = ob_get_contents();
+            ob_end_clean(); 		
+            $this->assertGreaterThan(0,strlen($renderedContent));
  			
  		
- 		//Fails with a fatal error, method creates editview without properly setting it up causing fatal errors.
- 		/*
+            //Fails with a fatal error, method creates editview without properly setting it up causing fatal errors.
+            /*
  		//test only with modules list in REQUEST object
  		$view = new ViewMultiedit();
  		$GLOBALS['current_language']= 'en_us';
@@ -51,6 +51,6 @@ 
  		$this->assertGreaterThan(0,strlen($renderedContent));
  		*/	
  		
- 	}
- }
+        }
+    }
 ?>

Spacing +9 added lines, -9 removed lines

@@ -1,18 +1,18 @@ 
 <?php
 
 
- class ViewMultieditTest  extends PHPUnit_Framework_TestCase{
+ class ViewMultieditTest  extends PHPUnit_Framework_TestCase {
 
- 	function testViewMultiedit(){
+ 	function testViewMultiedit() {
  		
  		//execute the contructor and check for the Object type and type attribute
  		$view = new ViewMultiedit();
- 		$this->assertInstanceOf('ViewMultiedit',$view);
- 		$this->assertInstanceOf('SugarView',$view);
- 		$this->assertAttributeEquals('edit','type', $view);
+ 		$this->assertInstanceOf('ViewMultiedit', $view);
+ 		$this->assertInstanceOf('SugarView', $view);
+ 		$this->assertAttributeEquals('edit', 'type', $view);
  	}
  	
- 	function testdisplay(){
+ 	function testdisplay() {
 
 
  		//test without action value and modules list in REQUEST object
@@ -21,7 +21,7 @@ 
  		$view->display();
  		$renderedContent = ob_get_contents();
  		ob_end_clean();
- 		$this->assertEquals(0,strlen($renderedContent));
+ 		$this->assertEquals(0, strlen($renderedContent));
  		
 
  		
@@ -30,12 +30,12 @@ 
  		$view->action = 'AjaxFormSave';
  		$view->module = 'Users';
  		$view->bean = new User();
- 		$view->bean->id =1;
+ 		$view->bean->id = 1;
  		ob_start(); 		
  		$view->display(); 		
  		$renderedContent = ob_get_contents();
  		ob_end_clean(); 		
- 		$this->assertGreaterThan(0,strlen($renderedContent));
+ 		$this->assertGreaterThan(0, strlen($renderedContent));
  			
  		
  		//Fails with a fatal error, method creates editview without properly setting it up causing fatal errors.