@@ -27,7 +27,7 @@ |
||
27 | 27 | */ |
28 | 28 | public static function setAllPossibleProperties($oData, $oTarget) |
29 | 29 | { |
30 | - foreach($oData as $attrName => $attrValue) |
|
30 | + foreach ($oData as $attrName => $attrValue) |
|
31 | 31 | { |
32 | 32 | // dont set array or objects. |
33 | 33 | // Because this would need further type information to properly set. |
@@ -35,14 +35,14 @@ |
||
35 | 35 | |
36 | 36 | if (isset($aData['docker'])) |
37 | 37 | { |
38 | - $this->docker = new Docker((array)$aData['docker']); |
|
38 | + $this->docker = new Docker((array) $aData['docker']); |
|
39 | 39 | } |
40 | 40 | |
41 | 41 | if (isset($aData['volumes'])) |
42 | 42 | { |
43 | 43 | foreach ($aData['volumes'] as $volume) |
44 | 44 | { |
45 | - $this->volumes[] = new ContainerVolume((array)$volume); |
|
45 | + $this->volumes[] = new ContainerVolume((array) $volume); |
|
46 | 46 | } |
47 | 47 | } |
48 | 48 |
@@ -39,17 +39,17 @@ |
||
39 | 39 | |
40 | 40 | if (isset($aData['portMappings'])) |
41 | 41 | { |
42 | - foreach($aData['portMappings'] as $portMapping) |
|
42 | + foreach ($aData['portMappings'] as $portMapping) |
|
43 | 43 | { |
44 | - $this->portMappings[] = new DockerPortMapping((array)$portMapping); |
|
44 | + $this->portMappings[] = new DockerPortMapping((array) $portMapping); |
|
45 | 45 | } |
46 | 46 | } |
47 | 47 | |
48 | 48 | if (isset($aData['parameters'])) |
49 | 49 | { |
50 | - foreach($aData['parameters'] as $parameter) |
|
50 | + foreach ($aData['parameters'] as $parameter) |
|
51 | 51 | { |
52 | - $this->parameters[] = new DockerParameters((array)$parameter); |
|
52 | + $this->parameters[] = new DockerParameters((array) $parameter); |
|
53 | 53 | } |
54 | 54 | } |
55 | 55 |
@@ -41,9 +41,9 @@ |
||
41 | 41 | { |
42 | 42 | MarathonEntityUtils::setAllPossibleProperties($aData, $this); |
43 | 43 | |
44 | - if(isset($aData['command'])) |
|
44 | + if (isset($aData['command'])) |
|
45 | 45 | { |
46 | - $this->command = new HealthCheckCommand((array)$aData['command']); |
|
46 | + $this->command = new HealthCheckCommand((array) $aData['command']); |
|
47 | 47 | } |
48 | 48 | } |
49 | 49 |
@@ -25,13 +25,13 @@ |
||
25 | 25 | { |
26 | 26 | MarathonEntityUtils::setAllPossibleProperties($aData, $this); |
27 | 27 | |
28 | - if(isset($aData['groups'])) |
|
28 | + if (isset($aData['groups'])) |
|
29 | 29 | { |
30 | 30 | $this->groups = $aData['groups']; |
31 | 31 | } |
32 | 32 | if (isset($aData['labels'])) |
33 | 33 | { |
34 | - $this->labels = (object)$aData['labels']; |
|
34 | + $this->labels = (object) $aData['labels']; |
|
35 | 35 | } |
36 | 36 | } |
37 | 37 | } |
38 | 38 | \ No newline at end of file |
@@ -37,7 +37,7 @@ |
||
37 | 37 | $this->oDiffCompare = $oDiffCompare; |
38 | 38 | } |
39 | 39 | |
40 | - protected function preCompareModifications(JobEntityInterface &$oLocalJob, JobEntityInterface &$oRemoteJob) |
|
40 | + protected function preCompareModifications(JobEntityInterface & $oLocalJob, JobEntityInterface & $oRemoteJob) |
|
41 | 41 | { |
42 | 42 | if ( |
43 | 43 | !$oLocalJob instanceof MarathonAppEntity || |
@@ -54,7 +54,7 @@ |
||
54 | 54 | } |
55 | 55 | |
56 | 56 | |
57 | - protected function preCompareModifications(JobEntityInterface &$oLocalJob, JobEntityInterface &$oRemoteJob) |
|
57 | + protected function preCompareModifications(JobEntityInterface & $oLocalJob, JobEntityInterface & $oRemoteJob) |
|
58 | 58 | { |
59 | 59 | // no modification needed |
60 | 60 | return; |
@@ -166,7 +166,7 @@ discard block |
||
166 | 166 | $_aDifferences[$_sProperty] = $this->oDiffCompare->compare( |
167 | 167 | $_oRemoteEntity->{$_sProperty}, |
168 | 168 | $_oLocalEntity->{$_sProperty} |
169 | - ) ; |
|
169 | + ); |
|
170 | 170 | } |
171 | 171 | |
172 | 172 | return $_aDifferences; |
@@ -219,7 +219,7 @@ discard block |
||
219 | 219 | * @param JobEntityInterface $oRemoteJob |
220 | 220 | * @return null |
221 | 221 | */ |
222 | - abstract protected function preCompareModifications(JobEntityInterface &$oLocalJob, JobEntityInterface &$oRemoteJob); |
|
222 | + abstract protected function preCompareModifications(JobEntityInterface & $oLocalJob, JobEntityInterface & $oRemoteJob); |
|
223 | 223 | |
224 | 224 | /** |
225 | 225 | * Gets entity for each system with defaults set |
@@ -72,7 +72,7 @@ discard block |
||
72 | 72 | } |
73 | 73 | |
74 | 74 | // check if dependency is satisfied |
75 | - if ( $_oJobEntityLocal->isDependencyJob()) |
|
75 | + if ($_oJobEntityLocal->isDependencyJob()) |
|
76 | 76 | { |
77 | 77 | try { |
78 | 78 | $circular = $this->isDependencyCircular($_oJobEntityLocal, count($_oJobEntityLocal->dependencies)); |
@@ -84,10 +84,10 @@ discard block |
||
84 | 84 | return false; |
85 | 85 | } |
86 | 86 | } |
87 | - catch(\Exception $e) |
|
87 | + catch (\Exception $e) |
|
88 | 88 | { |
89 | 89 | $this->oLogger->error(sprintf( |
90 | - 'Job %s cannot be added to remote : %s',$sAppId, $e->getMessage() |
|
90 | + 'Job %s cannot be added to remote : %s', $sAppId, $e->getMessage() |
|
91 | 91 | )); |
92 | 92 | return false; |
93 | 93 | } |
@@ -133,7 +133,7 @@ discard block |
||
133 | 133 | return !(count($arr) == count(array_unique($arr))); |
134 | 134 | } |
135 | 135 | |
136 | - private function isDependencyCircular(MarathonAppEntity $oEntity, $iImmediateChildren, &$path=[]) |
|
136 | + private function isDependencyCircular(MarathonAppEntity $oEntity, $iImmediateChildren, &$path = []) |
|
137 | 137 | { |
138 | 138 | // Invariant: path will not have duplicates for acyclic dependency tree |
139 | 139 | if ($this->hasDuplicates($path)) |
@@ -186,7 +186,7 @@ discard block |
||
186 | 186 | // for B intermediate Child will be 2. |
187 | 187 | // when we process D, it will be reduced to 1 and with C to 0 |
188 | 188 | // then we will pop B to generate path [A, E] when we reach E. |
189 | - $iImmediateChildren = $iImmediateChildren -1; |
|
189 | + $iImmediateChildren = $iImmediateChildren - 1; |
|
190 | 190 | if ($iImmediateChildren == 0) |
191 | 191 | { |
192 | 192 | array_pop($path); |