Merge pull request #225 from CraigMason/mssql-alter-primary-key

Fix SQLServerPlatform adds unique constraint to ADD PRIMARY KEY statement

lib/Doctrine/DBAL/Platforms/SQLServerPlatform.php

@@ -256,7 +256,7 @@ class SQLServerPlatform extends AbstractPlatform
     {
         $constraint = parent::getCreateIndexSQL($index, $table);
 
-        if ($index->isUnique()) {
+        if ($index->isUnique() && !$index->isPrimary()) {
             $constraint = $this->_appendUniqueConstraintDefinition($constraint, $index);
         }
 

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

@@ -221,4 +221,10 @@ class SQLServerPlatformTest extends AbstractPlatformTestCase
         $idx->addFlag('nonclustered');
         $this->assertEquals('ALTER TABLE tbl ADD PRIMARY KEY NONCLUSTERED (id)', $this->_platform->getCreatePrimaryKeySQL($idx, 'tbl'));
     }
+
+    public function testAlterAddPrimaryKey()
+    {
+        $idx = new \Doctrine\DBAL\Schema\Index('idx', array('id'), false, true);
+        $this->assertEquals('ALTER TABLE tbl ADD PRIMARY KEY (id)', $this->_platform->getCreateIndexSQL($idx, 'tbl'));
+    }
 }