PhileCMS / Phile

lib/Phile/Core/Utility.php

Doc Comments +8 added lines, -3 removed lines

@@ -55,7 +55,7 @@ 
     /**
      * resolve a file path by replace the mod: prefix
      *
-     * @param $path
+     * @param string $path
      *
      * @return string|null the full filepath or null if file does not exists
      */
@@ -76,7 +76,7 @@ 
     /**
      * load files e.g. config files
      *
-     * @param $file
+     * @param string $file
      *
      * @return mixed|null
      */
@@ -110,7 +110,7 @@ 
     /**
      * static method to get files by directory and file filter
      *
-     * @param $directory
+     * @param string $directory
      * @param string    $filter
      *
      * @return array
@@ -202,6 +202,11 @@ 
      * @return mixed
      */
     // @codingStandardsIgnoreStart
+
+    /**
+     * @param integer $min
+     * @param integer $max
+     */
     public static function crypto_rand_secure($min, $max)
     {
         // @codingStandardsIgnoreEnd

Indentation +1 added lines, -1 removed lines

@@ -128,7 +128,7 @@
         $result = array();
         foreach ($files as $file) {
             /**
- * @var \SplFileInfo $file
+             * @var \SplFileInfo $file
 */
             $result[] = $file->getPathname();
         }

Spacing +1 added lines, -1 removed lines

@@ -159,7 +159,7 @@
     {
         $config = Registry::get('Phile_Settings');
 
-        return md5($config['encryptionKey'] . $value);
+        return md5($config['encryptionKey'].$value);
     }
 
     /**

lib/Phile/Model/Page.php

Doc Comments +1 added lines, -1 removed lines

@@ -133,7 +133,7 @@
     /**
      * set content of page
      *
-     * @param $content
+     * @param string $content
      */
     public function setContent($content)
     {

lib/Phile/Plugin/AbstractPlugin.php

Doc Comments +1 added lines, -1 removed lines

@@ -128,7 +128,7 @@
      * get file path to plugin root (trailing slash) or to a sub-item
      *
      * @param  string $subPath
-     * @return null|string null if item does not exist
+     * @return string null if item does not exist
      */
     protected function getPluginPath($subPath = '')
     {

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -117,7 +117,7 @@
         if (!is_callable([$this, $method])) {
             $class = get_class($this);
             throw new \RuntimeException(
-                "Event $eventKey can't invoke $class::$method(). Not callable.",
+                "event $eventKey can't invoke $class::$method(). Not callable.",
                 1428564865
             );
         }

Indentation +3 added lines, -3 removed lines

@@ -21,17 +21,17 @@
 {
 
     /**
- * @var string plugin attributes
+     * @var string plugin attributes
 */
     protected $plugin = [];
 
     /**
- * @var array subscribed Phile events ['eventName' => 'classMethodToCall']
+     * @var array subscribed Phile events ['eventName' => 'classMethodToCall']
 */
     protected $events = [];
 
     /**
- * @var array the plugin settings
+     * @var array the plugin settings
 */
     protected $settings = [];
 

Spacing +2 added lines, -2 removed lines

@@ -52,7 +52,7 @@ 
         $this->plugin['key'] = $pluginKey;
         list($vendor, $name) = explode('\\', $this->plugin['key']);
         $DS = DIRECTORY_SEPARATOR;
-        $this->plugin['dir'] = PLUGINS_DIR . $vendor . $DS . $name . $DS;
+        $this->plugin['dir'] = PLUGINS_DIR.$vendor.$DS.$name.$DS;
 
         /**
          * init events
@@ -132,6 +132,6 @@ 
      */
     protected function getPluginPath($subPath = '')
     {
-        return $this->plugin['dir'] . ltrim($subPath, DIRECTORY_SEPARATOR);
+        return $this->plugin['dir'].ltrim($subPath, DIRECTORY_SEPARATOR);
     }
 }

plugins/phile/templateTwig/Classes/Template/Twig.php

Doc Comments +1 added lines, -1 removed lines

@@ -79,7 +79,7 @@
     /**
      * wrapper to call the render engine
      *
-     * @param  $engine
+     * @param  \Twig_Environment $engine
      * @param  $vars
      * @return mixed
      */

Indentation +1 added lines, -1 removed lines

@@ -175,7 +175,7 @@
         ];
 
         /**
- * @var array $templateVars
+         * @var array $templateVars
 */
         $templateVars = Registry::get('templateVars');
         $templateVars += $defaults;

Spacing +6 added lines, -6 removed lines

@@ -123,7 +123,7 @@ 
             $template = 'index';
         }
         if (!empty($this->config['template-extension'])) {
-            $template .= '.' . $this->config['template-extension'];
+            $template .= '.'.$this->config['template-extension'];
         }
         $templatePath = $this->getTemplatePath($template);
         if (!file_exists($templatePath)) {
@@ -143,9 +143,9 @@ 
      */
     protected function getTemplatePath($sub = '')
     {
-        $themePath = THEMES_DIR . $this->settings['theme'];
+        $themePath = THEMES_DIR.$this->settings['theme'];
         if (!empty($sub)) {
-            $themePath .= '/' . ltrim($sub, DIRECTORY_SEPARATOR);
+            $themePath .= '/'.ltrim($sub, DIRECTORY_SEPARATOR);
         }
         return $themePath;
     }
@@ -167,11 +167,11 @@ 
         'base_url' => $this->settings['base_url'],
         'config' => $this->settings,
         'content_dir' => CONTENT_DIR,
-        'content_url' => $this->settings['base_url'] . '/' . basename(CONTENT_DIR),
+        'content_url' => $this->settings['base_url'].'/'.basename(CONTENT_DIR),
         'pages' => $repository->findAll(),
         'site_title' => $this->settings['site_title'],
-        'theme_dir' => THEMES_DIR . $this->settings['theme'],
-        'theme_url' => $this->settings['base_url'] . '/' . basename(THEMES_DIR) . '/' . $this->settings['theme'],
+        'theme_dir' => THEMES_DIR.$this->settings['theme'],
+        'theme_url' => $this->settings['base_url'].'/'.basename(THEMES_DIR).'/'.$this->settings['theme'],
         ];
 
         /**

lib/Phile/FilterIterator/ContentFileFilterIterator.php

Indentation +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
     public function accept()
     {
         /**
- * @var \SplFileInfo $this
+         * @var \SplFileInfo $this
 */
         return (preg_match('/^[^\.]{1}.*'.CONTENT_EXT.'/', $this->getFilename()) > 0);
     }

