Change default serialization type to none as it implicitly was before

Fixes #417
snc · Apr 23, 2018 · ed4c925 · ed4c925
1 parent 59f1630
commit ed4c925
Showing 1 changed file with 3 additions and 11 deletions.
diff --git a/DependencyInjection/SncRedisExtension.php b/DependencyInjection/SncRedisExtension.php
@@ -443,6 +443,7 @@ protected function loadSwiftMailer(array $config, ContainerBuilder $container)
     public function loadSerializationType($type)
     {
         $types = array(
+            'default' => \Redis::SERIALIZER_NONE,
             'none' => \Redis::SERIALIZER_NONE,
             'php' => \Redis::SERIALIZER_PHP
         );
@@ -451,20 +452,11 @@ public function loadSerializationType($type)
             $types['igbinary'] = \Redis::SERIALIZER_IGBINARY;
         }
 
-        // allow user to pass in default serialization in which case we should automatically decide for them
-        if ('default' == $type) {
-            return isset($types['igbinary']) ? $types['igbinary'] : $types['php'];
-        } elseif (array_key_exists($type, $types)) {
+        if (array_key_exists($type, $types)) {
             return $types[$type];
         }
 
-        throw new InvalidConfigurationException(
-            sprintf(
-                '%s in not a valid serializer. Valid serializers: %s',
-                $type,
-                implode(", ", array_keys($types))
-            )
-        );
+        throw new InvalidConfigurationException(sprintf('%s in not a valid serializer. Valid serializers: %s', $type, implode(", ", array_keys($types))));
     }
 
      /* Loads the profiler storage configuration.