Skip to content

Commit

Permalink
feat: add commands for exporting current and expected database schema
Browse files Browse the repository at this point in the history
Signed-off-by: Robin Appelman <robin@icewind.nl>
  • Loading branch information
icewind1991 committed Jun 28, 2024
1 parent 8ec5360 commit 6c467c3
Show file tree
Hide file tree
Showing 8 changed files with 223 additions and 3 deletions.
68 changes: 68 additions & 0 deletions core/Command/Db/ExpectedSchema.php
Original file line number Diff line number Diff line change
@@ -0,0 +1,68 @@
<?php

declare(strict_types=1);
/**
* SPDX-FileCopyrightText: 2024 Robin Appelman <robin@icewind.nl>
* SPDX-License-Identifier: AGPL-3.0-or-later
*/

namespace OC\Core\Command\Db;

use Doctrine\DBAL\Schema\Schema;
use OC\Core\Command\Base;
use OC\DB\Connection;
use OC\DB\MigrationService;
use OC\DB\SchemaWrapper;
use OC\Migration\NullOutput;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Input\InputOption;
use Symfony\Component\Console\Output\OutputInterface;

class ExpectedSchema extends Base {
public function __construct(
protected Connection $connection,
) {
parent::__construct();
}

protected function configure(): void {
$this
->setName('db:schema:expected')
->setDescription('Export the expected database schema for a fresh installation')
->setHelp("Note that the expected schema might not exactly match the exported live schema as the expected schema doesn't take into account any database wide settings or defaults.")
->addOption('sql', null, InputOption::VALUE_NONE, 'Dump the SQL statements for creating the expected schema');
parent::configure();
}

protected function execute(InputInterface $input, OutputInterface $output): int {
$schema = new Schema();

$this->applyMigrations('core', $schema);

$apps = \OC_App::getEnabledApps();
foreach ($apps as $app) {
$this->applyMigrations($app, $schema);
}

$sql = $input->getOption('sql');
if ($sql) {
$output->writeln($schema->toSql($this->connection->getDatabasePlatform()));
} else {
$encoder = new SchemaEncoder();
$this->writeArrayInOutputFormat($input, $output, $encoder->encodeSchema($schema, $this->connection->getDatabasePlatform()));
}

return 0;
}

private function applyMigrations(string $app, Schema $schema): void {
$output = new NullOutput();
$ms = new MigrationService($app, $this->connection, $output);
foreach ($ms->getAvailableVersions() as $version) {
$migration = $ms->createInstance($version);
$migration->changeSchema($output, function() use (&$schema) {
return new SchemaWrapper($this->connection, $schema);
}, []);
}
}
}
44 changes: 44 additions & 0 deletions core/Command/Db/ExportSchema.php
Original file line number Diff line number Diff line change
@@ -0,0 +1,44 @@
<?php

declare(strict_types=1);
/**
* SPDX-FileCopyrightText: 2024 Robin Appelman <robin@icewind.nl>
* SPDX-License-Identifier: AGPL-3.0-or-later
*/

namespace OC\Core\Command\Db;

use OC\Core\Command\Base;
use OCP\IDBConnection;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Input\InputOption;
use Symfony\Component\Console\Output\OutputInterface;

class ExportSchema extends Base {
public function __construct(
protected IDBConnection $connection,
) {
parent::__construct();
}

protected function configure(): void {
$this
->setName('db:schema:export')
->setDescription('Export the current database schema')
->addOption('sql', null, InputOption::VALUE_NONE, 'Dump the SQL statements for creating a copy of the schema');
parent::configure();
}

protected function execute(InputInterface $input, OutputInterface $output): int {
$schema = $this->connection->createSchema();
$sql = $input->getOption('sql');
if ($sql) {
$output->writeln($schema->toSql($this->connection->getDatabasePlatform()));
} else {
$encoder = new SchemaEncoder();
$this->writeArrayInOutputFormat($input, $output, $encoder->encodeSchema($schema, $this->connection->getDatabasePlatform()));
}

return 0;
}
}
96 changes: 96 additions & 0 deletions core/Command/Db/SchemaEncoder.php
Original file line number Diff line number Diff line change
@@ -0,0 +1,96 @@
<?php

declare(strict_types=1);
/**
* SPDX-FileCopyrightText: 2024 Robin Appelman <robin@icewind.nl>
* SPDX-License-Identifier: AGPL-3.0-or-later
*/

namespace OC\Core\Command\Db;

use Doctrine\DBAL\Platforms\AbstractMySQLPlatform;
use Doctrine\DBAL\Platforms\AbstractPlatform;
use Doctrine\DBAL\Platforms\PostgreSQLPlatform;
use Doctrine\DBAL\Schema\Schema;
use Doctrine\DBAL\Schema\Table;
use Doctrine\DBAL\Types\PhpIntegerMappingType;

