Completed
Pull Request — master (#41)
by Vladimir
02:38
created
src/allejo/stakx/FrontMatter/FrontMatterParser.php 3 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.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -61,7 +61,7 @@
 block discarded – undo
61 61
      *
62 62
      * @param array $rawFrontMatter
63 63
      */
64
-    public function __construct(&$rawFrontMatter)
64
+    public function __construct (&$rawFrontMatter)
65 65
     {
66 66
         $this->expansionUsed = false;
67 67
         $this->nestingLevel = 0;
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 3 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.
Upper-Lower-Casing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -84,7 +84,7 @@
 block discarded – undo
84 84
 
85 85
         if (!$this->fs->exists($filePath))
86 86
         {
87
-            throw new FileNotFoundException("The following file could not be found: ${filePath}");
87
+            throw new FileNotFoundException("The following file could not be found: ${filepath}");
88 88
         }
89 89
 
90 90
         $this->extension = strtolower($this->fs->getExtension($filePath));
Please login to merge, or discard this patch.
src/allejo/stakx/Object/Website.php 2 patches
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.
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -154,11 +154,11 @@  discard block
 block discarded – undo
154 154
         $twigEnv->configureTwig($this->getConfiguration(), array(
155 155
             'safe'    => $this->safeMode,
156 156
             'globals' => array(
157
-                array('name' => 'site',        'value' => $this->getConfiguration()->getConfiguration()),
157
+                array('name' => 'site', 'value' => $this->getConfiguration()->getConfiguration()),
158 158
                 array('name' => 'collections', 'value' => $this->cm->getJailedCollections()),
159
-                array('name' => 'menu',        'value' => $this->mm->getSiteMenu()),
160
-                array('name' => 'pages',       'value' => $this->pm->getJailedStaticPageViews()),
161
-                array('name' => 'data',        'value' => $this->dm->getDataItems())
159
+                array('name' => 'menu', 'value' => $this->mm->getSiteMenu()),
160
+                array('name' => 'pages', 'value' => $this->pm->getJailedStaticPageViews()),
161
+                array('name' => 'data', 'value' => $this->dm->getDataItems())
162 162
             )
163 163
         ));
164 164
 
@@ -331,7 +331,7 @@  discard block
 block discarded – undo
331 331
     /**
332 332
      * @return boolean
333 333
      */
334
-    public function isNoClean()
334
+    public function isNoClean ()
335 335
     {
336 336
         return $this->noClean;
337 337
     }
@@ -339,7 +339,7 @@  discard block
 block discarded – undo
339 339
     /**
340 340
      * @param boolean $noClean
341 341
      */
342
-    public function setNoClean($noClean)
342
+    public function setNoClean ($noClean)
343 343
     {
344 344
         $this->noClean = $noClean;
345 345
     }
Please login to merge, or discard this patch.
src/allejo/stakx/System/FileExplorer.php 3 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.
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -20,7 +20,7 @@  discard block
 block discarded – undo
20 20
     /**
21 21
      * A bitwise flag to have FileExplorer search files starting with a period as well
22 22
      */
23
-    const ALLOW_DOT_FILES    = 0x2;
23
+    const ALLOW_DOT_FILES = 0x2;
24 24
 
25 25
     /**
26 26
      * A list of common version control folders to ignore.
@@ -36,7 +36,7 @@  discard block
 block discarded – undo
36 36
      *
37 37
      * @var string[]
38 38
      */
39
-    public static $vcsPatterns =  array('.git', '.hg', '.svn', '_svn');
39
+    public static $vcsPatterns = array('.git', '.hg', '.svn', '_svn');
40 40
 
41 41
     /**
42 42
      * A list of phrases to exclude from the search
@@ -67,7 +67,7 @@  discard block
 block discarded – undo
67 67
      * @param array              $includes
68 68
      * @param int|null           $flags
69 69
      */
70
-    public function __construct(\RecursiveIterator $iterator, array $excludes = array(), array $includes = array(), $flags = null)
70
+    public function __construct (\RecursiveIterator $iterator, array $excludes = array(), array $includes = array(), $flags = null)
71 71
     {
72 72
         parent::__construct($iterator);
73 73
 
@@ -79,7 +79,7 @@  discard block
 block discarded – undo
79 79
     /**
80 80
      * @return string
81 81
      */
82
-    public function __toString()
82
+    public function __toString ()
83 83
     {
84 84
         return $this->current()->getFilename();
85 85
     }
@@ -105,7 +105,7 @@  discard block
 block discarded – undo
105 105
      *
106 106
      * @return SplFileInfo
107 107
      */
108
-    public function current()
108
+    public function current ()
109 109
     {
110 110
         /** @var \SplFileInfo $current */
111 111
         $current = parent::current();
@@ -120,7 +120,7 @@  discard block
 block discarded – undo
120 120
     /**
121 121
      * {@inheritdoc}
122 122
      */
123
-    public function getChildren()
123
+    public function getChildren ()
124 124
     {
125 125
         return (new self(
126 126
             $this->getInnerIterator()->getChildren(),
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 (self::strpos_array($filePath, $this->includes)) { return true; }
147
-        if (($this->flags & self::INCLUDE_ONLY_FILES) && !$this->current()->isDir()) { return false; }
146
+        if (self::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 (self::strpos_array($filePath, $this->excludes) === false);
153 159
     }
@@ -193,10 +199,13 @@  discard block
 block discarded – undo
193 199
                 return true;
194 200
             }
195 201
 
196
-            if (strpos($haystack, $query, $offset) !== false) // stop on first true result
202
+            if (strpos($haystack, $query, $offset) !== false)
203
+            {
204
+                // stop on first true result
197 205
             {
198 206
                 return true;
199 207
             }
208
+            }
200 209
         }
201 210
 
202 211
         return false;
Please login to merge, or discard this patch.
src/allejo/stakx/Twig/TextExtension.php 2 patches
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.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -20,7 +20,7 @@  discard block
 block discarded – undo
20 20
     public function getFilters ()
21 21
     {
22 22
         return array(
23
-            new Twig_SimpleFilter('summary',  'twig_summary_filter'),
23
+            new Twig_SimpleFilter('summary', 'twig_summary_filter'),
24 24
             new Twig_SimpleFilter('truncate', 'twig_truncate_filter', array('needs_environment' => true)),
25 25
             new Twig_SimpleFilter('wordwrap', 'twig_wordwrap_filter', array('needs_environment' => true)),
26 26
         );
@@ -73,7 +73,7 @@  discard block
 block discarded – undo
73 73
             $length = $breakpoint;
74 74
         }
75 75
 
76
-        return rtrim(mb_substr($value, 0, $length, $env->getCharset())).$separator;
76
+        return rtrim(mb_substr($value, 0, $length, $env->getCharset())) . $separator;
77 77
     }
78 78
 
79 79
     return $value;
Please login to merge, or discard this patch.
src/allejo/stakx/Object/JailObject.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -103,7 +103,7 @@  discard block
 block discarded – undo
103 103
     /**
104 104
      * {@inheritdoc}
105 105
      */
106
-    public function offsetExists($offset)
106
+    public function offsetExists ($offset)
107 107
     {
108 108
         return $this->object->offsetExists($offset);
109 109
     }
@@ -111,7 +111,7 @@  discard block
 block discarded – undo
111 111
     /**
112 112
      * {@inheritdoc}
113 113
      */
114
-    public function offsetGet($offset)
114
+    public function offsetGet ($offset)
115 115
     {
116 116
         return $this->object->offsetGet($offset);
117 117
     }
@@ -119,7 +119,7 @@  discard block
 block discarded – undo
119 119
     /**
120 120
      * {@inheritdoc}
121 121
      */
122
-    public function offsetSet($offset, $value)
122
+    public function offsetSet ($offset, $value)
123 123
     {
124 124
         return $this->object->offsetSet($offset, $value);
125 125
     }
@@ -127,7 +127,7 @@  discard block
 block discarded – undo
127 127
     /**
128 128
      * {@inheritdoc}
129 129
      */
130
-    public function offsetUnset($offset)
130
+    public function offsetUnset ($offset)
131 131
     {
132 132
         return $this->object->offsetUnset($offset);
133 133
     }
Please login to merge, or discard this patch.
src/allejo/stakx/Object/Configuration.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -34,7 +34,7 @@  discard block
 block discarded – undo
34 34
     /**
35 35
      * Configuration constructor.
36 36
      */
37
-    public function __construct()
37
+    public function __construct ()
38 38
     {
39 39
         $this->configuration = array();
40 40
         $this->fs            = new Filesystem();
@@ -64,7 +64,7 @@  discard block
 block discarded – undo
64 64
     /**
65 65
      * {@inheritdoc}
66 66
      */
67
-    public function setLogger(LoggerInterface $logger)
67
+    public function setLogger (LoggerInterface $logger)
68 68
     {
69 69
         $this->output = $logger;
70 70
     }
Please login to merge, or discard this patch.
Doc Comments   +4 added lines, -1 removed lines patch added patch discarded remove patch
@@ -92,7 +92,7 @@  discard block
 block discarded – undo
92 92
     /**
93 93
      * @TODO 1.0.0 Remove support for 'base' in next major release; it has been replaced by 'baseurl'
94 94
      *
95
-     * @return mixed|null
95
+     * @return string
96 96
      */
97 97
     public function getBaseUrl ()
98 98
     {
@@ -163,6 +163,9 @@  discard block
 block discarded – undo
163 163
         return $this->configuration['twig']['autoescape'];
164 164
     }
165 165
 
166
+    /**
167
+     * @return string|false
168
+     */
166 169
     public function getRedirectTemplate ()
167 170
     {
168 171
         return $this->configuration['templates']['redirect'];
Please login to merge, or discard this patch.
src/allejo/stakx/Object/DynamicPageView.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -19,7 +19,7 @@
 block discarded – undo
19 19
     /**
20 20
      * {@inheritdoc}
21 21
      */
22
-    public function __construct($filePath)
22
+    public function __construct ($filePath)
23 23
     {
24 24
         parent::__construct($filePath);
25 25
 
Please login to merge, or discard this patch.
src/allejo/stakx/Object/RepeaterPageView.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -19,7 +19,7 @@
 block discarded – undo
19 19
     /**
20 20
      * {@inheritdoc}
21 21
      */
22
-    public function __construct($filePath)
22
+    public function __construct ($filePath)
23 23
     {
24 24
         parent::__construct($filePath);
25 25
 
Please login to merge, or discard this patch.