| @@ -194,7 +194,7 @@ | ||
| 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 | * | 
| @@ -61,7 +61,7 @@ | ||
| 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; | 
| @@ -100,7 +100,9 @@ discard block | ||
| 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 | ||
| 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 | ||
| 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 | 
| @@ -318,7 +318,7 @@ | ||
| 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 |      { | 
| @@ -392,7 +392,9 @@ | ||
| 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 | } | 
| @@ -84,7 +84,7 @@ | ||
| 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)); | 
| @@ -322,6 +322,9 @@ discard block | ||
| 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 | ||
| 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)); | 
| @@ -154,11 +154,11 @@ discard block | ||
| 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 | ||
| 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 | ||
| 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 | } | 
| @@ -63,8 +63,8 @@ | ||
| 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) | 
| @@ -20,7 +20,7 @@ discard block | ||
| 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 | ||
| 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 | ||
| 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 | ||
| 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 | ||
| 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 | ||
| 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(), | 
| @@ -143,11 +143,17 @@ discard block | ||
| 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 | ||
| 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; | 
| @@ -15,7 +15,7 @@ | ||
| 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 |      { | 
| @@ -20,7 +20,7 @@ discard block | ||
| 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 | ||
| 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; | 
| @@ -103,7 +103,7 @@ discard block | ||
| 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 | ||
| 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 | ||
| 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 | ||
| 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 | } | 
| @@ -34,7 +34,7 @@ discard block | ||
| 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 | ||
| 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 | } | 
| @@ -163,6 +163,9 @@ | ||
| 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']; | 
| @@ -19,7 +19,7 @@ | ||
| 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 | |
| @@ -19,7 +19,7 @@ | ||
| 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 | |