Passed
Pull Request — develop (#311)
by Schlaefer
02:21
created
lib/Phile/Model/Page.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -133,7 +133,7 @@
 block discarded – undo
133 133
     /**
134 134
      * set content of page
135 135
      *
136
-     * @param $content
136
+     * @param string $content
137 137
      */
138 138
     public function setContent($content)
139 139
     {
Please login to merge, or discard this patch.
plugins/phile/templateTwig/Classes/Template/Twig.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -79,7 +79,7 @@
 block discarded – undo
79 79
     /**
80 80
      * wrapper to call the render engine
81 81
      *
82
-     * @param  $engine
82
+     * @param  \Twig_Environment $engine
83 83
      * @param  $vars
84 84
      * @return mixed
85 85
      */
Please login to merge, or discard this patch.
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -123,7 +123,7 @@  discard block
 block discarded – undo
123 123
             $template = 'index';
124 124
         }
125 125
         if (!empty($this->config['template-extension'])) {
126
-            $template .= '.' . $this->config['template-extension'];
126
+            $template .= '.'.$this->config['template-extension'];
127 127
         }
128 128
         $templatePath = $this->getTemplatePath($template);
129 129
         if (!file_exists($templatePath)) {
@@ -143,9 +143,9 @@  discard block
 block discarded – undo
143 143
      */
144 144
     protected function getTemplatePath($sub = '')
145 145
     {
146
-        $themePath = THEMES_DIR . $this->settings['theme'];
146
+        $themePath = THEMES_DIR.$this->settings['theme'];
147 147
         if (!empty($sub)) {
148
-            $themePath .= '/' . ltrim($sub, DIRECTORY_SEPARATOR);
148
+            $themePath .= '/'.ltrim($sub, DIRECTORY_SEPARATOR);
149 149
         }
150 150
         return $themePath;
151 151
     }
@@ -167,11 +167,11 @@  discard block
 block discarded – undo
167 167
         'base_url' => $this->settings['base_url'],
168 168
         'config' => $this->settings,
169 169
         'content_dir' => $this->settings['content_dir'],
170
-        'content_url' => $this->settings['base_url'] . '/' . basename($this->settings['content_dir']),
170
+        'content_url' => $this->settings['base_url'].'/'.basename($this->settings['content_dir']),
171 171
         'pages' => $repository->findAll(),
172 172
         'site_title' => $this->settings['site_title'],
173
-        'theme_dir' => THEMES_DIR . $this->settings['theme'],
174
-        'theme_url' => $this->settings['base_url'] . '/' . basename(THEMES_DIR) . '/' . $this->settings['theme'],
173
+        'theme_dir' => THEMES_DIR.$this->settings['theme'],
174
+        'theme_url' => $this->settings['base_url'].'/'.basename(THEMES_DIR).'/'.$this->settings['theme'],
175 175
         ];
176 176
 
177 177
         /**
Please login to merge, or discard this patch.
lib/Phile/Repository/PageCollection.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -76,7 +76,7 @@
 block discarded – undo
76 76
     public function offsetSet($offset, $value)
77 77
     {
78 78
         $this->load();
79
-        $this->pages[$offset] =    $value;
79
+        $this->pages[$offset] = $value;
80 80
     }
81 81
 
82 82
     public function offsetUnset($offset)
Please login to merge, or discard this patch.
plugins/phile/errorHandler/Classes/Plugin.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -21,8 +21,8 @@
 block discarded – undo
21 21
  */
22 22
 class Plugin extends AbstractPlugin
23 23
 {
24
-    const HANDLER_ERROR_LOG        = 'error_log';
25
-    const HANDLER_DEVELOPMENT    = 'development';
24
+    const HANDLER_ERROR_LOG = 'error_log';
25
+    const HANDLER_DEVELOPMENT = 'development';
26 26
 
27 27
     protected $events = ['plugins_loaded' => 'onPluginsLoaded'];
28 28
 
Please login to merge, or discard this patch.
lib/Phile/Plugin/AbstractPlugin.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -52,7 +52,7 @@  discard block
 block discarded – undo
52 52
         $this->plugin['key'] = $pluginKey;
53 53
         list($vendor, $name) = explode('\\', $this->plugin['key']);
54 54
         $DS = DIRECTORY_SEPARATOR;
55
-        $this->plugin['dir'] = PLUGINS_DIR . $vendor . $DS . $name . $DS;
55
+        $this->plugin['dir'] = PLUGINS_DIR.$vendor.$DS.$name.$DS;
56 56
 
57 57
         /**
58 58
          * init events
@@ -132,6 +132,6 @@  discard block
 block discarded – undo
132 132
      */
133 133
     protected function getPluginPath($subPath = '')
134 134
     {
135
-        return $this->plugin['dir'] . ltrim($subPath, DIRECTORY_SEPARATOR);
135
+        return $this->plugin['dir'].ltrim($subPath, DIRECTORY_SEPARATOR);
136 136
     }
137 137
 }
Please login to merge, or discard this patch.
lib/Phile/Plugin/PluginRepository.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -72,7 +72,7 @@  discard block
 block discarded – undo
72 72
     {
73 73
         list($vendor, $pluginName) = explode('\\', $pluginKey);
74 74
         // uppercase first letter convention
75
-        $pluginClassName = '\\Phile\\Plugin\\' . ucfirst($vendor) . '\\' . ucfirst($pluginName) . '\\Plugin';
75
+        $pluginClassName = '\\Phile\\Plugin\\'.ucfirst($vendor).'\\'.ucfirst($pluginName).'\\Plugin';
76 76
 
77 77
         if (!class_exists($pluginClassName)) {
78 78
             throw new PluginException(
@@ -125,7 +125,7 @@  discard block
 block discarded – undo
125 125
             $classNameParts
126 126
         );
127 127
 
128
-        $fileName = PLUGINS_DIR . implode(DIRECTORY_SEPARATOR, $classPath) . '.php';
128
+        $fileName = PLUGINS_DIR.implode(DIRECTORY_SEPARATOR, $classPath).'.php';
129 129
         if (file_exists($fileName)) {
130 130
             include_once $fileName;
131 131
         }
Please login to merge, or discard this patch.
lib/Phile/Core/Utility.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -159,7 +159,7 @@
 block discarded – undo
159 159
     {
160 160
         $config = Registry::get('Phile_Settings');
161 161
 
162
-        return md5($config['encryptionKey'] . $value);
162
+        return md5($config['encryptionKey'].$value);
163 163
     }
164 164
 
165 165
     /**
Please login to merge, or discard this patch.
Doc Comments   +7 added lines, -2 removed lines patch added patch discarded remove patch
@@ -55,7 +55,7 @@  discard block
 block discarded – undo
55 55
     /**
56 56
      * resolve a file path by replace the mod: prefix
57 57
      *
58
-     * @param $path
58
+     * @param string $path
59 59
      *
60 60
      * @return string|null the full filepath or null if file does not exists
61 61
      */
@@ -76,7 +76,7 @@  discard block
 block discarded – undo
76 76
     /**
77 77
      * load files e.g. config files
78 78
      *
79
-     * @param $file
79
+     * @param string|null $file
80 80
      *
81 81
      * @return mixed|null
82 82
      */
@@ -202,6 +202,11 @@  discard block
 block discarded – undo
202 202
      * @return mixed
203 203
      */
204 204
     // @codingStandardsIgnoreStart
205
+
206
+    /**
207
+     * @param integer $min
208
+     * @param integer $max
209
+     */
205 210
     public static function crypto_rand_secure($min, $max)
206 211
     {
207 212
         // @codingStandardsIgnoreEnd
Please login to merge, or discard this patch.
lib/Phile/Core/Response.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -119,7 +119,7 @@
 block discarded – undo
119 119
     public function send()
120 120
     {
121 121
         if (!isset($this->headers['Content-Type'])) {
122
-            $this->setHeader('Content-Type', 'text/html; charset=' . $this->charset);
122
+            $this->setHeader('Content-Type', 'text/html; charset='.$this->charset);
123 123
         }
124 124
         $this->outputHeader();
125 125
         http_response_code($this->statusCode);
Please login to merge, or discard this patch.
lib/Phile/Core/Router.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -83,7 +83,7 @@  discard block
 block discarded – undo
83 83
         if (isset($this->server['HTTP_HOST'])) {
84 84
             $host = $this->server['HTTP_HOST'];
85 85
             $protocol = $this->getProtocol();
86
-            $url = $protocol . '://' . $host . $url;
86
+            $url = $protocol.'://'.$host.$url;
87 87
         }
88 88
 
89 89
         $url = rtrim($url, '/');
@@ -118,7 +118,7 @@  discard block
 block discarded – undo
118 118
      */
119 119
     public function url($url)
120 120
     {
121
-        return $this->getBaseUrl() . '/' . ltrim($url, '/');
121
+        return $this->getBaseUrl().'/'.ltrim($url, '/');
122 122
     }
123 123
 
124 124
     /**
Please login to merge, or discard this patch.