Completed
Pull Request — final (#396)
by Georges
04:22 queued 02:15
created
src/phpFastCache/Drivers/Memcached/Driver.php 2 patches
Indentation   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -131,7 +131,7 @@  discard block
 block discarded – undo
131 131
         $servers = (!empty($this->config[ 'memcache' ]) && is_array($this->config[ 'memcache' ]) ? $this->config[ 'memcache' ] : []);
132 132
         if (count($servers) < 1) {
133 133
             $servers = [
134
-              ['127.0.0.1', 11211],
134
+                ['127.0.0.1', 11211],
135 135
             ];
136 136
         }
137 137
 
@@ -161,9 +161,9 @@  discard block
 block discarded – undo
161 161
         $date = (new \DateTime())->setTimestamp(time() - $stats[ 'uptime' ]);
162 162
 
163 163
         return (new driverStatistic())
164
-          ->setData(implode(', ', array_keys($this->itemInstances)))
165
-          ->setInfo(sprintf("The memcache daemon v%s is up since %s.\n For more information see RawData.", $stats[ 'version' ], $date->format(DATE_RFC2822)))
166
-          ->setRawData($stats)
167
-          ->setSize($stats[ 'bytes' ]);
164
+            ->setData(implode(', ', array_keys($this->itemInstances)))
165
+            ->setInfo(sprintf("The memcache daemon v%s is up since %s.\n For more information see RawData.", $stats[ 'version' ], $date->format(DATE_RFC2822)))
166
+            ->setRawData($stats)
167
+            ->setSize($stats[ 'bytes' ]);
168 168
     }
169 169
 }
170 170
\ No newline at end of file
Please login to merge, or discard this patch.
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -127,7 +127,7 @@  discard block
 block discarded – undo
127 127
      */
128 128
     protected function driverConnect()
