@@ -105,7 +105,7 @@ discard block |
||
105 | 105 | /** |
106 | 106 | * Handle onBefore, onAfter and name |
107 | 107 | * |
108 | - * @param mixed $offset The name of the variable |
|
108 | + * @param string $offset The name of the variable |
|
109 | 109 | * @param mixed $value The value |
110 | 110 | * |
111 | 111 | * @return void |
@@ -158,7 +158,7 @@ discard block |
||
158 | 158 | /** |
159 | 159 | * Generate name if it doesn't exist |
160 | 160 | * |
161 | - * @param mixed $offset The name of the variable |
|
161 | + * @param string $offset The name of the variable |
|
162 | 162 | * |
163 | 163 | * @return mixed |
164 | 164 | */ |
@@ -58,7 +58,7 @@ discard block |
||
58 | 58 | if (substr($method, 0, 5) === 'check' && $method[5] === strtoupper($method[5])) { |
59 | 59 | $check = substr($method, 5); |
60 | 60 | $this->checks[] = $check; |
61 | - if (method_exists($this, 'fix' . $check)) { |
|
61 | + if (method_exists($this, 'fix'.$check)) { |
|
62 | 62 | $this->fixes[] = $check; |
63 | 63 | } |
64 | 64 | } |
@@ -68,12 +68,12 @@ discard block |
||
68 | 68 | 'check' => array( |
69 | 69 | 'type' => 'array', |
70 | 70 | 'option' => true, |
71 | - 'label' => 'Only execute these checks - available checks are ' . implode(', ', $this->checks), |
|
71 | + 'label' => 'Only execute these checks - available checks are '.implode(', ', $this->checks), |
|
72 | 72 | ), |
73 | 73 | 'fix' => array( |
74 | 74 | 'type' => 'boolean|array', |
75 | 75 | 'option' => true, |
76 | - 'label' => 'Enable fixes and optionally reduce to certain fixes - available fixes are ' . implode(', ', $this->fixes), |
|
76 | + 'label' => 'Enable fixes and optionally reduce to certain fixes - available fixes are '.implode(', ', $this->fixes), |
|
77 | 77 | ), |
78 | 78 | ) + parent::configureVariables(); |
79 | 79 | } |
@@ -86,7 +86,7 @@ discard block |
||
86 | 86 | public function assemble() |
87 | 87 | { |
88 | 88 | $this->callback( |
89 | - function () { |
|
89 | + function() { |
|
90 | 90 | $fix = $this->get('fix'); |
91 | 91 | $fixes = ($fix === true) ? $this->fixes : (array) $fix; |
92 | 92 | $checks = $this->get('check') ?: $this->checks; |
@@ -120,11 +120,11 @@ discard block |
||
120 | 120 | if ($packageNames && !in_array($package->name, $packageNames, true)) { |
121 | 121 | continue; |
122 | 122 | } |
123 | - if (is_string($message = $this->{'check' . $check}($package))) { |
|
123 | + if (is_string($message = $this->{'check'.$check}($package))) { |
|
124 | 124 | if (!$packageNames && !$errors) { |
125 | 125 | $this->console->output( |
126 | 126 | str_repeat(chr(8), strlen($check)) |
127 | - . '<fg=red;bg=black>' . $check . '</>' |
|
127 | + . '<fg=red;bg=black>'.$check.'</>' |
|
128 | 128 | ); |
129 | 129 | } |
130 | 130 | $message = sprintf($message, "package <comment>$package->name</>"); |
@@ -133,7 +133,7 @@ discard block |
||
133 | 133 | if ($fix) { |
134 | 134 | $this->dontCheckCurrentPackageAgain = false; |
135 | 135 | $this->console->indent(); |
136 | - $this->{'fix' . $check}($package); |
|
136 | + $this->{'fix'.$check}($package); |
|
137 | 137 | $this->console->outdent(); |
138 | 138 | if (!$this->dontCheckCurrentPackageAgain) { |
139 | 139 | $rerunForPackages[] = $package->name; |
@@ -155,7 +155,7 @@ discard block |
||
155 | 155 | if (!$errors) { |
156 | 156 | $this->console->output( |
157 | 157 | str_repeat(chr(8), strlen($check)) |
158 | - . '<fg=green;bg=black>' . $check . '</>' |
|
158 | + . '<fg=green;bg=black>'.$check.'</>' |
|
159 | 159 | ); |
160 | 160 | } |
161 | 161 | $this->console->outdent(); |
@@ -212,7 +212,7 @@ discard block |
||
212 | 212 | $this->git('reset', $package->path); |
213 | 213 | $args = 'save -u'; |
214 | 214 | if ($message = $this->answer('Message for stash:')) { |
215 | - $args .= ' ' . escapeshellarg($message); |
|
215 | + $args .= ' '.escapeshellarg($message); |
|
216 | 216 | } |
217 | 217 | $this->git('stash', $package->path, $args); |
218 | 218 | break; |
@@ -283,7 +283,7 @@ discard block |
||
283 | 283 | case 2: |
284 | 284 | foreach ($commands as $command) { |
285 | 285 | $pck = "<comment>{$package->name}</comment>"; |
286 | - $this->console->output($msg = ucfirst($command) . "ing $pck...", false); |
|
286 | + $this->console->output($msg = ucfirst($command)."ing $pck...", false); |
|
287 | 287 | $this->git($command, $package->path, isset($options) ? $options : null); |
288 | 288 | $this->console->output( |
289 | 289 | str_repeat(chr(8), strlen(strip_tags($msg))) |
@@ -330,8 +330,8 @@ discard block |
||
330 | 330 | } |
331 | 331 | if ($package->requiredBranch && $package->requiredBranch !== $requiredBranch) { |
332 | 332 | $package->invalidRequirements = true; |
333 | - return '<error>' . array_pop($dependentPackages) |
|
334 | - . ' and ' . array_pop($dependentPackages) |
|
333 | + return '<error>'.array_pop($dependentPackages) |
|
334 | + . ' and '.array_pop($dependentPackages) |
|
335 | 335 | . ' require %s in different branches</error>'; |
336 | 336 | } |
337 | 337 | $package->requiredBranch = $requiredBranch; |
@@ -343,7 +343,7 @@ discard block |
||
343 | 343 | } |
344 | 344 | } |
345 | 345 | } |
346 | - $constraint = $package->tag ?: 'dev-' . $package->branch; |
|
346 | + $constraint = $package->tag ?: 'dev-'.$package->branch; |
|
347 | 347 | if ($package->requiredBranch && $package->requiredBranch !== $package->branch) { |
348 | 348 | return "%s is at <comment>$constraint</comment> but is required at <comment>dev-{$package->requiredBranch}</comment>"; |
349 | 349 | } |
@@ -367,8 +367,8 @@ discard block |
||
367 | 367 | if ($package->invalidRequirements) { |
368 | 368 | $this->doExit('Can not fix that', 1); |
369 | 369 | } |
370 | - $currentConstraint = $package->tag ?: 'dev-' . $package->branch; |
|
371 | - $requiredConstraint = 'dev-' . $package->requiredBranch; |
|
370 | + $currentConstraint = $package->tag ?: 'dev-'.$package->branch; |
|
371 | + $requiredConstraint = 'dev-'.$package->requiredBranch; |
|
372 | 372 | if ($package->requiredBranch) { |
373 | 373 | $actions = array( |
374 | 374 | 1 => "Show divergent commits between <comment>$currentConstraint</comment> and <comment>$requiredConstraint</comment> (and ask again)", |
@@ -419,7 +419,7 @@ discard block |
||
419 | 419 | protected function checkDivergeFromLock($package) |
420 | 420 | { |
421 | 421 | if ($package->git && !$package->isRoot) { |
422 | - $constraint = $package->tag ? ltrim($package->tag, 'v') : 'dev-' . $package->branch; |
|
422 | + $constraint = $package->tag ? ltrim($package->tag, 'v') : 'dev-'.$package->branch; |
|
423 | 423 | if (($package->tag || $package->branch) && $package->version !== $constraint) { |
424 | 424 | if ($this->git('rev-parse', $package->path, 'HEAD') === $package->source->reference) { |
425 | 425 | // HEAD is tip of branch and tag - so only branch was detected |
@@ -432,9 +432,9 @@ discard block |
||
432 | 432 | if ($counts[0] || $counts[1]) { |
433 | 433 | $num = $counts[0] ?: $counts[1]; |
434 | 434 | $type = $counts[0] ? 'behind</>' : 'ahead</> of'; |
435 | - return '%s is <comment>' . $num . ' commit' . ($num > 1 ? 's ' : ' ') |
|
436 | - . $type . ' locked commit <comment>' |
|
437 | - . substr($package->source->reference, 0, 7) . '</>'; |
|
435 | + return '%s is <comment>'.$num.' commit'.($num > 1 ? 's ' : ' ') |
|
436 | + . $type.' locked commit <comment>' |
|
437 | + . substr($package->source->reference, 0, 7).'</>'; |
|
438 | 438 | } |
439 | 439 | } |
440 | 440 | return null; |
@@ -552,12 +552,12 @@ discard block |
||
552 | 552 | $side = $i ? 'left' : 'right'; |
553 | 553 | $otherSide = $i ? 'right' : 'left'; |
554 | 554 | $args = "--$side-only --cherry-pick --pretty=format:'%C(yellow)%h %Cgreen%cd %an%Creset%n %s' --abbrev-commit --date=local "; |
555 | - $args .= $ref . '...'; |
|
555 | + $args .= $ref.'...'; |
|
556 | 556 | $log = $this->git('log', $package->path, $args, array('shy' => true)); |
557 | 557 | if ($log) { |
558 | 558 | $this->console->output(''); |
559 | 559 | |
560 | - $title = "<info>{${$side . 'Title'}}</info> > <info>{${$otherSide . 'Title'}}</info>"; |
|
560 | + $title = "<info>{${$side.'Title'}}</info> > <info>{${$otherSide.'Title'}}</info>"; |
|
561 | 561 | $this->output($title); |
562 | 562 | $this->console->output(str_repeat('-', strlen(strip_tags($title)))); |
563 | 563 |
@@ -39,8 +39,8 @@ discard block |
||
39 | 39 | 'type' => 'shell', |
40 | 40 | 'command' => [ |
41 | 41 | 'rm -rf typo3temp/Cache/*', |
42 | - '{config["php"]} ' . __DIR__ . '/typo3/clear-cache.php', |
|
43 | - '{config["php"]} ' . __DIR__ . '/typo3/schema-migration.php', |
|
42 | + '{config["php"]} '.__DIR__.'/typo3/clear-cache.php', |
|
43 | + '{config["php"]} '.__DIR__.'/typo3/schema-migration.php', |
|
44 | 44 | ], |
45 | 45 | 'processSettings' => ['pt' => true] |
46 | 46 | ] |
@@ -63,7 +63,7 @@ discard block |
||
63 | 63 | ['workflow' => 'clearCodeCaches'], |
64 | 64 | [ |
65 | 65 | 'type' => 'scp', |
66 | - 'from' => __DIR__ . '/typo3', |
|
66 | + 'from' => __DIR__.'/typo3', |
|
67 | 67 | 'to' => '{node}:{node.deployPath}/current/{config["workspace"]}/typo3' |
68 | 68 | ], |
69 | 69 | [ |