pierre/wallabag
1
0
Fork 0
forked from wallabag/wallabag
Code Issues Pull Requests Packages Projects Releases 1 Wiki Activity

Added checks on migrations

Browse Source
This commit is contained in:
Nicolas Lœuillet
2016-11-25 17:43:28 +01:00
parent 986cb53645
commit 18d7bc3a35
6 changed files with 25 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
app/DoctrineMigrations/Version20161122203647.php
View File

@ -8,7 +8,7 @@ use Symfony\Component\DependencyInjection\ContainerAwareInterface;
use Symfony\Component\DependencyInjection\ContainerInterface;
/**
* Methods and properties removed from `FOS\UserBundle\Model\User`
* Methods and properties removed from `FOS\UserBundle\Model\User`.
*
* - `$expired`
* - `$credentialsExpired`
@ -32,7 +32,7 @@ class Version20161122203647 extends AbstractMigration implements ContainerAwareI
private function getTable($tableName)
{
return $this->container->getParameter('database_table_prefix') . $tableName;
return $this->container->getParameter('database_table_prefix').$tableName;
}
/**
@ -42,7 +42,12 @@ class Version20161122203647 extends AbstractMigration implements ContainerAwareI
{
$this->abortIf($this->connection->getDatabasePlatform()->getName() === 'sqlite', 'This up migration can\'t be executed on SQLite databases, because SQLite don\'t support DROP COLUMN.');
$this->skipIf(false === $schema->getTable($this->getTable('user'))->hasColumn('expired'), 'It seems that you already played this migration.');
$this->addSql('ALTER TABLE '.$this->getTable('user').' DROP expired');
$this->skipIf(false === $schema->getTable($this->getTable('user'))->hasColumn('credentials_expired'), 'It seems that you already played this migration.');
$this->addSql('ALTER TABLE '.$this->getTable('user').' DROP credentials_expired');
}