zicht / z

res/features/bootstrap/FeatureContext.php

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 use Behat\Behat\Context\Context;
 use Behat\Behat\Context\SnippetAcceptingContext;
 use Behat\Gherkin\Node\PyStringNode;
-use Behat\Gherkin\Node\TableNode;
 
 /**
  * Defines application features from the specific context.

src/Zicht/Tool/Application.php

Doc Comments +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
      * Construct the application with the specified name, version and config loader.
      *
      * @param string $name
-     * @param string $version
+     * @param Version $version
      * @param Configuration\ConfigurationLoader $loader
      */
     public function __construct($name, Version $version, ConfigurationLoader $loader = null)

Unused Use Statements -3 removed lines

@@ -6,16 +6,13 @@
 namespace Zicht\Tool;
 
 use Symfony\Component\Console\Application as BaseApplication;
-use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Input\ArrayInput;
 use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputOption;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Input\InputDefinition;
 use Symfony\Component\Console\Output\OutputInterface;
-use Symfony\Component\Yaml\Yaml;
 use Zicht\Version\Version;
-
 use Zicht\Tool\Command as Cmd;
 use Zicht\Tool\Configuration\ConfigurationLoader;
 use Zicht\Tool\Container\Container;

Spacing +11 added lines, -11 removed lines

@@ -87,12 +87,12 @@ 
                     $output->writeln(
                         sprintf(
                             '%s%-40s %s',
-                            ($depth > 0 ? str_repeat('   ', $depth - 1) . '-> ' : ''),
+                            ($depth > 0 ? str_repeat('   ', $depth-1) . '-> ' : ''),
                             '<fg=red;options=bold>' . $e->getMessage() . '</fg=red;options=bold>',
-                            $depth == count($ancestry) -1 ? str_pad('[' . get_class($e) . ']', $maxLength + 15, ' ') : ''
+                            $depth == count($ancestry)-1 ? str_pad('[' . get_class($e) . ']', $maxLength+15, ' ') : ''
                         )
                     );
-                    $depth ++;
+                    $depth++;
                 }
             }
         }
@@ -156,10 +156,10 @@ 
             array(
                 new InputArgument('command', InputArgument::REQUIRED, 'The command to execute'),
 
-                new InputOption('--help',           '-h', InputOption::VALUE_NONE, 'Display this help message.'),
-                new InputOption('--verbose',        '-v|vv|vvv', InputOption::VALUE_NONE, 'Increase the verbosity of messages: 1 for normal output, 2 for more verbose output and 3 for debug'),
-                new InputOption('--no-cache',       '-c', InputOption::VALUE_NONE, 'Force recompilation of container code'),
-                new InputOption('--version',        '-V', InputOption::VALUE_NONE, 'Display this application version.'),
+                new InputOption('--help', '-h', InputOption::VALUE_NONE, 'Display this help message.'),
+                new InputOption('--verbose', '-v|vv|vvv', InputOption::VALUE_NONE, 'Increase the verbosity of messages: 1 for normal output, 2 for more verbose output and 3 for debug'),
+                new InputOption('--no-cache', '-c', InputOption::VALUE_NONE, 'Force recompilation of container code'),
+                new InputOption('--version', '-V', InputOption::VALUE_NONE, 'Display this application version.'),
             )
         );
     }
@@ -195,10 +195,10 @@ 
 
         $container->output = $output;
 
