Completed
Pull Request — master (#41)
by Vladimir
03:10
created
src/allejo/stakx/FrontMatter/FrontMatterParser.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -194,7 +194,7 @@
 block discarded – undo
194 194
      *
195 195
      * @param  string $frontMatterKey     The current hierarchy of the Front Matter keys being used
196 196
      * @param  string $expandableValue    The Front Matter value that will be expanded
197
-     * @param  array  $arrayVariableNames The Front Matter variable names that reference arrays
197
+     * @param  string[]  $arrayVariableNames The Front Matter variable names that reference arrays
198 198
      *
199 199
      * @return array
200 200
      *
Please login to merge, or discard this patch.
Braces   +9 added lines, -3 removed lines patch added patch discarded remove patch
@@ -100,7 +100,9 @@  discard block
 block discarded – undo
100 100
      */
101 101
     private function handleDateField ()
102 102
     {
103
-        if (!isset($this->frontMatter['date'])) { return; }
103
+        if (!isset($this->frontMatter['date']))
104
+        {
105
+return; }
104 106
 
105 107
         $date = &$this->frontMatter['date'];
106 108
         $itemDate = $this->guessDateTime($date);
@@ -258,7 +260,9 @@  discard block
 block discarded – undo
258 260
 
259 261
             if (is_array($value) || is_bool($value))
260 262
             {
261
-                if ($ignoreArrays) { continue; }
263
+                if ($ignoreArrays)
264
+                {
265
+continue; }
262 266
 
263 267
                 throw new YamlUnsupportedVariableException("Yaml variable `$variable` for `$key` is not a supported data type.");
264 268
             }
@@ -348,6 +352,8 @@  discard block
 block discarded – undo
348 352
         {
349 353
             return (new \DateTime($guess));
350 354
         }
351
-        catch (\Exception $e) { return false; }
355
+        catch (\Exception $e)
356
+        {
357
+return false; }
352 358
     }
353 359
 }
354 360
\ No newline at end of file
Please login to merge, or discard this patch.
src/allejo/stakx/Object/FrontMatterObject.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -318,7 +318,7 @@
 block discarded – undo
318 318
     /**
319 319
      * Get the original file path
320 320
      *
321
-     * @return string
321
+     * @return SplFileInfo
322 322
      */
323 323
     final public function getFilePath ()
324 324
     {
Please login to merge, or discard this patch.
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -392,7 +392,9 @@
 block discarded – undo
392 392
      */
393 393
     final public function deleteFrontMatter ($key)
394 394
     {
395
-        if (!isset($this->writableFrontMatter[$key])) { return; }
395
+        if (!isset($this->writableFrontMatter[$key]))
396
+        {
397
+return; }
396 398
 
397 399
         unset($this->writableFrontMatter[$key]);
398 400
     }
Please login to merge, or discard this patch.
src/allejo/stakx/Object/Website.php 1 patch
Doc Comments   +6 added lines patch added patch discarded remove patch
@@ -322,6 +322,9 @@  discard block
 block discarded – undo
322 322
         $this->noClean = $noClean;
323 323
     }
324 324
 
325
+    /**
326
+     * @param string $filePath
327
+     */
325 328
     private function creationWatcher ($filePath)
326 329
     {
327 330
         $this->output->writeln(sprintf("File creation detected: %s", $filePath));
@@ -363,6 +366,9 @@  discard block
 block discarded – undo
363 366
         }
364 367
     }
365 368
 
369
+    /**
370
+     * @param string $filePath
371
+     */
366 372
     private function modificationWatcher ($filePath)
367 373
     {
368 374
         $this->output->writeln(sprintf("File change detected: %s", $filePath));
Please login to merge, or discard this patch.
src/allejo/stakx/System/FileExplorer.php 2 patches
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -63,8 +63,8 @@
 block discarded – undo
63 63
      * FileExplorer constructor.
64 64
      *
65 65
      * @param \RecursiveIterator $iterator
66
-     * @param array              $excludes
67
-     * @param array              $includes
66
+     * @param string[]              $excludes
67
+     * @param string[]              $includes
68 68
      * @param int|null           $flags
69 69
      */
70 70
     public function __construct(\RecursiveIterator $iterator, array $excludes = array(), array $includes = array(), $flags = null)
Please login to merge, or discard this patch.
Braces   +13 added lines, -4 removed lines patch added patch discarded remove patch
@@ -143,11 +143,17 @@  discard block
 block discarded – undo
143 143
      */
144 144
     public function matchesPattern ($filePath)
145 145
     {
146
-        if ($this->strpos_array($filePath, $this->includes)) { return true; }
147
-        if (($this->flags & self::INCLUDE_ONLY_FILES) && !$this->current()->isDir()) { return false; }
146
+        if ($this->strpos_array($filePath, $this->includes))
147
+        {
148
+return true; }
149
+        if (($this->flags & self::INCLUDE_ONLY_FILES) && !$this->current()->isDir())
150
+        {
151
+return false; }
148 152
 
149 153
         if (!($this->flags & self::ALLOW_DOT_FILES) &&
150
-            preg_match('#(^|\\\\|\/)\..+(\\\\|\/|$)#', $filePath) === 1) { return false; }
154
+            preg_match('#(^|\\\\|\/)\..+(\\\\|\/|$)#', $filePath) === 1)
155
+        {
156
+return false; }
151 157
 
152 158
         return ($this->strpos_array($filePath, $this->excludes) === false);
153 159
     }
