Make db_driver parameter optional with "no_driver" default value

Author: covex-nn

DependencyInjection/Configuration.php

@@ -34,17 +34,17 @@ public function getConfigTreeBuilder()
         $treeBuilder = new TreeBuilder();
         $rootNode = $treeBuilder->root('fos_user');
 
-        $supportedDrivers = array('orm', 'mongodb', 'couchdb', 'custom');
+        $supportedDrivers = array('orm', 'mongodb', 'couchdb', 'custom', 'no_driver');
 
         $rootNode
             ->children()
                 ->scalarNode('db_driver')
+                    ->defaultValue('no_driver')
                     ->validate()
                         ->ifNotInArray($supportedDrivers)
                         ->thenInvalid('The driver %s is not supported. Please choose one of '.json_encode($supportedDrivers))
                     ->end()
                     ->cannotBeOverwritten()
-                    ->isRequired()
                     ->cannotBeEmpty()
                 ->end()
                 ->scalarNode('user_class')->isRequired()->cannotBeEmpty()->end()

Model/GroupManagerNone.php

@@ -0,0 +1,45 @@
+<?php
+
+/*
+ * This file is part of the FOSUserBundle package.
+ *
+ * (c) FriendsOfSymfony <http://friendsofsymfony.github.com/>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace FOS\UserBundle\Model;
+
+/**
+ * Fallback Group Manager implementation when db_driver is not configured.
+ *
+ * @author Andrey F. Mindubaev <[email protected]>
+ */
+class GroupManagerNone extends GroupManager
+{
+    public function deleteGroup(GroupInterface $group)
+    {
+        throw new \RuntimeException('The child node "db_driver" at path "fos_user" must be configured.');
+    }
+
+    public function findGroupBy(array $criteria)
+    {
+        throw new \RuntimeException('The child node "db_driver" at path "fos_user" must be configured.');
+    }
+
+    public function findGroups()
+    {
+        throw new \RuntimeException('The child node "db_driver" at path "fos_user" must be configured.');
+    }
+
+    public function getClass()
+    {
+        throw new \RuntimeException('The child node "db_driver" at path "fos_user" must be configured.');
+    }
+
+    public function updateGroup(GroupInterface $group)
+    {
+        throw new \RuntimeException('The child node "db_driver" at path "fos_user" must be configured.');
+    }
+}

Model/UserManagerNone.php

@@ -0,0 +1,50 @@
+<?php
+
+/*
+ * This file is part of the FOSUserBundle package.
+ *
+ * (c) FriendsOfSymfony <http://friendsofsymfony.github.com/>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace FOS\UserBundle\Model;
+
+/**
+ * Fallback User Manager implementation when db_driver is not configured.
+ *
+ * @author Andrey F. Mindubaev <[email protected]>
+ */
+class UserManagerNone extends UserManager
+{
+    public function deleteUser(UserInterface $user)
+    {
+        throw new \RuntimeException('The child node "db_driver" at path "fos_user" must be configured.');
+    }
+
+    public function findUserBy(array $criteria)
+    {
+        throw new \RuntimeException('The child node "db_driver" at path "fos_user" must be configured.');
+    }
+
+    public function findUsers()
+    {
+        throw new \RuntimeException('The child node "db_driver" at path "fos_user" must be configured.');
+    }
+
+    public function getClass()
+    {
+        throw new \RuntimeException('The child node "db_driver" at path "fos_user" must be configured.');
+    }
+
+    public function reloadUser(UserInterface $user)
+    {
+        throw new \RuntimeException('The child node "db_driver" at path "fos_user" must be configured.');
+    }
+
+    public function updateUser(UserInterface $user)
+    {
+        throw new \RuntimeException('The child node "db_driver" at path "fos_user" must be configured.');
+    }
+}

Resources/config/no_driver.xml

@@ -0,0 +1,13 @@
+<?xml version="1.0" encoding="UTF-8" ?>
+
+<container xmlns="http://symfony.com/schema/dic/services"
+           xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+           xsi:schemaLocation="http://symfony.com/schema/dic/services http://symfony.com/schema/dic/services/services-1.0.xsd">
+
+    <services>
+        <service id="fos_user.user_manager.default" class="FOS\UserBundle\Model\UserManagerNone" public="false">
+            <argument type="service" id="fos_user.util.password_updater" />
+            <argument type="service" id="fos_user.util.canonical_fields_updater" />
+        </service>
+    </services>
+</container>

Resources/config/no_driver_group.xml

@@ -0,0 +1,10 @@
+<?xml version="1.0" encoding="UTF-8" ?>
+
+<container xmlns="http://symfony.com/schema/dic/services"
+           xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+           xsi:schemaLocation="http://symfony.com/schema/dic/services http://symfony.com/schema/dic/services/services-1.0.xsd">
+
+    <services>
+        <service id="fos_user.group_manager.default" class="FOS\UserBundle\Model\GroupManagerNone" public="false" />
+    </services>
+</container>

Resources/config/storage-validation/no_driver.xml

@@ -0,0 +1,10 @@
+<?xml version="1.0" ?>
+<constraint-mapping xmlns="http://symfony.com/schema/dic/constraint-mapping"
+    xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+    xsi:schemaLocation="http://symfony.com/schema/dic/constraint-mapping
+        http://symfony.com/schema/dic/constraint-mapping/constraint-mapping-1.0.xsd">
+
+    <class name="FOS\UserBundle\Model\User" />
+
+    <class name="FOS\UserBundle\Model\Group" />
+</constraint-mapping>

Tests/DependencyInjection/FOSUserExtensionTest.php

@@ -26,17 +26,6 @@ protected function tearDown()
         $this->configuration = null;
     }
 
-    /**
-     * @expectedException \Symfony\Component\Config\Definition\Exception\InvalidConfigurationException
-     */
-    public function testUserLoadThrowsExceptionUnlessDatabaseDriverSet()
-    {
-        $loader = new FOSUserExtension();
-        $config = $this->getEmptyConfig();
-        unset($config['db_driver']);
-        $loader->load(array($config), new ContainerBuilder());
-    }
-
     /**
      * @expectedException \Symfony\Component\Config\Definition\Exception\InvalidConfigurationException
      */