-        $container->set('VERBOSE',  $input->hasParameterOption(array('--verbose', '-v')));
-        $container->set('FORCE',    $input->hasParameterOption(array('--force', '-f')));
-        $container->set('EXPLAIN',  $input->hasParameterOption(array('--explain')));
-        $container->set('DEBUG',    $input->hasParameterOption(array('--debug')));
+        $container->set('VERBOSE', $input->hasParameterOption(array('--verbose', '-v')));
+        $container->set('FORCE', $input->hasParameterOption(array('--force', '-f')));
+        $container->set('EXPLAIN', $input->hasParameterOption(array('--explain')));
+        $container->set('DEBUG', $input->hasParameterOption(array('--debug')));
 
         foreach ($container->getCommands() as $task) {
             $this->add($task);

src/Zicht/Tool/Command/Descriptor/TextDescriptor.php

Doc Comments +1 added lines, -1 removed lines

@@ -70,7 +70,7 @@
      * the former representing the options that are hidden.
      *
      * @param \Symfony\Component\Console\Input\InputDefinition $definition
-     * @return array
+     * @return InputDefinition[]
      */
     public function splitDefinition(InputDefinition $definition)
     {

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 
 use Symfony\Component\Console\Application;
 use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputDefinition;
 use Symfony\Component\Console\Input\InputOption;
 use Symfony\Component\Console\Descriptor\TextDescriptor as BaseDescriptor;

Spacing +5 added lines, -5 removed lines

@@ -37,12 +37,12 @@ 
 
         $this->writeText('<comment>Usage:</comment>', $options);
         $this->writeText("\n");
-        $this->writeText(' '.$synopsis->getSynopsis(), $options);
+        $this->writeText(' ' . $synopsis->getSynopsis(), $options);
         $this->writeText("\n");
 
         if (count($command->getAliases()) > 0) {
             $this->writeText("\n");
-            $this->writeText('<comment>Aliases:</comment> <info>'.implode(', ', $command->getAliases()).'</info>', $options);
+            $this->writeText('<comment>Aliases:</comment> <info>' . implode(', ', $command->getAliases()) . '</info>', $options);
         }
 
         $this->writeText("\n");
@@ -58,7 +58,7 @@ 
         if ($help = $command->getProcessedHelp()) {
             $this->writeText('<comment>Help:</comment>', $options);
             $this->writeText("\n");
-            $this->writeText(' '.str_replace("\n", "\n ", $help), $options);
+            $this->writeText(' ' . str_replace("\n", "\n ", $help), $options);
             $this->writeText("\n");
         }
 
@@ -115,7 +115,7 @@ 
         foreach ($description->getNamespaces() as $namespace) {
             if (!$describedNamespace && ApplicationDescription::GLOBAL_NAMESPACE !== $namespace['id']) {
                 $this->writeText("\n");
-                $this->writeText('<comment>'.$namespace['id'].'</comment>', $options);
+                $this->writeText('<comment>' . $namespace['id'] . '</comment>', $options);
             }
 
             foreach ($namespace['commands'] as $name) {
@@ -151,6 +151,6 @@ 
             $width = strlen($command->getName()) > $width ? strlen($command->getName()) : $width;
         }
 
-        return $width + 2;
+        return $width+2;
     }
 }
\ No newline at end of file

src/Zicht/Tool/Command/TaskCommand.php

Doc Comments +9 added lines

@@ -69,6 +69,10 @@ 
         }
     }
 
+    /**
+     * @param string $shortcut
+     * @param string $help
+     */
     public function addOption($name, $shortcut = null, $mode = null, $help = null, $default = null)
     {
         $helpTag = ($mode === InputOption::VALUE_NONE) ? "--$name" : "--$name=" . strtoupper($name);
@@ -104,6 +108,11 @@ 
     }
 
 
