Passed
Push — master ( 804eb4...5e2840 )
by Julito
10:59
created

Version20200821224242::up()   D

Complexity

Conditions 10
Paths 512

Size

Total Lines 63
Code Lines 38

Duplication

Lines 0
Ratio 0 %

Importance

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

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
namespace Chamilo\CoreBundle\Migrations\Schema\V200;
4
5
use Chamilo\CoreBundle\Migrations\AbstractMigrationChamilo;
6
use Doctrine\DBAL\Schema\Schema;
7
8
/**
9
 * Messages.
10
 */
11
final class Version20200821224242 extends AbstractMigrationChamilo
12
{
13
    public function up(Schema $schema): void
14
    {
15
        $this->addSql('DELETE FROM message WHERE user_sender_id IS NULL OR user_sender_id = 0');
16
        $this->addSql('ALTER TABLE message CHANGE user_receiver_id user_receiver_id INT DEFAULT NULL');
17
        $this->addSql('UPDATE message SET user_receiver_id = NULL WHERE user_receiver_id = 0');
18
        $this->addSql('DELETE FROM message WHERE user_sender_id NOT IN (SELECT id FROM user)');
19
        $this->addSql(
20
            'DELETE FROM message WHERE user_receiver_id IS NOT NULL AND user_receiver_id NOT IN (SELECT id FROM user)'
21
        );
22
23
        $table = $schema->getTable('message');
24
        if (false === $table->hasForeignKey('FK_B6BD307FF6C43E79')) {
25
            $this->addSql(
26
                'ALTER TABLE message ADD CONSTRAINT FK_B6BD307FF6C43E79 FOREIGN KEY (user_sender_id) REFERENCES user (id)'
27
            );
28
        }
29
        if (false === $table->hasForeignKey('FK_B6BD307F64482423')) {
30
            $this->addSql(
31
                'ALTER TABLE message ADD CONSTRAINT FK_B6BD307F64482423 FOREIGN KEY (user_receiver_id) REFERENCES user (id)'
32
            );
33
        }
34
        if (!$table->hasIndex('idx_message_user_receiver_status')) {
35
            $this->addSql('CREATE INDEX idx_message_user_receiver_status ON message (user_receiver_id, msg_status)');
36
        }
37
38
        if (!$table->hasIndex('idx_message_status')) {
39
            $this->addSql('CREATE INDEX idx_message_status ON message (msg_status)');
40
        }
41
42
        if (!$table->hasIndex('idx_message_receiver_status_send_date')) {
43
            $this->addSql(
44
                'CREATE INDEX idx_message_receiver_status_send_date ON message (user_receiver_id, msg_status, send_date)'
45
            );
46
        }
47
48
        $this->addSql('ALTER TABLE message CHANGE msg_status msg_status SMALLINT NOT NULL;');
49
50
        $table = $schema->hasTable('message_feedback');
51
        if (false === $table) {
52
            $this->addSql(
53
                'CREATE TABLE message_feedback (id BIGINT AUTO_INCREMENT NOT NULL, message_id BIGINT NOT NULL, user_id INT NOT NULL, liked TINYINT(1) DEFAULT 0 NOT NULL, disliked TINYINT(1) DEFAULT 0 NOT NULL, updated_at DATETIME NOT NULL, INDEX IDX_DB0F8049537A1329 (message_id), INDEX IDX_DB0F8049A76ED395 (user_id), INDEX idx_message_feedback_uid_mid (message_id, user_id), PRIMARY KEY(id)) DEFAULT CHARACTER SET utf8mb4 COLLATE utf8mb4_unicode_ci ENGINE = InnoDB ROW_FORMAT = DYNAMIC;'
54
            );
55
            $this->addSql(
56
                'ALTER TABLE message_feedback ADD CONSTRAINT FK_DB0F8049537A1329 FOREIGN KEY (message_id) REFERENCES message (id) ON DELETE CASCADE'
57
            );
58
            $this->addSql(
59
                'ALTER TABLE message_feedback ADD CONSTRAINT FK_DB0F8049A76ED395 FOREIGN KEY (user_id) REFERENCES user (id) ON DELETE CASCADE;'
60
            );
61
        }
62
63
        $table = $schema->getTable('message_attachment');
64
        if (false === $table->hasIndex('IDX_B68FF524537A1329')) {
65
            $this->addSql('CREATE INDEX IDX_B68FF524537A1329 ON message_attachment (message_id)');
66
        }
67
        $this->addSql('ALTER TABLE message_attachment CHANGE message_id message_id BIGINT NOT NULL');
68
69
        if (false === $table->hasForeignKey('FK_B68FF524537A1329')) {
70
            $this->addSql('ALTER TABLE message_attachment ADD CONSTRAINT FK_B68FF524537A1329 FOREIGN KEY (message_id) REFERENCES message (id)');
71
        }
72
73
        if (false === $schema->hasTable('c_chat_conversation')) {
74
            $this->addSql('CREATE TABLE c_chat_conversation (id INT AUTO_INCREMENT NOT NULL, resource_node_id INT DEFAULT NULL, name VARCHAR(255) DEFAULT NULL, UNIQUE INDEX UNIQ_CD09E33F1BAD783F (resource_node_id), PRIMARY KEY(id)) DEFAULT CHARACTER SET utf8mb4 COLLATE `utf8mb4_unicode_ci` ENGINE = InnoDB ROW_FORMAT = DYNAMIC;');
75
            $this->addSql('ALTER TABLE c_chat_conversation ADD CONSTRAINT FK_CD09E33F1BAD783F FOREIGN KEY (resource_node_id) REFERENCES resource_node (id) ON DELETE CASCADE');
76
        }
77
    }
78
79
    public function down(Schema $schema): void
80
    {
81
    }
82
}
83