class SchemaEncoder {
/**
* Encode a DBAL schema to json, performing some normalization based on the database platform
*
* @param Schema $schema
* @param AbstractPlatform $platform
* @return array
*/
public function encodeSchema(Schema $schema, AbstractPlatform $platform): array {
$encoded = ['tables' => [], 'sequences' => []];
foreach ($schema->getTables() as $table) {
$encoded[$table->getName()] = $this->encodeTable($table, $platform);
}
ksort($encoded);
return $encoded;
}

private function encodeTable(Table $table, AbstractPlatform $platform): array {
$encoded = ['columns' => [], 'indexes' => []];
if ($table->getName() === 'oc_filecache_extended') {
$a = 1;
}
foreach ($table->getColumns() as $column) {
$data = $column->toArray();
$data['type'] = $column->getType()->getName();

Check notice

Code scanning / Psalm

DeprecatedMethod Note

The method Doctrine\DBAL\Types\Type::getName has been marked as deprecated
$data['default'] = $column->getType()->convertToPHPValue($column->getDefault(), $platform);
if ($platform instanceof PostgreSQLPlatform) {
$data['unsigned'] = false;
if ($column->getType() instanceof PhpIntegerMappingType) {
$data['length'] = null;
}
unset($data['jsonb']);
} elseif ($platform instanceof AbstractMySqlPlatform) {
if ($column->getType() instanceof PhpIntegerMappingType) {
$data['length'] = null;
} elseif (in_array($data['type'], ['text', 'blob', 'datetime', 'float', 'json'])) {
$data['length'] = 0;
}
unset($data['collation']);
unset($data['charset']);
}
if ($data['type'] === 'string' && $data['length'] === null) {

Check notice

Code scanning / Psalm

PossiblyUndefinedArrayOffset Note

Possibly undefined array key $data['length'] on array{default: mixed, length?: 0|mixed|null, type: string, unsigned?: false|mixed, ...<array-key, mixed>}
$data['length'] = 255;
}
$encoded['columns'][$column->getName()] = $data;
}
ksort($encoded['columns']);
foreach ($table->getIndexes() as $index) {
$options = $index->getOptions();
if (isset($options['lengths']) && count(array_filter($options['lengths'])) === 0) {
unset($options['lengths']);
}
if ($index->isPrimary()) {
if ($platform instanceof PostgreSqlPlatform) {
$name = $table->getName() . '_pkey';
} elseif ($platform instanceof AbstractMySQLPlatform) {
$name = "PRIMARY";
} else {
$name = $index->getName();
}
} else {
$name = $index->getName();
}
if ($platform instanceof PostgreSqlPlatform) {
$name = strtolower($name);
}
$encoded['indexes'][$name] = [
'name' => $name,
'columns' => $index->getColumns(),
'unique' => $index->isUnique(),
'primary' => $index->isPrimary(),
'flags' => $index->getFlags(),
'options' => $options,
];
}
ksort($encoded['indexes']);
return $encoded;
}
}
2 changes: 2 additions & 0 deletions core/register_command.php
Original file line number Diff line number Diff line change
Expand Up @@ -66,6 +66,8 @@
$application->add(Server::get(Command\Db\AddMissingColumns::class));
$application->add(Server::get(Command\Db\AddMissingIndices::class));
$application->add(Server::get(Command\Db\AddMissingPrimaryKeys::class));
$application->add(Server::get(Command\Db\ExpectedSchema::class));
$application->add(Server::get(Command\Db\ExportSchema::class));