+    /**
+     * @param string $name
+     *
+     * @return string
+     */
     protected function nameToVar($name)
     {
         return str_replace('-', '_', $name);

Spacing +9 added lines, -9 removed lines

@@ -46,11 +46,11 @@ 
         $this->setHelp(trim($help));
 
         foreach ($arguments as $name => $required) {
-            $mode  = $required ? InputArgument::REQUIRED : InputArgument::OPTIONAL;
+            $mode = $required ? InputArgument::REQUIRED : InputArgument::OPTIONAL;
 
             if ($multiple = ('[]' === substr($name, -2))) {
                 $name = substr($name, 0, -2);
-                $mode |=  InputArgument::IS_ARRAY;
+                $mode |= InputArgument::IS_ARRAY;
             }
 
             $this->addArgument($name, $mode);
@@ -87,7 +87,7 @@ 
         $this->setHelp(
             preg_replace_callback(
                 sprintf('/\s*%s: (.*)(\n|$)/', $name),
-                function ($m) use (&$ret) {
+                function($m) use (&$ret) {
                     $ret = trim($m[1]);
                     return '';
                 },
@@ -122,8 +122,8 @@ 
             $ret = preg_replace_callback(
                 '/\[--(no-)?' . $name . '\]/',
                 function() use(&$i, $name) {
-                    if ($i ++ == 0) {
-                        return '[--[no|]-' . $name .']';
+                    if ($i++ == 0) {
+                        return '[--[no|]-' . $name . ']';
                     }
                     return '';
                 },
@@ -143,10 +143,10 @@ 
     protected function configure()
     {
         $this
-            ->addOption('explain',  '',     InputOption::VALUE_NONE,        'Explains the commands that would be executed.')
-            ->addOption('force',    'f',    InputOption::VALUE_NONE,        'Force execution of otherwise skipped tasks.')
-            ->addOption('plugin',   '',     InputOption::VALUE_REQUIRED,    'Load additional plugins on-the-fly')
-            ->addOption('debug',    '',     InputOption::VALUE_NONE,        "Set the debug flag")
+            ->addOption('explain', '', InputOption::VALUE_NONE, 'Explains the commands that would be executed.')
+            ->addOption('force', 'f', InputOption::VALUE_NONE, 'Force execution of otherwise skipped tasks.')
+            ->addOption('plugin', '', InputOption::VALUE_REQUIRED, 'Load additional plugins on-the-fly')
+            ->addOption('debug', '', InputOption::VALUE_NONE, "Set the debug flag")
         ;
     }
 

src/Zicht/Tool/Configuration/PathDefaultFileLocator.php

Doc Comments +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
     /**
      * Construct the locator based on the passed environment variable.
      *
-     * @param array|string $envName
+     * @param string $envName
      * @param array $defaultPaths
      */
     public function __construct($envName, $defaultPaths = array())

src/Zicht/Tool/Container/Container.php

Doc Comments +4 added lines, -4 removed lines

@@ -200,7 +200,7 @@ 
      * @param array $context
      * @param array $path
      * @param bool $require
-     * @return mixed
+     * @return string
      *
      * @throws \RuntimeException
      * @throws \InvalidArgumentException
@@ -222,7 +222,7 @@ 
      *
      * @param array $id
      * @param bool $required
-     * @return mixed
+     * @return string
      *
      * @throws \RuntimeException
      * @throws CircularReferenceException
@@ -389,7 +389,7 @@ 
      * @param string $expression
      * @param string &$code
      *
-     * @return mixed
+     * @return string
      */
     public function evaluate($expression, &$code = null)
     {
@@ -496,7 +496,7 @@ 
      * Execute a command. This is a wrapper for 'exec', so that a task prefixed with '@' can be passed as well.
      *
      * @param string $cmd
-     * @return int
+     * @return string|null
      */
     public function cmd($cmd)
     {

Unused Use Statements -3 removed lines

@@ -9,16 +9,13 @@
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Output\NullOutput;
 use Symfony\Component\Console\Output\OutputInterface;
-
 use Zicht\Tool\Debug;
 use Zicht\Tool\PropertyPath\PropertyAccessor;
 use Zicht\Tool\PluginInterface;
 use Zicht\Tool\Script\Compiler as ScriptCompiler;
-use Zicht\Tool\Script;
 use Zicht\Tool\Util;
 use Zicht\Tool\Script\Parser\Expression as ExpressionParser;
 use Zicht\Tool\Script\Tokenizer\Expression as ExpressionTokenizer;
-
 use UnexpectedValueException;
 
 /**

Spacing +10 added lines, -10 removed lines

@@ -67,7 +67,7 @@ 
                 $opts = array();
                 foreach (array('FORCE', 'VERBOSE', 'EXPLAIN', 'DEBUG') as $opt) {
                     if ($z->has($opt) && $z->get($opt)) {
-                        $opts[]= '--' . strtolower($opt);
+                        $opts[] = '--' . strtolower($opt);
                     }
                 }
                 return join(' ', $opts);
@@ -76,14 +76,14 @@ 
         $this->set(array('z', 'cmd'), $_SERVER['argv'][0]);
         $this->decl(
             'STDIN',
-            function () {
+            function() {
                 return stream_get_contents(STDIN);
             }
         );
         $this->fn('confirm', function() {
             return false;
         });
-        $this->set('cwd',  getcwd());
+        $this->set('cwd', getcwd());
         $this->set('user', getenv('USER'));
 
 
@@ -99,7 +99,7 @@ 
         $this->fn('str_replace', 'str_replace');
         $this->fn(
             'sha1',
-            function () {
+            function() {
                 return sha1(join("", func_get_args()));
             }
         );
@@ -156,7 +156,7 @@ 
 
         // -----------------------------------------------------------------
         // encoding / decoding
-        $this->fn('json_encode', function ($v) {
+        $this->fn('json_encode', function($v) {
             return json_encode($v, JSON_UNESCAPED_SLASHES);
         });
         $this->fn('json_decode');
@@ -393,7 +393,7 @@ 
      */
     public function evaluate($expression, &$code = null)
     {
-        $exprcompiler  = new ScriptCompiler(new ExpressionParser(), new ExpressionTokenizer());
+        $exprcompiler = new ScriptCompiler(new ExpressionParser(), new ExpressionTokenizer());
 
         $z = $this;
         $_value = null;
@@ -537,10 +537,10 @@ 
     public function str($value)
     {
         if (is_array($value)) {
-            $allScalar = function ($a, $b) {
+            $allScalar = function($a, $b) {
                 return $a && is_scalar($b);
             };
-            if (! array_reduce($value, $allScalar, true)) {
+            if (!array_reduce($value, $allScalar, true)) {
                 throw new UnexpectedValueException("Unexpected complex type " . Util::toPhp($value));
             }
             return join(' ', $value);
@@ -557,7 +557,7 @@ 
      */
     public function addPlugin(PluginInterface $plugin)
     {
-        $this->plugins[]= $plugin;
+        $this->plugins[] = $plugin;
         $plugin->setContainer($this);
     }
 
@@ -570,7 +570,7 @@ 
      */
     public function addCommand(Command $command)
     {
-        $this->commands[]= $command;
+        $this->commands[] = $command;
     }
 
 

src/Zicht/Tool/Container/ContainerBuilder.php

Doc Comments +1 added lines, -1 removed lines

@@ -188,7 +188,7 @@
      *
      * @param array $path
      * @param string $node
-     * @return \Zicht\Tool\Script\Node\Task\OptNode
+     * @return SetNode
      */
     public function createSetNode($path, $node)
     {

Unused Use Statements -1 removed lines

@@ -11,7 +11,6 @@
 use Zicht\Tool\Script\Node\Task\OptNode;
 use Zicht\Tool\Script\Compiler;
 use Zicht\Tool\Script\Node\Task\SetNode;
-use Zicht\Tool\Script\Parser;
 use Zicht\Tool\Debug;
 use Zicht\Tool\Script\Parser\Expression as ExpressionParser;
 use Zicht\Tool\Script\Tokenizer\Expression as ExpressionTokenizer;

Spacing +4 added lines, -4 removed lines

@@ -32,7 +32,7 @@ 
     {
         $this->config = $config;
 
-        $this->exprcompiler  = new Compiler(new ExpressionParser(), new ExpressionTokenizer());
+        $this->exprcompiler = new Compiler(new ExpressionParser(), new ExpressionTokenizer());
         $this->scriptcompiler = new Compiler();
     }
 
@@ -45,7 +45,7 @@ 
      */
     public function addExpressionPath($callable)
     {
-        $this->expressionPaths[]= $callable;
+        $this->expressionPaths[] = $callable;
     }
 
 
@@ -103,7 +103,7 @@ 
             function($path, $node) use($containerNode) {
                 $containerNode->append($node);
             },
-            function ($path, $node) {
+            function($path, $node) {
                 return $node instanceof Node;
             },
             Traverser::AFTER
@@ -271,7 +271,7 @@ 
                 return
                     count($path) == 4
                     && $path[0] == 'tasks'
-                    && in_array($path[2], array('do', 'pre' ,'post'))
+                    && in_array($path[2], array('do', 'pre', 'post'))
                 ;
             },
             Traverser::BEFORE

src/Zicht/Tool/Container/Declaration.php

Doc Comments +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
      * Constructor.
      *
      * @param array $path
-     * @param mixed $value
+     * @param null|Node $value
      */
     public function __construct(array $path, Node $value = null)
     {

src/Zicht/Tool/Container/Executor.php

Doc Comments +1 added lines

@@ -30,6 +30,7 @@
      *
      * @param string $cmd
      * @param null &$captureOutput
+     * @param string $captureOutput
      * @return int
      *
      * @throws ExecutionAbortedException

Unused Use Statements -1 removed lines

@@ -7,7 +7,6 @@
 namespace Zicht\Tool\Container;
 
 use Symfony\Component\Process\Process;
-use Symfony\Component\Process\ProcessBuilder;
 
 /**
  * Runs the commands in the shell.