Fix altering sqlite table with indexes or triggers

Signed-off-by: Matthew Peveler <[email protected]>

Author: MasterOdin

src/Phinx/Db/Adapter/SQLiteAdapter.php

@@ -812,13 +812,24 @@ protected function copyAndDropTmpTable($instructions, $tableName)
                 $state['selectColumns']
             );
 
+            $rows = $this->fetchAll(
+                sprintf(
+                    "SELECT * FROM sqlite_master WHERE `type` = 'index' OR `type` = 'trigger' AND tbl_name = %s",
+                    $this->quoteValue($tableName)
+                )
+            );
+
             $this->execute(sprintf('DROP TABLE %s', $this->quoteTableName($tableName)));
             $this->execute(sprintf(
                 'ALTER TABLE %s RENAME TO %s',
                 $this->quoteTableName($state['tmpTableName']),
                 $this->quoteTableName($tableName)
             ));
 
+            foreach ($rows as $row) {
+                $this->execute($row['sql']);
+            }
+
             return $state;
         });
 

tests/Phinx/Db/Adapter/SQLiteAdapterTest.php

@@ -600,6 +600,24 @@ public function testChangeColumnWithForeignKey()
         $this->assertTrue($this->adapter->hasForeignKey($table->getName(), ['ref_table_id']));
     }
 
+    public function testChangeColumnWithIndex()
+    {
+        $table = new \Phinx\Db\Table('t', [], $this->adapter);
+        $table
+            ->addColumn('indexcol', 'integer')
+            ->addIndex(
+                'indexcol',
+                ['unique' => true]
+            )
+            ->create();
+
+        $this->assertTrue($this->adapter->hasIndex($table->getName(), 'indexcol'));
+
+        $table->changeColumn('indexcol', 'integer', ['null' => false])->update();
+
+        $this->assertTrue($this->adapter->hasIndex($table->getName(), 'indexcol'));
+    }
+
     public function testChangeColumnDefaultToZero()
     {
         $table = new \Phinx\Db\Table('t', [], $this->adapter);