Drupal
/
drupal
mirror of https://git.drupalcode.org/project/drupal.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Issue #3107472 by ravi.shankar, alexpott, kishor_kolekar, jibran: DbDumpCommand should not hardcode the version string

merge-requests/64/head
xjm 2020-05-05 08:59:34 -05:00
parent 8433377461
commit f75a4b42cb
4 changed files with 7 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
core/lib/Drupal/Core/Command/DbDumpCommand.php
View File

@ -93,6 +93,8 @@ class DbDumpCommand extends DbCommandBase {
$tables .= $this->getTableScript($table, $schema, $data);
}
$script = $this->getTemplate();
// Substitute in the version.
$script = str_replace('{{VERSION}}', \Drupal::VERSION, $script);
// Substitute in the tables.
$script = str_replace('{{TABLES}}', trim($tables), $script);
return trim($script);
@ -392,7 +394,7 @@ class DbDumpCommand extends DbCommandBase {
* @file
* A database agnostic dump for testing purposes.
*
* This file was generated by the Drupal 8.0 db-tools.php script.
* This file was generated by the Drupal {{VERSION}} db-tools.php script.
*/
use Drupal\Core\Database\Database;

2
core/lib/Drupal/Core/Command/DbToolsApplication.php
View File

@ -13,7 +13,7 @@ class DbToolsApplication extends Application {
* {@inheritdoc}
*/
public function __construct() {
parent::__construct('Database Tools', '8.0.x');
parent::__construct('Database Tools', \Drupal::VERSION);
}
/**

2
core/modules/system/tests/src/Kernel/Scripts/DbDumpCommandTest.php
View File

@ -54,6 +54,8 @@ class DbDumpCommandTest extends KernelTestBase {
$this->assertContains("'path' => 'test", $output, 'Insert path field found');
$this->assertContains("'pattern_outline' => 'test", $output, 'Insert pattern_outline field found');
$this->assertContains("// @codingStandardsIgnoreFile", $output);
$version = \Drupal::VERSION;
$this->assertContains("This file was generated by the Drupal {$version} db-tools.php script.", $output);
}
/**

1
core/modules/system/tests/src/Kernel/Scripts/DbToolsApplicationTest.php
View File

@ -22,6 +22,7 @@ class DbToolsApplicationTest extends KernelTestBase {
$application = new DbToolsApplication();
$command = $application->find('dump');
$this->assertInstanceOf('\Drupal\Core\Command\DbDumpCommand', $command);
$this->assertSame(\Drupal::VERSION, $application->getVersion());
}
/**