View GitHub Repository deploynaut

Schedule Inspection
Subscribe
Completed
Push — master ( 5f85fd...06b769 )
by Sean
05:34
created
Status
Category
Toggle Patches
code/control/DNDeploymentDispatcher.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch 
@@ -19,7 +19,7 @@  discard block
 
 block discarded – undo
19 19
20 20 
 	public function getModel($name) {
21 21 
 		$deployment = $this->getDeployment();
22 
-		if (!$deployment) {
22 
+		if(!$deployment) {
23 23 
 			return [];
24 24 
 		}
25 25 
 
@@ -31,7 +31,7 @@  discard block
 
 block discarded – undo
31 31 
 	public function getDeployment() {
32 32 
 		$id = $this->request->param('Id');
33 33 
 		$deployment = DNDeployment::get()->byId($id);
34 
-		if (!$deployment || !$deployment->exists()) {
34 
+		if(!$deployment || !$deployment->exists()) {
35 35 
 			return false;
36 36 
 		}
37 37 
 		return $deployment;
 
@@ -40,7 +40,7 @@  discard block
 
 block discarded – undo
40 40 
 	public function apply(SS_HTTPRequest $request) {
41 41 
 		$this->checkSecurityToken();
42 42 
 		$response = $this->checkRequest();
43 
-		if ($response instanceof SS_HTTPResponse) {
43 
+		if($response instanceof SS_HTTPResponse) {
44 44 
 			return $response;
45 45 
 		}
46 46 
 
@@ -54,7 +54,7 @@  discard block
 
 block discarded – undo
54 54 
 	public function canApply(SS_HTTPRequest $request) {
55 55 
 		$this->checkSecurityToken();
56 56 
 		$response = $this->checkRequest();
57 
-		if ($response instanceof SS_HTTPResponse) {
57 
+		if($response instanceof SS_HTTPResponse) {
58 58 
 			return $response;
59 59 
 		}
60 60 
 
@@ -75,7 +75,7 @@  discard block
 
 block discarded – undo
75 75 
 		}
76 76
77 77 
 		$deployment = $this->getDeployment();
78 
-		if (!$deployment) {
78 
+		if(!$deployment) {
79 79 
 			return new SS_HTTPResponse('Deployment not found', 404);
80 80 
 		}
81 81 
 
@@ -84,7 +84,7 @@  discard block
 
 block discarded – undo
84 84
85 85 
 	protected function getMachine() {
86 86 
 		$deployment = $this->getDeployment();
87 
-		if (!$deployment) {
87 
+		if(!$deployment) {
88 88 
 			return null;
89 89 
 		}
90 90 
 
@@ -96,10 +96,10 @@  discard block
 
 block discarded – undo
96 96 
 			// It gives a chance to perform a dry-run, and reject the transition, for example
97 97 
 			// in case the user is unauthorised.
98 98 
 			'finite.test_transition',
99 
-			function (Finite\Event\TransitionEvent $e) use ($project) {
99 
+			function(Finite\Event\TransitionEvent $e) use ($project) {
100 100 
 				$code = null;
101 101
102 
-				switch ($e->getTransition()->getName()) {
102 
+				switch($e->getTransition()->getName()) {
103 103 
 				case 'approve':
104 104 
 					$code = self::ALLOW_DNDEPLOYMENT_APPROVE;
105 105 
 					break;
 
@@ -108,7 +108,7 @@  discard block
 
 block discarded – undo
108 108 
 					break;
109 109 
 				}
110 110
111 
-				if (!$code || !$project->allowed($code)) {
111 
+				if(!$code || !$project->allowed($code)) {
112 112 
 					$e->reject();
113 113 
 				}
114 114 
 			}
Please login to merge, or discard this patch.