129 129
     {
130
-        $servers = (!empty($this->config[ 'memcache' ]) && is_array($this->config[ 'memcache' ]) ? $this->config[ 'memcache' ] : []);
130
+        $servers = (!empty($this->config['memcache']) && is_array($this->config['memcache']) ? $this->config['memcache'] : []);
131 131
         if (count($servers) < 1) {
132 132
             $servers = [
133 133
               ['127.0.0.1', 11211],
@@ -136,11 +136,11 @@  discard block
 block discarded – undo
136 136
 
137 137
         foreach ($servers as $server) {
138 138
             try {
139
-                if (!$this->instance->addServer($server[ 0 ], $server[ 1 ])) {
139
+                if (!$this->instance->addServer($server[0], $server[1])) {
140 140
                     $this->fallback = true;
141 141
                 }
142
-                if(!empty($server[ 'sasl_user' ]) && !empty($server[ 'sasl_password'])){
143
-                    $this->instance->setSaslAuthData($server[ 'sasl_user' ], $server[ 'sasl_password']);
142
+                if (!empty($server['sasl_user']) && !empty($server['sasl_password'])) {
143
+                    $this->instance->setSaslAuthData($server['sasl_user'], $server['sasl_password']);
144 144
                 }
145 145
             } catch (\Exception $e) {
146 146
                 $this->fallback = true;
@@ -160,16 +160,16 @@  discard block
 block discarded – undo
160 160
     public function getStats()
161 161
     {
162 162
         $stats = (array) $this->instance->getStats();
163
-        $stats[ 'uptime' ] = (isset($stats[ 'uptime' ]) ? $stats[ 'uptime' ] : 0);
164
-        $stats[ 'version' ] = (isset($stats[ 'version' ]) ? $stats[ 'version' ] : 'UnknownVersion');
165
-        $stats[ 'bytes' ] = (isset($stats[ 'bytes' ]) ? $stats[ 'version' ] : 0);
163
+        $stats['uptime'] = (isset($stats['uptime']) ? $stats['uptime'] : 0);
164
+        $stats['version'] = (isset($stats['version']) ? $stats['version'] : 'UnknownVersion');
165
+        $stats['bytes'] = (isset($stats['bytes']) ? $stats['version'] : 0);
166 166
 
167
-        $date = (new \DateTime())->setTimestamp(time() - $stats[ 'uptime' ]);
167
+        $date = (new \DateTime())->setTimestamp(time() - $stats['uptime']);
168 168
 
169 169
         return (new driverStatistic())
170 170
           ->setData(implode(', ', array_keys($this->itemInstances)))
171
-          ->setInfo(sprintf("The memcache daemon v%s is up since %s.\n For more information see RawData.", $stats[ 'version' ], $date->format(DATE_RFC2822)))
171
+          ->setInfo(sprintf("The memcache daemon v%s is up since %s.\n For more information see RawData.", $stats['version'], $date->format(DATE_RFC2822)))
172 172
           ->setRawData($stats)
173
-          ->setSize($stats[ 'bytes' ]);
173
+          ->setSize($stats['bytes']);
174 174
     }
175 175
 }
176 176
\ No newline at end of file
Please login to merge, or discard this patch.
src/phpFastCache/Proxy/phpFastCacheAbstractProxy.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -74,9 +74,9 @@
 block discarded – undo
74 74
      */
75 75
     public function __call($name, $args)
76 76
     {
77
-        if(method_exists($this->instance, $name)){
77
+        if (method_exists($this->instance, $name)) {
78 78
             return call_user_func_array([$this->instance, $name], $args);
79
-        }else{
79
+        } else {
80 80
             throw new \BadMethodCallException(sprintf('Method %s does not exists', $name));
81 81
         }
82 82
     }
Please login to merge, or discard this patch.
Braces   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -273,7 +273,7 @@
 block discarded – undo
273 273
         
274 274
         if (!is_dir($path)) {
275 275
             throw new phpFastCacheDriverException("Can't read PATH:" . $path, 94);
276
-        }else{
276
+        } else{
277 277
             $size = Directory::dirSize($path);
278 278
         }
279 279
 
Please login to merge, or discard this patch.
src/phpFastCache/CacheManager.php 3 patches
Braces   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -95,13 +95,13 @@  discard block
 block discarded – undo
95 95
             $class = self::getNamespacePath() . $driver . '\Driver';
96 96
             try{
97 97
                 self::$instances[ $instance ] = new $class($config);
98
-            }catch(phpFastCacheDriverCheckException $e){
98
+            } catch(phpFastCacheDriverCheckException $e){
99 99
                 $fallback = self::standardizeDriverName($config['fallback']);
100 100
                 if($fallback && $fallback !== $driver){
101 101
                     $class = self::getNamespacePath() . $fallback . '\Driver';
102 102
                     self::$instances[ $instance ] = new $class($config);
103 103
                     trigger_error(sprintf('The "%s" driver is unavailable at the moment, the fallback driver "%s" has been used instead.', $driver, $fallback), E_USER_WARNING);
104
-                }else{
104
+                } else{
105 105
                     throw new phpFastCacheDriverCheckException($e->getMessage(), $e->getCode(), $e);
106 106
                 }
107 107
             }
@@ -198,7 +198,7 @@  discard block
 block discarded – undo
198 198
             self::$config = array_merge(self::$config, $name);
199 199
         } else if (is_string($name)){
200 200
             self::$config[ $name ] = $value;
201
-        }else{
201
+        } else{
202 202
             throw new \InvalidArgumentException('Invalid variable type: $name');
203 203
         }
204 204
     }
Please login to merge, or discard this patch.
Indentation   +27 added lines, -27 removed lines patch added patch discarded remove patch
@@ -57,15 +57,15 @@  discard block
 block discarded – undo
57 57
      * @var array
58 58
      */
59 59
     protected static $config = [
60
-      'securityKey' => 'auto',// The securityKey that will be used to create sub-directory
61
-      'ignoreSymfonyNotice' => false,// Ignore Symfony notice for Symfony project which do not makes use of PhpFastCache's Symfony Bundle
62
-      'defaultTtl' => 900,// Default time-to-live in second
63
-      'htaccess' => true,// Auto-generate .htaccess if tit is missing
64
-      'default_chmod' => 0777, // 0777 recommended
65
-      'path' => '',// if not set will be the value of sys_get_temp_dir()
66
-      'fallback' => false, //Fall back when old driver is not support
67
-      'limited_memory_each_object' => 4096, // maximum size (bytes) of object store in memory
68
-      'compress_data' => false, // compress stored data, if the backend supports it
60
+        'securityKey' => 'auto',// The securityKey that will be used to create sub-directory
61
+        'ignoreSymfonyNotice' => false,// Ignore Symfony notice for Symfony project which do not makes use of PhpFastCache's Symfony Bundle
62
+        'defaultTtl' => 900,// Default time-to-live in second
63
+        'htaccess' => true,// Auto-generate .htaccess if tit is missing
64
+        'default_chmod' => 0777, // 0777 recommended
65
+        'path' => '',// if not set will be the value of sys_get_temp_dir()
66
+        'fallback' => false, //Fall back when old driver is not support
67
+        'limited_memory_each_object' => 4096, // maximum size (bytes) of object store in memory
68
+        'compress_data' => false, // compress stored data, if the backend supports it
69 69
     ];
70 70
 
71 71
     /**
@@ -116,7 +116,7 @@  discard block
 block discarded – undo
116 116
                 }
117 117
             }
118 118
         } else if(++$badPracticeOmeter[$driver] >= 5){
119
-           trigger_error('[' . $driver . '] Calling many times CacheManager::getInstance() for already instanced drivers is a bad practice and have a significant impact on performances.
119
+            trigger_error('[' . $driver . '] Calling many times CacheManager::getInstance() for already instanced drivers is a bad practice and have a significant impact on performances.
120 120
            See https://github.com/PHPSocialNetwork/phpfastcache/wiki/[V5]-Why-calling-getInstance%28%29-each-time-is-a-bad-practice-%3F');
121 121
         }
122 122
 
@@ -227,23 +227,23 @@  discard block
 block discarded – undo
227 227
     public static function getStaticSystemDrivers()
228 228
     {
229 229
         return [
230
-          'Sqlite',
231
-          'Files',
232
-          'Apc',
233
-          'Apcu',
234
-          'Memcache',
235
-          'Memcached',
236
-          'Couchbase',
237
-          'Mongodb',
238
-          'Predis',
239
-          'Redis',
240
-          'Ssdb',
241
-          'Leveldb',
242
-          'Wincache',
243
-          'Xcache',
244
-          'Zenddisk',
245
-          'Zendshm',
246
-          'Devnull',
230
+            'Sqlite',
231
+            'Files',
232
+            'Apc',
233
+            'Apcu',
234
+            'Memcache',
235
+            'Memcached',
236
+            'Couchbase',
237
+            'Mongodb',
238
+            'Predis',
239
+            'Redis',
240
+            'Ssdb',
241
+            'Leveldb',
242
+            'Wincache',
243
+            'Xcache',
244
+            'Zenddisk',
245
+            'Zendshm',
246
+            'Devnull',
247 247
         ];
248 248
     }
249 249
 
Please login to merge, or discard this patch.
Spacing   +19 added lines, -19 removed lines patch added patch discarded remove patch
@@ -57,12 +57,12 @@  discard block
 block discarded – undo
57 57
      * @var array
58 58
      */
59 59
     protected static $config = [
60
-      'securityKey' => 'auto',// The securityKey that will be used to create sub-directory
61
-      'ignoreSymfonyNotice' => false,// Ignore Symfony notice for Symfony project which do not makes use of PhpFastCache's Symfony Bundle
62
-      'defaultTtl' => 900,// Default time-to-live in second
63
-      'htaccess' => true,// Auto-generate .htaccess if tit is missing
60
+      'securityKey' => 'auto', // The securityKey that will be used to create sub-directory
61
+      'ignoreSymfonyNotice' => false, // Ignore Symfony notice for Symfony project which do not makes use of PhpFastCache's Symfony Bundle
62
+      'defaultTtl' => 900, // Default time-to-live in second
63
+      'htaccess' => true, // Auto-generate .htaccess if tit is missing
64 64
       'default_chmod' => 0777, // 0777 recommended
65
-      'path' => '',// if not set will be the value of sys_get_temp_dir()
65
+      'path' => '', // if not set will be the value of sys_get_temp_dir()
66 66
       'fallback' => false, //Fall back when old driver is not support
67 67
       'limited_memory_each_object' => 4096, // maximum size (bytes) of object store in memory
68 68
       'compress_data' => false, // compress stored data, if the backend supports it
@@ -97,30 +97,30 @@  discard block
 block discarded – undo
97 97
         }
98 98
 
99 99
         $instance = crc32($driver . serialize($config));
100
-        if (!isset(self::$instances[ $instance ])) {
100
+        if (!isset(self::$instances[$instance])) {
101 101
             $badPracticeOmeter[$driver] = 1;
102
-            if(!$config['ignoreSymfonyNotice'] && interface_exists('Symfony\Component\HttpKernel\KernelInterface') && !class_exists('phpFastCache\Bundle\phpFastCacheBundle')){
102
+            if (!$config['ignoreSymfonyNotice'] && interface_exists('Symfony\Component\HttpKernel\KernelInterface') && !class_exists('phpFastCache\Bundle\phpFastCacheBundle')) {
103 103
                 trigger_error('A Symfony Bundle to make the PhpFastCache integration more easier is now available here: https://github.com/PHPSocialNetwork/phpfastcache-bundle', E_USER_NOTICE);
104 104
             }
105 105
             $class = self::getNamespacePath() . $driver . '\Driver';
106
-            try{
107
-                self::$instances[ $instance ] = new $class($config);
108
-            }catch(phpFastCacheDriverCheckException $e){
106
+            try {
107
+                self::$instances[$instance] = new $class($config);
108
+            } catch (phpFastCacheDriverCheckException $e) {
109 109
                 $fallback = self::standardizeDriverName($config['fallback']);
110
-                if($fallback && $fallback !== $driver){
110
+                if ($fallback && $fallback !== $driver) {
111 111
                     $class = self::getNamespacePath() . $fallback . '\Driver';
112
-                    self::$instances[ $instance ] = new $class($config);
112
+                    self::$instances[$instance] = new $class($config);
113 113
                     trigger_error(sprintf('The "%s" driver is unavailable at the moment, the fallback driver "%s" has been used instead.', $driver, $fallback), E_USER_WARNING);
114
-                }else{
114
+                } else {
115 115
                     throw new phpFastCacheDriverCheckException($e->getMessage(), $e->getCode(), $e);
116 116
                 }
117 117
             }
118
-        } else if(++$badPracticeOmeter[$driver] >= 5){
118
+        } else if (++$badPracticeOmeter[$driver] >= 5) {
119 119
            trigger_error('[' . $driver . '] Calling many times CacheManager::getInstance() for already instanced drivers is a bad practice and have a significant impact on performances.
120 120
            See https://github.com/PHPSocialNetwork/phpfastcache/wiki/[V5]-Why-calling-getInstance%28%29-each-time-is-a-bad-practice-%3F');
121 121
         }
122 122
 
123
-        return self::$instances[ $instance ];
123
+        return self::$instances[$instance];
124 124
     }
125 125
 
126 126
     /**
@@ -153,7 +153,7 @@  discard block
 block discarded – undo
153 153
      */
154 154
     public static function __callStatic($name, $arguments)
155 155
     {
156
-        $options = (array_key_exists(0, $arguments) && is_array($arguments) ? $arguments[ 0 ] : []);
156
+        $options = (array_key_exists(0, $arguments) && is_array($arguments) ? $arguments[0] : []);
157 157
 
158 158
         return self::getInstance($name, $options);
159 159
     }
@@ -206,9 +206,9 @@  discard block
 block discarded – undo
206 206
     {
207 207
         if (is_array($name)) {
208 208
             self::$config = array_merge(self::$config, $name);
209
-        } else if (is_string($name)){
210
-            self::$config[ $name ] = $value;
211
-        }else{
209
+        } else if (is_string($name)) {
210
+            self::$config[$name] = $value;
211
+        } else {
212 212
             throw new \InvalidArgumentException('Invalid variable type: $name');
213 213
         }
214 214
     }
Please login to merge, or discard this patch.
src/phpFastCache/Core/PathSeekerTrait.php 4 patches
Indentation   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -185,9 +185,9 @@
 block discarded – undo
185 185
     protected static function cleanFileName($filename)
186 186
     {
187 187
         $regex = [
188
-          '/[\?\[\]\/\\\=\<\>\:\;\,\'\"\&\$\#\*\(\)\|\~\`\!\{\}]/',
189
-          '/\.$/',
190
-          '/^\./',
188
+            '/[\?\[\]\/\\\=\<\>\:\;\,\'\"\&\$\#\*\(\)\|\~\`\!\{\}]/',
189
+            '/\.$/',
190
+            '/^\./',
191 191
         ];
192 192
         $replace = ['-', '', ''];
193 193
 
Please login to merge, or discard this patch.
Doc Comments   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -141,7 +141,7 @@  discard block
 block discarded – undo
141 141
     }
142 142
 
143 143
     /**
144
-     * @param $keyword
144
+     * @param string|false $keyword
145 145
      * @param bool $skip
146 146
      * @return string
147 147
      * @throws phpFastCacheDriverException
@@ -187,7 +187,7 @@  discard block
 block discarded – undo
187 187
 
188 188
     /**
189 189
      * @param $filename
190
-     * @return mixed
190
+     * @return string
191 191
      */
192 192
     protected static function cleanFileName($filename)
193 193
     {
@@ -202,7 +202,7 @@  discard block
 block discarded – undo
202 202
     }
203 203
 
204 204
     /**
205
-     * @param $path
205
+     * @param string $path
206 206
      * @param bool $create
207 207
      * @throws \Exception
208 208
      */
Please login to merge, or discard this patch.
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -40,10 +40,10 @@  discard block
 block discarded – undo
40 40
          * Calculate the security key
41 41
          */
42 42
         {
43
-            $securityKey = array_key_exists('securityKey', $this->config) ? $this->config[ 'securityKey' ] : '';
43
+            $securityKey = array_key_exists('securityKey', $this->config) ? $this->config['securityKey'] : '';
44 44
             if (!$securityKey || $securityKey === 'auto') {
45
-                if (isset($_SERVER[ 'HTTP_HOST' ])) {
46
-                    $securityKey = preg_replace('/^www./', '', strtolower(str_replace(':', '_', $_SERVER[ 'HTTP_HOST' ])));
45
+                if (isset($_SERVER['HTTP_HOST'])) {
46
+                    $securityKey = preg_replace('/^www./', '', strtolower(str_replace(':', '_', $_SERVER['HTTP_HOST'])));
47 47
                 } else {
48 48
                     $securityKey = ($this->isPHPModule() ? 'web' : 'cli');
49 49
                 }
@@ -58,10 +58,10 @@  discard block
 block discarded – undo
58 58
          * with the security key and the driver name
59 59
          */
60 60
         $tmp_dir = rtrim($tmp_dir, '/') . DIRECTORY_SEPARATOR;
61
-        if (empty($this->config[ 'path' ]) || !is_string($this->config[ 'path' ])) {
61
+        if (empty($this->config['path']) || !is_string($this->config['path'])) {
62 62
             $path = $tmp_dir;
63 63
         } else {
64
-            $path = rtrim($this->config[ 'path' ], '/') . DIRECTORY_SEPARATOR;
64
+            $path = rtrim($this->config['path'], '/') . DIRECTORY_SEPARATOR;
65 65
         }
66 66
         $path_suffix = $securityKey . DIRECTORY_SEPARATOR . $this->getDriverName();
67 67
         $full_path = Directory::getAbsolutePath($path . $path_suffix);
@@ -74,12 +74,12 @@  discard block
 block discarded – undo
74 74
          * return the temp dir
75 75
          */
76 76
         if ($readonly === true) {
77
-            if(!@file_exists($full_path) || !@is_writable($full_path)){
77
+            if (!@file_exists($full_path) || !@is_writable($full_path)) {
78 78
                 return $full_path_tmp;
79 79
             }
80 80
             return $full_path;
81
-        }else{
82
-            if (!isset($this->tmp[ $full_path_hash ]) || (!@file_exists($full_path) || !@is_writable($full_path))) {
81
+        } else {
82
+            if (!isset($this->tmp[$full_path_hash]) || (!@file_exists($full_path) || !@is_writable($full_path))) {
83 83
                 if (!@file_exists($full_path)) {
84 84
                     @mkdir($full_path, $this->setChmodAuto(), true);
85 85
                 } elseif (!@is_writable($full_path)) {
@@ -99,8 +99,8 @@  discard block
 block discarded – undo
99 99
                     }
100 100
                 }
101 101
 
102
-                $this->tmp[ $full_path_hash ] = $full_path;
103
-                $this->htaccessGen($full_path, array_key_exists('htaccess', $this->config) ? $this->config[ 'htaccess' ] : false);
102
+                $this->tmp[$full_path_hash] = $full_path;
103
+                $this->htaccessGen($full_path, array_key_exists('htaccess', $this->config) ? $this->config['htaccess'] : false);
104 104
             }
105 105
         }
106 106
         return realpath($full_path);
@@ -174,10 +174,10 @@  discard block
 block discarded – undo
174 174
      */
175 175
     public function setChmodAuto()
176 176
     {
177
-        if (!isset($this->config[ 'default_chmod' ]) || $this->config[ 'default_chmod' ] == '' || is_null($this->config[ 'default_chmod' ])) {
177
+        if (!isset($this->config['default_chmod']) || $this->config['default_chmod'] == '' || is_null($this->config['default_chmod'])) {
178 178
             return 0777;
179 179
         } else {
180
-            return $this->config[ 'default_chmod' ];
180
+            return $this->config['default_chmod'];
181 181
         }
182 182
     }
183 183
 
@@ -207,7 +207,7 @@  discard block
 block discarded – undo
207 207
         if ($create === true) {
208 208
             if (!is_writable($path)) {
209 209
                 try {
210
-                    if(!chmod($path, 0777)){
210
+                    if (!chmod($path, 0777)) {
211 211
                         throw new phpFastCacheDriverException('Chmod failed on : ' . $path);
212 212
                     }
213 213
                 } catch (phpFastCacheDriverException $e) {
Please login to merge, or discard this patch.
Braces   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -78,11 +78,11 @@
 block discarded – undo
78 78
                 return $full_path_tmp;
79 79
             }
80 80
             return $full_path;
81
-        }else{
81
+        } else{
82 82
             if (!isset($this->tmp[ $full_path_hash ]) || (!@file_exists($full_path) || !@is_writable($full_path))) {
83 83
                 if (!@file_exists($full_path)) {
84 84
                     @mkdir($full_path, $this->setChmodAuto(), true);
85
-                }elseif (!@is_writable($full_path)) {
85
+                } elseif (!@is_writable($full_path)) {
86 86
                     if (!@chmod($full_path, $this->setChmodAuto()))
87 87
                     {
88 88
                         /**
Please login to merge, or discard this patch.
src/autoload.php 3 patches
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -18,9 +18,9 @@  discard block
 block discarded – undo
18 18
 /**
19 19
  * Register Autoload
20 20
  */
21
-spl_autoload_register(function ($entity) {
21
+spl_autoload_register(function($entity) {
22 22
     $module = explode('\\', $entity, 2);
23
-    if (!in_array($module[ 0 ], ['phpFastCache', 'Psr'])) {
23
+    if (!in_array($module[0], ['phpFastCache', 'Psr'])) {
24 24
         /**
25 25
          * Not a part of phpFastCache file
26 26
          * then we return here.
@@ -32,7 +32,7 @@  discard block
 block discarded – undo
32 32
 
33 33
         if (is_readable($path)) {
34 34
             require_once $path;
35
-        }else{
35
+        } else {
36 36
             trigger_error('Cannot locate the Psr/Cache files', E_USER_ERROR);
37 37
             return;
38 38
         }
Please login to merge, or discard this patch.
Braces   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -273,7 +273,7 @@
 block discarded – undo
273 273
         
274 274
         if (!is_dir($path)) {
275 275
             throw new phpFastCacheDriverException("Can't read PATH:" . $path, 94);
276
-        }else{
276
+        } else{
277 277
             $size = Directory::dirSize($path);
278 278
         }
279 279
 
Please login to merge, or discard this patch.
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -46,6 +46,6 @@
 block discarded – undo
46 46
 });
47 47
 
48 48
 if ((!defined('PFC_IGNORE_COMPOSER_WARNING') || !PFC_IGNORE_COMPOSER_WARNING) && class_exists('Composer\Autoload\ClassLoader')) {
49
-  trigger_error('Your project already makes use of Composer. You SHOULD use the composer dependency "phpfastcache/phpfastcache" instead of hard-autoloading.',
49
+    trigger_error('Your project already makes use of Composer. You SHOULD use the composer dependency "phpfastcache/phpfastcache" instead of hard-autoloading.',
50 50
     E_USER_WARNING);
51 51
 }
52 52
\ No newline at end of file
Please login to merge, or discard this patch.
tests/Autoload.test.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -18,7 +18,7 @@  discard block
 block discarded – undo
18 18
 if (!class_exists('phpFastCache\CacheManager')) {
19 19
     echo "[FAIL] Autoload failed to find the CacheManager\n";
20 20
     $status = 255;
21
-}else{
21
+} else {
22 22
     echo "[PASS] Autoload successfully found the CacheManager\n";
23 23
 }
24 24
 
@@ -28,7 +28,7 @@  discard block
 block discarded – undo
28 28
 if (!interface_exists('Psr\Cache\CacheItemInterface')) {
29 29
     echo "[FAIL] Autoload failed to find the Psr CacheItemInterface\n";
30 30
     $status = 255;
31
-}else{
31
+} else {
32 32
     echo "[PASS] Autoload successfully found the Psr CacheItemInterface\n";
33 33
 }
34 34
 
Please login to merge, or discard this patch.
Braces   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -18,7 +18,7 @@  discard block
 block discarded – undo
18 18
 if (!class_exists('phpFastCache\CacheManager')) {
19 19
     echo "[FAIL] Autoload failed to find the CacheManager\n";
20 20
     $status = 255;
21
-}else{
21
+} else{
22 22
     echo "[PASS] Autoload successfully found the CacheManager\n";
23 23
 }
24 24
 
@@ -28,7 +28,7 @@  discard block
 block discarded – undo
28 28
 if (!interface_exists('Psr\Cache\CacheItemInterface')) {
29 29
     echo "[FAIL] Autoload failed to find the Psr CacheItemInterface\n";
30 30
     $status = 255;
31
-}else{
31
+} else{
32 32
     echo "[PASS] Autoload successfully found the Psr CacheItemInterface\n";
33 33
 }
34 34
 
Please login to merge, or discard this patch.
tests/AbstractProxy.test.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -42,10 +42,10 @@
 block discarded – undo
42 42
 if (!is_object($driverInstance->getItem('test'))) {
43 43
     echo '[FAIL] $driverInstance->getItem() returned an invalid var type:' . gettype($driverInstance) . "\n";
44 44
     $status = 1;
45
-}else if(!($driverInstance->getItem('test') instanceof ExtendedCacheItemInterface)){
45
+} else if (!($driverInstance->getItem('test') instanceof ExtendedCacheItemInterface)) {
46 46
     echo '[FAIL] $driverInstance->getItem() returned an invalid class that does not implements ExtendedCacheItemInterface: ' . get_class($driverInstance) . "\n";
47 47
     $status = 1;
48
-}else{
48
+} else {
49 49
     echo '[PASS] $driverInstance->getItem() returned a valid class that implements ExtendedCacheItemInterface: ' . get_class($driverInstance) . "\n";
50 50
 }
51 51
 
Please login to merge, or discard this patch.
Braces   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -42,10 +42,10 @@
 block discarded – undo
42 42
 if (!is_object($driverInstance->getItem('test'))) {
43 43
     echo '[FAIL] $driverInstance->getItem() returned an invalid var type:' . gettype($driverInstance) . "\n";
44 44
     $status = 1;
45
-}else if(!($driverInstance->getItem('test') instanceof ExtendedCacheItemInterface)){
45
+} else if(!($driverInstance->getItem('test') instanceof ExtendedCacheItemInterface)){
46 46
     echo '[FAIL] $driverInstance->getItem() returned an invalid class that does not implements ExtendedCacheItemInterface: ' . get_class($driverInstance) . "\n";
47 47
     $status = 1;
48
-}else{
48
+} else{
49 49
     echo '[PASS] $driverInstance->getItem() returned a valid class that implements ExtendedCacheItemInterface: ' . get_class($driverInstance) . "\n";
50 50
 }
51 51
 
Please login to merge, or discard this patch.
src/phpFastCache/Entities/driverStatistic.php 1 patch
Doc Comments   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -52,7 +52,7 @@  discard block
 block discarded – undo
52 52
     }
53 53
 
54 54
     /**
55
-     * @return int|bool Return size in octet or false if no information available
55
+     * @return string Return size in octet or false if no information available
56 56
      */
57 57
     public function getSize()
58 58
     {
@@ -60,7 +60,7 @@  discard block
 block discarded – undo
60 60
     }
61 61
 
62 62
     /**
63
-     * @return mixed
63
+     * @return string
64 64
      */
65 65
     public function getData()
66 66
     {
@@ -68,7 +68,7 @@  discard block
 block discarded – undo
68 68
     }
69 69
 
70 70
     /**
71
-     * @param $info
71
+     * @param string $info
72 72
      * @return $this
73 73
      */
74 74
     public function setInfo($info)
Please login to merge, or discard this patch.
tests/AttachingDetachingMethods.test.php 2 patches
Braces   +4 added lines, -5 removed lines patch added patch discarded remove patch
@@ -24,10 +24,10 @@  discard block
 block discarded – undo
24 24
 if (!is_object($driverInstance)) {
25 25
     echo '[FAIL] CacheManager::getInstance() returned an invalid variable type:' . gettype($driverInstance) . "\n";
26 26
     $status = 1;
27
-}else if(!($driverInstance instanceof CacheItemPoolInterface)){
27
+} else if(!($driverInstance instanceof CacheItemPoolInterface)){
28 28
     echo '[FAIL] CacheManager::getInstance() returned an invalid class:' . get_class($driverInstance) . "\n";
29 29
     $status = 1;
30
-}else{
30
+} else{
31 31
     $key = 'test_attaching_detaching';
32 32
 
33 33
     $itemDetached = $driverInstance->getItem($key);
@@ -37,8 +37,7 @@  discard block
 block discarded – undo
37 37
     if(!$driverInstance->isAttached($itemDetached))
38 38
     {
39 39
         echo '[PASS] ExtendedCacheItemPoolInterface::isAttached() identified $itemDetached as being detached.' . "\n";
40
-    }
41
-    else
40
+    } else
42 41
     {
43 42
         echo '[FAIL] ExtendedCacheItemPoolInterface::isAttached() failed to identify $itemDetached as to be detached.' . "\n";
44 43
         $status = 1;
@@ -48,7 +47,7 @@  discard block
 block discarded – undo
48 47
         $driverInstance->attachItem($itemDetached);
49 48
         echo '[FAIL] ExtendedCacheItemPoolInterface::attachItem() attached $itemDetached without trowing an error.' . "\n";
50 49
         $status = 1;
51
-    }catch(\LogicException $e){
50
+    } catch(\LogicException $e){
52 51
         echo '[PASS] ExtendedCacheItemPoolInterface::attachItem() failed to attach $itemDetached by trowing a LogicException exception.' . "\n";
53 52
     }
54 53
 
Please login to merge, or discard this patch.
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -24,17 +24,17 @@  discard block
 block discarded – undo
24 24
 if (!is_object($driverInstance)) {
25 25
     echo '[FAIL] CacheManager::getInstance() returned an invalid variable type:' . gettype($driverInstance) . "\n";
26 26
     $status = 1;
27
-}else if(!($driverInstance instanceof CacheItemPoolInterface)){
27
+} else if (!($driverInstance instanceof CacheItemPoolInterface)) {
28 28
     echo '[FAIL] CacheManager::getInstance() returned an invalid class:' . get_class($driverInstance) . "\n";
29 29
     $status = 1;
30
-}else{
30
+} else {
31 31
     $key = 'test_attaching_detaching';
32 32
 
33 33
     $itemDetached = $driverInstance->getItem($key);
34 34
     $driverInstance->detachItem($itemDetached);
35 35
     $itemAttached = $driverInstance->getItem($key);
36 36
 
37
-    if($driverInstance->isAttached($itemDetached) !== true)
37
+    if ($driverInstance->isAttached($itemDetached) !== true)
38 38
     {
39 39
         echo '[PASS] ExtendedCacheItemPoolInterface::isAttached() identified $itemDetached as being detached.' . "\n";
40 40
     }
@@ -44,11 +44,11 @@  discard block
 block discarded – undo
44 44
         $status = 1;
45 45
     }
46 46
 
47
-    try{
47
+    try {
48 48
         $driverInstance->attachItem($itemDetached);
49 49
         echo '[FAIL] ExtendedCacheItemPoolInterface::attachItem() attached $itemDetached without trowing an error.' . "\n";
50 50
         $status = 1;
51
-    }catch(\LogicException $e){
51
+    } catch (\LogicException $e) {
52 52
         echo '[PASS] ExtendedCacheItemPoolInterface::attachItem() failed to attach $itemDetached by trowing a LogicException exception.' . "\n";
53 53
     }
54 54
 
Please login to merge, or discard this patch.