Merge pull request #504 from Cosmologist/patch-2

Simplify CacheManager::validateConfig

Author: Geolim4

src/phpFastCache/CacheManager.php

@@ -438,21 +438,11 @@ protected static function validateConfig(array $config)
         foreach ($config as $configName => $configValue) {
             switch ($configName) {
                 case 'itemDetailedDate':
-                    if (!is_bool($configValue)) {
-                        throw new phpFastCacheInvalidConfigurationException("{$configName} must be a boolean");
-                    }
-                    break;
                 case 'autoTmpFallback':
-                    if (!is_bool($configValue)) {
-                        throw new phpFastCacheInvalidConfigurationException("{$configName} must be a boolean");
-                    }
-                    break;
                 case 'secureFileManipulation':
-                    if (!is_bool($configValue)) {
-                        throw new phpFastCacheInvalidConfigurationException("{$configName} must be a boolean");
-                    }
-                    break;
                 case 'ignoreSymfonyNotice':
+                case 'htaccess':
+                case 'compress_data':
                     if (!is_bool($configValue)) {
                         throw new phpFastCacheInvalidConfigurationException("{$configName} must be a boolean");
                     }
@@ -468,40 +458,22 @@ protected static function validateConfig(array $config)
                     }
                     break;
                 case 'securityKey':
+                case 'path':
                     if (!is_string($configValue)) {
                         throw new phpFastCacheInvalidConfigurationException("{$configName} must be a string");
                     }
                     break;
-                case 'htaccess':
-                    if (!is_bool($configValue)) {
-                        throw new phpFastCacheInvalidConfigurationException("{$configName} must be a boolean");
-                    }
-                    break;
                 case 'default_chmod':
+                case 'limited_memory_each_object':
                     if (!is_int($configValue)) {
                         throw new phpFastCacheInvalidConfigurationException("{$configName} must be an integer");
                     }
                     break;
-                case 'path':
-                    if (!is_string($configValue)) {
-                        throw new phpFastCacheInvalidConfigurationException("{$configName} must be a string");
-                    }
-                    break;
                 case 'fallback':
                     if (!is_bool($configValue) && !is_string($configValue)) {
                         throw new phpFastCacheInvalidConfigurationException("{$configName} must be a boolean or string");
                     }
                     break;
-                case 'limited_memory_each_object':
-                    if (!is_int($configValue)) {
-                        throw new phpFastCacheInvalidConfigurationException("{$configName} must be an integer");
-                    }
-                    break;
-                case 'compress_data':
-                    if (!is_bool($configValue)) {
-                        throw new phpFastCacheInvalidConfigurationException("{$configName} must be a boolean");
-                    }
-                    break;
                 case 'cacheFileExtension':
                     if (!is_string($configValue)) {
                         throw new phpFastCacheInvalidConfigurationException("{$configName} must be a boolean");
@@ -511,7 +483,7 @@ protected static function validateConfig(array $config)
                     }
                     if (!in_array($configValue, self::$safeFileExtensions)) {
                         throw new phpFastCacheInvalidConfigurationException(
-                          "{$configName} is not a safe extension, currently allowed extension: " . implode(', ', self::$safeFileExtensions)
+                            "{$configName} is not a safe extension, currently allowed extension: " . implode(', ', self::$safeFileExtensions)
                         );
                     }
                     break;