feat(ModuleInstaller): Add method to rename tables in the DB

setup/moduleinstaller.class.inc.php

@@ -259,7 +259,7 @@ public static function MoveColumnInDB($sOrigTable, $sOrigColumn, $sDstTable, $sD
 			return;
 		}
 
-		if (!CMDBSource::IsTable($sOrigTable) || !CMDBSource::IsField($sOrigTable, $sOrigColumn))
+		if (!CMDBSource::IsField($sOrigTable, $sOrigColumn))
 		{
 			// Original field is not present
 			return;
@@ -291,4 +291,47 @@ public static function MoveColumnInDB($sOrigTable, $sOrigColumn, $sDstTable, $sD
 		CMDBSource::CacheReset($sDstTable);
 	}
 
+	/**
+	 * Rename a table providing:
+	 * - The original name exists
+	 * - The destination name does not exist
+	 *
+	 * @param string $sOrigTable
+	 * @param string $sDstTable
+	 *
+	 * @return void
+	 * @throws CoreException
+	 * @throws CoreUnexpectedValue
+	 * @throws MySQLException
+	 */
+	public static function RenameTableInDB(string $sOrigTable, string $sDstTable)
+	{
+		if ($sOrigTable == $sDstTable)
+		{
+			throw new CoreUnexpectedValue("Origin table and destination table are the same");
+		}
+
+		if (!MetaModel::DBExists(false))
+		{
+			// Install from scratch, no migration
+			return;
+		}
+
+		if (!CMDBSource::IsTable($sOrigTable))
+		{
+			// Origin table is not present
+			return;
+		}
+
+		if (CMDBSource::IsTable($sDstTable))
+		{
+			// Destination table is already present
+			return;
+		}
+
+		$sQueryRename = sprintf(/** @lang MariaDB */ "RENAME TABLE `%s` TO `%s`;", $sOrigTable, $sDstTable);
+		CMDBSource::Query($sQueryRename);
+
+		CMDBSource::CacheReset($sOrigTable);
+	}
 }

tests/php-unit-tests/unitary-tests/setup/ModuleInstallerAPITest.php

@@ -241,4 +241,30 @@ public function testMoveColumnInDB_MoveMultipleTable(): void
 		$this->assertEquals('from table 2', $sFromTable2Data, "Data was not moved as expected");
 	}
 
+	/**
+	 * Test that the table has been renamed
+	 *
+	 * @covers ModuleInstallerAPI::RenameTableInDB
+	 *
+	 * @return void
+	 * @throws \CoreException
+	 * @throws \MySQLException
+	 */
+	public function testRenameTableInDB()
+	{
+		$sOrigTable = MetaModel::DBGetTable('Person');
+		$aOrigTableInfo = CMDBSource::GetTableInfo($sOrigTable);
+		$this->assertNotEmpty($aOrigTableInfo, 'Origin table does not exist');
+
+		$sDstTable = static::$sWorkTable;
+		$this->assertFalse(CMDBSource::IsTable($sDstTable), 'Work table already exists');
+
+		ModuleInstallerAPI::RenameTableInDB($sOrigTable, $sDstTable);
+
+		$this->assertEquals($aOrigTableInfo, CMDBSource::GetTableInfo($sDstTable), 'Table was not renamed');
+
+		// Revert
+		ModuleInstallerAPI::RenameTableInDB($sDstTable, $sOrigTable);
+		$this->assertEquals($aOrigTableInfo, CMDBSource::GetTableInfo($sOrigTable));
+	}
 }