@@ -29,44 +29,44 @@ |
||
| 29 | 29 | |
| 30 | 30 | class InfoChecker extends BasicEmitter { |
| 31 | 31 | |
| 32 | - /** @var IAppManager */ |
|
| 33 | - private $appManager; |
|
| 32 | + /** @var IAppManager */ |
|
| 33 | + private $appManager; |
|
| 34 | 34 | |
| 35 | - public function __construct(IAppManager $appManager) { |
|
| 36 | - $this->appManager = $appManager; |
|
| 37 | - } |
|
| 35 | + public function __construct(IAppManager $appManager) { |
|
| 36 | + $this->appManager = $appManager; |
|
| 37 | + } |
|
| 38 | 38 | |
| 39 | - /** |
|
| 40 | - * @param string $appId |
|
| 41 | - * @return array |
|
| 42 | - */ |
|
| 43 | - public function analyse($appId): array { |
|
| 44 | - try { |
|
| 45 | - $appPath = $this->appManager->getAppPath($appId); |
|
| 46 | - } catch (AppPathNotFoundException $e) { |
|
| 47 | - throw new \RuntimeException("No app with given id $appId known."); |
|
| 48 | - } |
|
| 39 | + /** |
|
| 40 | + * @param string $appId |
|
| 41 | + * @return array |
|
| 42 | + */ |
|
| 43 | + public function analyse($appId): array { |
|
| 44 | + try { |
|
| 45 | + $appPath = $this->appManager->getAppPath($appId); |
|
| 46 | + } catch (AppPathNotFoundException $e) { |
|
| 47 | + throw new \RuntimeException("No app with given id $appId known."); |
|
| 48 | + } |
|
| 49 | 49 | |
| 50 | - $xml = new \DOMDocument(); |
|
| 51 | - $xml->load($appPath . '/appinfo/info.xml'); |
|
| 50 | + $xml = new \DOMDocument(); |
|
| 51 | + $xml->load($appPath . '/appinfo/info.xml'); |
|
| 52 | 52 | |
| 53 | - $schema = \OC::$SERVERROOT . '/resources/app-info.xsd'; |
|
| 54 | - if ($this->appManager->isShipped($appId)) { |
|
| 55 | - // Shipped apps are allowed to have the public and default_enabled tags |
|
| 56 | - $schema = \OC::$SERVERROOT . '/resources/app-info-shipped.xsd'; |
|
| 57 | - } |
|
| 53 | + $schema = \OC::$SERVERROOT . '/resources/app-info.xsd'; |
|
| 54 | + if ($this->appManager->isShipped($appId)) { |
|
| 55 | + // Shipped apps are allowed to have the public and default_enabled tags |
|
| 56 | + $schema = \OC::$SERVERROOT . '/resources/app-info-shipped.xsd'; |
|
| 57 | + } |
|
| 58 | 58 | |
| 59 | - $errors = []; |
|
| 60 | - if (!$xml->schemaValidate($schema)) { |
|
| 61 | - foreach (libxml_get_errors() as $error) { |
|
| 62 | - $errors[] = [ |
|
| 63 | - 'type' => 'parseError', |
|
| 64 | - 'field' => $error->message, |
|
| 65 | - ]; |
|
| 66 | - $this->emit('InfoChecker', 'parseError', [$error->message]); |
|
| 67 | - } |
|
| 68 | - } |
|
| 59 | + $errors = []; |
|
| 60 | + if (!$xml->schemaValidate($schema)) { |
|
| 61 | + foreach (libxml_get_errors() as $error) { |
|
| 62 | + $errors[] = [ |
|
| 63 | + 'type' => 'parseError', |
|
| 64 | + 'field' => $error->message, |
|
| 65 | + ]; |
|
| 66 | + $this->emit('InfoChecker', 'parseError', [$error->message]); |
|
| 67 | + } |
|
| 68 | + } |
|
| 69 | 69 | |
| 70 | - return $errors; |
|
| 71 | - } |
|
| 70 | + return $errors; |
|
| 71 | + } |
|
| 72 | 72 | } |
@@ -48,12 +48,12 @@ |
||
| 48 | 48 | } |
| 49 | 49 | |
| 50 | 50 | $xml = new \DOMDocument(); |
| 51 | - $xml->load($appPath . '/appinfo/info.xml'); |
|
| 51 | + $xml->load($appPath.'/appinfo/info.xml'); |
|
| 52 | 52 | |
| 53 | - $schema = \OC::$SERVERROOT . '/resources/app-info.xsd'; |
|
| 53 | + $schema = \OC::$SERVERROOT.'/resources/app-info.xsd'; |
|
| 54 | 54 | if ($this->appManager->isShipped($appId)) { |
| 55 | 55 | // Shipped apps are allowed to have the public and default_enabled tags |
| 56 | - $schema = \OC::$SERVERROOT . '/resources/app-info-shipped.xsd'; |
|
| 56 | + $schema = \OC::$SERVERROOT.'/resources/app-info-shipped.xsd'; |
|
| 57 | 57 | } |
| 58 | 58 | |
| 59 | 59 | $errors = []; |
@@ -44,168 +44,168 @@ |
||
| 44 | 44 | |
| 45 | 45 | class CheckCode extends Command implements CompletionAwareInterface { |
| 46 | 46 | |
| 47 | - protected $checkers = [ |
|
| 48 | - 'private' => PrivateCheck::class, |
|
| 49 | - 'deprecation' => DeprecationCheck::class, |
|
| 50 | - 'strong-comparison' => StrongComparisonCheck::class, |
|
| 51 | - ]; |
|
| 52 | - |
|
| 53 | - protected function configure() { |
|
| 54 | - $this |
|
| 55 | - ->setName('app:check-code') |
|
| 56 | - ->setDescription('check code to be compliant') |
|
| 57 | - ->addArgument( |
|
| 58 | - 'app-id', |
|
| 59 | - InputArgument::REQUIRED, |
|
| 60 | - 'check the specified app' |
|
| 61 | - ) |
|
| 62 | - ->addOption( |
|
| 63 | - 'checker', |
|
| 64 | - 'c', |
|
| 65 | - InputOption::VALUE_REQUIRED | InputOption::VALUE_IS_ARRAY, |
|
| 66 | - 'enable the specified checker(s)', |
|
| 67 | - [ 'private', 'deprecation', 'strong-comparison' ] |
|
| 68 | - ) |
|
| 69 | - ->addOption( |
|
| 70 | - '--skip-checkers', |
|
| 71 | - null, |
|
| 72 | - InputOption::VALUE_NONE, |
|
| 73 | - 'skips the the code checkers to only check info.xml, language and database schema' |
|
| 74 | - ) |
|
| 75 | - ->addOption( |
|
| 76 | - '--skip-validate-info', |
|
| 77 | - null, |
|
| 78 | - InputOption::VALUE_NONE, |
|
| 79 | - 'skips the info.xml/version check' |
|
| 80 | - ); |
|
| 81 | - } |
|
| 82 | - |
|
| 83 | - protected function execute(InputInterface $input, OutputInterface $output) { |
|
| 84 | - $appId = $input->getArgument('app-id'); |
|
| 85 | - |
|
| 86 | - $checkList = new EmptyCheck(); |
|
| 87 | - foreach ($input->getOption('checker') as $checker) { |
|
| 88 | - if (!isset($this->checkers[$checker])) { |
|
| 89 | - throw new \InvalidArgumentException('Invalid checker: '.$checker); |
|
| 90 | - } |
|
| 91 | - $checkerClass = $this->checkers[$checker]; |
|
| 92 | - $checkList = new $checkerClass($checkList); |
|
| 93 | - } |
|
| 94 | - |
|
| 95 | - $codeChecker = new CodeChecker($checkList, !$input->getOption('skip-validate-info')); |
|
| 96 | - |
|
| 97 | - $codeChecker->listen('CodeChecker', 'analyseFileBegin', function($params) use ($output) { |
|
| 98 | - if(OutputInterface::VERBOSITY_VERBOSE <= $output->getVerbosity()) { |
|
| 99 | - $output->writeln("<info>Analysing {$params}</info>"); |
|
| 100 | - } |
|
| 101 | - }); |
|
| 102 | - $codeChecker->listen('CodeChecker', 'analyseFileFinished', function($filename, $errors) use ($output) { |
|
| 103 | - $count = count($errors); |
|
| 104 | - |
|
| 105 | - // show filename if the verbosity is low, but there are errors in a file |
|
| 106 | - if($count > 0 && OutputInterface::VERBOSITY_VERBOSE > $output->getVerbosity()) { |
|
| 107 | - $output->writeln("<info>Analysing {$filename}</info>"); |
|
| 108 | - } |
|
| 109 | - |
|
| 110 | - // show error count if there are errors present or the verbosity is high |
|
| 111 | - if($count > 0 || OutputInterface::VERBOSITY_VERBOSE <= $output->getVerbosity()) { |
|
| 112 | - $output->writeln(" {$count} errors"); |
|
| 113 | - } |
|
| 114 | - usort($errors, function($a, $b) { |
|
| 115 | - return $a['line'] >$b['line']; |
|
| 116 | - }); |
|
| 117 | - |
|
| 118 | - foreach($errors as $p) { |
|
| 119 | - $line = sprintf("%' 4d", $p['line']); |
|
| 120 | - $output->writeln(" <error>line $line: {$p['disallowedToken']} - {$p['reason']}</error>"); |
|
| 121 | - } |
|
| 122 | - }); |
|
| 123 | - $errors = []; |
|
| 124 | - if(!$input->getOption('skip-checkers')) { |
|
| 125 | - $errors = $codeChecker->analyse($appId); |
|
| 126 | - } |
|
| 127 | - |
|
| 128 | - if(!$input->getOption('skip-validate-info')) { |
|
| 129 | - // Can not inject because of circular dependency |
|
| 130 | - $infoChecker = new InfoChecker(\OC::$server->getAppManager()); |
|
| 131 | - $infoChecker->listen('InfoChecker', 'parseError', function($error) use ($output) { |
|
| 132 | - $output->writeln("<error>Invalid appinfo.xml file found: $error</error>"); |
|
| 133 | - }); |
|
| 134 | - |
|
| 135 | - $infoErrors = $infoChecker->analyse($appId); |
|
| 136 | - |
|
| 137 | - $errors = array_merge($errors, $infoErrors); |
|
| 138 | - |
|
| 139 | - $languageParser = new LanguageParseChecker(); |
|
| 140 | - $languageErrors = $languageParser->analyse($appId); |
|
| 141 | - |
|
| 142 | - foreach ($languageErrors as $languageError) { |
|
| 143 | - $output->writeln("<error>$languageError</error>"); |
|
| 144 | - } |
|
| 145 | - |
|
| 146 | - $errors = array_merge($errors, $languageErrors); |
|
| 147 | - |
|
| 148 | - $databaseSchema = new DatabaseSchemaChecker(); |
|
| 149 | - $schemaErrors = $databaseSchema->analyse($appId); |
|
| 150 | - |
|
| 151 | - foreach ($schemaErrors['errors'] as $schemaError) { |
|
| 152 | - $output->writeln("<error>$schemaError</error>"); |
|
| 153 | - } |
|
| 154 | - foreach ($schemaErrors['warnings'] as $schemaWarning) { |
|
| 155 | - $output->writeln("<comment>$schemaWarning</comment>"); |
|
| 156 | - } |
|
| 157 | - |
|
| 158 | - $errors = array_merge($errors, $schemaErrors['errors']); |
|
| 159 | - } |
|
| 160 | - |
|
| 161 | - $this->analyseUpdateFile($appId, $output); |
|
| 162 | - |
|
| 163 | - if (empty($errors)) { |
|
| 164 | - $output->writeln('<info>App is compliant - awesome job!</info>'); |
|
| 165 | - return 0; |
|
| 166 | - } else { |
|
| 167 | - $output->writeln('<error>App is not compliant</error>'); |
|
| 168 | - return 101; |
|
| 169 | - } |
|
| 170 | - } |
|
| 171 | - |
|
| 172 | - /** |
|
| 173 | - * @param string $appId |
|
| 174 | - * @param $output |
|
| 175 | - */ |
|
| 176 | - private function analyseUpdateFile($appId, OutputInterface $output) { |
|
| 177 | - $appPath = \OC_App::getAppPath($appId); |
|
| 178 | - if ($appPath === false) { |
|
| 179 | - throw new \RuntimeException("No app with given id <$appId> known."); |
|
| 180 | - } |
|
| 181 | - |
|
| 182 | - $updatePhp = $appPath . '/appinfo/update.php'; |
|
| 183 | - if (file_exists($updatePhp)) { |
|
| 184 | - $output->writeln("<info>Deprecated file found: $updatePhp - please use repair steps</info>"); |
|
| 185 | - } |
|
| 186 | - } |
|
| 187 | - |
|
| 188 | - /** |
|
| 189 | - * @param string $optionName |
|
| 190 | - * @param CompletionContext $context |
|
| 191 | - * @return string[] |
|
| 192 | - */ |
|
| 193 | - public function completeOptionValues($optionName, CompletionContext $context) { |
|
| 194 | - if ($optionName === 'checker') { |
|
| 195 | - return ['private', 'deprecation', 'strong-comparison']; |
|
| 196 | - } |
|
| 197 | - return []; |
|
| 198 | - } |
|
| 199 | - |
|
| 200 | - /** |
|
| 201 | - * @param string $argumentName |
|
| 202 | - * @param CompletionContext $context |
|
| 203 | - * @return string[] |
|
| 204 | - */ |
|
| 205 | - public function completeArgumentValues($argumentName, CompletionContext $context) { |
|
| 206 | - if ($argumentName === 'app-id') { |
|
| 207 | - return \OC_App::getAllApps(); |
|
| 208 | - } |
|
| 209 | - return []; |
|
| 210 | - } |
|
| 47 | + protected $checkers = [ |
|
| 48 | + 'private' => PrivateCheck::class, |
|
| 49 | + 'deprecation' => DeprecationCheck::class, |
|
| 50 | + 'strong-comparison' => StrongComparisonCheck::class, |
|
| 51 | + ]; |
|
| 52 | + |
|
| 53 | + protected function configure() { |
|
| 54 | + $this |
|
| 55 | + ->setName('app:check-code') |
|
| 56 | + ->setDescription('check code to be compliant') |
|
| 57 | + ->addArgument( |
|
| 58 | + 'app-id', |
|
| 59 | + InputArgument::REQUIRED, |
|
| 60 | + 'check the specified app' |
|
| 61 | + ) |
|
| 62 | + ->addOption( |
|
| 63 | + 'checker', |
|
| 64 | + 'c', |
|
| 65 | + InputOption::VALUE_REQUIRED | InputOption::VALUE_IS_ARRAY, |
|
| 66 | + 'enable the specified checker(s)', |
|
| 67 | + [ 'private', 'deprecation', 'strong-comparison' ] |
|
| 68 | + ) |
|
| 69 | + ->addOption( |
|
| 70 | + '--skip-checkers', |
|
| 71 | + null, |
|
| 72 | + InputOption::VALUE_NONE, |
|
| 73 | + 'skips the the code checkers to only check info.xml, language and database schema' |
|
| 74 | + ) |
|
| 75 | + ->addOption( |
|
| 76 | + '--skip-validate-info', |
|
| 77 | + null, |
|
| 78 | + InputOption::VALUE_NONE, |
|
| 79 | + 'skips the info.xml/version check' |
|
| 80 | + ); |
|
| 81 | + } |
|
| 82 | + |
|
| 83 | + protected function execute(InputInterface $input, OutputInterface $output) { |
|
| 84 | + $appId = $input->getArgument('app-id'); |
|
| 85 | + |
|
| 86 | + $checkList = new EmptyCheck(); |
|
| 87 | + foreach ($input->getOption('checker') as $checker) { |
|
| 88 | + if (!isset($this->checkers[$checker])) { |
|
| 89 | + throw new \InvalidArgumentException('Invalid checker: '.$checker); |
|
| 90 | + } |
|
| 91 | + $checkerClass = $this->checkers[$checker]; |
|
| 92 | + $checkList = new $checkerClass($checkList); |
|
| 93 | + } |
|
| 94 | + |
|
| 95 | + $codeChecker = new CodeChecker($checkList, !$input->getOption('skip-validate-info')); |
|
| 96 | + |
|
| 97 | + $codeChecker->listen('CodeChecker', 'analyseFileBegin', function($params) use ($output) { |
|
| 98 | + if(OutputInterface::VERBOSITY_VERBOSE <= $output->getVerbosity()) { |
|
| 99 | + $output->writeln("<info>Analysing {$params}</info>"); |
|
| 100 | + } |
|
| 101 | + }); |
|
| 102 | + $codeChecker->listen('CodeChecker', 'analyseFileFinished', function($filename, $errors) use ($output) { |
|
| 103 | + $count = count($errors); |
|
| 104 | + |
|
| 105 | + // show filename if the verbosity is low, but there are errors in a file |
|
| 106 | + if($count > 0 && OutputInterface::VERBOSITY_VERBOSE > $output->getVerbosity()) { |
|
| 107 | + $output->writeln("<info>Analysing {$filename}</info>"); |
|
| 108 | + } |
|
| 109 | + |
|
| 110 | + // show error count if there are errors present or the verbosity is high |
|
| 111 | + if($count > 0 || OutputInterface::VERBOSITY_VERBOSE <= $output->getVerbosity()) { |
|
| 112 | + $output->writeln(" {$count} errors"); |
|
| 113 | + } |
|
| 114 | + usort($errors, function($a, $b) { |
|
| 115 | + return $a['line'] >$b['line']; |
|
| 116 | + }); |
|
| 117 | + |
|
| 118 | + foreach($errors as $p) { |
|
| 119 | + $line = sprintf("%' 4d", $p['line']); |
|
| 120 | + $output->writeln(" <error>line $line: {$p['disallowedToken']} - {$p['reason']}</error>"); |
|
| 121 | + } |
|
| 122 | + }); |
|
| 123 | + $errors = []; |
|
| 124 | + if(!$input->getOption('skip-checkers')) { |
|
| 125 | + $errors = $codeChecker->analyse($appId); |
|
| 126 | + } |
|
| 127 | + |
|
| 128 | + if(!$input->getOption('skip-validate-info')) { |
|
| 129 | + // Can not inject because of circular dependency |
|
| 130 | + $infoChecker = new InfoChecker(\OC::$server->getAppManager()); |
|
| 131 | + $infoChecker->listen('InfoChecker', 'parseError', function($error) use ($output) { |
|
| 132 | + $output->writeln("<error>Invalid appinfo.xml file found: $error</error>"); |
|
| 133 | + }); |
|
| 134 | + |
|
| 135 | + $infoErrors = $infoChecker->analyse($appId); |
|
| 136 | + |
|
| 137 | + $errors = array_merge($errors, $infoErrors); |
|
| 138 | + |
|
| 139 | + $languageParser = new LanguageParseChecker(); |
|
| 140 | + $languageErrors = $languageParser->analyse($appId); |
|
| 141 | + |
|
| 142 | + foreach ($languageErrors as $languageError) { |
|
| 143 | + $output->writeln("<error>$languageError</error>"); |
|
| 144 | + } |
|
| 145 | + |
|
| 146 | + $errors = array_merge($errors, $languageErrors); |
|
| 147 | + |
|
| 148 | + $databaseSchema = new DatabaseSchemaChecker(); |
|
| 149 | + $schemaErrors = $databaseSchema->analyse($appId); |
|
| 150 | + |
|
| 151 | + foreach ($schemaErrors['errors'] as $schemaError) { |
|
| 152 | + $output->writeln("<error>$schemaError</error>"); |
|
| 153 | + } |
|
| 154 | + foreach ($schemaErrors['warnings'] as $schemaWarning) { |
|
| 155 | + $output->writeln("<comment>$schemaWarning</comment>"); |
|
| 156 | + } |
|
| 157 | + |
|
| 158 | + $errors = array_merge($errors, $schemaErrors['errors']); |
|
| 159 | + } |
|
| 160 | + |
|
| 161 | + $this->analyseUpdateFile($appId, $output); |
|
| 162 | + |
|
| 163 | + if (empty($errors)) { |
|
| 164 | + $output->writeln('<info>App is compliant - awesome job!</info>'); |
|
| 165 | + return 0; |
|
| 166 | + } else { |
|
| 167 | + $output->writeln('<error>App is not compliant</error>'); |
|
| 168 | + return 101; |
|
| 169 | + } |
|
| 170 | + } |
|
| 171 | + |
|
| 172 | + /** |
|
| 173 | + * @param string $appId |
|
| 174 | + * @param $output |
|
| 175 | + */ |
|
| 176 | + private function analyseUpdateFile($appId, OutputInterface $output) { |
|
| 177 | + $appPath = \OC_App::getAppPath($appId); |
|
| 178 | + if ($appPath === false) { |
|
| 179 | + throw new \RuntimeException("No app with given id <$appId> known."); |
|
| 180 | + } |
|
| 181 | + |
|
| 182 | + $updatePhp = $appPath . '/appinfo/update.php'; |
|
| 183 | + if (file_exists($updatePhp)) { |
|
| 184 | + $output->writeln("<info>Deprecated file found: $updatePhp - please use repair steps</info>"); |
|
| 185 | + } |
|
| 186 | + } |
|
| 187 | + |
|
| 188 | + /** |
|
| 189 | + * @param string $optionName |
|
| 190 | + * @param CompletionContext $context |
|
| 191 | + * @return string[] |
|
| 192 | + */ |
|
| 193 | + public function completeOptionValues($optionName, CompletionContext $context) { |
|
| 194 | + if ($optionName === 'checker') { |
|
| 195 | + return ['private', 'deprecation', 'strong-comparison']; |
|
| 196 | + } |
|
| 197 | + return []; |
|
| 198 | + } |
|
| 199 | + |
|
| 200 | + /** |
|
| 201 | + * @param string $argumentName |
|
| 202 | + * @param CompletionContext $context |
|
| 203 | + * @return string[] |
|
| 204 | + */ |
|
| 205 | + public function completeArgumentValues($argumentName, CompletionContext $context) { |
|
| 206 | + if ($argumentName === 'app-id') { |
|
| 207 | + return \OC_App::getAllApps(); |
|
| 208 | + } |
|
| 209 | + return []; |
|
| 210 | + } |
|
| 211 | 211 | } |
@@ -42,7 +42,7 @@ discard block |
||
| 42 | 42 | use OC\App\CodeChecker\DeprecationCheck; |
| 43 | 43 | use OC\App\CodeChecker\PrivateCheck; |
| 44 | 44 | |
| 45 | -class CheckCode extends Command implements CompletionAwareInterface { |
|
| 45 | +class CheckCode extends Command implements CompletionAwareInterface { |
|
| 46 | 46 | |
| 47 | 47 | protected $checkers = [ |
| 48 | 48 | 'private' => PrivateCheck::class, |
@@ -64,7 +64,7 @@ discard block |
||
| 64 | 64 | 'c', |
| 65 | 65 | InputOption::VALUE_REQUIRED | InputOption::VALUE_IS_ARRAY, |
| 66 | 66 | 'enable the specified checker(s)', |
| 67 | - [ 'private', 'deprecation', 'strong-comparison' ] |
|
| 67 | + ['private', 'deprecation', 'strong-comparison'] |
|
| 68 | 68 | ) |
| 69 | 69 | ->addOption( |
| 70 | 70 | '--skip-checkers', |
@@ -95,7 +95,7 @@ discard block |
||
| 95 | 95 | $codeChecker = new CodeChecker($checkList, !$input->getOption('skip-validate-info')); |
| 96 | 96 | |
| 97 | 97 | $codeChecker->listen('CodeChecker', 'analyseFileBegin', function($params) use ($output) { |
| 98 | - if(OutputInterface::VERBOSITY_VERBOSE <= $output->getVerbosity()) { |
|
| 98 | + if (OutputInterface::VERBOSITY_VERBOSE <= $output->getVerbosity()) { |
|
| 99 | 99 | $output->writeln("<info>Analysing {$params}</info>"); |
| 100 | 100 | } |
| 101 | 101 | }); |
@@ -103,29 +103,29 @@ discard block |
||
| 103 | 103 | $count = count($errors); |
| 104 | 104 | |
| 105 | 105 | // show filename if the verbosity is low, but there are errors in a file |
| 106 | - if($count > 0 && OutputInterface::VERBOSITY_VERBOSE > $output->getVerbosity()) { |
|
| 106 | + if ($count > 0 && OutputInterface::VERBOSITY_VERBOSE > $output->getVerbosity()) { |
|
| 107 | 107 | $output->writeln("<info>Analysing {$filename}</info>"); |
| 108 | 108 | } |
| 109 | 109 | |
| 110 | 110 | // show error count if there are errors present or the verbosity is high |
| 111 | - if($count > 0 || OutputInterface::VERBOSITY_VERBOSE <= $output->getVerbosity()) { |
|
| 111 | + if ($count > 0 || OutputInterface::VERBOSITY_VERBOSE <= $output->getVerbosity()) { |
|
| 112 | 112 | $output->writeln(" {$count} errors"); |
| 113 | 113 | } |
| 114 | 114 | usort($errors, function($a, $b) { |
| 115 | - return $a['line'] >$b['line']; |
|
| 115 | + return $a['line'] > $b['line']; |
|
| 116 | 116 | }); |
| 117 | 117 | |
| 118 | - foreach($errors as $p) { |
|
| 118 | + foreach ($errors as $p) { |
|
| 119 | 119 | $line = sprintf("%' 4d", $p['line']); |
| 120 | 120 | $output->writeln(" <error>line $line: {$p['disallowedToken']} - {$p['reason']}</error>"); |
| 121 | 121 | } |
| 122 | 122 | }); |
| 123 | 123 | $errors = []; |
| 124 | - if(!$input->getOption('skip-checkers')) { |
|
| 124 | + if (!$input->getOption('skip-checkers')) { |
|
| 125 | 125 | $errors = $codeChecker->analyse($appId); |
| 126 | 126 | } |
| 127 | 127 | |
| 128 | - if(!$input->getOption('skip-validate-info')) { |
|
| 128 | + if (!$input->getOption('skip-validate-info')) { |
|
| 129 | 129 | // Can not inject because of circular dependency |
| 130 | 130 | $infoChecker = new InfoChecker(\OC::$server->getAppManager()); |
| 131 | 131 | $infoChecker->listen('InfoChecker', 'parseError', function($error) use ($output) { |
@@ -179,7 +179,7 @@ discard block |
||
| 179 | 179 | throw new \RuntimeException("No app with given id <$appId> known."); |
| 180 | 180 | } |
| 181 | 181 | |
| 182 | - $updatePhp = $appPath . '/appinfo/update.php'; |
|
| 182 | + $updatePhp = $appPath.'/appinfo/update.php'; |
|
| 183 | 183 | if (file_exists($updatePhp)) { |
| 184 | 184 | $output->writeln("<info>Deprecated file found: $updatePhp - please use repair steps</info>"); |
| 185 | 185 | } |
@@ -43,123 +43,123 @@ |
||
| 43 | 43 | $application->add(new OC\Core\Command\App\CheckCode()); |
| 44 | 44 | $application->add(new OC\Core\Command\L10n\CreateJs()); |
| 45 | 45 | $application->add(new \OC\Core\Command\Integrity\SignApp( |
| 46 | - \OC::$server->getIntegrityCodeChecker(), |
|
| 47 | - new \OC\IntegrityCheck\Helpers\FileAccessHelper(), |
|
| 48 | - \OC::$server->getURLGenerator() |
|
| 46 | + \OC::$server->getIntegrityCodeChecker(), |
|
| 47 | + new \OC\IntegrityCheck\Helpers\FileAccessHelper(), |
|
| 48 | + \OC::$server->getURLGenerator() |
|
| 49 | 49 | )); |
| 50 | 50 | $application->add(new \OC\Core\Command\Integrity\SignCore( |
| 51 | - \OC::$server->getIntegrityCodeChecker(), |
|
| 52 | - new \OC\IntegrityCheck\Helpers\FileAccessHelper() |
|
| 51 | + \OC::$server->getIntegrityCodeChecker(), |
|
| 52 | + new \OC\IntegrityCheck\Helpers\FileAccessHelper() |
|
| 53 | 53 | )); |
| 54 | 54 | $application->add(new \OC\Core\Command\Integrity\CheckApp( |
| 55 | - \OC::$server->getIntegrityCodeChecker() |
|
| 55 | + \OC::$server->getIntegrityCodeChecker() |
|
| 56 | 56 | )); |
| 57 | 57 | $application->add(new \OC\Core\Command\Integrity\CheckCore( |
| 58 | - \OC::$server->getIntegrityCodeChecker() |
|
| 58 | + \OC::$server->getIntegrityCodeChecker() |
|
| 59 | 59 | )); |
| 60 | 60 | |
| 61 | 61 | |
| 62 | 62 | if (\OC::$server->getConfig()->getSystemValue('installed', false)) { |
| 63 | - $application->add(new OC\Core\Command\App\Disable(\OC::$server->getAppManager())); |
|
| 64 | - $application->add(new OC\Core\Command\App\Enable(\OC::$server->getAppManager())); |
|
| 65 | - $application->add(new OC\Core\Command\App\Install()); |
|
| 66 | - $application->add(new OC\Core\Command\App\GetPath()); |
|
| 67 | - $application->add(new OC\Core\Command\App\ListApps(\OC::$server->getAppManager())); |
|
| 63 | + $application->add(new OC\Core\Command\App\Disable(\OC::$server->getAppManager())); |
|
| 64 | + $application->add(new OC\Core\Command\App\Enable(\OC::$server->getAppManager())); |
|
| 65 | + $application->add(new OC\Core\Command\App\Install()); |
|
| 66 | + $application->add(new OC\Core\Command\App\GetPath()); |
|
| 67 | + $application->add(new OC\Core\Command\App\ListApps(\OC::$server->getAppManager())); |
|
| 68 | 68 | |
| 69 | - $application->add(new OC\Core\Command\TwoFactorAuth\Enable( |
|
| 70 | - \OC::$server->getTwoFactorAuthManager(), \OC::$server->getUserManager() |
|
| 71 | - )); |
|
| 72 | - $application->add(new OC\Core\Command\TwoFactorAuth\Disable( |
|
| 73 | - \OC::$server->getTwoFactorAuthManager(), \OC::$server->getUserManager() |
|
| 74 | - )); |
|
| 69 | + $application->add(new OC\Core\Command\TwoFactorAuth\Enable( |
|
| 70 | + \OC::$server->getTwoFactorAuthManager(), \OC::$server->getUserManager() |
|
| 71 | + )); |
|
| 72 | + $application->add(new OC\Core\Command\TwoFactorAuth\Disable( |
|
| 73 | + \OC::$server->getTwoFactorAuthManager(), \OC::$server->getUserManager() |
|
| 74 | + )); |
|
| 75 | 75 | |
| 76 | - $application->add(new OC\Core\Command\Background\Cron(\OC::$server->getConfig())); |
|
| 77 | - $application->add(new OC\Core\Command\Background\WebCron(\OC::$server->getConfig())); |
|
| 78 | - $application->add(new OC\Core\Command\Background\Ajax(\OC::$server->getConfig())); |
|
| 76 | + $application->add(new OC\Core\Command\Background\Cron(\OC::$server->getConfig())); |
|
| 77 | + $application->add(new OC\Core\Command\Background\WebCron(\OC::$server->getConfig())); |
|
| 78 | + $application->add(new OC\Core\Command\Background\Ajax(\OC::$server->getConfig())); |
|
| 79 | 79 | |
| 80 | - $application->add(new OC\Core\Command\Config\App\DeleteConfig(\OC::$server->getConfig())); |
|
| 81 | - $application->add(new OC\Core\Command\Config\App\GetConfig(\OC::$server->getConfig())); |
|
| 82 | - $application->add(new OC\Core\Command\Config\App\SetConfig(\OC::$server->getConfig())); |
|
| 83 | - $application->add(new OC\Core\Command\Config\Import(\OC::$server->getConfig())); |
|
| 84 | - $application->add(new OC\Core\Command\Config\ListConfigs(\OC::$server->getSystemConfig(), \OC::$server->getAppConfig())); |
|
| 85 | - $application->add(new OC\Core\Command\Config\System\DeleteConfig(\OC::$server->getSystemConfig())); |
|
| 86 | - $application->add(new OC\Core\Command\Config\System\GetConfig(\OC::$server->getSystemConfig())); |
|
| 87 | - $application->add(new OC\Core\Command\Config\System\SetConfig(\OC::$server->getSystemConfig())); |
|
| 80 | + $application->add(new OC\Core\Command\Config\App\DeleteConfig(\OC::$server->getConfig())); |
|
| 81 | + $application->add(new OC\Core\Command\Config\App\GetConfig(\OC::$server->getConfig())); |
|
| 82 | + $application->add(new OC\Core\Command\Config\App\SetConfig(\OC::$server->getConfig())); |
|
| 83 | + $application->add(new OC\Core\Command\Config\Import(\OC::$server->getConfig())); |
|
| 84 | + $application->add(new OC\Core\Command\Config\ListConfigs(\OC::$server->getSystemConfig(), \OC::$server->getAppConfig())); |
|
| 85 | + $application->add(new OC\Core\Command\Config\System\DeleteConfig(\OC::$server->getSystemConfig())); |
|
| 86 | + $application->add(new OC\Core\Command\Config\System\GetConfig(\OC::$server->getSystemConfig())); |
|
| 87 | + $application->add(new OC\Core\Command\Config\System\SetConfig(\OC::$server->getSystemConfig())); |
|
| 88 | 88 | |
| 89 | - $application->add(new OC\Core\Command\Db\ConvertType(\OC::$server->getConfig(), new \OC\DB\ConnectionFactory(\OC::$server->getSystemConfig()))); |
|
| 90 | - $application->add(new OC\Core\Command\Db\ConvertMysqlToMB4(\OC::$server->getConfig(), \OC::$server->getDatabaseConnection(), \OC::$server->getURLGenerator(), \OC::$server->getLogger())); |
|
| 91 | - $application->add(new OC\Core\Command\Db\ConvertFilecacheBigInt(\OC::$server->getDatabaseConnection())); |
|
| 92 | - $application->add(new OC\Core\Command\Db\AddMissingIndices(\OC::$server->getDatabaseConnection())); |
|
| 93 | - $application->add(new OC\Core\Command\Db\Migrations\StatusCommand(\OC::$server->getDatabaseConnection())); |
|
| 94 | - $application->add(new OC\Core\Command\Db\Migrations\MigrateCommand(\OC::$server->getDatabaseConnection())); |
|
| 95 | - $application->add(new OC\Core\Command\Db\Migrations\GenerateCommand(\OC::$server->getDatabaseConnection())); |
|
| 96 | - $application->add(new OC\Core\Command\Db\Migrations\GenerateFromSchemaFileCommand(\OC::$server->getConfig(), \OC::$server->getAppManager(), \OC::$server->getDatabaseConnection())); |
|
| 97 | - $application->add(new OC\Core\Command\Db\Migrations\ExecuteCommand(\OC::$server->getDatabaseConnection(), \OC::$server->getConfig())); |
|
| 89 | + $application->add(new OC\Core\Command\Db\ConvertType(\OC::$server->getConfig(), new \OC\DB\ConnectionFactory(\OC::$server->getSystemConfig()))); |
|
| 90 | + $application->add(new OC\Core\Command\Db\ConvertMysqlToMB4(\OC::$server->getConfig(), \OC::$server->getDatabaseConnection(), \OC::$server->getURLGenerator(), \OC::$server->getLogger())); |
|
| 91 | + $application->add(new OC\Core\Command\Db\ConvertFilecacheBigInt(\OC::$server->getDatabaseConnection())); |
|
| 92 | + $application->add(new OC\Core\Command\Db\AddMissingIndices(\OC::$server->getDatabaseConnection())); |
|
| 93 | + $application->add(new OC\Core\Command\Db\Migrations\StatusCommand(\OC::$server->getDatabaseConnection())); |
|
| 94 | + $application->add(new OC\Core\Command\Db\Migrations\MigrateCommand(\OC::$server->getDatabaseConnection())); |
|
| 95 | + $application->add(new OC\Core\Command\Db\Migrations\GenerateCommand(\OC::$server->getDatabaseConnection())); |
|
| 96 | + $application->add(new OC\Core\Command\Db\Migrations\GenerateFromSchemaFileCommand(\OC::$server->getConfig(), \OC::$server->getAppManager(), \OC::$server->getDatabaseConnection())); |
|
| 97 | + $application->add(new OC\Core\Command\Db\Migrations\ExecuteCommand(\OC::$server->getDatabaseConnection(), \OC::$server->getConfig())); |
|
| 98 | 98 | |
| 99 | - $application->add(new OC\Core\Command\Encryption\Disable(\OC::$server->getConfig())); |
|
| 100 | - $application->add(new OC\Core\Command\Encryption\Enable(\OC::$server->getConfig(), \OC::$server->getEncryptionManager())); |
|
| 101 | - $application->add(new OC\Core\Command\Encryption\ListModules(\OC::$server->getEncryptionManager())); |
|
| 102 | - $application->add(new OC\Core\Command\Encryption\SetDefaultModule(\OC::$server->getEncryptionManager())); |
|
| 103 | - $application->add(new OC\Core\Command\Encryption\Status(\OC::$server->getEncryptionManager())); |
|
| 104 | - $application->add(new OC\Core\Command\Encryption\EncryptAll(\OC::$server->getEncryptionManager(), \OC::$server->getAppManager(), \OC::$server->getConfig(), new \Symfony\Component\Console\Helper\QuestionHelper())); |
|
| 105 | - $application->add(new OC\Core\Command\Encryption\DecryptAll( |
|
| 106 | - \OC::$server->getEncryptionManager(), |
|
| 107 | - \OC::$server->getAppManager(), |
|
| 108 | - \OC::$server->getConfig(), |
|
| 109 | - new \OC\Encryption\DecryptAll(\OC::$server->getEncryptionManager(), \OC::$server->getUserManager(), new \OC\Files\View()), |
|
| 110 | - new \Symfony\Component\Console\Helper\QuestionHelper()) |
|
| 111 | - ); |
|
| 99 | + $application->add(new OC\Core\Command\Encryption\Disable(\OC::$server->getConfig())); |
|
| 100 | + $application->add(new OC\Core\Command\Encryption\Enable(\OC::$server->getConfig(), \OC::$server->getEncryptionManager())); |
|
| 101 | + $application->add(new OC\Core\Command\Encryption\ListModules(\OC::$server->getEncryptionManager())); |
|
| 102 | + $application->add(new OC\Core\Command\Encryption\SetDefaultModule(\OC::$server->getEncryptionManager())); |
|
| 103 | + $application->add(new OC\Core\Command\Encryption\Status(\OC::$server->getEncryptionManager())); |
|
| 104 | + $application->add(new OC\Core\Command\Encryption\EncryptAll(\OC::$server->getEncryptionManager(), \OC::$server->getAppManager(), \OC::$server->getConfig(), new \Symfony\Component\Console\Helper\QuestionHelper())); |
|
| 105 | + $application->add(new OC\Core\Command\Encryption\DecryptAll( |
|
| 106 | + \OC::$server->getEncryptionManager(), |
|
| 107 | + \OC::$server->getAppManager(), |
|
| 108 | + \OC::$server->getConfig(), |
|
| 109 | + new \OC\Encryption\DecryptAll(\OC::$server->getEncryptionManager(), \OC::$server->getUserManager(), new \OC\Files\View()), |
|
| 110 | + new \Symfony\Component\Console\Helper\QuestionHelper()) |
|
| 111 | + ); |
|
| 112 | 112 | |
| 113 | - $application->add(new OC\Core\Command\Log\Manage(\OC::$server->getConfig())); |
|
| 114 | - $application->add(new OC\Core\Command\Log\File(\OC::$server->getConfig())); |
|
| 113 | + $application->add(new OC\Core\Command\Log\Manage(\OC::$server->getConfig())); |
|
| 114 | + $application->add(new OC\Core\Command\Log\File(\OC::$server->getConfig())); |
|
| 115 | 115 | |
| 116 | - $view = new \OC\Files\View(); |
|
| 117 | - $util = new \OC\Encryption\Util( |
|
| 118 | - $view, |
|
| 119 | - \OC::$server->getUserManager(), |
|
| 120 | - \OC::$server->getGroupManager(), |
|
| 121 | - \OC::$server->getConfig() |
|
| 122 | - ); |
|
| 123 | - $application->add(new OC\Core\Command\Encryption\ChangeKeyStorageRoot( |
|
| 124 | - $view, |
|
| 125 | - \OC::$server->getUserManager(), |
|
| 126 | - \OC::$server->getConfig(), |
|
| 127 | - $util, |
|
| 128 | - new \Symfony\Component\Console\Helper\QuestionHelper() |
|
| 129 | - ) |
|
| 130 | - ); |
|
| 131 | - $application->add(new OC\Core\Command\Encryption\ShowKeyStorageRoot($util)); |
|
| 116 | + $view = new \OC\Files\View(); |
|
| 117 | + $util = new \OC\Encryption\Util( |
|
| 118 | + $view, |
|
| 119 | + \OC::$server->getUserManager(), |
|
| 120 | + \OC::$server->getGroupManager(), |
|
| 121 | + \OC::$server->getConfig() |
|
| 122 | + ); |
|
| 123 | + $application->add(new OC\Core\Command\Encryption\ChangeKeyStorageRoot( |
|
| 124 | + $view, |
|
| 125 | + \OC::$server->getUserManager(), |
|
| 126 | + \OC::$server->getConfig(), |
|
| 127 | + $util, |
|
| 128 | + new \Symfony\Component\Console\Helper\QuestionHelper() |
|
| 129 | + ) |
|
| 130 | + ); |
|
| 131 | + $application->add(new OC\Core\Command\Encryption\ShowKeyStorageRoot($util)); |
|
| 132 | 132 | |
| 133 | - $application->add(new OC\Core\Command\Maintenance\DataFingerprint(\OC::$server->getConfig(), new \OC\AppFramework\Utility\TimeFactory())); |
|
| 134 | - $application->add(new OC\Core\Command\Maintenance\Mimetype\UpdateDB(\OC::$server->getMimeTypeDetector(), \OC::$server->getMimeTypeLoader())); |
|
| 135 | - $application->add(new OC\Core\Command\Maintenance\Mimetype\UpdateJS(\OC::$server->getMimeTypeDetector())); |
|
| 136 | - $application->add(new OC\Core\Command\Maintenance\Mode(\OC::$server->getConfig())); |
|
| 137 | - $application->add(new OC\Core\Command\Maintenance\UpdateHtaccess()); |
|
| 138 | - $application->add(new OC\Core\Command\Maintenance\UpdateTheme(\OC::$server->getMimeTypeDetector(), \OC::$server->getMemCacheFactory())); |
|
| 133 | + $application->add(new OC\Core\Command\Maintenance\DataFingerprint(\OC::$server->getConfig(), new \OC\AppFramework\Utility\TimeFactory())); |
|
| 134 | + $application->add(new OC\Core\Command\Maintenance\Mimetype\UpdateDB(\OC::$server->getMimeTypeDetector(), \OC::$server->getMimeTypeLoader())); |
|
| 135 | + $application->add(new OC\Core\Command\Maintenance\Mimetype\UpdateJS(\OC::$server->getMimeTypeDetector())); |
|
| 136 | + $application->add(new OC\Core\Command\Maintenance\Mode(\OC::$server->getConfig())); |
|
| 137 | + $application->add(new OC\Core\Command\Maintenance\UpdateHtaccess()); |
|
| 138 | + $application->add(new OC\Core\Command\Maintenance\UpdateTheme(\OC::$server->getMimeTypeDetector(), \OC::$server->getMemCacheFactory())); |
|
| 139 | 139 | |
| 140 | - $application->add(new OC\Core\Command\Upgrade(\OC::$server->getConfig(), \OC::$server->getLogger(), \OC::$server->query(\OC\Installer::class))); |
|
| 141 | - $application->add(new OC\Core\Command\Maintenance\Repair( |
|
| 142 | - new \OC\Repair(\OC\Repair::getRepairSteps(), \OC::$server->getEventDispatcher()), \OC::$server->getConfig(), |
|
| 143 | - \OC::$server->getEventDispatcher(), \OC::$server->getAppManager())); |
|
| 140 | + $application->add(new OC\Core\Command\Upgrade(\OC::$server->getConfig(), \OC::$server->getLogger(), \OC::$server->query(\OC\Installer::class))); |
|
| 141 | + $application->add(new OC\Core\Command\Maintenance\Repair( |
|
| 142 | + new \OC\Repair(\OC\Repair::getRepairSteps(), \OC::$server->getEventDispatcher()), \OC::$server->getConfig(), |
|
| 143 | + \OC::$server->getEventDispatcher(), \OC::$server->getAppManager())); |
|
| 144 | 144 | |
| 145 | - $application->add(new OC\Core\Command\User\Add(\OC::$server->getUserManager(), \OC::$server->getGroupManager())); |
|
| 146 | - $application->add(new OC\Core\Command\User\Delete(\OC::$server->getUserManager())); |
|
| 147 | - $application->add(new OC\Core\Command\User\Disable(\OC::$server->getUserManager())); |
|
| 148 | - $application->add(new OC\Core\Command\User\Enable(\OC::$server->getUserManager())); |
|
| 149 | - $application->add(new OC\Core\Command\User\LastSeen(\OC::$server->getUserManager())); |
|
| 150 | - $application->add(new OC\Core\Command\User\Report(\OC::$server->getUserManager())); |
|
| 151 | - $application->add(new OC\Core\Command\User\ResetPassword(\OC::$server->getUserManager())); |
|
| 152 | - $application->add(new OC\Core\Command\User\Setting(\OC::$server->getUserManager(), \OC::$server->getConfig(), \OC::$server->getDatabaseConnection())); |
|
| 153 | - $application->add(new OC\Core\Command\User\ListCommand(\OC::$server->getUserManager())); |
|
| 154 | - $application->add(new OC\Core\Command\User\Info(\OC::$server->getUserManager(), \OC::$server->getGroupManager())); |
|
| 145 | + $application->add(new OC\Core\Command\User\Add(\OC::$server->getUserManager(), \OC::$server->getGroupManager())); |
|
| 146 | + $application->add(new OC\Core\Command\User\Delete(\OC::$server->getUserManager())); |
|
| 147 | + $application->add(new OC\Core\Command\User\Disable(\OC::$server->getUserManager())); |
|
| 148 | + $application->add(new OC\Core\Command\User\Enable(\OC::$server->getUserManager())); |
|
| 149 | + $application->add(new OC\Core\Command\User\LastSeen(\OC::$server->getUserManager())); |
|
| 150 | + $application->add(new OC\Core\Command\User\Report(\OC::$server->getUserManager())); |
|
| 151 | + $application->add(new OC\Core\Command\User\ResetPassword(\OC::$server->getUserManager())); |
|
| 152 | + $application->add(new OC\Core\Command\User\Setting(\OC::$server->getUserManager(), \OC::$server->getConfig(), \OC::$server->getDatabaseConnection())); |
|
| 153 | + $application->add(new OC\Core\Command\User\ListCommand(\OC::$server->getUserManager())); |
|
| 154 | + $application->add(new OC\Core\Command\User\Info(\OC::$server->getUserManager(), \OC::$server->getGroupManager())); |
|
| 155 | 155 | |
| 156 | - $application->add(new OC\Core\Command\Group\ListCommand(\OC::$server->getGroupManager())); |
|
| 157 | - $application->add(new OC\Core\Command\Group\AddUser(\OC::$server->getUserManager(), \OC::$server->getGroupManager())); |
|
| 158 | - $application->add(new OC\Core\Command\Group\RemoveUser(\OC::$server->getUserManager(), \OC::$server->getGroupManager())); |
|
| 156 | + $application->add(new OC\Core\Command\Group\ListCommand(\OC::$server->getGroupManager())); |
|
| 157 | + $application->add(new OC\Core\Command\Group\AddUser(\OC::$server->getUserManager(), \OC::$server->getGroupManager())); |
|
| 158 | + $application->add(new OC\Core\Command\Group\RemoveUser(\OC::$server->getUserManager(), \OC::$server->getGroupManager())); |
|
| 159 | 159 | |
| 160 | - $application->add(new OC\Core\Command\Security\ListCertificates(\OC::$server->getCertificateManager(null), \OC::$server->getL10N('core'))); |
|
| 161 | - $application->add(new OC\Core\Command\Security\ImportCertificate(\OC::$server->getCertificateManager(null))); |
|
| 162 | - $application->add(new OC\Core\Command\Security\RemoveCertificate(\OC::$server->getCertificateManager(null))); |
|
| 160 | + $application->add(new OC\Core\Command\Security\ListCertificates(\OC::$server->getCertificateManager(null), \OC::$server->getL10N('core'))); |
|
| 161 | + $application->add(new OC\Core\Command\Security\ImportCertificate(\OC::$server->getCertificateManager(null))); |
|
| 162 | + $application->add(new OC\Core\Command\Security\RemoveCertificate(\OC::$server->getCertificateManager(null))); |
|
| 163 | 163 | } else { |
| 164 | - $application->add(new OC\Core\Command\Maintenance\Install(\OC::$server->getSystemConfig())); |
|
| 164 | + $application->add(new OC\Core\Command\Maintenance\Install(\OC::$server->getSystemConfig())); |
|
| 165 | 165 | } |