lib/Phile/Model/Meta.php

Indentation +1 added lines, -1 removed lines

@@ -81,7 +81,7 @@
     protected function parseRawData($rawData)
     {
         /**
- * @var \Phile\ServiceLocator\MetaInterface $metaParser
+         * @var \Phile\ServiceLocator\MetaInterface $metaParser
 */
         $metaParser = ServiceLocator::getService('Phile_Parser_Meta');
         $data       = $metaParser->parse($rawData);

lib/Phile/Repository/Page.php

Indentation +2 added lines, -2 removed lines

@@ -128,8 +128,8 @@
                     $column = array();
                     foreach ($pages as $page) {
                         /**
-            * @var \Phile\Model\Page $page
-            */
+                         * @var \Phile\Model\Page $page
+                         */
                         $meta = $page->getMeta();
                         if ($sort['type'] === 'page') {
                             $method = 'get' . ucfirst($key);

Spacing +7 added lines, -7 removed lines

@@ -64,7 +64,7 @@ 
             $pageId .= 'index';
         }
 
-        $file = $folder . $pageId . CONTENT_EXT;
+        $file = $folder.$pageId.CONTENT_EXT;
         if (!file_exists($file)) {
             if (substr($pageId, -6) === '/index') {
                 // try to resolve sub-directory 'sub/' to page 'sub'
@@ -73,7 +73,7 @@ 
                 // try to resolve page 'sub' to sub-directory 'sub/'
                 $pageId .= '/index';
             }
-            $file = $folder . $pageId . CONTENT_EXT;
+            $file = $folder.$pageId.CONTENT_EXT;
         }
         if (!file_exists($file)) {
             return null;
@@ -92,13 +92,13 @@ 
     public function findAll(array $options = array(), $folder = CONTENT_DIR)
     {
         return new PageCollection(
-            function () use ($options, $folder) {
+            function() use ($options, $folder) {
                 $options += $this->settings;
                 // ignore files with a leading '.' in its filename
                 $files = Utility::getFiles($folder, '\Phile\FilterIterator\ContentFileFilterIterator');
                 $pages = array();
                 foreach ($files as $file) {
-                    if (str_replace($folder, '', $file) == '404' . CONTENT_EXT) {
+                    if (str_replace($folder, '', $file) == '404'.CONTENT_EXT) {
                         // jump to next page if file is the 404 page
                         continue;
                     }
@@ -135,7 +135,7 @@ 
             */
                         $meta = $page->getMeta();
                         if ($sort['type'] === 'page') {
-                            $method = 'get' . ucfirst($key);
+                            $method = 'get'.ucfirst($key);
                             $value = $page->$method();
                         } elseif ($sort['type'] === 'meta') {
                             $value = $meta->get($key);
@@ -149,7 +149,7 @@ 
                         $column[] = $value;
                     }
                     $sortHelper[] = $column;
-                    $sortHelper[] = constant('SORT_' . strtoupper($sort['order']));
+                    $sortHelper[] = constant('SORT_'.strtoupper($sort['order']));
                 }
                 $sortHelper[] = &$pages;
 
@@ -191,7 +191,7 @@ 
      */
     protected function getPage($filePath, $folder = CONTENT_DIR)
     {
-        $key = 'Phile_Model_Page_' . md5($filePath);
+        $key = 'Phile_Model_Page_'.md5($filePath);
         if (isset($this->storage[$key])) {
             return $this->storage[$key];
         }

lib/Phile/Repository/PageCollection.php

Spacing +1 added lines, -1 removed lines

@@ -76,7 +76,7 @@
     public function offsetSet($offset, $value)
     {
         $this->load();
-        $this->pages[$offset] =    $value;
+        $this->pages[$offset] = $value;
     }
 
     public function offsetUnset($offset)

lib/Phile/Plugin/PluginRepository.php

Indentation +3 added lines, -3 removed lines

@@ -16,12 +16,12 @@ 
 {
 
     /**
- * @var array of AbstractPlugin
+     * @var array of AbstractPlugin
 */
     protected $plugins = [];
 
     /**
- * @var array errors during load; keys: 'message' and 'code'
+     * @var array errors during load; keys: 'message' and 'code'
 */
     protected $loadErrors = [];
 
@@ -82,7 +82,7 @@ 
         }
 
         /**
- * @var \Phile\Plugin\AbstractPlugin $plugin
+         * @var \Phile\Plugin\AbstractPlugin $plugin
 */
         $plugin = new $pluginClassName;
         if (($plugin instanceof AbstractPlugin) === false) {

Spacing +2 added lines, -2 removed lines

@@ -72,7 +72,7 @@ 
     {
         list($vendor, $pluginName) = explode('\\', $pluginKey);
         // uppercase first letter convention
-        $pluginClassName = '\\Phile\\Plugin\\' . ucfirst($vendor) . '\\' . ucfirst($pluginName) . '\\Plugin';
+        $pluginClassName = '\\Phile\\Plugin\\'.ucfirst($vendor).'\\'.ucfirst($pluginName).'\\Plugin';
 
         if (!class_exists($pluginClassName)) {
             throw new PluginException(
@@ -125,7 +125,7 @@ 
             $classNameParts
         );
 
-        $fileName = PLUGINS_DIR . implode(DIRECTORY_SEPARATOR, $classPath) . '.php';
+        $fileName = PLUGINS_DIR.implode(DIRECTORY_SEPARATOR, $classPath).'.php';
         if (file_exists($fileName)) {
             include_once $fileName;
         }