@@ -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 | } |
@@ -186,9 +186,9 @@ discard block |
||
| 186 | 186 | protected static function cleanFileName($filename) |
| 187 | 187 | { |
| 188 | 188 | $regex = [ |
| 189 | - '/[\?\[\]\/\\\=\<\>\:\;\,\'\"\&\$\#\*\(\)\|\~\`\!\{\}]/', |
|
| 190 | - '/\.$/', |
|
| 191 | - '/^\./', |
|
| 189 | + '/[\?\[\]\/\\\=\<\>\:\;\,\'\"\&\$\#\*\(\)\|\~\`\!\{\}]/', |
|
| 190 | + '/\.$/', |
|
| 191 | + '/^\./', |
|
| 192 | 192 | ]; |
| 193 | 193 | $replace = ['-', '', '']; |
| 194 | 194 | |
@@ -277,7 +277,7 @@ discard block |
||
| 277 | 277 | $tmpFilename = Directory::getAbsolutePath(dirname($file) . '/tmp_' . md5( |
| 278 | 278 | str_shuffle(uniqid($this->getDriverName(), false)) |
| 279 | 279 | . str_shuffle(uniqid($this->getDriverName(), false)) |
| 280 | - )); |
|
| 280 | + )); |
|
| 281 | 281 | |
| 282 | 282 | $f = fopen($tmpFilename, 'w+'); |
| 283 | 283 | flock($f, LOCK_EX); |
@@ -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 | */ |
@@ -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); |
@@ -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,10 +63,10 @@ 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 | $path = $tmp_dir; |
| 68 | 68 | } else { |
| 69 | - $path = rtrim($this->config[ 'path' ], '/') . DIRECTORY_SEPARATOR; |
|
| 69 | + $path = rtrim($this->config['path'], '/') . DIRECTORY_SEPARATOR; |
|
| 70 | 70 | } |
| 71 | 71 | |
| 72 | 72 | $path_suffix = $securityKey . DIRECTORY_SEPARATOR . $this->getDriverName(); |
@@ -81,19 +81,19 @@ discard block |
||
| 81 | 81 | * return the temp dir |
| 82 | 82 | */ |
| 83 | 83 | if ($readonly === true) { |
| 84 | - if($this->config[ 'autoTmpFallback' ] && (!@file_exists($full_path) || !@is_writable($full_path))){ |
|
| 84 | + if ($this->config['autoTmpFallback'] && (!@file_exists($full_path) || !@is_writable($full_path))) { |
|
| 85 | 85 | return $full_path_tmp; |
| 86 | 86 | } |
| 87 | 87 | return $full_path; |
| 88 | - }else{ |
|
| 89 | - if (!isset($this->tmp[ $full_path_hash ]) || (!@file_exists($full_path) || !@is_writable($full_path))) { |
|
| 88 | + } else { |
|
| 89 | + if (!isset($this->tmp[$full_path_hash]) || (!@file_exists($full_path) || !@is_writable($full_path))) { |
|
| 90 | 90 | if (!@file_exists($full_path)) { |
| 91 | 91 | @mkdir($full_path, $this->setChmodAuto(), true); |
| 92 | - }else if (!@is_writable($full_path)) { |
|
| 92 | + } else if (!@is_writable($full_path)) { |
|
| 93 | 93 | @chmod($full_path, $this->setChmodAuto()); |
| 94 | 94 | } |
| 95 | 95 | |
| 96 | - if ($this->config[ 'autoTmpFallback' ] && !@is_writable($full_path)) { |
|
| 96 | + if ($this->config['autoTmpFallback'] && !@is_writable($full_path)) { |
|
| 97 | 97 | /** |
| 98 | 98 | * Switch back to tmp dir |
| 99 | 99 | * again if the path is not writable |
@@ -113,8 +113,8 @@ discard block |
||
| 113 | 113 | throw new phpFastCacheIOException('PLEASE CREATE OR CHMOD ' . $full_path . ' - 0777 OR ANY WRITABLE PERMISSION!'); |
| 114 | 114 | } |
| 115 | 115 | |
| 116 | - $this->tmp[ $full_path_hash ] = $full_path; |
|
| 117 | - $this->htaccessGen($full_path, array_key_exists('htaccess', $this->config) ? $this->config[ 'htaccess' ] : false); |
|
| 116 | + $this->tmp[$full_path_hash] = $full_path; |
|
| 117 | + $this->htaccessGen($full_path, array_key_exists('htaccess', $this->config) ? $this->config['htaccess'] : false); |
|
| 118 | 118 | } |
| 119 | 119 | } |
| 120 | 120 | |
@@ -181,10 +181,10 @@ discard block |
||
| 181 | 181 | */ |
| 182 | 182 | public function setChmodAuto() |
| 183 | 183 | { |
| 184 | - if (!isset($this->config[ 'default_chmod' ]) || $this->config[ 'default_chmod' ] == '' || is_null($this->config[ 'default_chmod' ])) { |
|
| 184 | + if (!isset($this->config['default_chmod']) || $this->config['default_chmod'] == '' || is_null($this->config['default_chmod'])) { |
|
| 185 | 185 | return 0777; |
| 186 | 186 | } else { |
| 187 | - return $this->config[ 'default_chmod' ]; |
|
| 187 | + return $this->config['default_chmod']; |
|
| 188 | 188 | } |
| 189 | 189 | } |
| 190 | 190 | |
@@ -214,7 +214,7 @@ discard block |
||
| 214 | 214 | if ($create === true) { |
| 215 | 215 | if (!is_writable($path)) { |
| 216 | 216 | try { |
| 217 | - if(!chmod($path, 0777)){ |
|
| 217 | + if (!chmod($path, 0777)) { |
|
| 218 | 218 | throw new phpFastCacheIOException('Chmod failed on : ' . $path); |
| 219 | 219 | } |
| 220 | 220 | } catch (phpFastCacheIOException $e) { |
@@ -285,7 +285,7 @@ discard block |
||
| 285 | 285 | */ |
| 286 | 286 | $this->eventManager->dispatch('CacheWriteFileOnDisk', $this, $file, $secureFileManipulation); |
| 287 | 287 | |
| 288 | - if($secureFileManipulation){ |
|
| 288 | + if ($secureFileManipulation) { |
|
| 289 | 289 | $tmpFilename = Directory::getAbsolutePath(dirname($file) . '/tmp_' . md5( |
| 290 | 290 | str_shuffle(uniqid($this->getDriverName(), false)) |
| 291 | 291 | . str_shuffle(uniqid($this->getDriverName(), false)) |
@@ -297,10 +297,10 @@ discard block |
||
| 297 | 297 | flock($f, LOCK_UN); |
| 298 | 298 | fclose($f); |
| 299 | 299 | |
| 300 | - if(!rename($tmpFilename, $file)){ |
|
| 300 | + if (!rename($tmpFilename, $file)) { |
|
| 301 | 301 | throw new phpFastCacheIOException(sprintf('Failed to rename %s to %s', $tmpFilename, $file)); |
| 302 | 302 | } |
| 303 | - }else{ |
|
| 303 | + } else { |
|
| 304 | 304 | $f = fopen($file, 'w+'); |
| 305 | 305 | $octetWritten = fwrite($f, $data); |
| 306 | 306 | fclose($f); |