Passed
Push — master ( 221dc3...6b1522 )
by Julito
15:24
created

Version20170625122900::up()   D

Complexity

Conditions 10
Paths 512

Size

Total Lines 63
Code Lines 38

Duplication

Lines 0
Ratio 0 %

Importance

Changes 1
Bugs 1 Features 0
Metric Value
cc 10
eloc 38
nc 512
nop 1
dl 0
loc 63
rs 4.1777
c 1
b 1
f 0

How to fix   Long Method    Complexity   

Long Method

Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.

For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.

Commonly applied refactorings include:

1
<?php
2
3
/* For licensing terms, see /license.txt */
4
5
namespace Chamilo\CoreBundle\Migrations\Schema\V200;
6
7
use Chamilo\CoreBundle\Migrations\AbstractMigrationChamilo;
8
use Chamilo\CoreBundle\ToolChain;
9
use Doctrine\DBAL\Schema\Schema;
10
11
/**
12
 * c_document.
13
 */
14
class Version20170625122900 extends AbstractMigrationChamilo
15
{
16
    public function up(Schema $schema): void
17
    {
18
        // Install tools.
19
        $em = $this->getEntityManager();
20
        $container = $this->getContainer();
21
        $toolChain = $container->get(ToolChain::class);
22
        $toolChain->createTools($em);
23
24
        $table = $schema->getTable('c_document');
25
        if (false === $table->hasColumn('resource_node_id')) {
26
            $this->addSql('ALTER TABLE c_document ADD resource_node_id INT DEFAULT NULL');
27
            $this->addSql(
28
                'ALTER TABLE c_document ADD CONSTRAINT FK_C9FA0CBD1BAD783F FOREIGN KEY (resource_node_id) REFERENCES resource_node (id) ON DELETE CASCADE;'
29
            );
30
            $this->addSql('CREATE UNIQUE INDEX UNIQ_C9FA0CBD1BAD783F ON c_document (resource_node_id);');
31
        }
32
33
        if (false === $table->hasColumn('template')) {
34
            $this->addSql('ALTER TABLE c_document ADD template TINYINT(1) NOT NULL');
35
        }
36
37
        if ($table->hasColumn('id')) {
38
            $this->addSql('ALTER TABLE c_document DROP id');
39
            //$this->addSql('ALTER TABLE c_document DROP id, DROP c_id, DROP path, DROP size, DROP session_id');
40
        }
41
42
        if (false === $table->hasIndex('idx_cdoc_type')) {
43
            $this->addSql('CREATE INDEX idx_cdoc_type ON c_document (filetype)');
44
        }
45
46
        //$this->addSql('ALTER TABLE c_document CHANGE path path VARCHAR(255) DEFAULT NULL;');
47
        $table = $schema->getTable('c_announcement');
48
        if (false === $table->hasColumn('resource_node_id')) {
49
            $this->addSql('ALTER TABLE c_announcement ADD resource_node_id INT DEFAULT NULL;');
50
            $this->addSql(
51
                'ALTER TABLE c_announcement ADD CONSTRAINT FK_39912E021BAD783F FOREIGN KEY (resource_node_id) REFERENCES resource_node (id) ON DELETE CASCADE;'
52
            );
53
            $this->addSql('CREATE UNIQUE INDEX UNIQ_39912E021BAD783F ON c_announcement (resource_node_id);');
54
        }
55
        if ($table->hasIndex('course')) {
56
            $this->addSql('DROP INDEX course ON c_announcement');
57
        }
58
        if ($table->hasIndex('session_id')) {
59
            $this->addSql('DROP INDEX session_id ON c_announcement');
60
        }
61
62
        $table = $schema->getTable('c_announcement_attachment');
63
        if ($table->hasIndex('course')) {
64
            $this->addSql('DROP INDEX course ON c_announcement_attachment');
65
        }
66
67
        $this->addSql('ALTER TABLE c_announcement_attachment CHANGE announcement_id announcement_id INT DEFAULT NULL');
68
69
        if (false === $table->hasColumn('resource_node_id')) {
70
            $this->addSql('ALTER TABLE c_announcement_attachment ADD resource_node_id INT DEFAULT NULL');
71
            $this->addSql(
72
                'ALTER TABLE c_announcement_attachment ADD CONSTRAINT FK_5480BD4A913AEA17 FOREIGN KEY (announcement_id) REFERENCES c_announcement (iid) ON DELETE CASCADE'
73
            );
74
            $this->addSql(
75
                'ALTER TABLE c_announcement_attachment ADD CONSTRAINT FK_5480BD4A1BAD783F FOREIGN KEY (resource_node_id) REFERENCES resource_node (id) ON DELETE CASCADE'
76
            );
77
            $this->addSql('CREATE INDEX IDX_5480BD4A913AEA17 ON c_announcement_attachment (announcement_id)');
78
            $this->addSql('CREATE UNIQUE INDEX UNIQ_5480BD4A1BAD783F ON c_announcement_attachment (resource_node_id)');
79
        }
80
    }
81
82
    public function down(Schema $schema): void
83
    {
84
    }
85
}
86