@@ -47,7 +47,7 @@ |
||
| 47 | 47 | } |
| 48 | 48 | |
| 49 | 49 | /** |
| 50 | - * @return array |
|
| 50 | + * @return string[] |
|
| 51 | 51 | */ |
| 52 | 52 | private function parseIsoString() |
| 53 | 53 | { |
@@ -317,8 +317,7 @@ |
||
| 317 | 317 | { |
| 318 | 318 | $_oDateTime = new \DateTime(str_replace('Z', '', $_oIso8601Entity->sStartTime)); |
| 319 | 319 | $_oDateTime->setTimezone(new \DateTimeZone($sTimeZone)); |
| 320 | - } |
|
| 321 | - else |
|
| 320 | + } else |
|
| 322 | 321 | { |
| 323 | 322 | $_oDateTime = new \DateTime($_oIso8601Entity->sStartTime); |
| 324 | 323 | } |
@@ -85,8 +85,7 @@ discard block |
||
| 85 | 85 | 'Job "%s" successfully added to chronos', |
| 86 | 86 | $_sJobName |
| 87 | 87 | )); |
| 88 | - } |
|
| 89 | - else |
|
| 88 | + } else |
|
| 90 | 89 | { |
| 91 | 90 | $this->oLogger->error(sprintf( |
| 92 | 91 | 'Failed to add job "%s" to chronos', |
@@ -109,8 +108,7 @@ discard block |
||
| 109 | 108 | 'Job "%s" successfully removed from chronos', |
| 110 | 109 | $_sJobName |
| 111 | 110 | )); |
| 112 | - } |
|
| 113 | - else |
|
| 111 | + } else |
|
| 114 | 112 | { |
| 115 | 113 | $this->oLogger->error(sprintf( |
| 116 | 114 | 'Failed to remove job "%s" from chronos', |
@@ -134,8 +132,7 @@ discard block |
||
| 134 | 132 | 'Job "%s" successfully updated in chronos', |
| 135 | 133 | $_sJobName |
| 136 | 134 | )); |
| 137 | - } |
|
| 138 | - else |
|
| 135 | + } else |
|
| 139 | 136 | { |
| 140 | 137 | $this->oLogger->error(sprintf( |
| 141 | 138 | 'Failed to update job "%s" in chronos', |
@@ -154,8 +151,7 @@ discard block |
||
| 154 | 151 | if (empty($aJobNames)) |
| 155 | 152 | { |
| 156 | 153 | $_aChronosJobs = $this->oJobRepositoryChronos->getJobs(); |
| 157 | - } |
|
| 158 | - else |
|
| 154 | + } else |
|
| 159 | 155 | { |
| 160 | 156 | $_aChronosJobs = []; |
| 161 | 157 | foreach ($aJobNames as $_sJobName) |
@@ -177,8 +173,7 @@ discard block |
||
| 177 | 173 | 'Job "%s" successfully stored in local repository', |
| 178 | 174 | $_oJobEntity->name |
| 179 | 175 | )); |
| 180 | - } |
|
| 181 | - else |
|
| 176 | + } else |
|
| 182 | 177 | { |
| 183 | 178 | $this->oLogger->error(sprintf( |
| 184 | 179 | 'Failed to store job "%s" in local repository', |
@@ -209,8 +204,7 @@ discard block |
||
| 209 | 204 | 'Job "%s" successfully updated in local repository', |
| 210 | 205 | $_oJobEntity->name |
| 211 | 206 | )); |
| 212 | - } |
|
| 213 | - else |
|
| 207 | + } else |
|
| 214 | 208 | { |
| 215 | 209 | $this->oLogger->error(sprintf( |
| 216 | 210 | 'Failed to update job "%s" in local repository', |
@@ -37,8 +37,7 @@ discard block |
||
| 37 | 37 | if (!empty($_sJobName)) |
| 38 | 38 | { |
| 39 | 39 | $this->printJobDiff($_sJobName); |
| 40 | - } |
|
| 41 | - else |
|
| 40 | + } else |
|
| 42 | 41 | { |
| 43 | 42 | $_aLocalJobUpdates = $_oJobComparisonBusinessCase->getLocalJobUpdates(); |
| 44 | 43 | if (!empty($_aLocalJobUpdates)) |
@@ -76,12 +75,10 @@ discard block |
||
| 76 | 75 | if ($_sDiffSign == '+') |
| 77 | 76 | { |
| 78 | 77 | $this->oOutput->writeln(sprintf("<info>%s\t%s: %s</info>", $_sDiffSign, $_sProperty, substr($_sDiffLine, 1))); |
| 79 | - } |
|
| 80 | - elseif ($_sDiffSign == '-') |
|
| 78 | + } elseif ($_sDiffSign == '-') |
|
| 81 | 79 | { |
| 82 | 80 | $this->oOutput->writeln(sprintf("<fg=red>%s\t%s: %s</>", $_sDiffSign, $_sProperty, substr($_sDiffLine, 1))); |
| 83 | - } |
|
| 84 | - else |
|
| 81 | + } else |
|
| 85 | 82 | { |
| 86 | 83 | $this->oOutput->writeln(sprintf("\t%s: %s", $_sProperty, $_sDiffLine)); |
| 87 | 84 | } |
@@ -49,8 +49,7 @@ |
||
| 49 | 49 | $_mValue = (!empty($_mValue)) |
| 50 | 50 | ? '[ ' . implode(', ', $_mValue) . ' ]' |
| 51 | 51 | : '[ ]'; |
| 52 | - } |
|
| 53 | - elseif (is_bool($_mValue)) |
|
| 52 | + } elseif (is_bool($_mValue)) |
|
| 54 | 53 | { |
| 55 | 54 | $_mValue = (true === $_mValue) |
| 56 | 55 | ? 'true' |
@@ -50,12 +50,10 @@ |
||
| 50 | 50 | { |
| 51 | 51 | |
| 52 | 52 | $this->oOutput->writeln(sprintf("\t<fg=red>%s</>", $_oJobEntity->name)); |
| 53 | - } |
|
| 54 | - elseif ($_oJobEntity->errorCount > 0) |
|
| 53 | + } elseif ($_oJobEntity->errorCount > 0) |
|
| 55 | 54 | { |
| 56 | 55 | $this->oOutput->writeln(sprintf("\t<comment>%s</comment>", $_oJobEntity->name)); |
| 57 | - } |
|
| 58 | - else |
|
| 56 | + } else |
|
| 59 | 57 | { |
| 60 | 58 | $this->oOutput->writeln(sprintf("\t<info>%s</info>", $_oJobEntity->name)); |
| 61 | 59 | } |
@@ -49,8 +49,7 @@ |
||
| 49 | 49 | if (JobUtils::isWildcard($aJobNames)) |
| 50 | 50 | { |
| 51 | 51 | $_oJobIndexService->resetJobIndex(); |
| 52 | - } |
|
| 53 | - else |
|
| 52 | + } else |
|
| 54 | 53 | { |
| 55 | 54 | $_oJobIndexService->removeJobs($aJobNames); |
| 56 | 55 | } |
@@ -250,13 +250,11 @@ discard block |
||
| 250 | 250 | { |
| 251 | 251 | $_sTimeline .= $this->parseTimeLineStrMark($_bPrintJobStart, $_bPrintJobEnd, $_bHasToCloseFinalTag, $_oTime->format('H>i'), $_oTime->format('H:i')); |
| 252 | 252 | |
| 253 | - } |
|
| 254 | - else |
|
| 253 | + } else |
|
| 255 | 254 | { |
| 256 | 255 | $_sTimeline .= $this->parseTimeLineStrMark($_bPrintJobStart, $_bPrintJobEnd, $_bHasToCloseFinalTag, str_repeat('>', 5), str_repeat($_sSpacer, 5)); |
| 257 | 256 | } |
| 258 | - } |
|
| 259 | - else |
|
| 257 | + } else |
|
| 260 | 258 | { |
| 261 | 259 | $_sTimeline .= $this->parseTimeLineStrMark($_bPrintJobStart, $_bPrintJobEnd, $_bHasToCloseFinalTag, '>', $_sSpacer); |
| 262 | 260 | } |
@@ -284,18 +282,15 @@ discard block |
||
| 284 | 282 | if ($bPrintJobStart && $bPrintJobEnd) |
| 285 | 283 | { |
| 286 | 284 | $_sTimelineSnippet = sprintf('<comment>%s</comment>', $sStartStopMark); |
| 287 | - } |
|
| 288 | - elseif ($bPrintJobStart) |
|
| 285 | + } elseif ($bPrintJobStart) |
|
| 289 | 286 | { |
| 290 | 287 | $_sTimelineSnippet = sprintf('<comment>%s', $sStartStopMark); |
| 291 | 288 | $bHasToCloseFinalTag = true; |
| 292 | - } |
|
| 293 | - elseif ($bPrintJobEnd) |
|
| 289 | + } elseif ($bPrintJobEnd) |
|
| 294 | 290 | { |
| 295 | 291 | $_sTimelineSnippet = sprintf('%s</comment>', $sStartStopMark); |
| 296 | 292 | $bHasToCloseFinalTag = false; |
| 297 | - } |
|
| 298 | - else |
|
| 293 | + } else |
|
| 299 | 294 | { |
| 300 | 295 | $_sTimelineSnippet = $sSpacer; |
| 301 | 296 | } |
@@ -115,8 +115,7 @@ |
||
| 115 | 115 | { |
| 116 | 116 | $_aFilteredJobList[] = $_sJobName; |
| 117 | 117 | } |
| 118 | - } |
|
| 119 | - else |
|
| 118 | + } else |
|
| 120 | 119 | { |
| 121 | 120 | if (!$this->oJobIndexService->isJobInIndex($_sJobName)) |
| 122 | 121 | { |