Merge pull request #528 from doctrine/hotfix/sqlite-auto-increment

Applying patch suggested by @guilhermeblanco for SQLite's auto-inc integer PKs

lib/Doctrine/DBAL/Platforms/SqlitePlatform.php

@@ -289,6 +289,11 @@ class SqlitePlatform extends AbstractPlatform
      */
     protected function _getCommonIntegerTypeDeclarationSQL(array $columnDef)
     {
+        // sqlite autoincrement is implicit for integer PKs, but not when the field is unsigned
+        if ( ! empty($columnDef['autoincrement'])) {
+            return '';
+        }
+
         return ! empty($columnDef['unsigned']) ? ' UNSIGNED' : '';
     }
 

tests/Doctrine/Tests/DBAL/Platforms/SqlitePlatformTest.php

@@ -63,6 +63,14 @@ class SqlitePlatformTest extends AbstractPlatformTestCase
         $this->assertTrue($this->_platform->prefersIdentityColumns());
     }
 
+    public function testIgnoresUnsignedIntegerDeclarationForAutoIncrementalIntegers()
+    {
+        $this->assertSame(
+            'INTEGER',
+            $this->_platform->getIntegerTypeDeclarationSQL(array('autoincrement' => true, 'unsigned' => true))
+        );
+    }
+
     /**
      * @group DBAL-752
      */