@@ -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 | { |
@@ -1,12 +1,12 @@ |
||
1 | 1 | <?php |
2 | 2 | /** |
3 | - * @package: chapi |
|
4 | - * |
|
5 | - * @author: msiebeneicher |
|
6 | - * @since: 2015-09-25 |
|
7 | - * |
|
8 | - * @link: https://github.com/msiebeneicher/chapi/issues/36 |
|
9 | - */ |
|
3 | + * @package: chapi |
|
4 | + * |
|
5 | + * @author: msiebeneicher |
|
6 | + * @since: 2015-09-25 |
|
7 | + * |
|
8 | + * @link: https://github.com/msiebeneicher/chapi/issues/36 |
|
9 | + */ |
|
10 | 10 | |
11 | 11 | |
12 | 12 | namespace Chapi\Entity\DatePeriod; |
@@ -1,11 +1,11 @@ |
||
1 | 1 | <?php |
2 | 2 | /** |
3 | - * @package: Chapi |
|
4 | - * |
|
5 | - * @author: msiebeneicher |
|
6 | - * @since: 2015-07-28 |
|
7 | - * |
|
8 | - */ |
|
3 | + * @package: Chapi |
|
4 | + * |
|
5 | + * @author: msiebeneicher |
|
6 | + * @since: 2015-07-28 |
|
7 | + * |
|
8 | + */ |
|
9 | 9 | namespace Chapi; |
10 | 10 | |
11 | 11 | use Symfony\Component\Console\Application; |
@@ -1,11 +1,11 @@ discard block |
||
1 | 1 | <?php |
2 | 2 | /** |
3 | - * @package: chapi |
|
4 | - * |
|
5 | - * @author: msiebeneicher |
|
6 | - * @since: 2015-07-21 |
|
7 | - * |
|
8 | - */ |
|
3 | + * @package: chapi |
|
4 | + * |
|
5 | + * @author: msiebeneicher |
|
6 | + * @since: 2015-07-21 |
|
7 | + * |
|
8 | + */ |
|
9 | 9 | |
10 | 10 | |
11 | 11 | namespace Chapi\Commands; |
@@ -44,22 +44,22 @@ discard block |
||
44 | 44 | private static $sHomeDir = ''; |
45 | 45 | |
46 | 46 | /** |
47 | - * Executes the current command. |
|
48 | - * |
|
49 | - * This method is not abstract because you can use this class |
|
50 | - * as a concrete class. In this case, instead of defining the |
|
51 | - * execute() method, you set the code to execute by passing |
|
52 | - * a Closure to the setCode() method. |
|
53 | - * |
|
54 | - * @param InputInterface $oInput An InputInterface instance |
|
55 | - * @param OutputInterface $oOutput An OutputInterface instance |
|
56 | - * |
|
57 | - * @return integer null or 0 if everything went fine, or an error code |
|
58 | - * |
|
59 | - * @throws \LogicException When this abstract method is not implemented |
|
60 | - * |
|
61 | - * @see setCode() |
|
62 | - */ |
|
47 | + * Executes the current command. |
|
48 | + * |
|
49 | + * This method is not abstract because you can use this class |
|
50 | + * as a concrete class. In this case, instead of defining the |
|
51 | + * execute() method, you set the code to execute by passing |
|
52 | + * a Closure to the setCode() method. |
|
53 | + * |
|
54 | + * @param InputInterface $oInput An InputInterface instance |
|
55 | + * @param OutputInterface $oOutput An OutputInterface instance |
|
56 | + * |
|
57 | + * @return integer null or 0 if everything went fine, or an error code |
|
58 | + * |
|
59 | + * @throws \LogicException When this abstract method is not implemented |
|
60 | + * |
|
61 | + * @see setCode() |
|
62 | + */ |
|
63 | 63 | protected function execute(InputInterface $oInput, OutputInterface $oOutput) |
64 | 64 | { |
65 | 65 | $this->oInput = $oInput; |
@@ -125,12 +125,14 @@ |
||
125 | 125 | if ( |
126 | 126 | !file_exists($this->getHomeDir() . DIRECTORY_SEPARATOR . 'parameters.yml') |
127 | 127 | AND !file_exists(getcwd() . DIRECTORY_SEPARATOR . '.chapiconfig') |
128 | - ) // one file have to exist |
|
128 | + ) { |
|
129 | + // one file have to exist |
|
129 | 130 | { |
130 | 131 | $this->oOutput->writeln(sprintf( |
131 | 132 | '<error>%s</error>', |
132 | 133 | 'No parameter file found. Please run "configure" command for initial setup or add a local `.chapiconfig` to your working directory.' |
133 | 134 | )); |
135 | + } |
|
134 | 136 | return false; |
135 | 137 | } |
136 | 138 |
@@ -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 | } |
@@ -1,11 +1,11 @@ |
||
1 | 1 | <?php |
2 | 2 | /** |
3 | - * @package: chapi |
|
4 | - * |
|
5 | - * @author: msiebeneicher |
|
6 | - * @since: 2015-07-30 |
|
7 | - * |
|
8 | - */ |
|
3 | + * @package: chapi |
|
4 | + * |
|
5 | + * @author: msiebeneicher |
|
6 | + * @since: 2015-07-30 |
|
7 | + * |
|
8 | + */ |
|
9 | 9 | namespace Chapi\Component\Comparison; |
10 | 10 | |
11 | 11 |
@@ -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' |
@@ -1,11 +1,11 @@ |
||
1 | 1 | <?php |
2 | 2 | /** |
3 | - * @package: chapi |
|
4 | - * |
|
5 | - * @author: msiebeneicher |
|
6 | - * @since: 2015-07-30 |
|
7 | - * |
|
8 | - */ |
|
3 | + * @package: chapi |
|
4 | + * |
|
5 | + * @author: msiebeneicher |
|
6 | + * @since: 2015-07-30 |
|
7 | + * |
|
8 | + */ |
|
9 | 9 | namespace Chapi\Component\Comparison; |
10 | 10 | |
11 | 11 |
@@ -1,11 +1,11 @@ |
||
1 | 1 | <?php |
2 | 2 | /** |
3 | - * @package: chapi |
|
4 | - * |
|
5 | - * @author: msiebeneicher |
|
6 | - * @since: 2015-08-02 |
|
7 | - * |
|
8 | - */ |
|
3 | + * @package: chapi |
|
4 | + * |
|
5 | + * @author: msiebeneicher |
|
6 | + * @since: 2015-08-02 |
|
7 | + * |
|
8 | + */ |
|
9 | 9 | |
10 | 10 | |
11 | 11 | namespace Chapi\Commands; |
@@ -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 | } |
@@ -1,11 +1,11 @@ |
||
1 | 1 | <?php |
2 | 2 | /** |
3 | - * @package: chapi |
|
4 | - * |
|
5 | - * @author: msiebeneicher |
|
6 | - * @since: 2015-07-31 |
|
7 | - * |
|
8 | - */ |
|
3 | + * @package: chapi |
|
4 | + * |
|
5 | + * @author: msiebeneicher |
|
6 | + * @since: 2015-07-31 |
|
7 | + * |
|
8 | + */ |
|
9 | 9 | |
10 | 10 | |
11 | 11 | namespace Chapi\Service\JobRepository; |
@@ -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 | } |
@@ -1,12 +1,12 @@ |
||
1 | 1 | <?php |
2 | 2 | /** |
3 | - * @package: chapi |
|
4 | - * |
|
5 | - * @author: msiebeneicher |
|
6 | - * @since: 2015-09-09 |
|
7 | - * |
|
8 | - * @link: https://github.com/msiebeneicher/chapi/issues/24 |
|
9 | - */ |
|
3 | + * @package: chapi |
|
4 | + * |
|
5 | + * @author: msiebeneicher |
|
6 | + * @since: 2015-09-09 |
|
7 | + * |
|
8 | + * @link: https://github.com/msiebeneicher/chapi/issues/24 |
|
9 | + */ |
|
10 | 10 | |
11 | 11 | namespace Chapi\Service\Chronos; |
12 | 12 |
@@ -1,12 +1,12 @@ |
||
1 | 1 | <?php |
2 | 2 | /** |
3 | - * @package: chapi |
|
4 | - * |
|
5 | - * @author: msiebeneicher |
|
6 | - * @since: 2015-09-20 |
|
7 | - * |
|
8 | - * @link: http:// |
|
9 | - */ |
|
3 | + * @package: chapi |
|
4 | + * |
|
5 | + * @author: msiebeneicher |
|
6 | + * @since: 2015-09-20 |
|
7 | + * |
|
8 | + * @link: http:// |
|
9 | + */ |
|
10 | 10 | |
11 | 11 | namespace Chapi\Commands; |
12 | 12 |