Completed
Push — master ( e8fa7c...f772f6 )
by Amine
03:09
created
src/Command.php 2 patches
Doc Comments   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -185,7 +185,7 @@  discard block
 block discarded – undo
185 185
      * @param  string $name
186 186
      * @param  string $description
187 187
      * @param  mixed  $default
188
-     * @return Tarsana\Command\Syntax\SyntaxBuilder
188
+     * @return Command
189 189
      */
190 190
     public function describe($name, $description, $default = null)
191 191
     {
@@ -212,7 +212,7 @@  discard block
 block discarded – undo
212 212
      * adds/overrides or gets a sub command.
213 213
      *
214 214
      * @param  string  $name
215
-     * @param  Tarsana\Command\Command|null $cmd
215
+     * @param  null|Command $cmd
216 216
      * @return Tarsana\Command\Command
217 217
      */
218 218
     public function command ($name, Command $cmd = null)
@@ -299,7 +299,7 @@  discard block
 block discarded – undo
299 299
      * Runs the command.
300 300
      *
301 301
      * @param  string|null $args
302
-     * @param  League\CLImate\CLImate|null $console
302
+     * @param  null|CLImate $console
303 303
      * @return void
304 304
      */
305 305
     public function run ($args = null, CLImate $console = null)
Please login to merge, or discard this patch.
Braces   +33 added lines, -24 removed lines patch added patch discarded remove patch
@@ -109,8 +109,9 @@  discard block
 block discarded – undo
109 109
      */
110 110
     public function name ($value = null)
111 111
     {
112
-        if (null === $value)
113
-            return $this->name;
112
+        if (null === $value) {
113
+                    return $this->name;
114
+        }
114 115
 
115 116
         $this->name = $value;
116 117
         return $this;
@@ -124,8 +125,9 @@  discard block
 block discarded – undo
124 125
      */
125 126
     public function description ($value = null)
126 127
     {
127
-        if (null === $value)
128
-            return $this->description;
128
+        if (null === $value) {
129
+                    return $this->description;
130
+        }
129 131
 
130 132
         $this->description = $value;
131 133
         return $this;
@@ -139,8 +141,9 @@  discard block
 block discarded – undo
139 141
      */
140 142
     public function version ($value = null)
141 143
     {
142
-        if (null === $value)
143
-            return $this->version;
144
+        if (null === $value) {
145
+                    return $this->version;
146
+        }
144 147
 
145 148
         $this->version = $value;
146 149
         return $this;
@@ -154,8 +157,9 @@  discard block
 block discarded – undo
154 157
      */
155 158
     public function console ($value = null)
156 159
     {
157
-        if (null === $value)
158
-            return $this->console;
160
+        if (null === $value) {
161
+                    return $this->console;
162
+        }
159 163
 
160 164
         $this->console = $value;
161 165
         return $this;
@@ -170,8 +174,9 @@  discard block
 block discarded – undo
170 174
      */
171 175
     public function syntax ($value = null)
172 176
     {
173
-        if (null === $value)
174
-            return $this->syntaxBuilder->get();
177
+        if (null === $value) {
178
+                    return $this->syntaxBuilder->get();
179
+        }
175 180
 
176 181
         $this->syntaxBuilder = SyntaxBuilder::of($value);
177 182
 
@@ -218,8 +223,9 @@  discard block
 block discarded – undo
218 223
     public function command ($name, Command $cmd = null)
219 224
     {
220 225
         if (null === $cmd) {
221
-            if (!$this->hasCommand($name))
222
-                throw new CommandNotFound("Command '{$name}' not found");
226
+            if (!$this->hasCommand($name)) {
227
+                            throw new CommandNotFound("Command '{$name}' not found");
228
+            }
223 229
             return $this->subCommands[$name];
224 230
         }
225 231
 
@@ -245,8 +251,9 @@  discard block
 block discarded – undo
245 251
      */
246 252
     public function args($value = null)
247 253
     {
248
-        if (null === $value)
249
-            return $this->args;
254
+        if (null === $value) {
255
+                    return $this->args;
256
+        }
250 257
 
251 258
         $this->args = $value;
252 259
         return $this;
@@ -306,10 +313,12 @@  discard block
 block discarded – undo
306 313
     {
307 314
         try {
308 315
 
309
-            if (null === $args)
310
-                $args = $this->consoleArguments();
311
-            if (null === $console)
312
-                $console = new CLImate;
316
+            if (null === $args) {
317
+                            $args = $this->consoleArguments();
318
+            }
319
+            if (null === $console) {
320
+                            $console = new CLImate;
321
+            }
313 322
 
314 323
             $firstArg = F\head(F\split(' ', $args));
315 324
             if ($this->hasCommand($firstArg)) {
@@ -323,11 +332,10 @@  discard block
 block discarded – undo
323 332
                         ->then(F\join(PHP_EOL))
324 333
                         ->get();
325 334
                     $this->error($errors);
326
-                }
327
-
328
-                else
329
-                    $this->args($this->syntax()->parse($args))
335
+                } else {
336
+                                    $this->args($this->syntax()->parse($args))
330 337
                          ->execute();
338
+                }
331 339
             }
332 340
         } catch (\Exception $e) {
333 341
             $this->error(
@@ -346,8 +354,9 @@  discard block
 block discarded – undo
346 354
      */
347 355
     protected function template($name)
348 356
     {
349
-        if (null === $this->templateLoader)
350
-            throw new NullPropertyAccess("The templatesLoader is not initialized; Please set it before trying to load a template !");
357
+        if (null === $this->templateLoader) {
358
+                    throw new NullPropertyAccess("The templatesLoader is not initialized; Please set it before trying to load a template !");
359
+        }
351 360
 
352 361
         return $this->templateLoader->load($name);
353 362
     }
Please login to merge, or discard this patch.
src/Commands/HelpCommand.php 2 patches
Doc Comments   +6 added lines patch added patch discarded remove patch
@@ -74,6 +74,9 @@  discard block
 block discarded – undo
74 74
         }
75 75
     }
76 76
 
77
+    /**
78
+     * @param integer $level
79
+     */
77 80
     protected function showSyntax($name, $syntax, $level)
78 81
     {
79 82
         $type = $this->typeOf($syntax);
@@ -102,6 +105,9 @@  discard block
 block discarded – undo
102 105
         }
103 106
     }
104 107
 
108
+    /**
109
+     * @return integer
110
+     */
105 111
     protected function typeOf(Syntax $syntax)
106 112
     {
107 113
         if ($syntax instanceof ObjectSyntax) {
Please login to merge, or discard this patch.
Braces   +5 added lines, -4 removed lines patch added patch discarded remove patch
@@ -24,10 +24,11 @@
 block discarded – undo
24 24
     protected function execute ()
25 25
     {
26 26
         $command = trim($this->args->command);
27
-        if (empty($command) || !$this->parent()->hasCommand($command))
28
-            $this->showHelpOf($this->parent);
29
-        else
30
-            $this->showHelpOf($this->parent->command($command));
27
+        if (empty($command) || !$this->parent()->hasCommand($command)) {
28
+                    $this->showHelpOf($this->parent);
29
+        } else {
30
+                    $this->showHelpOf($this->parent->command($command));
31
+        }
31 32
     }
32 33
 
33 34
     protected function showHelpOf (Command $command)
Please login to merge, or discard this patch.
src/Syntax/SyntaxBuilder.php 2 patches
Doc Comments   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -23,7 +23,7 @@  discard block
 block discarded – undo
23 23
     /**
24 24
      * A constructor alias; If you don't like the 'new' keyword.
25 25
      * @param  string|Tarsana\Syntax\ObjectSyntax $syntax
26
-     * @return Tarsana\Command\Syntax\SyntaxBuilder
26
+     * @return SyntaxBuilder
27 27
      * @throws Tarsana\Command\Exceptions\SyntaxBuilderException
28 28
      */
29 29
     public static function of($syntax)
@@ -88,7 +88,7 @@  discard block
 block discarded – undo
88 88
      * @param  string $name
89 89
      * @param  string $description
90 90
      * @param  mixed  $default
91
-     * @return Tarsana\Command\Syntax\SyntaxBuilder
91
+     * @return SyntaxBuilder
92 92
      */
93 93
     public function describe($name, $description, $default = null)
94 94
     {
@@ -104,7 +104,7 @@  discard block
 block discarded – undo
104 104
     /**
105 105
      * Gets a field or subfield from an Syntax.
106 106
      *
107
-     * @param  Tarsana\Syntax\Syntax $syntax
107
+     * @param  Syntax $syntax
108 108
      * @param  array $names
109 109
      * @return Tarsana\Syntax\Syntax
110 110
      * @throws Tarsana\Command\Exceptions\SyntaxBuilderException
@@ -122,8 +122,8 @@  discard block
 block discarded – undo
122 122
      * Ensures that `$syntax` is an ObjectSyntax or an ArraySyntax of ObjectSyntaxes,
123 123
      * and returns the ObjectSyntax or throws a SyntaxBuilderException.
124 124
      *
125
-     * @param  Tarsana\Syntax\Syntax $syntax
126
-     * @return Tarsana\Syntax\Syntax
125
+     * @param  Syntax $syntax
126
+     * @return ObjectSyntax
127 127
      * @throws Tarsana\Command\Exceptions\SyntaxBuilderException
128 128
      */
129 129
     protected function ensureObject(Syntax $syntax)
Please login to merge, or discard this patch.
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -75,8 +75,9 @@
 block discarded – undo
75 75
 
76 76
         $ss = S::syntax();
77 77
 
78
-        if (!$ss->canParse($syntax))
79
-            throw new SyntaxBuilderException("Invalid Syntax: '{$syntax}'");
78
+        if (!$ss->canParse($syntax)) {
79
+                    throw new SyntaxBuilderException("Invalid Syntax: '{$syntax}'");
80
+        }
80 81
 
81 82
         return $ss->parse($syntax);
82 83
     }
Please login to merge, or discard this patch.
src/TemplateLoader.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -58,7 +58,7 @@
 block discarded – undo
58 58
     /**
59 59
      * Filesystem getter and setter.
60 60
      *
61
-     * @param  Tarsana\IO\Filesystem|void $value
61
+     * @param  null|Filesystem $value
62 62
      * @return Tarsana\IO\Filesystem|self
63 63
      */
64 64
     public function fs (Filesystem $value = null)
Please login to merge, or discard this patch.
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -63,8 +63,9 @@
 block discarded – undo
63 63
      */
64 64
     public function fs (Filesystem $value = null)
65 65
     {
66
-        if (null === $value)
67
-            return $this->fs;
66
+        if (null === $value) {
67
+                    return $this->fs;
68
+        }
68 69
         $this->fs = $value;
69 70
         return $this;
70 71
     }
Please login to merge, or discard this patch.
src/Templates/TwigTemplateLoader.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -38,7 +38,7 @@
 block discarded – undo
38 38
      * Loads a template.
39 39
      *
40 40
      * @param  string $name
41
-     * @return Tarsana\Command\Templates\TwigTemplate
41
+     * @return TwigTemplate
42 42
      */
43 43
     public function load($name)
44 44
     {
Please login to merge, or discard this patch.