@@ -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; |
@@ -84,7 +84,7 @@ discard block |
||
84 | 84 | } |
85 | 85 | |
86 | 86 | // check if dependency is satisfied |
87 | - if ( $_oJobEntityLocal->isDependencyJob()) |
|
87 | + if ($_oJobEntityLocal->isDependencyJob()) |
|
88 | 88 | { |
89 | 89 | try { |
90 | 90 | $circular = $this->isDependencyCircular($_oJobEntityLocal, count($_oJobEntityLocal->dependencies)); |
@@ -96,10 +96,10 @@ discard block |
||
96 | 96 | return false; |
97 | 97 | } |
98 | 98 | } |
99 | - catch(\Exception $e) |
|
99 | + catch (\Exception $e) |
|
100 | 100 | { |
101 | 101 | $this->oLogger->error(sprintf( |
102 | - 'Job %s cannot be added to remote : %s',$sAppId, $e->getMessage() |
|
102 | + 'Job %s cannot be added to remote : %s', $sAppId, $e->getMessage() |
|
103 | 103 | )); |
104 | 104 | return false; |
105 | 105 | } |
@@ -157,7 +157,7 @@ discard block |
||
157 | 157 | * @return bool |
158 | 158 | * @throws \Exception |
159 | 159 | */ |
160 | - private function isDependencyCircular(MarathonAppEntity $oEntity, $iImmediateChildren, &$path=[]) |
|
160 | + private function isDependencyCircular(MarathonAppEntity $oEntity, $iImmediateChildren, &$path = []) |
|
161 | 161 | { |
162 | 162 | // Invariant: path will not have duplicates for acyclic dependency tree |
163 | 163 | if ($this->hasDuplicates($path)) |
@@ -211,7 +211,7 @@ discard block |
||
211 | 211 | // for B intermediate Child will be 2. |
212 | 212 | // when we process D, it will be reduced to 1 and with C to 0 |
213 | 213 | // then we will pop B to generate path [A, E] when we reach E. |
214 | - $iImmediateChildren = $iImmediateChildren -1; |
|
214 | + $iImmediateChildren = $iImmediateChildren - 1; |
|
215 | 215 | if ($iImmediateChildren == 0) |
216 | 216 | { |
217 | 217 | array_pop($path); |
@@ -219,7 +219,7 @@ |
||
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 |