Updated the styles of the Twig commands
This commit is contained in:
parent
bee1faaa95
commit
fdeccd5849
@ -16,6 +16,7 @@ use Symfony\Component\Console\Input\InputArgument;
|
|||||||
use Symfony\Component\Console\Input\InputOption;
|
use Symfony\Component\Console\Input\InputOption;
|
||||||
use Symfony\Component\Console\Input\InputInterface;
|
use Symfony\Component\Console\Input\InputInterface;
|
||||||
use Symfony\Component\Console\Output\OutputInterface;
|
use Symfony\Component\Console\Output\OutputInterface;
|
||||||
|
use Symfony\Component\Console\Style\SymfonyStyle;
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Lists twig functions, filters, globals and tests present in the current project.
|
* Lists twig functions, filters, globals and tests present in the current project.
|
||||||
@ -82,10 +83,11 @@ EOF
|
|||||||
|
|
||||||
protected function execute(InputInterface $input, OutputInterface $output)
|
protected function execute(InputInterface $input, OutputInterface $output)
|
||||||
{
|
{
|
||||||
|
$output = new SymfonyStyle($input, $output);
|
||||||
$twig = $this->getTwigEnvironment();
|
$twig = $this->getTwigEnvironment();
|
||||||
|
|
||||||
if (null === $twig) {
|
if (null === $twig) {
|
||||||
$output->writeln('<error>The Twig environment needs to be set.</error>');
|
$output->error('The Twig environment needs to be set.');
|
||||||
|
|
||||||
return 1;
|
return 1;
|
||||||
}
|
}
|
||||||
@ -118,14 +120,11 @@ EOF
|
|||||||
if (!$items) {
|
if (!$items) {
|
||||||
continue;
|
continue;
|
||||||
}
|
}
|
||||||
if ($index > 0) {
|
|
||||||
$output->writeln('');
|
$output->section(ucfirst($type));
|
||||||
}
|
|
||||||
$output->writeln('<info>'.ucfirst($type).'</info>');
|
|
||||||
ksort($items);
|
ksort($items);
|
||||||
foreach ($items as $item) {
|
$output->listing($items);
|
||||||
$output->writeln(' '.$item);
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
return 0;
|
return 0;
|
||||||
|
@ -16,6 +16,7 @@ use Symfony\Component\Console\Input\InputArgument;
|
|||||||
use Symfony\Component\Console\Input\InputInterface;
|
use Symfony\Component\Console\Input\InputInterface;
|
||||||
use Symfony\Component\Console\Input\InputOption;
|
use Symfony\Component\Console\Input\InputOption;
|
||||||
use Symfony\Component\Console\Output\OutputInterface;
|
use Symfony\Component\Console\Output\OutputInterface;
|
||||||
|
use Symfony\Component\Console\Style\SymfonyStyle;
|
||||||
use Symfony\Component\Finder\Finder;
|
use Symfony\Component\Finder\Finder;
|
||||||
|
|
||||||
/**
|
/**
|
||||||
@ -84,14 +85,17 @@ EOF
|
|||||||
|
|
||||||
protected function execute(InputInterface $input, OutputInterface $output)
|
protected function execute(InputInterface $input, OutputInterface $output)
|
||||||
{
|
{
|
||||||
|
$stdout = $output;
|
||||||
|
$output = new SymfonyStyle($input, $output);
|
||||||
|
|
||||||
if (false !== strpos($input->getFirstArgument(), ':l')) {
|
if (false !== strpos($input->getFirstArgument(), ':l')) {
|
||||||
$output->writeln('<comment>The use of "twig:lint" command is deprecated since version 2.7 and will be removed in 3.0. Use the "lint:twig" instead.</comment>');
|
$output->caution('The use of "twig:lint" command is deprecated since version 2.7 and will be removed in 3.0. Use the "lint:twig" instead.');
|
||||||
}
|
}
|
||||||
|
|
||||||
$twig = $this->getTwigEnvironment();
|
$twig = $this->getTwigEnvironment();
|
||||||
|
|
||||||
if (null === $twig) {
|
if (null === $twig) {
|
||||||
$output->writeln('<error>The Twig environment needs to be set.</error>');
|
$output->error('The Twig environment needs to be set.');
|
||||||
|
|
||||||
return 1;
|
return 1;
|
||||||
}
|
}
|
||||||
@ -108,12 +112,12 @@ EOF
|
|||||||
$template .= fread(STDIN, 1024);
|
$template .= fread(STDIN, 1024);
|
||||||
}
|
}
|
||||||
|
|
||||||
return $this->display($input, $output, array($this->validate($twig, $template, uniqid('sf_'))));
|
return $this->display($input, $stdout, $output, array($this->validate($twig, $template, uniqid('sf_'))));
|
||||||
}
|
}
|
||||||
|
|
||||||
$filesInfo = $this->getFilesInfo($twig, $filenames);
|
$filesInfo = $this->getFilesInfo($twig, $filenames);
|
||||||
|
|
||||||
return $this->display($input, $output, $filesInfo);
|
return $this->display($input, $stdout, $output, $filesInfo);
|
||||||
}
|
}
|
||||||
|
|
||||||
private function getFilesInfo(\Twig_Environment $twig, array $filenames)
|
private function getFilesInfo(\Twig_Environment $twig, array $filenames)
|
||||||
@ -157,32 +161,36 @@ EOF
|
|||||||
return array('template' => $template, 'file' => $file, 'valid' => true);
|
return array('template' => $template, 'file' => $file, 'valid' => true);
|
||||||
}
|
}
|
||||||
|
|
||||||
private function display(InputInterface $input, OutputInterface $output, $files)
|
private function display(InputInterface $input, OutputInterface $stdout, $output, $files)
|
||||||
{
|
{
|
||||||
switch ($input->getOption('format')) {
|
switch ($input->getOption('format')) {
|
||||||
case 'txt':
|
case 'txt':
|
||||||
return $this->displayTxt($output, $files);
|
return $this->displayTxt($stdout, $output, $files);
|
||||||
case 'json':
|
case 'json':
|
||||||
return $this->displayJson($output, $files);
|
return $this->displayJson($stdout, $files);
|
||||||
default:
|
default:
|
||||||
throw new \InvalidArgumentException(sprintf('The format "%s" is not supported.', $input->getOption('format')));
|
throw new \InvalidArgumentException(sprintf('The format "%s" is not supported.', $input->getOption('format')));
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
private function displayTxt(OutputInterface $output, $filesInfo)
|
private function displayTxt(OutputInterface $stdout, $output, $filesInfo)
|
||||||
{
|
{
|
||||||
$errors = 0;
|
$errors = 0;
|
||||||
|
|
||||||
foreach ($filesInfo as $info) {
|
foreach ($filesInfo as $info) {
|
||||||
if ($info['valid'] && $output->isVerbose()) {
|
if ($info['valid'] && $stdout->isVerbose()) {
|
||||||
$output->writeln('<info>OK</info>'.($info['file'] ? sprintf(' in %s', $info['file']) : ''));
|
$output->comment('<info>OK</info>'.($info['file'] ? sprintf(' in %s', $info['file']) : ''));
|
||||||
} elseif (!$info['valid']) {
|
} elseif (!$info['valid']) {
|
||||||
++$errors;
|
++$errors;
|
||||||
$this->renderException($output, $info['template'], $info['exception'], $info['file']);
|
$this->renderException($output, $info['template'], $info['exception'], $info['file']);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
$output->writeln(sprintf('<comment>%d/%d valid files</comment>', count($filesInfo) - $errors, count($filesInfo)));
|
if ($errors === 0) {
|
||||||
|
$output->success(sprintf('All %d Twig files contain valid syntax.', count($filesInfo)));
|
||||||
|
} else {
|
||||||
|
$output->warning(sprintf('%d Twig files have valid syntax and %d contain errors.', count($filesInfo) - $errors, $errors));
|
||||||
|
}
|
||||||
|
|
||||||
return min($errors, 1);
|
return min($errors, 1);
|
||||||
}
|
}
|
||||||
@ -211,20 +219,20 @@ EOF
|
|||||||
$line = $exception->getTemplateLine();
|
$line = $exception->getTemplateLine();
|
||||||
|
|
||||||
if ($file) {
|
if ($file) {
|
||||||
$output->writeln(sprintf('<error>KO</error> in %s (line %s)', $file, $line));
|
$output->text(sprintf('<error> ERROR </error> in %s (line %s)', $file, $line));
|
||||||
} else {
|
} else {
|
||||||
$output->writeln(sprintf('<error>KO</error> (line %s)', $line));
|
$output->text(sprintf('<error> ERROR </error> (line %s)', $line));
|
||||||
}
|
}
|
||||||
|
|
||||||
foreach ($this->getContext($template, $line) as $no => $code) {
|
foreach ($this->getContext($template, $line) as $lineNumber => $code) {
|
||||||
$output->writeln(sprintf(
|
$output->text(sprintf(
|
||||||
'%s %-6s %s',
|
'%s %-6s %s',
|
||||||
$no == $line ? '<error>>></error>' : ' ',
|
$lineNumber === $line ? '<error> >> </error>' : ' ',
|
||||||
$no,
|
$lineNumber,
|
||||||
$code
|
$code
|
||||||
));
|
));
|
||||||
if ($no == $line) {
|
if ($lineNumber === $line) {
|
||||||
$output->writeln(sprintf('<error>>> %s</error> ', $exception->getRawMessage()));
|
$output->text(sprintf('<error> >> %s</error> ', $exception->getRawMessage()));
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
@ -31,7 +31,7 @@ class LintCommandTest extends \PHPUnit_Framework_TestCase
|
|||||||
$ret = $tester->execute(array('filename' => array($filename)), array('verbosity' => OutputInterface::VERBOSITY_VERBOSE, 'decorated' => false));
|
$ret = $tester->execute(array('filename' => array($filename)), array('verbosity' => OutputInterface::VERBOSITY_VERBOSE, 'decorated' => false));
|
||||||
|
|
||||||
$this->assertEquals(0, $ret, 'Returns 0 in case of success');
|
$this->assertEquals(0, $ret, 'Returns 0 in case of success');
|
||||||
$this->assertRegExp('/^OK in /', $tester->getDisplay());
|
$this->assertRegExp('/^\/\/ OK in /', trim($tester->getDisplay()));
|
||||||
}
|
}
|
||||||
|
|
||||||
public function testLintIncorrectFile()
|
public function testLintIncorrectFile()
|
||||||
@ -42,7 +42,7 @@ class LintCommandTest extends \PHPUnit_Framework_TestCase
|
|||||||
$ret = $tester->execute(array('filename' => array($filename)), array('decorated' => false));
|
$ret = $tester->execute(array('filename' => array($filename)), array('decorated' => false));
|
||||||
|
|
||||||
$this->assertEquals(1, $ret, 'Returns 1 in case of error');
|
$this->assertEquals(1, $ret, 'Returns 1 in case of error');
|
||||||
$this->assertRegExp('/^KO in /', $tester->getDisplay());
|
$this->assertRegExp('/^ERROR in /', trim($tester->getDisplay()));
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
@ -65,7 +65,7 @@ class LintCommandTest extends \PHPUnit_Framework_TestCase
|
|||||||
$ret = $tester->execute(array('filename' => array($filename)), array('decorated' => false));
|
$ret = $tester->execute(array('filename' => array($filename)), array('decorated' => false));
|
||||||
|
|
||||||
$this->assertEquals(1, $ret, 'Returns 1 in case of error');
|
$this->assertEquals(1, $ret, 'Returns 1 in case of error');
|
||||||
$this->assertRegExp('/^KO in /', $tester->getDisplay());
|
$this->assertRegExp('/^ERROR in /', trim($tester->getDisplay()));
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
@ -14,6 +14,7 @@ namespace Symfony\Bundle\TwigBundle\Command;
|
|||||||
use Symfony\Bridge\Twig\Command\DebugCommand as BaseDebugCommand;
|
use Symfony\Bridge\Twig\Command\DebugCommand as BaseDebugCommand;
|
||||||
use Symfony\Component\Console\Input\InputInterface;
|
use Symfony\Component\Console\Input\InputInterface;
|
||||||
use Symfony\Component\Console\Output\OutputInterface;
|
use Symfony\Component\Console\Output\OutputInterface;
|
||||||
|
use Symfony\Component\Console\Style\SymfonyStyle;
|
||||||
use Symfony\Component\DependencyInjection\ContainerInterface;
|
use Symfony\Component\DependencyInjection\ContainerInterface;
|
||||||
use Symfony\Component\DependencyInjection\ContainerAwareInterface;
|
use Symfony\Component\DependencyInjection\ContainerAwareInterface;
|
||||||
|
|
||||||
@ -57,8 +58,9 @@ class DebugCommand extends BaseDebugCommand implements ContainerAwareInterface
|
|||||||
|
|
||||||
protected function execute(InputInterface $input, OutputInterface $output)
|
protected function execute(InputInterface $input, OutputInterface $output)
|
||||||
{
|
{
|
||||||
|
$output = new SymfonyStyle($input, $output);
|
||||||
if (false !== strpos($input->getFirstArgument(), ':d')) {
|
if (false !== strpos($input->getFirstArgument(), ':d')) {
|
||||||
$output->writeln('<comment>The use of "twig:debug" command is deprecated since version 2.7 and will be removed in 3.0. Use the "debug:twig" instead.</comment>');
|
$output->caution('The use of "twig:debug" command is deprecated since version 2.7 and will be removed in 3.0. Use the "debug:twig" instead.');
|
||||||
}
|
}
|
||||||
|
|
||||||
parent::execute($input, $output);
|
parent::execute($input, $output);
|
||||||
|
Reference in New Issue
Block a user