tarsana / command

src/Commands/HelpCommand.php

Doc Comments +6 added lines

@@ -74,6 +74,9 @@ 
         }
     }
 
+    /**
+     * @param integer $level
+     */
     protected function showSyntax($name, $syntax, $level)
     {
         $type = $this->typeOf($syntax);
@@ -102,6 +105,9 @@ 
         }
     }
 
+    /**
+     * @return integer
+     */
     protected function typeOf(Syntax $syntax)
     {
         if ($syntax instanceof ObjectSyntax) {

Indentation +14 added lines, -14 removed lines

@@ -35,15 +35,15 @@ 
         $c = $this->console();
 
         $c->br()
-          ->green()
-          ->inline($command->name() . ' ')
-          ->yellow()
-          ->out($command->version())
-          ->br()
-          ->out($command->description())
-          ->br()
-          ->yellow()
-          ->inline('Arguments: ');
+            ->green()
+            ->inline($command->name() . ' ')
+            ->yellow()
+            ->out($command->version())
+            ->br()
+            ->out($command->description())
+            ->br()
+            ->yellow()
+            ->inline('Arguments: ');
 
         $args = F\s(array_keys($command->syntax()->fields()))
             ->then(F\join(' '))
@@ -67,9 +67,9 @@ 
                 ->get() * 2;
             foreach ($command->subCommands() as $name => $cmd) {
                 $c->tab()
-                  ->padding($padding)->char(' ')
-                  ->label("<green>{$name}</green>")
-                  ->result($cmd->description());
+                    ->padding($padding)->char(' ')
+                    ->label("<green>{$name}</green>")
+                    ->result($cmd->description());
             }
         }
     }
@@ -86,8 +86,8 @@ 
             $c->out("(Required)");
         } else {
             $c->inline("(default: ")
-              ->white()->inline(json_encode($syntax->getDefault()))
-              ->yellow()->out(" )");
+                ->white()->inline(json_encode($syntax->getDefault()))
+                ->yellow()->out(" )");
         }
 
         if ($syntax instanceof ArraySyntax) {

Spacing +3 added lines, -3 removed lines

@@ -12,7 +12,7 @@ 
 
 class HelpCommand extends SubCommand {
 
-    protected function init ()
+    protected function init()
     {
         $this
             ->name('Help')
@@ -21,7 +21,7 @@ 
             ->syntax('[command]');
     }
 
-    protected function execute ()
+    protected function execute()
     {
         $command = trim($this->args->command);
         if (empty($command) || !$this->parent()->hasCommand($command))
@@ -30,7 +30,7 @@ 
             $this->showHelpOf($this->parent->command($command));
     }
 
-    protected function showHelpOf (Command $command)
+    protected function showHelpOf(Command $command)
     {
         $c = $this->console();
 

Braces +5 added lines, -4 removed lines

@@ -24,10 +24,11 @@
     protected function execute ()
     {
         $command = trim($this->args->command);
-        if (empty($command) || !$this->parent()->hasCommand($command))
-            $this->showHelpOf($this->parent);
-        else
-            $this->showHelpOf($this->parent->command($command));
+        if (empty($command) || !$this->parent()->hasCommand($command)) {
+                    $this->showHelpOf($this->parent);
+        } else {
+                    $this->showHelpOf($this->parent->command($command));
+        }
     }
 
     protected function showHelpOf (Command $command)

src/SubCommand.php

Unused Use Statements -1 removed lines

@@ -1,7 +1,6 @@
 <?php namespace Tarsana\Command;
 
 use Tarsana\Command\Command;
-use Tarsana\Command\Commands\HelpCommand;
 
 /**
  * A SubCommand which has a parent command and can access it.

src/Syntax/SyntaxBuilder.php

Doc Comments +5 added lines, -5 removed lines

@@ -23,7 +23,7 @@ 
     /**
      * A constructor alias; If you don't like the 'new' keyword.
      * @param  string|Tarsana\Syntax\ObjectSyntax $syntax
-     * @return Tarsana\Command\Syntax\SyntaxBuilder
+     * @return SyntaxBuilder
      * @throws Tarsana\Command\Exceptions\SyntaxBuilderException
      */
     public static function of($syntax)
@@ -88,7 +88,7 @@ 
      * @param  string $name
      * @param  string $description
      * @param  mixed  $default
-     * @return Tarsana\Command\Syntax\SyntaxBuilder
+     * @return SyntaxBuilder
      */
     public function describe($name, $description, $default = null)
     {
@@ -104,7 +104,7 @@ 
     /**
      * Gets a field or subfield from an Syntax.
      *
-     * @param  Tarsana\Syntax\Syntax $syntax
+     * @param  Syntax $syntax
      * @param  array $names
      * @return Tarsana\Syntax\Syntax
      * @throws Tarsana\Command\Exceptions\SyntaxBuilderException
@@ -122,8 +122,8 @@ 
      * Ensures that `$syntax` is an ObjectSyntax or an ArraySyntax of ObjectSyntaxes,
      * and returns the ObjectSyntax or throws a SyntaxBuilderException.
      *
-     * @param  Tarsana\Syntax\Syntax $syntax
-     * @return Tarsana\Syntax\Syntax
+     * @param  Syntax $syntax
+     * @return ObjectSyntax
      * @throws Tarsana\Command\Exceptions\SyntaxBuilderException
      */
     protected function ensureObject(Syntax $syntax)

Spacing +6 added lines, -7 removed lines

@@ -41,7 +41,7 @@ 
     {
         if (is_string($syntax)) {
             $syntax = self::syntaxFromString($syntax);
-        } else if (! ($syntax instanceof ObjectSyntax)) {
+        } else if (!($syntax instanceof ObjectSyntax)) {
             throw new SyntaxBuilderException("The command syntax should be an insatnce of ObjectSyntax");
         }
 
@@ -65,11 +65,11 @@ 
      * @return Tarsana\Syntax\Syntax
      * @throws Tarsana\Command\Exceptions\SyntaxBuilderException
      */
-    protected static function syntaxFromString ($value)
+    protected static function syntaxFromString($value)
     {
         $syntax = F\s($value)->then(        // "first name" [#age]
-            F\chunks('(){}[]""', ' '),      // ["first name", [#age]]
-            F\join(','),                    // "first name",[#age]
+            F\chunks('(){}[]""', ' '), // ["first name", [#age]]
+            F\join(','), // "first name",[#age]
             F\prepend('{ ,'), F\append('}') // { ,"first name",[#age]}
         )->get();
 
@@ -111,8 +111,7 @@ 
      */
     protected function field(Syntax $syntax, $names)
     {
-        return empty($names) ? $syntax :
-            $this->field(
+        return empty($names) ? $syntax : $this->field(
                 $this->ensureObject($syntax)->field(F\head($names)),
                 F\tail($names)
             );
@@ -131,7 +130,7 @@ 
         while ($syntax instanceof ArraySyntax) {
             $syntax = $syntax->itemSyntax();
         }
-        if (! ($syntax instanceof ObjectSyntax)) {
+        if (!($syntax instanceof ObjectSyntax)) {
             throw new SyntaxBuilderException("Could not retreive field of non object syntax '{$syntax}'");
         }
         return $syntax;

Braces +3 added lines, -2 removed lines

@@ -75,8 +75,9 @@
 
         $ss = S::syntax();
 
-        if (!$ss->canParse($syntax))
-            throw new SyntaxBuilderException("Invalid Syntax: '{$syntax}'");
+        if (!$ss->canParse($syntax)) {
+                    throw new SyntaxBuilderException("Invalid Syntax: '{$syntax}'");
+        }
 
         return $ss->parse($syntax);
     }

src/TemplateLoader.php

Doc Comments +1 added lines, -1 removed lines

@@ -58,7 +58,7 @@
     /**
      * Filesystem getter and setter.
      *
-     * @param  Tarsana\IO\Filesystem|void $value
+     * @param  null|Filesystem $value
      * @return Tarsana\IO\Filesystem|self
      */
     public function fs (Filesystem $value = null)

Spacing +2 added lines, -2 removed lines

@@ -61,7 +61,7 @@ 
      * @param  Tarsana\IO\Filesystem|void $value
      * @return Tarsana\IO\Filesystem|self
      */
-    public function fs (Filesystem $value = null)
+    public function fs(Filesystem $value = null)
     {
         if (null === $value)
             return $this->fs;
@@ -81,7 +81,7 @@ 
      * @throws Tarsana\Command\Exceptions\TemplateNotFound
      * @throws Tarsana\Command\Exceptions\TemplateNameConflict
      */
-    public function load ($name)
+    public function load($name)
     {
         $suportedExtensions = array_keys(self::$providers);
 

Braces +3 added lines, -2 removed lines

@@ -63,8 +63,9 @@
      */
     public function fs (Filesystem $value = null)
     {
-        if (null === $value)
-            return $this->fs;
+        if (null === $value) {
+                    return $this->fs;
+        }
         $this->fs = $value;
         return $this;
     }

src/Templates/TwigTemplateLoader.php

Doc Comments +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
      * Loads a template.
      *
      * @param  string $name
-     * @return Tarsana\Command\Templates\TwigTemplate
+     * @return TwigTemplate
      */
     public function load($name)
     {

src/Templates/TwigTemplate.php

Spacing +3 added lines, -3 removed lines

@@ -21,7 +21,7 @@ 
      */
     protected $data;
 
-    public function __construct (\Twig_TemplateInterface $twig)
+    public function __construct(\Twig_TemplateInterface $twig)
     {
         $this->twig = $twig;
         $this->data = [];
@@ -33,7 +33,7 @@ 
      * @param  array $data
      * @return self
      */
-    public function bind ($data)
+    public function bind($data)
     {
         $this->data = array_merge($this->data, $data);
         return $this;
@@ -57,7 +57,7 @@ 
      *
      * @return self
      */
-    public function clean ()
+    public function clean()
     {
         $this->data = [];
     }

src/Environment.php

Spacing +1 added lines, -1 removed lines

@@ -73,7 +73,7 @@
     public function command($name, $value = null)
     {
         if (null === $value) {
-            if (! $this->hasCommand($name)) {
+            if (!$this->hasCommand($name)) {
                 throw new EnvironmentException("Unable to find the command '{$name}'");
             }
             return $this->store->get($name);

src/Commands/VersionCommand.php

Indentation +3 added lines, -3 removed lines

@@ -19,8 +19,8 @@
     {
         $c = $this->parent();
         $this->console
-             ->green()->inline($c->name())
-             ->white()->inline(' version ')
-             ->yellow()->out($c->version());
+                ->green()->inline($c->name())
+                ->white()->inline(' version ')
+                ->yellow()->out($c->version());
     }
 }

Spacing +2 added lines, -2 removed lines

@@ -7,7 +7,7 @@ 
 
 class VersionCommand extends SubCommand {
 
-    protected function init ()
+    protected function init()
     {
         $this
             ->name('Version')
@@ -15,7 +15,7 @@ 
             ->description('Shows the version of the command');
     }
 
-    protected function execute ()
+    protected function execute()
     {
         $c = $this->parent();
         $this->console

src/Utils/Store.php

Spacing +3 added lines, -3 removed lines

@@ -71,7 +71,7 @@ 
     public function store($name, $value)
     {
         // Validates the name
-        if (! is_string($name)) {
+        if (!is_string($name)) {
             $this->error('The name must be a string');
         }
         if ($this->contains($name)) {
@@ -80,7 +80,7 @@ 
 
         // Validates the value
         if (is_string($value)) {
-            if (! class_exists($value)) {
+            if (!class_exists($value)) {
                 $this->error("Trying to store an unknow class '{$value}'");
             }
             if ($value != $this->type && !is_subclass_of($value, $this->type)) {
@@ -120,7 +120,7 @@ 
      */
     public function get($name)
     {
-        if (! $this->contains($name)) {
+        if (!$this->contains($name)) {
             $this->error("The element with name '{$name}' is missing");
         }