@@ -52,7 +52,8 @@ |
||
| 52 | 52 | |
| 53 | 53 | if (!empty($timezone)) { |
| 54 | 54 | $dateStart = new \DateTime(str_replace('Z', '', $iso8601Entity->startTime), new \DateTimeZone($timezone)); |
| 55 | - } else { |
|
| 55 | + } |
|
| 56 | + else { |
|
| 56 | 57 | // todo: use a defined chronos time zone here? |
| 57 | 58 | $dateStart = new \DateTime($iso8601Entity->startTime); |
| 58 | 59 | } |
@@ -237,10 +237,12 @@ discard block |
||
| 237 | 237 | if ($mod == 0) { |
| 238 | 238 | if ($printTime) { |
| 239 | 239 | $timeline .= $this->parseTimeLineStrMark($printJobStart, $printJobEnd, $hasToCloseFinalTag, $time->format('H>i'), $time->format('H:i')); |
| 240 | - } else { |
|
| 240 | + } |
|
| 241 | + else { |
|
| 241 | 242 | $timeline .= $this->parseTimeLineStrMark($printJobStart, $printJobEnd, $hasToCloseFinalTag, str_repeat('>', 5), str_repeat($spacer, 5)); |
| 242 | 243 | } |
| 243 | - } else { |
|
| 244 | + } |
|
| 245 | + else { |
|
| 244 | 246 | $timeline .= $this->parseTimeLineStrMark($printJobStart, $printJobEnd, $hasToCloseFinalTag, '>', $spacer); |
| 245 | 247 | } |
| 246 | 248 | } |
@@ -265,13 +267,16 @@ discard block |
||
| 265 | 267 | { |
| 266 | 268 | if ($printJobStart && $printJobEnd) { |
| 267 | 269 | $timelineSnippet = sprintf('<comment>%s</comment>', $startStopMark); |
| 268 | - } elseif ($printJobStart) { |
|
| 270 | + } |
|
| 271 | + elseif ($printJobStart) { |
|
| 269 | 272 | $timelineSnippet = sprintf('<comment>%s', $startStopMark); |
| 270 | 273 | $bHasToCloseFinalTag = true; |
| 271 | - } elseif ($printJobEnd) { |
|
| 274 | + } |
|
| 275 | + elseif ($printJobEnd) { |
|
| 272 | 276 | $timelineSnippet = sprintf('%s</comment>', $startStopMark); |
| 273 | 277 | $bHasToCloseFinalTag = false; |
| 274 | - } else { |
|
| 278 | + } |
|
| 279 | + else { |
|
| 275 | 280 | $timelineSnippet = $spacer; |
| 276 | 281 | } |
| 277 | 282 | |
@@ -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'; |
@@ -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) { |
@@ -68,9 +69,11 @@ discard block |
||
| 68 | 69 | |
| 69 | 70 | if ($diffSign == '+') { |
| 70 | 71 | $this->output->writeln(sprintf("<info>%s\t%s: %s</info>", $diffSign, $property, substr($diffLine, 1))); |
| 71 | - } elseif ($diffSign == '-') { |
|
| 72 | + } |
|
| 73 | + elseif ($diffSign == '-') { |
|
| 72 | 74 | $this->output->writeln(sprintf("<fg=red>%s\t%s: %s</>", $diffSign, $property, substr($diffLine, 1))); |
| 73 | - } else { |
|
| 75 | + } |
|
| 76 | + else { |
|
| 74 | 77 | $this->output->writeln(sprintf("\t%s: %s", $property, $diffLine)); |
| 75 | 78 | } |
| 76 | 79 | } |
@@ -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 | } |
@@ -37,7 +37,7 @@ |
||
| 37 | 37 | public function jsonSerialize() |
| 38 | 38 | { |
| 39 | 39 | $return = (array) $this; |
| 40 | - $return = array_filter($return, function ($value, $key) { |
|
| 40 | + $return = array_filter($return, function($value, $key) { |
|
| 41 | 41 | return !is_null($value); |
| 42 | 42 | }, ARRAY_FILTER_USE_BOTH); |
| 43 | 43 | return $return; |
@@ -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 | } |