Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed cs migration template #1943

Merged
merged 3 commits into from
Jan 19, 2023
Merged

Fixed cs migration template #1943

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
e7339b8
Adjusted template
michbeck Jan 18, 2023
356b07c
Adjusted return types
michbeck Jan 18, 2023
ae3350e
Merge branch 'master' into bugfix/fix-cs-migration-template
michbeck Jan 19, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
27 changes: 25 additions & 2 deletions templates/Manager/migration_template.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,23 +9,46 @@
*/
class <?= $migrationClassName ?>
{
/**
* @var string
*/
public $comment = '<?= $commentString ?>';

/**
* @param \Propel\Generator\Manager\MigrationManager $manager
*
* @return null|false|void
*/
public function preUp(MigrationManager $manager)
{
// add the pre-migration code here
}

/**
* @param \Propel\Generator\Manager\MigrationManager $manager
*
* @return null|false|void
*/
public function postUp(MigrationManager $manager)
{
// add the post-migration code here
}

/**
* @param \Propel\Generator\Manager\MigrationManager $manager
*
* @return null|false|void
*/
public function preDown(MigrationManager $manager)
{
// add the pre-migration code here
}

/**
* @param \Propel\Generator\Manager\MigrationManager $manager
*
* @return null|false|void
*/
public function postDown(MigrationManager $manager)
{
// add the post-migration code here
Expand All @@ -37,7 +60,7 @@ public function postDown(MigrationManager $manager)
* @return array list of the SQL strings to execute for the Up migration
* the keys being the datasources
*/
public function getUpSQL()
public function getUpSQL(): array
{
<?php foreach($migrationsUp as $connectionName => $sql): ?>
<?= $connectionToVariableName[$connectionName] ?> = <<< 'EOT'
Expand All @@ -58,7 +81,7 @@ public function getUpSQL()
* @return array list of the SQL strings to execute for the Down migration
* the keys being the datasources
*/
public function getDownSQL()
public function getDownSQL(): array
{
<?php foreach($migrationsDown as $connectionName => $sql): ?>
<?= $connectionToVariableName[$connectionName] ?> = <<< 'EOT'
Expand Down