Completed
Push — scrutinizer ( 84e9d0...6a9613 )
by Fabio
22:48
created
tests/simple_unit/ActiveRecord/BaseActiveRecordTestCase.php 1 patch
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -22,8 +22,7 @@
 block discarded – undo
22 22
 		{
23 23
 			$obj->save();
24 24
 			$this->fail();
25
-		}
26
-		catch(TActiveRecordException $e)
25
+		} catch(TActiveRecordException $e)
27 26
 		{
28 27
 			$this->pass();
29 28
 		}
Please login to merge, or discard this patch.
tests/simple_unit/ActiveRecord/ViewRecordTestCase.php 1 patch
Braces   +4 added lines, -8 removed lines patch added patch discarded remove patch
@@ -25,8 +25,7 @@  discard block
 block discarded – undo
25 25
 		{
26 26
 			$user->save();
27 27
 			$this->fail();
28
-		}
29
-		catch(TActiveRecordException $e)
28
+		} catch(TActiveRecordException $e)
30 29
 		{
31 30
 			$this->pass();
32 31
 		}
@@ -40,8 +39,7 @@  discard block
 block discarded – undo
40 39
 		{
41 40
 			$user->save();
42 41
 			$this->fail();
43
-		}
44
-		catch(TActiveRecordException $e)
42
+		} catch(TActiveRecordException $e)
45 43
 		{
46 44
 			$this->pass();
47 45
 		}
@@ -53,8 +51,7 @@  discard block
 block discarded – undo
53 51
 		{
54 52
 			$user = SimpleUser::finder()->findByPk('admin');
55 53
 			$this->fail();
56
-		}
57
-		catch(TDbException $e)
54
+		} catch(TDbException $e)
58 55
 		{
59 56
 			$this->pass();
60 57
 		}
@@ -66,8 +63,7 @@  discard block
 block discarded – undo
66 63
 		{
67 64
 			SimpleUser::finder()->deleteByPk('admin');
68 65
 			$this->fail();
69
-		}
70
-		catch(TDbException $e)
66
+		} catch(TDbException $e)
71 67
 		{
72 68
 			$this->pass();
73 69
 		}
Please login to merge, or discard this patch.
tests/simple_unit/ActiveRecord/ActiveRecordMySql5TestCase.php 1 patch
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -38,8 +38,9 @@
 block discarded – undo
38 38
 	function assertSameBlog($check, $blog)
39 39
 	{
40 40
 		$props = array('blog_id', 'blog_name', 'blog_author');
41
-		foreach($props as $prop)
42
-			$this->assertEqual($check->{$prop}, $blog->{$prop});
41
+		foreach($props as $prop) {
42
+					$this->assertEqual($check->{$prop}, $blog->{$prop});
43
+		}
43 44
 	}
44 45
 
45 46
 }
46 47
\ No newline at end of file
Please login to merge, or discard this patch.
tests/simple_unit/ActiveRecord/ForeignObjectUpdateTest.php 1 patch
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -68,8 +68,7 @@
 block discarded – undo
68 68
 		{
69 69
 			//lazy load the skill records
70 70
 			$this->setSkills($this->withSkills()->findByPk($this->player_id)->skills);
71
-		}
72
-		else if($this->_skills===null)
71
+		} else if($this->_skills===null)
73 72
 		{
74 73
 			//create new TList;
75 74
 			$this->setSkills(new TList());
Please login to merge, or discard this patch.
tests/simple_unit/Soap/SoapTestCase.php 1 patch
Braces   +2 added lines, -4 removed lines patch added patch discarded remove patch
@@ -37,8 +37,7 @@  discard block
 block discarded – undo
37 37
 		{
38 38
 			$result = $this->getClient()->getContact(1);
39 39
 			$this->fail();
40
-		}
41
-		catch (SoapFault $f)
40
+		} catch (SoapFault $f)
42 41
 		{
43 42
 			$this->pass();
44 43
 		}
@@ -80,8 +79,7 @@  discard block
 block discarded – undo
80 79
 		{
81 80
 			$this->getClient()->test();
82 81
 			$this->fail();
83
-		}
84
-		catch (SoapFault $f)
82
+		} catch (SoapFault $f)
85 83
 		{
86 84
 			$this->pass();
87 85
 		}
Please login to merge, or discard this patch.
tests/FunctionalTests/active-controlstests/ActiveRatingListTestCase.php 1 patch
Braces   +5 added lines, -4 removed lines patch added patch discarded remove patch
@@ -251,10 +251,11 @@
 block discarded – undo
251 251
 	{
252 252
 		for($i = 0; $i < $total; $i++)
253 253
 		{
254
-			if(in_array($i, $checks))
255
-				$this->assertTrue($this->byId("{$clientID}_c{$i}")->selected());
256
-			else
257
-				$this->assertFalse($this->byId("{$clientID}_c{$i}")->selected());
254
+			if(in_array($i, $checks)) {
255
+							$this->assertTrue($this->byId("{$clientID}_c{$i}")->selected());
256
+			} else {
257
+							$this->assertFalse($this->byId("{$clientID}_c{$i}")->selected());
258
+			}
258 259
 		}
259 260
 	}
260 261
 }
261 262
\ No newline at end of file
Please login to merge, or discard this patch.
tests/FunctionalTests/tickets/protected/pages/Ticket225.php 1 patch
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -10,8 +10,9 @@
 block discarded – undo
10 10
 	private function getGroupIDs($radio)
11 11
 	{
12 12
 		$ids = '';
13
-		foreach($radio->getRadioButtonsInGroup() as $control)
14
-			$ids .= " ".$control->getUniqueID();
13
+		foreach($radio->getRadioButtonsInGroup() as $control) {
14
+					$ids .= " ".$control->getUniqueID();
15
+		}
15 16
 		return $ids;
16 17
 	}
17 18
 }
Please login to merge, or discard this patch.
tests/FunctionalTests/tickets/protected/pages/Ticket585.php 1 patch
Braces   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -8,9 +8,9 @@
 block discarded – undo
8 8
 		if ($param->Value && date('d-m-Y', $param->Value) == "15-03-2007")
9 9
 		{
10 10
 			$param->IsValid=false;
11
+		} else {
12
+					$param->IsValid=true;
11 13
 		}
12
-		else
13
-			$param->IsValid=true;
14 14
 	}
15 15
 
16 16
 }
Please login to merge, or discard this patch.
tests/FunctionalTests/tickets/protected/pages/Ticket191.php 1 patch
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -4,8 +4,9 @@
 block discarded – undo
4 4
 {
5 5
 	public function buttonClicked($sender,$param)
6 6
 	{
7
-		if($this->IsValid)
8
-			$this->Application->clearGlobalState('ticket190');
7
+		if($this->IsValid) {
8
+					$this->Application->clearGlobalState('ticket190');
9
+		}
9 10
 	}
10 11
 
11 12
 	public function customValidation($sender,$param)
Please login to merge, or discard this patch.