Merge pull request #3211 from morozov/issues/3167

Removed support for DB-generated UUIDs
doctrine · Aug 7, 2018 · 89fbe15 · 89fbe15
2 parents 5465c6c + b269bc5
commit 89fbe15
Show file tree

Hide file tree

Showing 10 changed files with 5 additions and 130 deletions.
diff --git a/UPGRADE.md b/UPGRADE.md
@@ -1,5 +1,10 @@
 # Upgrade to 3.0
 
+## BC BREAK: Removed support for DB-generated UUIDs
+
+The support for DB-generated UUIDs was removed as non-portable.
+Please generate UUIDs on the application side (e.g. using [ramsey/uuid](https://packagist.org/packages/ramsey/uuid)).
+
 ## BC BREAK: Removed MsSQLKeywords class
 
 The `Doctrine\DBAL\Platforms\MsSQLKeywords` has been removed.

diff --git a/lib/Doctrine/DBAL/Platforms/AbstractPlatform.php b/lib/Doctrine/DBAL/Platforms/AbstractPlatform.php
@@ -649,20 +649,6 @@ public function getRegexpExpression()
         throw DBALException::notSupported(__METHOD__);
     }
 
-    /**
-     * Returns the global unique identifier expression.
-     *
-     * @return string
-     *
-     * @throws \Doctrine\DBAL\DBALException If not supported on this platform.
-     *
-     * @deprecated Use application-generated UUIDs instead
-     */
-    public function getGuidExpression()
-    {
-        throw DBALException::notSupported(__METHOD__);
-    }
-
     /**
      * Returns the SQL snippet to get the average value of a column.
      *

diff --git a/lib/Doctrine/DBAL/Platforms/MySqlPlatform.php b/lib/Doctrine/DBAL/Platforms/MySqlPlatform.php
@@ -80,16 +80,6 @@ public function getRegexpExpression()
         return 'RLIKE';
     }
 
-    /**
-     * {@inheritDoc}
-     *
-     * @deprecated Use application-generated UUIDs instead
-     */
-    public function getGuidExpression()
-    {
-        return 'UUID()';
-    }
-
     /**
      * {@inheritDoc}
      */

diff --git a/lib/Doctrine/DBAL/Platforms/OraclePlatform.php b/lib/Doctrine/DBAL/Platforms/OraclePlatform.php
@@ -89,16 +89,6 @@ public function getLocateExpression($str, $substr, $startPos = false)
         return 'INSTR('.$str.', '.$substr.', '.$startPos.')';
     }
 
-    /**
-     * {@inheritDoc}
-     *
-     * @deprecated Use application-generated UUIDs instead
-     */
-    public function getGuidExpression()
-    {
-        return 'SYS_GUID()';
-    }
-
     /**
      * {@inheritdoc}
      */

diff --git a/lib/Doctrine/DBAL/Platforms/PostgreSqlPlatform.php b/lib/Doctrine/DBAL/Platforms/PostgreSqlPlatform.php
@@ -1002,16 +1002,6 @@ public function getTimeTypeDeclarationSQL(array $fieldDeclaration)
         return 'TIME(0) WITHOUT TIME ZONE';
     }
 
-    /**
-     * {@inheritDoc}
-     *
-     * @deprecated Use application-generated UUIDs instead
-     */
-    public function getGuidExpression()
-    {
-        return 'UUID_GENERATE_V4()';
-    }
-
     /**
      * {@inheritDoc}
      */

diff --git a/lib/Doctrine/DBAL/Platforms/SQLAnywherePlatform.php b/lib/Doctrine/DBAL/Platforms/SQLAnywherePlatform.php
@@ -673,16 +673,6 @@ public function getForUpdateSQL()
         return '';
     }
 
-    /**
-     * {@inheritdoc}
-     *
-     * @deprecated Use application-generated UUIDs instead
-     */
-    public function getGuidExpression()
-    {
-        return 'NEWID()';
-    }
-
     /**
      * {@inheritdoc}
      */

diff --git a/lib/Doctrine/DBAL/Platforms/SQLServerPlatform.php b/lib/Doctrine/DBAL/Platforms/SQLServerPlatform.php
@@ -994,16 +994,6 @@ public function getDropViewSQL($name)
         return 'DROP VIEW ' . $name;
     }
 
-    /**
-     * {@inheritDoc}
-     *
-     * @deprecated Use application-generated UUIDs instead
-     */
-    public function getGuidExpression()
-    {
-        return 'NEWID()';
-    }
-
     /**
      * {@inheritDoc}
      */

diff --git a/lib/Doctrine/DBAL/Platforms/SqlitePlatform.php b/lib/Doctrine/DBAL/Platforms/SqlitePlatform.php
@@ -44,19 +44,6 @@ public function getRegexpExpression()
         return 'REGEXP';
     }
 
-    /**
-     * {@inheritDoc}
-     *
-     * @deprecated Use application-generated UUIDs instead
-     */
-    public function getGuidExpression()
-    {
-        return "HEX(RANDOMBLOB(4)) || '-' || HEX(RANDOMBLOB(2)) || '-4' || "
-            . "SUBSTR(HEX(RANDOMBLOB(2)), 2) || '-' || "
-            . "SUBSTR('89AB', 1 + (ABS(RANDOM()) % 4), 1) || "
-            . "SUBSTR(HEX(RANDOMBLOB(2)), 2) || '-' || HEX(RANDOMBLOB(6))";
-    }
-
     /**
      * {@inheritDoc}
      */

diff --git a/tests/Doctrine/Tests/DBAL/Functional/Ticket/DBAL421Test.php b/tests/Doctrine/Tests/DBAL/Functional/Ticket/DBAL421Test.php
diff --git a/tests/Doctrine/Tests/DBAL/Platforms/SQLAnywherePlatformTest.php b/tests/Doctrine/Tests/DBAL/Platforms/SQLAnywherePlatformTest.php
@@ -649,7 +649,6 @@ public function testGeneratesSQLSnippets()
         self::assertEquals("Y-m-d H:i:s.u", $this->_platform->getDateTimeFormatString());
         self::assertEquals("H:i:s.u", $this->_platform->getTimeFormatString());
         self::assertEquals('', $this->_platform->getForUpdateSQL());
-        self::assertEquals('NEWID()', $this->_platform->getGuidExpression());
         self::assertEquals('LOCATE(string_column, substring_column)', $this->_platform->getLocateExpression('string_column', 'substring_column'));
         self::assertEquals('LOCATE(string_column, substring_column, 1)', $this->_platform->getLocateExpression('string_column', 'substring_column', 1));
         self::assertEquals("HASH(column, 'MD5')", $this->_platform->getMd5Expression('column'));