@@ -192,10 +198,13 @@  discard block
 block discarded – undo
192 198
                 return true;
193 199
             }
194 200
 
195
-            if (strpos($haystack, $query, $offset) !== false) // stop on first true result
201
+            if (strpos($haystack, $query, $offset) !== false)
202
+            {
203
+                // stop on first true result
196 204
             {
197 205
                 return true;
198 206
             }
207
+            }
199 208
         }
200 209
 
201 210
         return false;
Please login to merge, or discard this patch.
src/allejo/stakx/Twig/TextExtension.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -15,7 +15,7 @@
 block discarded – undo
15 15
     /**
16 16
      * Returns a list of filters.
17 17
      *
18
-     * @return array
18
+     * @return Twig_SimpleFilter[]
19 19
      */
20 20
     public function getFilters ()
21 21
     {
Please login to merge, or discard this patch.
src/allejo/stakx/Utilities/ArrayUtilities.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -9,7 +9,9 @@
 block discarded – undo
9 9
         foreach ($array as $element)
10 10
         {
11 11
             if (is_array($element))
12
-                return true;
12
+            {
13
+                            return true;
14
+            }
13 15
         }
14 16
 
15 17
         return false;
Please login to merge, or discard this patch.
src/allejo/stakx/Core/StakxLogger.php 1 patch
Braces   +6 added lines, -3 removed lines patch added patch discarded remove patch
@@ -93,7 +93,8 @@  discard block
 block discarded – undo
93 93
      */
94 94
     public function log ($level, $message, array $context = array())
95 95
     {
96
-        if (!isset($this->verbosityLevelMap[$level])) {
96
+        if (!isset($this->verbosityLevelMap[$level]))
97
+        {
97 98
             throw new InvalidArgumentException(sprintf('The log level "%s" does not exist.', $level));
98 99
         }
99 100
 
@@ -144,8 +145,10 @@  discard block
 block discarded – undo
144 145
     {
145 146
         // build a replacement array with braces around the context keys
146 147
         $replace = array();
147
-        foreach ($context as $key => $val) {
148
-            if (!is_array($val) && (!is_object($val) || method_exists($val, '__toString'))) {
148
+        foreach ($context as $key => $val)
149
+        {
150
+            if (!is_array($val) && (!is_object($val) || method_exists($val, '__toString')))
151
+            {
149 152
                 $replace[sprintf('{%s}', $key)] = $val;
150 153
             }
151 154
         }
Please login to merge, or discard this patch.
src/allejo/stakx/Twig/GroupByFilter.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -10,7 +10,9 @@
 block discarded – undo
10 10
 
11 11
         foreach ($array as $key => $item)
12 12
         {
13
-            if (!isset($item[$sortKey])) { continue; }
13
+            if (!isset($item[$sortKey]))
14
+            {
15
+continue; }
14 16
 
15 17
             $groupBy = $item[$sortKey];
16 18
 
Please login to merge, or discard this patch.
src/allejo/stakx/Twig/WhereFilter.php 1 patch
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch
@@ -136,8 +136,12 @@
 block discarded – undo
136 136
 
137 137
         if ($array->coreInstanceOf(FrontMatterObject::class) && !isset($array[$key]))
138 138
         {
139
-            if ($comparison == '==' && is_null($value))  { return true; }
140
-            if ($comparison == '!=' && !is_null($value)) { return true; }
139
+            if ($comparison == '==' && is_null($value))
140
+            {
141
+return true; }
142
+            if ($comparison == '!=' && !is_null($value))
143
+            {
144
+return true; }
141 145
         }
142 146
 
143 147
         return false;
Please login to merge, or discard this patch.