@@ -60,7 +60,7 @@ discard block |
||
60 | 60 | |
61 | 61 | /** |
62 | 62 | * @param \Psr\Cache\CacheItemInterface $item |
63 | - * @return mixed |
|
63 | + * @return boolean |
|
64 | 64 | * @throws \InvalidArgumentException |
65 | 65 | */ |
66 | 66 | protected function driverWrite(CacheItemInterface $item) |
@@ -146,7 +146,7 @@ discard block |
||
146 | 146 | /** |
147 | 147 | * @param string $optionName |
148 | 148 | * @param mixed $optionValue |
149 | - * @return bool |
|
149 | + * @return boolean|null |
|
150 | 150 | * @throws \InvalidArgumentException |
151 | 151 | */ |
152 | 152 | public static function isValidOption($optionName, $optionValue) |
@@ -174,7 +174,7 @@ discard block |
||
174 | 174 | } |
175 | 175 | |
176 | 176 | /** |
177 | - * @return array |
|
177 | + * @return string[] |
|
178 | 178 | */ |
179 | 179 | public static function getValidOptions() |
180 | 180 | { |
@@ -182,7 +182,7 @@ discard block |
||
182 | 182 | } |
183 | 183 | |
184 | 184 | /** |
185 | - * @return array |
|
185 | + * @return string[] |
|
186 | 186 | */ |
187 | 187 | public static function getRequiredOptions() |
188 | 188 | { |
@@ -215,11 +215,11 @@ |
||
215 | 215 | } |
216 | 216 | |
217 | 217 | $stat->setData(implode(', ', array_keys($this->itemInstances))) |
218 | - ->setRawData([ |
|
218 | + ->setRawData([ |
|
219 | 219 | 'tmp' => $this->tmp |
220 | - ]) |
|
221 | - ->setSize(Directory::dirSize($path)) |
|
222 | - ->setInfo('Number of files used to build the cache: ' . Directory::getFileCount($path)); |
|
220 | + ]) |
|
221 | + ->setSize(Directory::dirSize($path)) |
|
222 | + ->setInfo('Number of files used to build the cache: ' . Directory::getFileCount($path)); |
|
223 | 223 | |
224 | 224 | return $stat; |
225 | 225 | } |
@@ -130,7 +130,7 @@ discard block |
||
130 | 130 | } |
131 | 131 | |
132 | 132 | /** |
133 | - * @param $keyword |
|
133 | + * @param string|false $keyword |
|
134 | 134 | * @param bool $skip |
135 | 135 | * @return string |
136 | 136 | * @throws phpFastCacheIOException |
@@ -176,7 +176,7 @@ discard block |
||
176 | 176 | |
177 | 177 | /** |
178 | 178 | * @param $filename |
179 | - * @return mixed |
|
179 | + * @return string |
|
180 | 180 | */ |
181 | 181 | protected static function cleanFileName($filename) |
182 | 182 | { |
@@ -191,7 +191,7 @@ discard block |
||
191 | 191 | } |
192 | 192 | |
193 | 193 | /** |
194 | - * @param $path |
|
194 | + * @param string $path |
|
195 | 195 | * @param bool $create |
196 | 196 | * @throws phpFastCacheIOException |
197 | 197 | */ |
@@ -225,7 +225,7 @@ discard block |
||
225 | 225 | |
226 | 226 | |
227 | 227 | /** |
228 | - * @param $file |
|
228 | + * @param string $file |
|
229 | 229 | * @return string |
230 | 230 | * @throws phpFastCacheIOException |
231 | 231 | */ |
@@ -80,10 +80,10 @@ discard block |
||
80 | 80 | } |
81 | 81 | |
82 | 82 | $full_path = rtrim($path, '/') |
83 | - . DIRECTORY_SEPARATOR |
|
84 | - . $securityKey |
|
85 | - . DIRECTORY_SEPARATOR |
|
86 | - . $this->getDriverName(); |
|
83 | + . DIRECTORY_SEPARATOR |
|
84 | + . $securityKey |
|
85 | + . DIRECTORY_SEPARATOR |
|
86 | + . $this->getDriverName(); |
|
87 | 87 | $full_path_hash = md5($full_path); |
88 | 88 | |
89 | 89 | /** |
@@ -207,9 +207,9 @@ discard block |
||
207 | 207 | protected static function cleanFileName($filename) |
208 | 208 | { |
209 | 209 | $regex = [ |
210 | - '/[\?\[\]\/\\\=\<\>\:\;\,\'\"\&\$\#\*\(\)\|\~\`\!\{\}]/', |
|
211 | - '/\.$/', |
|
212 | - '/^\./', |
|
210 | + '/[\?\[\]\/\\\=\<\>\:\;\,\'\"\&\$\#\*\(\)\|\~\`\!\{\}]/', |
|
211 | + '/\.$/', |
|
212 | + '/^\./', |
|
213 | 213 | ]; |
214 | 214 | $replace = ['-', '', '']; |
215 | 215 | |
@@ -301,7 +301,7 @@ discard block |
||
301 | 301 | $tmpFilename = Directory::getAbsolutePath(dirname($file) . '/tmp_' . md5( |
302 | 302 | str_shuffle(uniqid($this->getDriverName(), false)) |
303 | 303 | . str_shuffle(uniqid($this->getDriverName(), false)) |
304 | - )); |
|
304 | + )); |
|
305 | 305 | |
306 | 306 | $f = fopen($tmpFilename, 'w+'); |
307 | 307 | flock($f, LOCK_EX); |
@@ -41,10 +41,10 @@ discard block |
||
41 | 41 | * Calculate the security key |
42 | 42 | */ |
43 | 43 | { |
44 | - $securityKey = array_key_exists('securityKey', $this->config) ? $this->config[ 'securityKey' ] : ''; |
|
44 | + $securityKey = array_key_exists('securityKey', $this->config) ? $this->config['securityKey'] : ''; |
|
45 | 45 | if (!$securityKey || $securityKey === 'auto') { |
46 | - if (isset($_SERVER[ 'HTTP_HOST' ])) { |
|
47 | - $securityKey = preg_replace('/^www./', '', strtolower(str_replace(':', '_', $_SERVER[ 'HTTP_HOST' ]))); |
|
46 | + if (isset($_SERVER['HTTP_HOST'])) { |
|
47 | + $securityKey = preg_replace('/^www./', '', strtolower(str_replace(':', '_', $_SERVER['HTTP_HOST']))); |
|
48 | 48 | } else { |
49 | 49 | $securityKey = ($this->isPHPModule() ? 'web' : 'cli'); |
50 | 50 | } |
@@ -63,20 +63,20 @@ discard block |
||
63 | 63 | */ |
64 | 64 | $tmp_dir = rtrim($tmp_dir, '/') . DIRECTORY_SEPARATOR; |
65 | 65 | |
66 | - if (empty($this->config[ 'path' ]) || !is_string($this->config[ 'path' ])) { |
|
66 | + if (empty($this->config['path']) || !is_string($this->config['path'])) { |
|
67 | 67 | if (self::isPHPModule()) { |
68 | 68 | $path = $tmp_dir; |
69 | 69 | } else { |
70 | - $document_root_path = rtrim($_SERVER[ 'DOCUMENT_ROOT' ], '/') . '/../'; |
|
71 | - $path = isset($_SERVER[ 'DOCUMENT_ROOT' ]) && is_writable($document_root_path) ? $document_root_path : rtrim(__DIR__, '/') . DIRECTORY_SEPARATOR; |
|
70 | + $document_root_path = rtrim($_SERVER['DOCUMENT_ROOT'], '/') . '/../'; |
|
71 | + $path = isset($_SERVER['DOCUMENT_ROOT']) && is_writable($document_root_path) ? $document_root_path : rtrim(__DIR__, '/') . DIRECTORY_SEPARATOR; |
|
72 | 72 | } |
73 | 73 | |
74 | - if ($this->config[ 'path' ] != '') { |
|
75 | - $path = $this->config[ 'path' ]; |
|
74 | + if ($this->config['path'] != '') { |
|
75 | + $path = $this->config['path']; |
|
76 | 76 | } |
77 | 77 | |
78 | 78 | } else { |
79 | - $path = $this->config[ 'path' ]; |
|
79 | + $path = $this->config['path']; |
|
80 | 80 | } |
81 | 81 | |
82 | 82 | $full_path = rtrim($path, '/') |
@@ -93,19 +93,19 @@ discard block |
||
93 | 93 | * return the temp dir |
94 | 94 | */ |
95 | 95 | if ($readonly === true) { |
96 | - if($this->config[ 'autoTmpFallback' ] && (@file_exists($full_path) || !@is_writable($full_path))){ |
|
96 | + if ($this->config['autoTmpFallback'] && (@file_exists($full_path) || !@is_writable($full_path))) { |
|
97 | 97 | return $tmp_dir; |
98 | 98 | } |
99 | 99 | return $full_path; |
100 | - }else{ |
|
101 | - if (!isset($this->tmp[ $full_path_hash ]) || (!@file_exists($full_path) || !@is_writable($full_path))) { |
|
100 | + } else { |
|
101 | + if (!isset($this->tmp[$full_path_hash]) || (!@file_exists($full_path) || !@is_writable($full_path))) { |
|
102 | 102 | if (!@file_exists($full_path)) { |
103 | 103 | @mkdir($full_path, $this->setChmodAuto(), true); |
104 | - }else if (!@is_writable($full_path)) { |
|
104 | + } else if (!@is_writable($full_path)) { |
|
105 | 105 | @chmod($full_path, $this->setChmodAuto()); |
106 | 106 | } |
107 | 107 | |
108 | - if ($this->config[ 'autoTmpFallback' ] && !@is_writable($full_path)) { |
|
108 | + if ($this->config['autoTmpFallback'] && !@is_writable($full_path)) { |
|
109 | 109 | /** |
110 | 110 | * Switch back to tmp dir |
111 | 111 | * again if the path is not writable |
@@ -125,8 +125,8 @@ discard block |
||
125 | 125 | throw new phpFastCacheIOException('PLEASE CREATE OR CHMOD ' . $full_path . ' - 0777 OR ANY WRITABLE PERMISSION!'); |
126 | 126 | } |
127 | 127 | |
128 | - $this->tmp[ $full_path_hash ] = $full_path; |
|
129 | - $this->htaccessGen($full_path, array_key_exists('htaccess', $this->config) ? $this->config[ 'htaccess' ] : false); |
|
128 | + $this->tmp[$full_path_hash] = $full_path; |
|
129 | + $this->htaccessGen($full_path, array_key_exists('htaccess', $this->config) ? $this->config['htaccess'] : false); |
|
130 | 130 | } |
131 | 131 | } |
132 | 132 | |
@@ -193,10 +193,10 @@ discard block |
||
193 | 193 | */ |
194 | 194 | public function setChmodAuto() |
195 | 195 | { |
196 | - if (!isset($this->config[ 'default_chmod' ]) || $this->config[ 'default_chmod' ] == '' || is_null($this->config[ 'default_chmod' ])) { |
|
196 | + if (!isset($this->config['default_chmod']) || $this->config['default_chmod'] == '' || is_null($this->config['default_chmod'])) { |
|
197 | 197 | return 0777; |
198 | 198 | } else { |
199 | - return $this->config[ 'default_chmod' ]; |
|
199 | + return $this->config['default_chmod']; |
|
200 | 200 | } |
201 | 201 | } |
202 | 202 | |
@@ -226,7 +226,7 @@ discard block |
||
226 | 226 | if ($create === true) { |
227 | 227 | if (!is_writable($path)) { |
228 | 228 | try { |
229 | - if(!chmod($path, 0777)){ |
|
229 | + if (!chmod($path, 0777)) { |
|
230 | 230 | throw new phpFastCacheIOException('Chmod failed on : ' . $path); |
231 | 231 | } |
232 | 232 | } catch (phpFastCacheIOException $e) { |
@@ -297,7 +297,7 @@ discard block |
||
297 | 297 | */ |
298 | 298 | $this->eventManager->dispatch('CacheWriteFileOnDisk', $this, $file, $secureFileManipulation); |
299 | 299 | |
300 | - if($secureFileManipulation){ |
|
300 | + if ($secureFileManipulation) { |
|
301 | 301 | $tmpFilename = Directory::getAbsolutePath(dirname($file) . '/tmp_' . md5( |
302 | 302 | str_shuffle(uniqid($this->getDriverName(), false)) |
303 | 303 | . str_shuffle(uniqid($this->getDriverName(), false)) |
@@ -309,10 +309,10 @@ discard block |
||
309 | 309 | flock($f, LOCK_UN); |
310 | 310 | fclose($f); |
311 | 311 | |
312 | - if(!rename($tmpFilename, $file)){ |
|
312 | + if (!rename($tmpFilename, $file)) { |
|
313 | 313 | throw new phpFastCacheIOException(sprintf('Failed to rename %s to %s', $tmpFilename, $file)); |
314 | 314 | } |
315 | - }else{ |
|
315 | + } else { |
|
316 | 316 | $f = fopen($file, 'w+'); |
317 | 317 | $octetWritten = fwrite($f, $data); |
318 | 318 | fclose($f); |
@@ -97,11 +97,11 @@ discard block |
||
97 | 97 | return $tmp_dir; |
98 | 98 | } |
99 | 99 | return $full_path; |
100 | - }else{ |
|
100 | + } else{ |
|
101 | 101 | if (!isset($this->tmp[ $full_path_hash ]) || (!@file_exists($full_path) || !@is_writable($full_path))) { |
102 | 102 | if (!@file_exists($full_path)) { |
103 | 103 | @mkdir($full_path, $this->setChmodAuto(), true); |
104 | - }else if (!@is_writable($full_path)) { |
|
104 | + } else if (!@is_writable($full_path)) { |
|
105 | 105 | @chmod($full_path, $this->setChmodAuto()); |
106 | 106 | } |
107 | 107 | |
@@ -312,7 +312,7 @@ discard block |
||
312 | 312 | if(!rename($tmpFilename, $file)){ |
313 | 313 | throw new phpFastCacheIOException(sprintf('Failed to rename %s to %s', $tmpFilename, $file)); |
314 | 314 | } |
315 | - }else{ |
|
315 | + } else{ |
|
316 | 316 | $f = fopen($file, 'w+'); |
317 | 317 | $octetWritten = fwrite($f, $data); |
318 | 318 | fclose($f); |