@@ -48,7 +48,8 @@ |
||
48 | 48 | |
49 | 49 | if (JobUtils::isWildcard($jobNames)) { |
50 | 50 | $jobIndexService->resetJobIndex(); |
51 | - } else { |
|
51 | + } |
|
52 | + else { |
|
52 | 53 | $jobIndexService->removeJobs($jobNames); |
53 | 54 | } |
54 | 55 | } |
@@ -53,7 +53,8 @@ |
||
53 | 53 | $value = (!empty($value)) |
54 | 54 | ? json_encode($value, JSON_PRETTY_PRINT | JSON_UNESCAPED_SLASHES) |
55 | 55 | : $emptyString; |
56 | - } elseif (is_bool($value)) { |
|
56 | + } |
|
57 | + elseif (is_bool($value)) { |
|
57 | 58 | $value = (true === $value) |
58 | 59 | ? 'true' |
59 | 60 | : 'false'; |
@@ -91,12 +91,14 @@ discard block |
||
91 | 91 | if (property_exists($this, $key)) { |
92 | 92 | if ($key == 'container') { |
93 | 93 | $this->{$key} = new ContainerEntity($value); |
94 | - } else { |
|
94 | + } |
|
95 | + else { |
|
95 | 96 | $this->{$key} = $value; |
96 | 97 | } |
97 | 98 | } |
98 | 99 | } |
99 | - } else { |
|
100 | + } |
|
101 | + else { |
|
100 | 102 | throw new \InvalidArgumentException(sprintf('Argument 1 passed to "%s" must be an array or object', __METHOD__)); |
101 | 103 | } |
102 | 104 | } |
@@ -141,7 +143,8 @@ discard block |
||
141 | 143 | $return = (array) $this; |
142 | 144 | if (!empty($this->schedule)) { |
143 | 145 | unset($return['parents']); |
144 | - } else { |
|
146 | + } |
|
147 | + else { |
|
145 | 148 | unset($return['schedule']); |
146 | 149 | unset($return['scheduleTimeZone']); |
147 | 150 | } |
@@ -22,7 +22,8 @@ |
||
22 | 22 | $this->{$key} = $value; |
23 | 23 | } |
24 | 24 | } |
25 | - } else { |
|
25 | + } |
|
26 | + else { |
|
26 | 27 | throw new \InvalidArgumentException(sprintf('Argument 1 passed to "%s" must be an array or object', __METHOD__)); |
27 | 28 | } |
28 | 29 | } |
@@ -21,17 +21,20 @@ |
||
21 | 21 | if (property_exists($this, $key)) { |
22 | 22 | if ($key == 'type') { |
23 | 23 | $this->{$key} = strtolower($value); |
24 | - } elseif ($key == 'volumes') { |
|
24 | + } |
|
25 | + elseif ($key == 'volumes') { |
|
25 | 26 | foreach ($value as $valueVolume) { |
26 | 27 | $volume = new ContainerVolumeEntity($valueVolume); |
27 | 28 | $this->volumes[] = $volume; |
28 | 29 | } |
29 | - } else { |
|
30 | + } |
|
31 | + else { |
|
30 | 32 | $this->{$key} = $value; |
31 | 33 | } |
32 | 34 | } |
33 | 35 | } |
34 | - } else { |
|
36 | + } |
|
37 | + else { |
|
35 | 38 | throw new \InvalidArgumentException(sprintf('Argument 1 passed to "%s" must be an array or object', __METHOD__)); |
36 | 39 | } |
37 | 40 | } |
@@ -26,7 +26,8 @@ |
||
26 | 26 | MarathonEntityUtils::setAllPossibleProperties((array) $data, $this); |
27 | 27 | if (isset($data['labels'])) { |
28 | 28 | $this->labels = (object) $data['labels']; |
29 | - } else { |
|
29 | + } |
|
30 | + else { |
|
30 | 31 | $this->labels = (object) []; |
31 | 32 | } |
32 | 33 | } |
@@ -114,7 +114,8 @@ discard block |
||
114 | 114 | |
115 | 115 | if (isset($dataArray['upgradeStrategy'])) { |
116 | 116 | $this->upgradeStrategy = new UpgradeStrategy((array) $dataArray['upgradeStrategy']); |
117 | - } else { |
|
117 | + } |
|
118 | + else { |
|
118 | 119 | $this->upgradeStrategy = new UpgradeStrategy(); |
119 | 120 | } |
120 | 121 | |
@@ -129,13 +130,15 @@ discard block |
||
129 | 130 | ksort($env); |
130 | 131 | |
131 | 132 | $this->env = (object) $env; |
132 | - } else { |
|
133 | + } |
|
134 | + else { |
|
133 | 135 | $this->env = (object) []; |
134 | 136 | } |
135 | 137 | |
136 | 138 | if (isset($dataArray['labels'])) { |
137 | 139 | $this->labels = (object) $dataArray['labels']; |
138 | - } else { |
|
140 | + } |
|
141 | + else { |
|
139 | 142 | $this->labels = (object) []; |
140 | 143 | } |
141 | 144 | MarathonEntityUtils::setPropertyIfExist($dataArray, $this, 'constraints'); |
@@ -36,7 +36,8 @@ discard block |
||
36 | 36 | |
37 | 37 | if (!empty($jobName)) { |
38 | 38 | $this->printJobDiff($jobName); |
39 | - } else { |
|
39 | + } |
|
40 | + else { |
|
40 | 41 | $localJobUpdates = $jobComparisonBusinessCase->getLocalJobUpdates(); |
41 | 42 | if (!empty($localJobUpdates)) { |
42 | 43 | foreach ($localJobUpdates as $jobName) { |
@@ -79,9 +80,11 @@ discard block |
||
79 | 80 | |
80 | 81 | if ($diffSign == '+') { |
81 | 82 | $this->output->writeln(sprintf("<info>%s\t%s: %s</info>", $diffSign, $property, ' ' . substr($diffLine, 1))); |
82 | - } elseif ($diffSign == '-') { |
|
83 | + } |
|
84 | + elseif ($diffSign == '-') { |
|
83 | 85 | $this->output->writeln(sprintf("<fg=red>%s\t%s: %s</>", $diffSign, $property, ' ' . substr($diffLine, 1))); |
84 | - } else { |
|
86 | + } |
|
87 | + else { |
|
85 | 88 | $this->output->writeln(sprintf(" \t%s: %s", $property, $diffLine)); |
86 | 89 | } |
87 | 90 | } |