@@ -83,7 +83,7 @@ discard block |
||
| 83 | 83 | /** |
| 84 | 84 | * @param array $codeArgument |
| 85 | 85 | * @param bool $status |
| 86 | - * @return bool |
|
| 86 | + * @return boolean|null |
|
| 87 | 87 | */ |
| 88 | 88 | protected function saveCacheStatus($codeArgument, $status) |
| 89 | 89 | { |
@@ -288,8 +288,8 @@ discard block |
||
| 288 | 288 | /** |
| 289 | 289 | * brings locally cached repository up to date if it is missing the requested tag |
| 290 | 290 | * |
| 291 | - * @param $package |
|
| 292 | - * @param $targetFolder |
|
| 291 | + * @param PackageInterface $package |
|
| 292 | + * @param string $targetFolder |
|
| 293 | 293 | */ |
| 294 | 294 | protected function checkRepository($package, $targetFolder) |
| 295 | 295 | { |
@@ -203,7 +203,7 @@ discard block |
||
| 203 | 203 | if (!$silent) { |
| 204 | 204 | $editionString = ($this->_magentoEnterprise ? ' (Enterprise Edition) ' : ''); |
| 205 | 205 | $output->writeln( |
| 206 | - '<info>Found Magento '. $editionString . 'in folder "' . $this->_magentoRootFolder . '"</info>' |
|
| 206 | + '<info>Found Magento ' . $editionString . 'in folder "' . $this->_magentoRootFolder . '"</info>' |
|
| 207 | 207 | ); |
| 208 | 208 | } |
| 209 | 209 | |
@@ -272,7 +272,7 @@ discard block |
||
| 272 | 272 | $preferSource = true |
| 273 | 273 | ) { |
| 274 | 274 | $dm = $this->getComposerDownloadManager($input, $output); |
| 275 | - if (! $config instanceof PackageInterface) { |
|
| 275 | + if (!$config instanceof PackageInterface) { |
|
| 276 | 276 | $package = $this->createComposerPackageByConfig($config); |
| 277 | 277 | } else { |
| 278 | 278 | $package = $config; |
@@ -316,7 +316,7 @@ discard block |
||
| 316 | 316 | escapeshellarg($targetFolder), |
| 317 | 317 | escapeshellarg($package->getSourceReference()) |
| 318 | 318 | ); |
| 319 | - $existingTag = shell_exec($command); |
|
| 319 | + $existingTag = shell_exec($command); |
|
| 320 | 320 | if ($existingTag === $package->getSourceReference()) { |
| 321 | 321 | $command = sprintf('cd %s && hg pull', escapeshellarg($targetFolder)); |
| 322 | 322 | shell_exec($command); |
@@ -143,7 +143,7 @@ |
||
| 143 | 143 | } |
| 144 | 144 | |
| 145 | 145 | /** |
| 146 | - * @param $configBag |
|
| 146 | + * @param ConfigBag $configBag |
|
| 147 | 147 | */ |
| 148 | 148 | private function initConfigBagDefaultValues($configBag) |
| 149 | 149 | { |
@@ -252,7 +252,7 @@ |
||
| 252 | 252 | } |
| 253 | 253 | |
| 254 | 254 | /** |
| 255 | - * @param array $installArgs |
|
| 255 | + * @param string $installArgs |
|
| 256 | 256 | * |
| 257 | 257 | * @throws \Exception |
| 258 | 258 | */ |
@@ -9,7 +9,7 @@ |
||
| 9 | 9 | class InstallSampleData extends AbstractSubCommand |
| 10 | 10 | { |
| 11 | 11 | /** |
| 12 | - * @return bool |
|
| 12 | + * @return false|null |
|
| 13 | 13 | */ |
| 14 | 14 | public function execute() |
| 15 | 15 | { |
@@ -7,7 +7,7 @@ discard block |
||
| 7 | 7 | class RewriteHtaccessFile extends AbstractSubCommand |
| 8 | 8 | { |
| 9 | 9 | /** |
| 10 | - * @return bool |
|
| 10 | + * @return boolean|null |
|
| 11 | 11 | */ |
| 12 | 12 | public function execute() |
| 13 | 13 | { |
@@ -53,6 +53,9 @@ discard block |
||
| 53 | 53 | $this->_replaceContent($htaccessFile, $baseUrl); |
| 54 | 54 | } |
| 55 | 55 | |
| 56 | + /** |
|
| 57 | + * @param string $htaccesFile |
|
| 58 | + */ |
|
| 56 | 59 | protected function _backupOriginalFile($htaccesFile) |
| 57 | 60 | { |
| 58 | 61 | copy( |
@@ -44,10 +44,10 @@ |
||
| 44 | 44 | protected function replaceHtaccessFile($baseUrl) |
| 45 | 45 | { |
| 46 | 46 | $htaccessFile = $this->config->getString('installationFolder') |
| 47 | - . DIRECTORY_SEPARATOR |
|
| 48 | - . 'pub' |
|
| 49 | - . DIRECTORY_SEPARATOR |
|
| 50 | - . '.htaccess'; |
|
| 47 | + . DIRECTORY_SEPARATOR |
|
| 48 | + . 'pub' |
|
| 49 | + . DIRECTORY_SEPARATOR |
|
| 50 | + . '.htaccess'; |
|
| 51 | 51 | |
| 52 | 52 | $this->_backupOriginalFile($htaccessFile); |
| 53 | 53 | $this->_replaceContent($htaccessFile, $baseUrl); |
@@ -78,7 +78,7 @@ |
||
| 78 | 78 | /** |
| 79 | 79 | * Reads the first line. If it's a comment return it. |
| 80 | 80 | * |
| 81 | - * @param $file |
|
| 81 | + * @param string $file |
|
| 82 | 82 | * |
| 83 | 83 | * @return string |
| 84 | 84 | */ |
@@ -254,7 +254,7 @@ discard block |
||
| 254 | 254 | /** |
| 255 | 255 | * @param InputInterface $input |
| 256 | 256 | * @param OutputInterface $output |
| 257 | - * @param $commandString |
|
| 257 | + * @param string $commandString |
|
| 258 | 258 | * @throws \RuntimeException |
| 259 | 259 | */ |
| 260 | 260 | protected function runMagerunCommand(InputInterface $input, OutputInterface $output, $commandString) |
@@ -270,7 +270,7 @@ discard block |
||
| 270 | 270 | |
| 271 | 271 | /** |
| 272 | 272 | * @param string $commandString |
| 273 | - * @return mixed|string |
|
| 273 | + * @return string |
|
| 274 | 274 | */ |
| 275 | 275 | protected function _prepareShellCommand($commandString) |
| 276 | 276 | { |
@@ -320,7 +320,7 @@ discard block |
||
| 320 | 320 | |
| 321 | 321 | /** |
| 322 | 322 | * @param $commandString |
| 323 | - * @return mixed |
|
| 323 | + * @return string |
|
| 324 | 324 | */ |
| 325 | 325 | protected function _replaceScriptVars($commandString) |
| 326 | 326 | { |
@@ -171,7 +171,7 @@ |
||
| 171 | 171 | if (isset($parts[1])) { |
| 172 | 172 | $value = $parts[1]; |
| 173 | 173 | } |
| 174 | - $this->scriptVars['${' . $variable. '}'] = $value; |
|
| 174 | + $this->scriptVars['${' . $variable . '}'] = $value; |
|
| 175 | 175 | } |
| 176 | 176 | } |
| 177 | 177 | } |
@@ -10,21 +10,25 @@ discard block |
||
| 10 | 10 | { |
| 11 | 11 | /** |
| 12 | 12 | * @param ConfigBag $config |
| 13 | + * @return void |
|
| 13 | 14 | */ |
| 14 | 15 | public function setConfig(ConfigBag $config); |
| 15 | 16 | |
| 16 | 17 | /** |
| 17 | 18 | * @param array $commandConfig |
| 19 | + * @return void |
|
| 18 | 20 | */ |
| 19 | 21 | public function setCommandConfig(array $commandConfig); |
| 20 | 22 | |
| 21 | 23 | /** |
| 22 | 24 | * @param InputInterface $input |
| 25 | + * @return void |
|
| 23 | 26 | */ |
| 24 | 27 | public function setInput(InputInterface $input); |
| 25 | 28 | |
| 26 | 29 | /** |
| 27 | 30 | * @param OutputInterface $output |
| 31 | + * @return void |
|
| 28 | 32 | */ |
| 29 | 33 | public function setOutput(OutputInterface $output); |
| 30 | 34 | |
@@ -35,6 +39,7 @@ discard block |
||
| 35 | 39 | |
| 36 | 40 | /** |
| 37 | 41 | * @param AbstractMagentoCommand $command |
| 42 | + * @return void |
|
| 38 | 43 | */ |
| 39 | 44 | public function setCommand(AbstractMagentoCommand $command); |
| 40 | 45 | |
@@ -97,7 +97,7 @@ |
||
| 97 | 97 | * @param InputInterface $input |
| 98 | 98 | * @param OutputInterface $output |
| 99 | 99 | * @param array $jobs |
| 100 | - * @return mixed |
|
| 100 | + * @return string |
|
| 101 | 101 | * @throws \InvalidArgumentException |
| 102 | 102 | * @throws \Exception |
| 103 | 103 | */ |
@@ -48,7 +48,7 @@ discard block |
||
| 48 | 48 | |
| 49 | 49 | $jobConfig = $this->getJobConfig($jobCode); |
| 50 | 50 | |
| 51 | - if (empty($jobCode)|| !isset($jobConfig['instance'])) { |
|
| 51 | + if (empty($jobCode) || !isset($jobConfig['instance'])) { |
|
| 52 | 52 | throw new \InvalidArgumentException('No job config found!'); |
| 53 | 53 | } |
| 54 | 54 | |
@@ -106,7 +106,7 @@ discard block |
||
| 106 | 106 | protected function askJobCode(InputInterface $input, OutputInterface $output, $jobs) |
| 107 | 107 | { |
| 108 | 108 | foreach ($jobs as $key => $job) { |
| 109 | - $question[] = '<comment>[' . ($key+1) . ']</comment> ' . $job['Job'] . PHP_EOL; |
|
| 109 | + $question[] = '<comment>[' . ($key + 1) . ']</comment> ' . $job['Job'] . PHP_EOL; |
|
| 110 | 110 | } |
| 111 | 111 | $question[] = '<question>Please select job: </question>' . PHP_EOL; |
| 112 | 112 | |