2015-01-29 00:08:50 +03:00
|
|
|
<?php
|
|
|
|
/**
|
2015-06-25 12:43:55 +03:00
|
|
|
* @author Joas Schilling <nickvergessen@owncloud.com>
|
2015-03-26 13:44:34 +03:00
|
|
|
* @author Morris Jobke <hey@morrisjobke.de>
|
|
|
|
* @author Thomas Müller <thomas.mueller@tmit.eu>
|
2015-07-11 00:17:55 +03:00
|
|
|
* @author Robin McCorkell <rmccorkell@karoshi.org.uk>
|
2015-03-26 13:44:34 +03:00
|
|
|
*
|
|
|
|
* @copyright Copyright (c) 2015, ownCloud, Inc.
|
|
|
|
* @license AGPL-3.0
|
|
|
|
*
|
|
|
|
* This code is free software: you can redistribute it and/or modify
|
|
|
|
* it under the terms of the GNU Affero General Public License, version 3,
|
|
|
|
* as published by the Free Software Foundation.
|
|
|
|
*
|
|
|
|
* This program is distributed in the hope that it will be useful,
|
|
|
|
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
|
|
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
|
|
* GNU Affero General Public License for more details.
|
|
|
|
*
|
|
|
|
* You should have received a copy of the GNU Affero General Public License, version 3,
|
|
|
|
* along with this program. If not, see <http://www.gnu.org/licenses/>
|
|
|
|
*
|
2015-01-29 00:08:50 +03:00
|
|
|
*/
|
2015-02-26 13:37:37 +03:00
|
|
|
|
2015-01-29 00:08:50 +03:00
|
|
|
namespace OC\Core\Command\App;
|
|
|
|
|
2015-07-07 13:08:12 +03:00
|
|
|
use OC\App\CodeChecker\CodeChecker;
|
2015-07-07 16:37:56 +03:00
|
|
|
use OC\App\CodeChecker\EmptyCheck;
|
2015-01-29 00:08:50 +03:00
|
|
|
use Symfony\Component\Console\Command\Command;
|
|
|
|
use Symfony\Component\Console\Input\InputArgument;
|
|
|
|
use Symfony\Component\Console\Input\InputInterface;
|
2015-06-15 13:51:22 +03:00
|
|
|
use Symfony\Component\Console\Input\InputOption;
|
2015-01-29 00:08:50 +03:00
|
|
|
use Symfony\Component\Console\Output\OutputInterface;
|
|
|
|
|
|
|
|
class CheckCode extends Command {
|
2015-07-11 00:17:55 +03:00
|
|
|
protected $checkers = [
|
|
|
|
'private' => '\OC\App\CodeChecker\PrivateCheck',
|
|
|
|
'deprecation' => '\OC\App\CodeChecker\DeprecationCheck',
|
|
|
|
'strong-comparison' => '\OC\App\CodeChecker\StrongComparisonCheck',
|
|
|
|
];
|
|
|
|
|
2015-01-29 00:08:50 +03:00
|
|
|
protected function configure() {
|
|
|
|
$this
|
|
|
|
->setName('app:check-code')
|
|
|
|
->setDescription('check code to be compliant')
|
|
|
|
->addArgument(
|
|
|
|
'app-id',
|
|
|
|
InputArgument::REQUIRED,
|
2015-05-05 09:23:59 +03:00
|
|
|
'check the specified app'
|
2015-06-15 13:51:22 +03:00
|
|
|
)
|
|
|
|
->addOption(
|
2015-07-11 00:17:55 +03:00
|
|
|
'checker',
|
|
|
|
'c',
|
|
|
|
InputOption::VALUE_REQUIRED | InputOption::VALUE_IS_ARRAY,
|
2015-07-16 12:47:20 +03:00
|
|
|
'enable the specified checker(s)',
|
2015-07-11 00:17:55 +03:00
|
|
|
[ 'private', 'deprecation', 'strong-comparison' ]
|
2015-01-29 00:08:50 +03:00
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
protected function execute(InputInterface $input, OutputInterface $output) {
|
|
|
|
$appId = $input->getArgument('app-id');
|
2015-07-11 00:17:55 +03:00
|
|
|
|
2015-07-07 16:37:56 +03:00
|
|
|
$checkList = new EmptyCheck();
|
2015-07-11 00:17:55 +03:00
|
|
|
foreach ($input->getOption('checker') as $checker) {
|
|
|
|
if (!isset($this->checkers[$checker])) {
|
|
|
|
throw new \InvalidArgumentException('Invalid checker: '.$checker);
|
|
|
|
}
|
|
|
|
$checkerClass = $this->checkers[$checker];
|
|
|
|
$checkList = new $checkerClass($checkList);
|
2015-06-15 13:51:22 +03:00
|
|
|
}
|
2015-07-11 00:17:55 +03:00
|
|
|
|
2015-07-07 16:37:56 +03:00
|
|
|
$codeChecker = new CodeChecker($checkList);
|
2015-07-07 13:08:12 +03:00
|
|
|
|
2015-01-29 00:08:50 +03:00
|
|
|
$codeChecker->listen('CodeChecker', 'analyseFileBegin', function($params) use ($output) {
|
2015-05-05 14:57:23 +03:00
|
|
|
if(OutputInterface::VERBOSITY_VERBOSE <= $output->getVerbosity()) {
|
|
|
|
$output->writeln("<info>Analysing {$params}</info>");
|
|
|
|
}
|
2015-01-29 00:08:50 +03:00
|
|
|
});
|
2015-05-05 14:57:23 +03:00
|
|
|
$codeChecker->listen('CodeChecker', 'analyseFileFinished', function($filename, $errors) use ($output) {
|
|
|
|
$count = count($errors);
|
|
|
|
|
|
|
|
// show filename if the verbosity is low, but there are errors in a file
|
|
|
|
if($count > 0 && OutputInterface::VERBOSITY_VERBOSE > $output->getVerbosity()) {
|
|
|
|
$output->writeln("<info>Analysing {$filename}</info>");
|
|
|
|
}
|
|
|
|
|
|
|
|
// show error count if there are errros present or the verbosity is high
|
|
|
|
if($count > 0 || OutputInterface::VERBOSITY_VERBOSE <= $output->getVerbosity()) {
|
|
|
|
$output->writeln(" {$count} errors");
|
|
|
|
}
|
|
|
|
usort($errors, function($a, $b) {
|
2015-01-29 00:08:50 +03:00
|
|
|
return $a['line'] >$b['line'];
|
|
|
|
});
|
|
|
|
|
2015-05-05 14:57:23 +03:00
|
|
|
foreach($errors as $p) {
|
2015-01-29 00:08:50 +03:00
|
|
|
$line = sprintf("%' 4d", $p['line']);
|
|
|
|
$output->writeln(" <error>line $line: {$p['disallowedToken']} - {$p['reason']}</error>");
|
|
|
|
}
|
|
|
|
});
|
|
|
|
$errors = $codeChecker->analyse($appId);
|
|
|
|
if (empty($errors)) {
|
|
|
|
$output->writeln('<info>App is compliant - awesome job!</info>');
|
2015-07-11 00:17:55 +03:00
|
|
|
return 0;
|
2015-01-29 00:08:50 +03:00
|
|
|
} else {
|
|
|
|
$output->writeln('<error>App is not compliant</error>');
|
2015-07-07 12:51:06 +03:00
|
|
|
return 101;
|
2015-01-29 00:08:50 +03:00
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|