drop obsolete DB conversion scripts, fixes #3335

Closes #3335

Merge request studip/studip!2268

3 files changed, 0 insertions, 278 deletions

diff --git a/cli/Commands/DB/MigrateEngine.php b/cli/Commands/DB/MigrateEngine.php
deleted file mode 100644
index 6ae5219..0000000
--- a/cli/Commands/DB/MigrateEngine.php
+++ /dev/null
@@ -1,148 +0,0 @@
-<?php
-
-namespace Studip\Cli\Commands\DB;
-
-use DBManager;
-use StudipPDO;
-use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Output\OutputInterface;
-use Symfony\Component\Console\Style\SymfonyStyle;
-
-class MigrateEngine extends Command
-{
-    protected static $defaultName = 'db:migrate-engine';
-
-    protected function configure(): void
-    {
-        $this->setDescription('MyISAM to InnoDB');
-        $this->setHelp('Migrate the Engine from MyISAM to InnoDB.');
-    }
-
-    protected function execute(InputInterface $input, OutputInterface $output): int
-    {
-        $io = new SymfonyStyle($input, $output);
-        $io->info('Migration starting at ' . date('d.m.Y H:i:s'));
-        $start = microtime(true);
-        // Check if InnoDB is enabled in database server.
-        $engines = DBManager::get()->fetchAll('SHOW ENGINES');
-        $innodb = false;
-        foreach ($engines as $e) {
-            // InnoDB is found and enabled.
-            if ($e['Engine'] == 'InnoDB' && in_array(mb_strtolower($e['Support']), ['default', 'yes'])) {
-                $innodb = true;
-                break;
-            }
-        }
-        if ($innodb) {
-            // Get version of database system (MySQL/MariaDB/Percona)
-            $data = DBManager::get()->fetchFirst('SELECT VERSION() AS version');
-            $version = $data[0];
-
-            // Tables to ignore on engine conversion.
-            $ignore_tables = [];
-            // Fetch all tables that need to be converted.
-            $tables = DBManager::get()->fetchFirst(
-                "SELECT TABLE_NAME
-        FROM `information_schema`.TABLES
-        WHERE TABLE_SCHEMA=:database AND ENGINE=:oldengine
-        ORDER BY TABLE_NAME",
-                [
-                    ':database' => $GLOBALS['DB_STUDIP_DATABASE'],
-                    ':oldengine' => 'MyISAM',
-                ]
-            );
-
-            /*
-             * lit_catalog needs fulltext indices which InnoDB doesn't support
-             * in older versions.
-             */
-            if (version_compare($version, '5.6', '<')) {
-                $stmt_fulltext = DBManager::get()->prepare(
-                    "SHOW INDEX FROM :database.:table WHERE Index_type = 'FULLTEXT'"
-                );
-                foreach ($tables as $k => $t) {
-                    $stmt_fulltext->bindParam(':table', $t, StudipPDO::PARAM_COLUMN);
-                    $stmt_fulltext->bindParam(':database', $DB_STUDIP_DATABASE, StudipPDO::PARAM_COLUMN);
-                    $stmt_fulltext->execute();
-                    if ($stmt_fulltext->fetch()) {
-                        $ignore_tables[] = $t;
-                        unset($tables[$k]);
-                    }
-                }
-                if (count($ignore_tables)) {
-                    $io->info(
-                        'The following tables needs fulltext indices ' .
-                            'which are not supported for InnoDB in your database ' .
-                            'version, so the tables will be left untouched: ' .
-                            join(',', $ignore_tables)
-                    );
-                }
-            }
-
-            // Use Barracuda format if database supports it (5.5 upwards).
-            if (version_compare($version, '5.5', '>=')) {
-                $io->info('Found MySQL in version >= 5.5, checking if Barracuda file format is supported...');
-                // Get innodb_file_per_table setting
-                $data = DBManager::get()->fetchOne("SHOW VARIABLES LIKE 'innodb_file_per_table'");
-
-                $file_per_table = $data['Value'];
-
-                // Check if Barracuda file format is enabled
-                $data = DBManager::get()->fetchOne("SHOW VARIABLES LIKE 'innodb_file_format'");
-                $file_format = $data['Value'];
-
-                if (mb_strtolower($file_per_table) == 'on' && mb_strtolower($file_format) == 'barracuda') {
-                    $rowformat = 'DYNAMIC';
-                } else {
-                    if (mb_strtolower($file_per_table) != 'on') {
-                        $io->info('file_per_table not set');
-                    }
-                    if (mb_strtolower($file_format) != 'barracuda') {
-                        $io->info('file_format not set to Barracuda (but to ' . $file_format . ')');
-                    }
-                    $rowformat = 'COMPACT';
-                }
-            }
-
-            // Prepare query for table conversion.
-            $stmt = DBManager::get()->prepare('ALTER TABLE :database.:table ROW_FORMAT=:rowformat ENGINE=:newengine');
-            $stmt->bindParam(':database', $DB_STUDIP_DATABASE, StudipPDO::PARAM_COLUMN);
-            $stmt->bindParam(':rowformat', $rowformat, StudipPDO::PARAM_COLUMN);
-            $newengine = 'InnoDB';
-            $stmt->bindParam(':newengine', $newengine, StudipPDO::PARAM_COLUMN);
-
-            // Now convert the found tables.
-            foreach ($tables as $t) {
-                $local_start = microtime(true);
-                $stmt->bindParam(':table', $t, StudipPDO::PARAM_COLUMN);
-                $stmt->execute();
-                $local_end = microtime(true);
-                $local_duration = $local_end - $local_start;
-                $human_local_duration = sprintf(
-                    '%02d:%02d:%02d',
-                    ($local_duration / 60 / 60) % 24,
-                    ($local_duration / 60) % 60,
-                    $local_duration % 60
-                );
-                $io->info('Conversion of table ' . $t . ' took ' . $human_local_duration);
-            }
-
-            $end = microtime(true);
-
-            $duration = $end - $start;
-            $human_duration = sprintf(
-                '%02d:%02d:%02d',
-                ($duration / 60 / 60) % 24,
-                ($duration / 60) % 60,
-                $duration % 60
-            );
-
-            $io->info('Migration finished at ' . date('d.m.Y H:i:s') . ', duration ' . $human_duration);
-        } else {
-            $io->error(
-                'The storage engine InnoDB is not enabled in your database installation, tables cannot be converted.'
-            );
-        }
-    }
-}
diff --git a/cli/Commands/DB/MigrateFileFormat.php b/cli/Commands/DB/MigrateFileFormat.php
deleted file mode 100644
index 0c97824..0000000
--- a/cli/Commands/DB/MigrateFileFormat.php
+++ /dev/null
@@ -1,128 +0,0 @@
-<?php
-
-namespace Studip\Cli\Commands\DB;
-
-use DBManager;
-use StudipPDO;
-use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Output\OutputInterface;
-use Symfony\Component\Console\Style\SymfonyStyle;
-
-class MigrateFileFormat extends Command
-{
-    protected static $defaultName = 'db:migrate-file-format';
-
-    protected function configure(): void
-    {
-        $this->setDescription('Antelope to Barracuda');
-        $this->setHelp('Migrate the FileFormat from Antelope to Barracuda.');
-    }
-
-    protected function execute(InputInterface $input, OutputInterface $output): int
-    {
-        $io = new SymfonyStyle($input, $output);
-        $io->info('Migration starting at ' . date('d.m.Y H:i:s'));
-        $start = microtime(true);
-        // Check if InnoDB is enabled in database server.
-        $engines = DBManager::get()->fetchAll('SHOW ENGINES');
-        $innodb = false;
-        foreach ($engines as $e) {
-            // InnoDB is found and enabled.
-            if ($e['Engine'] == 'InnoDB' && in_array(mb_strtolower($e['Support']), ['default', 'yes'])) {
-                $innodb = true;
-                break;
-            }
-        }
-        if ($innodb) {
-            // Get version of database system (MySQL/MariaDB/Percona)
-            $data = DBManager::get()->fetchFirst('SELECT VERSION() AS version');
-            $version = $data[0];
-
-            // Use Barracuda format if database supports it (5.5 upwards).
-            if (version_compare($version, '5.5', '>=')) {
-                $io->info('Checking if Barracuda file format is supported...');
-                // Get innodb_file_per_table setting
-                $data = DBManager::get()->fetchOne("SHOW VARIABLES LIKE 'innodb_file_per_table'");
-                $file_per_table = $data['Value'];
-
-                // Check if Barracuda file format is enabled
-                $data = DBManager::get()->fetchOne("SHOW VARIABLES LIKE 'innodb_file_format'");
-                $file_format = $data['Value'];
-                if (mb_strtolower($file_per_table) == 'on' && mb_strtolower($file_format) == 'barracuda') {
-                    // Fetch all tables that need to be converted.
-                    $tables = DBManager::get()->fetchFirst(
-                        "SELECT TABLE_NAME
-                FROM `information_schema`.TABLES
-                WHERE TABLE_SCHEMA=:database AND ENGINE=:engine
-                    AND ROW_FORMAT IN (:rowformats)
-                ORDER BY TABLE_NAME",
-                        [
-                            ':database' => $GLOBALS['DB_STUDIP_DATABASE'],
-                            ':engine' => 'InnoDB',
-                            ':rowformats' => ['Compact', 'Redundant'],
-                        ]
-                    );
-
-                    $newformat = 'DYNAMIC';
-
-                    // Prepare query for table conversion.
-                    $stmt = DBManager::get()->prepare('ALTER TABLE :database.:table ROW_FORMAT=:newformat');
-                    $stmt->bindParam(':database', $DB_STUDIP_DATABASE, StudipPDO::PARAM_COLUMN);
-                    $stmt->bindParam(':newformat', $newformat, StudipPDO::PARAM_COLUMN);
-
-                    if (count($tables) > 0) {
-                        // Now convert the found tables.
-                        foreach ($tables as $t) {
-                            $local_start = microtime(true);
-                            $stmt->bindParam(':table', $t, StudipPDO::PARAM_COLUMN);
-                            $stmt->execute();
-                            $local_end = microtime(true);
-                            $local_duration = $local_end - $local_start;
-                            $human_local_duration = sprintf(
-                                '%02d:%02d:%02d',
-                                ($local_duration / 60 / 60) % 24,
-                                ($local_duration / 60) % 60,
-                                $local_duration % 60
-                            );
-                            $io->info('Converserion of table' . $t . ' took ' . $human_local_duration);
-                        }
-                    } else {
-                        $io->error('No Antelope format tables found');
-                        return Command::FAILURE;
-                    }
-                } else {
-                    if (mb_strtolower($file_per_table) != 'on') {
-                        $io->error('file_per_table not set');
-                        return Command::FAILURE;
-                    }
-                    if (mb_strtolower($file_format) != 'barracuda') {
-                        $io->error('file_format not set to Barracuda (but to ' . $file_format . ')');
-                        return Command::FAILURE;
-                    }
-                }
-                $end = microtime(true);
-                $duration = $end - $start;
-                $human_duration = sprintf(
-                    '%02d:%02d:%02d',
-                    ($duration / 60 / 60) % 24,
-                    ($duration / 60) % 60,
-                    $duration % 60
-                );
-
-                $io->info('Migration finished at ' . date('d.m.Y H:i:s') . ', duration ' . $human_duration);
-                return Command::SUCCESS;
-            } else {
-                $io->error(
-                    'Your database server does not yet support the Barracuda row format (you need at least 5.5)'
-                );
-                return Command::FAILURE;
-            }
-        } else {
-            $io->error(
-                'The storage engine InnoDB is not enabled in your database installation, tables cannot be converted.'
-            );
-            return Command::INVALID;
-        }
-    }
-}
diff --git a/cli/studip b/cli/studip
index ced9f0d..bef389c 100755
--- a/cli/studip
+++ b/cli/studip
@@ -31,8 +31,6 @@ $commands = [
     Commands\Cronjobs\CronjobWorker::class,
     Commands\Cronjobs\CronjobWorker::class,
     Commands\DB\Dump::class,
-    Commands\DB\MigrateEngine::class,
-    Commands\DB\MigrateFileFormat::class,
     Commands\Files\Dump::class,
     Commands\Fix\Biest7789::class,
     Commands\Fix\Biest7866::class,