Commit ea1d8d04 authored by Julien Fastré's avatar Julien Fastré
Browse files

add a default behaviour on adding scope column

To ensure backward-compatibility, add a default scope during schema modification, if reports are already present in database.
parent 03f1243c
......@@ -4,12 +4,33 @@ namespace Application\Migrations;
use Doctrine\DBAL\Migrations\AbstractMigration;
use Doctrine\DBAL\Schema\Schema;
use Symfony\Component\DependencyInjection\ContainerAwareInterface;
use Symfony\Component\DependencyInjection\ContainerInterface;
use Chill\MainBundle\Entity\Scope;
* Add a scope to report
class Version20150622233319 extends AbstractMigration
class Version20150622233319 extends AbstractMigration
implements ContainerAwareInterface
* @var ContainerInterface
private $container;
public function setContainer(ContainerInterface $container = null)
if ($container === NULL) {
throw new \RuntimeException('Container is not provided. This migration '
. 'need container to set a default center');
$this->container = $container;
* @param Schema $schema
......@@ -19,10 +40,52 @@ class Version20150622233319 extends AbstractMigration
'Migration can only be executed safely on \'postgresql\'.');
$this->addSql('ALTER TABLE report ADD scope_id INT DEFAULT NULL');
//add a default scope
$scopes = $this->container->get('doctrine.orm.default_entity_manager')
if (count($scopes) > 0) {
$defaultScopeId = $scopes[0]->getId();
} else {
//check if there are data in report table
$nbReports = $this->container->get('doctrine.orm.default_entity_manager')
->createQuery('SELECT count( FROM ChillReportBundle:Report r')
if ($nbReports > 0) {
//create a default scope
$scope = new Scope();
//create name according to installed languages
$locales = $this->container
$names = array();
foreach($locales as $locale) {
$names[$locale] = 'default';
//default scope is the newly-created one
$defaultScopeId = $scope->getId();
$this->addSql('ALTER TABLE report DROP scope'); //before this migration, scope was never used
$this->addSql('ALTER TABLE report ADD CONSTRAINT FK_report_scope '
. 'FOREIGN KEY (scope_id) '
if (isset($defaultScopeId)){
$this->addSql('UPDATE report SET scope_id = :id', array(
'id' => $defaultScopeId
$this->addSql('ALTER TABLE report ALTER COLUMN scope_id SET NOT NULL');
$this->addSql('CREATE INDEX IDX_report_scope ON report (scope_id)');
......@@ -34,8 +97,8 @@ class Version20150622233319 extends AbstractMigration
$this->abortIf($this->connection->getDatabasePlatform()->getName() != 'postgresql',
'Migration can only be executed safely on \'postgresql\'.');
$this->addSql('ALTER TABLE Report DROP CONSTRAINT FK_C38372B2682B5931');
$this->addSql('DROP INDEX IDX_C38372B2682B5931');
$this->addSql('ALTER TABLE Report DROP CONSTRAINT FK_report_scope');
$this->addSql('DROP INDEX IDX_report_scope');
$this->addSql('ALTER TABLE Report ADD scope VARCHAR(255) DEFAULT NULL');
$this->addSql('ALTER TABLE Report DROP scope_id');
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment