tarsana / command

src/Console/Console.php

Unused Use Statements -1 removed lines

@@ -7,7 +7,6 @@
 use Tarsana\IO\Interfaces\Resource\Writer as WriterInterface;
 use Tarsana\IO\Resource\Reader;
 use Tarsana\IO\Resource\Writer;
-use Tarsana\Syntax\Syntax;
 
 class Console implements ConsoleInterface {
 

src/Console/ExceptionPrinter.php

Braces +18 added lines, -10 removed lines

@@ -18,8 +18,9 @@ 
      */
     public function print(\Exception $e) : string
     {
-        if ($e instanceof ParseException)
-            return $this->printParseException($e);
+        if ($e instanceof ParseException) {
+                    return $this->printParseException($e);
+        }
 
         return "<error>{$e}</error>";
     }
@@ -36,10 +37,12 @@ 
         $error  = '';
         if ($syntax instanceof ObjectSyntax) {
             $i = $e->extra();
-            if ($i['type'] == 'invalid-field')
-                $error = "{$i['field']} is invalid!";
-            if ($i['type'] == 'missing-field')
-                $error = "{$i['field']} is missing!";
+            if ($i['type'] == 'invalid-field') {
+                            $error = "{$i['field']} is invalid!";
+            }
+            if ($i['type'] == 'missing-field') {
+                            $error = "{$i['field']} is missing!";
+            }
             if ($i['type'] == 'additional-items') {
                 $items = implode(', ', $i['items']);
                 $error = "additional items {$items}";
@@ -48,8 +51,9 @@ 
         $syntax = $this->printSyntax($e->syntax());
 
         $output = "<reset>Failed to parse <warn>'{$e->input()}'</warn> as <info>{$syntax}</info>";
-        if ('' != $error)
-            $output .= " <error>{$error}</error>";
+        if ('' != $error) {
+                    $output .= " <error>{$error}</error>";
+        }
 
         $previous = $e->previous();
         if ($previous) {
@@ -62,11 +66,15 @@ 
     protected function printSyntax(Syntax $s, bool $short = false) : string
     {
         if ($s instanceof ObjectSyntax) {
-            if ($short) return 'object';
+            if ($short) {
+                return 'object';
+            }
             return implode($s->separator(), array_keys($s->fields()));
         }
         if ($s instanceof ArraySyntax) {
-            if ($short) return 'array';
+            if ($short) {
+                return 'array';
+            }
             return $this->printSyntax($s->syntax()).$s->separator().'...';
         }
         return (string) $s;

src/Command.php

Braces +37 added lines, -26 removed lines

@@ -35,8 +35,9 @@ 
 
     public static function create(callable $action = null) {
         $command = new Command;
-        if (null !== $action)
-            $command->action($action);
+        if (null !== $action) {
+                    $command->action($action);
+        }
         return $command;
     }
 
@@ -122,8 +123,9 @@ 
      */
     public function syntax(string $syntax = null)
     {
-        if (null === $syntax)
-            return $this->syntax;
+        if (null === $syntax) {
+                    return $this->syntax;
+        }
 
         $this->syntax = S::syntax()->parse("{{$syntax}| }");
         return $this;
@@ -142,8 +144,9 @@ 
         }
 
         $this->options = [];
-        foreach($options as $option)
-            $this->options[$option] = false;
+        foreach($options as $option) {
+                    $this->options[$option] = false;
+        }
 
         return $this;
     }
@@ -156,8 +159,9 @@ 
      */
     public function option(string $name)
     {
-        if (!array_key_exists($name, $this->options))
-            throw new \InvalidArgumentException("Unknown option '{$name}'");
+        if (!array_key_exists($name, $this->options)) {
+                    throw new \InvalidArgumentException("Unknown option '{$name}'");
+        }
         return $this->options[$name];
     }
 
@@ -205,8 +209,9 @@ 
         if (null === $value) {
             return $this->fs;
         }
-        if (is_string($value))
-            $value = new Filesystem($value);
+        if (is_string($value)) {
+                    $value = new Filesystem($value);
+        }
         $this->fs = $value;
         foreach ($this->commands as $name => $command) {
             $command->fs = $value;
@@ -235,8 +240,9 @@ 
     }
 
     public function template(string $name) {
-        if (null === $this->templatesLoader)
-            throw new \Exception("Please initialize the templates loader before trying to load templates!");
+        if (null === $this->templatesLoader) {
+                    throw new \Exception("Please initialize the templates loader before trying to load templates!");
+        }
         return $this->templatesLoader->load($name);
     }
 
@@ -258,8 +264,9 @@ 
     public function command(string $name, Command $command = null)
     {
         if (null === $command) {
-            if (!array_key_exists($name, $this->commands))
-                throw new \InvalidArgumentException("subcommand '{$name}' not found!");
+            if (!array_key_exists($name, $this->commands)) {
+                            throw new \InvalidArgumentException("subcommand '{$name}' not found!");
+            }
             return $this->commands[$name];
         }
         $this->commands[$name] = $command;
@@ -274,9 +281,10 @@ 
 
     public function describe(string $name, string $description = null)
     {
-        if (null === $description)
-            return array_key_exists($name, $this->descriptions)
+        if (null === $description) {
+                    return array_key_exists($name, $this->descriptions)
                 ? $this->descriptions[$name] : '';
+        }
         if (substr($name, 0, 2) == '--' && array_key_exists($name, $this->options())) {
             $this->descriptions[$name] = $description;
             return $this;
@@ -312,16 +320,18 @@ 
             } else {
                 $this->args = (object) $args;
                 foreach ($options as $name) {
-                    if (!array_key_exists($name, $this->options))
-                        throw new \Exception("Unknown option '{$name}'");
+                    if (!array_key_exists($name, $this->options)) {
+                                            throw new \Exception("Unknown option '{$name}'");
+                    }
                     $this->options[$name] = true;
                 }
             }
 
-            if (null === $this->action)
-                $this->execute();
-            else
-                ($this->action)($this);
+            if (null === $this->action) {
+                            $this->execute();
+            } else {
+                            ($this->action)($this);
+            }
         } catch (\Exception $e) {
             $this->handleError($e);
         }
@@ -344,10 +354,11 @@ 
 
         $arguments = [];
         foreach ($args as &$arg) {
-            if (array_key_exists($arg, $this->options))
-                $this->options[$arg] = true;
-            else
-                $arguments[] = $arg;
+            if (array_key_exists($arg, $this->options)) {
+                            $this->options[$arg] = true;
+            } else {
+                            $arguments[] = $arg;
+            }
         }
         $arguments = T::join($arguments, ' ');
         $this->args = $this->syntax->parse($arguments);