tarsana / command

src/Commands/HelpCommand.php

Braces +20 added lines, -14 removed lines

@@ -35,22 +35,27 @@ 
 
     protected function formatSyntax(Syntax $s) : string
     {
-        if ($s instanceof ObjectSyntax)
-            return implode($s->separator(), array_keys($s->fields()));
-        if ($s instanceof ArraySyntax)
-            return $this->formatSyntax($s->syntax()) . $s->separator() . '...';
-        if ($s instanceof OptionalSyntax)
-            return $this->formatSyntax($s->syntax());
+        if ($s instanceof ObjectSyntax) {
+                    return implode($s->separator(), array_keys($s->fields()));
+        }
+        if ($s instanceof ArraySyntax) {
+                    return $this->formatSyntax($s->syntax()) . $s->separator() . '...';
+        }
+        if ($s instanceof OptionalSyntax) {
+                    return $this->formatSyntax($s->syntax());
+        }
 
         return (string) $s;
     }
 
     protected function getFields(Syntax $s) : array
     {
-        if ($s instanceof ObjectSyntax)
-            return $s->fields();
-        if ($s instanceof ArraySyntax || $s instanceof OptionalSyntax)
-            return $this->getFields($s->syntax());
+        if ($s instanceof ObjectSyntax) {
+                    return $s->fields();
+        }
+        if ($s instanceof ArraySyntax || $s instanceof OptionalSyntax) {
+                    return $this->getFields($s->syntax());
+        }
         return [];
     }
 
@@ -58,10 +63,11 @@ 
     {
         $tabs = str_repeat('<tab>', $level);
         $optional = ($s instanceof OptionalSyntax);
-        if ($optional)
-            $default = 'default: ' . json_encode($s->getDefault());
-        else
-            $default = 'required';
+        if ($optional) {
+                    $default = 'default: ' . json_encode($s->getDefault());
+        } else {
+                    $default = 'required';
+        }
         $description = $this->parent()->describe($prefix.$name);
         $syntax = $this->formatSyntax($s);
         $this->console()->line("{$tabs}<warn>{$name}</warn> <success>{$syntax}</success> {$description} <info>({$default})</info>");

src/Template/TemplateLoader.php

Braces +3 added lines, -2 removed lines

@@ -87,8 +87,9 @@
         $found = [];
         foreach ($files as $file) {
             $ext = $file->extension();
-            if (!in_array($ext, $supportedExtensions))
-                continue;
+            if (!in_array($ext, $supportedExtensions)) {
+                            continue;
+            }
             $found[] = [
                 'name' => substr($file->path(), $fsPathLength),
                 'extension' => $ext

src/Console/Console.php

Braces +9 added lines, -6 removed lines

@@ -41,24 +41,27 @@
 
     public function stdin(ReaderInterface $in = null)
     {
-        if (null === $in)
-            return $this->in;
+        if (null === $in) {
+                    return $this->in;
+        }
         $this->in = $in;
         return $this;
     }
 
     public function stdout(WriterInterface $out = null)
     {
-        if (null === $out)
-            return $this->out;
+        if (null === $out) {
+                    return $this->out;
+        }
         $this->out = $out;
         return $this;
     }
 
     public function stderr(WriterInterface $err = null)
     {
-        if (null === $err)
-            return $this->err;
+        if (null === $err) {
+                    return $this->err;
+        }
         $this->err = $err;
         return $this;
     }

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/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);

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($syntax->separator(), $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;