if ($config->getSystemValueBool('debug', false)) {
$application->add(Server::get(Command\Db\Migrations\StatusCommand::class));
Expand Down
3 changes: 3 additions & 0 deletions lib/composer/composer/autoload_classmap.php
Original file line number Diff line number Diff line change
Expand Up @@ -1107,10 +1107,13 @@
'OC\\Core\\Command\\Db\\ConvertFilecacheBigInt' => $baseDir . '/core/Command/Db/ConvertFilecacheBigInt.php',
'OC\\Core\\Command\\Db\\ConvertMysqlToMB4' => $baseDir . '/core/Command/Db/ConvertMysqlToMB4.php',
'OC\\Core\\Command\\Db\\ConvertType' => $baseDir . '/core/Command/Db/ConvertType.php',
'OC\\Core\\Command\\Db\\ExpectedSchema' => $baseDir . '/core/Command/Db/ExpectedSchema.php',
'OC\\Core\\Command\\Db\\ExportSchema' => $baseDir . '/core/Command/Db/ExportSchema.php',
'OC\\Core\\Command\\Db\\Migrations\\ExecuteCommand' => $baseDir . '/core/Command/Db/Migrations/ExecuteCommand.php',
'OC\\Core\\Command\\Db\\Migrations\\GenerateCommand' => $baseDir . '/core/Command/Db/Migrations/GenerateCommand.php',
'OC\\Core\\Command\\Db\\Migrations\\MigrateCommand' => $baseDir . '/core/Command/Db/Migrations/MigrateCommand.php',
'OC\\Core\\Command\\Db\\Migrations\\StatusCommand' => $baseDir . '/core/Command/Db/Migrations/StatusCommand.php',
'OC\\Core\\Command\\Db\\SchemaEncoder' => $baseDir . '/core/Command/Db/SchemaEncoder.php',
'OC\\Core\\Command\\Encryption\\ChangeKeyStorageRoot' => $baseDir . '/core/Command/Encryption/ChangeKeyStorageRoot.php',
'OC\\Core\\Command\\Encryption\\DecryptAll' => $baseDir . '/core/Command/Encryption/DecryptAll.php',
'OC\\Core\\Command\\Encryption\\Disable' => $baseDir . '/core/Command/Encryption/Disable.php',
Expand Down
3 changes: 3 additions & 0 deletions lib/composer/composer/autoload_static.php
Original file line number Diff line number Diff line change
Expand Up @@ -1140,10 +1140,13 @@ class ComposerStaticInit749170dad3f5e7f9ca158f5a9f04f6a2
'OC\\Core\\Command\\Db\\ConvertFilecacheBigInt' => __DIR__ . '/../../..' . '/core/Command/Db/ConvertFilecacheBigInt.php',
'OC\\Core\\Command\\Db\\ConvertMysqlToMB4' => __DIR__ . '/../../..' . '/core/Command/Db/ConvertMysqlToMB4.php',
'OC\\Core\\Command\\Db\\ConvertType' => __DIR__ . '/../../..' . '/core/Command/Db/ConvertType.php',
'OC\\Core\\Command\\Db\\ExpectedSchema' => __DIR__ . '/../../..' . '/core/Command/Db/ExpectedSchema.php',
'OC\\Core\\Command\\Db\\ExportSchema' => __DIR__ . '/../../..' . '/core/Command/Db/ExportSchema.php',
'OC\\Core\\Command\\Db\\Migrations\\ExecuteCommand' => __DIR__ . '/../../..' . '/core/Command/Db/Migrations/ExecuteCommand.php',
'OC\\Core\\Command\\Db\\Migrations\\GenerateCommand' => __DIR__ . '/../../..' . '/core/Command/Db/Migrations/GenerateCommand.php',
'OC\\Core\\Command\\Db\\Migrations\\MigrateCommand' => __DIR__ . '/../../..' . '/core/Command/Db/Migrations/MigrateCommand.php',
'OC\\Core\\Command\\Db\\Migrations\\StatusCommand' => __DIR__ . '/../../..' . '/core/Command/Db/Migrations/StatusCommand.php',
'OC\\Core\\Command\\Db\\SchemaEncoder' => __DIR__ . '/../../..' . '/core/Command/Db/SchemaEncoder.php',
'OC\\Core\\Command\\Encryption\\ChangeKeyStorageRoot' => __DIR__ . '/../../..' . '/core/Command/Encryption/ChangeKeyStorageRoot.php',
'OC\\Core\\Command\\Encryption\\DecryptAll' => __DIR__ . '/../../..' . '/core/Command/Encryption/DecryptAll.php',
'OC\\Core\\Command\\Encryption\\Disable' => __DIR__ . '/../../..' . '/core/Command/Encryption/Disable.php',
Expand Down
2 changes: 1 addition & 1 deletion lib/private/DB/MigrationService.php
Original file line number Diff line number Diff line change
Expand Up @@ -459,7 +459,7 @@ public function describeMigrationStep($to = 'latest') {
* @return IMigrationStep
* @throws \InvalidArgumentException
*/
protected function createInstance($version) {
public function createInstance($version) {
$class = $this->getClass($version);
try {
$s = \OCP\Server::get($class);
Expand Down
8 changes: 6 additions & 2 deletions lib/private/DB/SchemaWrapper.php
Original file line number Diff line number Diff line change
Expand Up @@ -20,9 +20,13 @@ class SchemaWrapper implements ISchemaWrapper {
/** @var array */
protected $tablesToDelete = [];

public function __construct(Connection $connection) {
public function __construct(Connection $connection, ?Schema $schema = null) {
$this->connection = $connection;
$this->schema = $this->connection->createSchema();
if ($schema) {
$this->schema = $schema;
} else {
$this->schema = $this->connection->createSchema();
}
}

public function getWrappedSchema() {
Expand Down

0 comments on commit 6c467c3

Please sign in to comment.