use "memory" connection parameter instead of "path" for SQLite URLs if possible

also refactors URL path evaluation into smaller methods

fixes #1106

lib/Doctrine/DBAL/DriverManager.php

@@ -259,15 +259,7 @@ final class DriverManager
         }
 
         if (isset($url['path'])) {
-            $nameKey = isset($url['scheme']) && strpos($url['scheme'], 'sqlite') !== false
-                ? 'path' // pdo_sqlite driver uses 'path' instead of 'dbname' key
-                : 'dbname';
-
-            if (!isset($url['scheme']) || (strpos($url['scheme'], 'sqlite') !== false && $url['path'] == ':memory:')) {
-                $params[$nameKey] = $url['path']; // if the URL was just "sqlite::memory:", which parses to scheme and path only
-            } else {
-                $params[$nameKey] = substr($url['path'], 1); // strip the leading slash from the URL
-            }
+            $params = self::parseDatabaseUrlPath($url, $params);
         }
 
         if (isset($url['query'])) {
@@ -278,4 +270,52 @@ final class DriverManager
 
         return $params;
     }
+
+    /**
+     * Parses the given URL and resolves the given connection parameters.
+     *
+     * @param array $url    The URL parts to evaluate.
+     * @param array $params The connection parameters to resolve.
+     *
+     * @return array The resolved connection parameters.
+     */
+    private static function parseDatabaseUrlPath(array $url, array $params)
+    {
+        if (!isset($url['scheme'])) {
+            $params['dbname'] = $url['path'];
+
+            return $params;
+        }
+
+        $url['path'] = substr($url['path'], 1);
+
+        if (strpos($url['scheme'], 'sqlite') !== false) {
+            return self::parseSqliteDatabaseUrlPath($url, $params);
+        }
+
+        $params['dbname'] = $url['path'];
+
+        return $params;
+    }
+
+    /**
+     * Parses the given SQLite URL and resolves the given connection parameters.
+     *
+     * @param array $url    The SQLite URL parts to evaluate.
+     * @param array $params The connection parameters to resolve.
+     *
+     * @return array The resolved connection parameters.
+     */
+    private static function parseSqliteDatabaseUrlPath(array $url, array $params)
+    {
+        if ($url['path'] === ':memory:') {
+            $params['memory'] = true;
+
+            return $params;
+        }
+
+        $params['path'] = $url['path']; // pdo_sqlite driver uses 'path' instead of 'dbname' key
+
+        return $params;
+    }
 }

tests/Doctrine/Tests/DBAL/DriverManagerTest.php

@@ -167,11 +167,11 @@ class DriverManagerTest extends \Doctrine\Tests\DbalTestCase
             ),
             'sqlite memory' => array(
                 'sqlite:///:memory:',
-                array('path' => ':memory:', 'driver' => 'Doctrine\DBAL\Driver\PDOSqlite\Driver'),
+                array('memory' => true, 'driver' => 'Doctrine\DBAL\Driver\PDOSqlite\Driver'),
             ),
             'sqlite memory with host' => array(
                 'sqlite://localhost/:memory:',
-                array('path' => ':memory:', 'driver' => 'Doctrine\DBAL\Driver\PDOSqlite\Driver'),
+                array('memory' => true, 'driver' => 'Doctrine\DBAL\Driver\PDOSqlite\Driver'),
             ),
             'params parsed from URL override individual params' => array(
                 array('url' => 'mysql://foo:bar@localhost/baz', 'password' => 'lulz'),