Version20190210182615::up() - Code Metrics - Inspection of "Sessions: Add UniqueEntity constraints, add phpuni..." - chamilo/chamilo-lms - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( c37697...7551f3 )

by Julito

created 2021-06-16 13:07 UTC

Version20190210182615::up() D

↳ Parent: Version20190210182615

Complexity

Conditions	10
Paths	512

Size

Total Lines	51
Code Lines	31

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

Metric	Value
cc	10
eloc	31
c	1
b	0
f	1
nc	512
nop	1
dl	0
loc	51
rs	4.1777

How to fix Long Method Complexity

<?php

declare(strict_types=1);

/* For licensing terms, see /license.txt */

namespace Chamilo\CoreBundle\Migrations\Schema\V200;

use Chamilo\CoreBundle\Migrations\AbstractMigrationChamilo;
use Doctrine\DBAL\Schema\Schema;

class Version20190210182615 extends AbstractMigrationChamilo
{
    public function getDescription(): string
    {
        return 'Session changes';
    }

    public function up(Schema $schema): void
    {
        $table = $schema->getTable('session');
        if (false === $table->hasColumn('position')) {
            $this->addSql('ALTER TABLE session ADD COLUMN position INT DEFAULT 0 NOT NULL');
        } else {
            $this->addSql('ALTER TABLE session CHANGE position position INT DEFAULT 0 NOT NULL');
        }

        $this->addSql('UPDATE session SET promotion_id = NULL WHERE promotion_id = 0');
        if (false === $table->hasForeignKey('FK_D044D5D4139DF194')) {
            $this->addSql('ALTER TABLE session ADD CONSTRAINT FK_D044D5D4139DF194 FOREIGN KEY (promotion_id) REFERENCES promotion (id) ON DELETE CASCADE');
            $this->addSql('CREATE INDEX IDX_D044D5D4139DF194 ON session (promotion_id);');
        }

        if (false === $table->hasColumn('status')) {
            $this->addSql('ALTER TABLE session ADD COLUMN status INT NOT NULL');
        }

        if (false === $table->hasForeignKey('FK_D044D5D4EF87E278')) {
            $this->addSql('ALTER TABLE session ADD CONSTRAINT FK_D044D5D4EF87E278 FOREIGN KEY(session_admin_id) REFERENCES user(id);');
        }

        $this->addSql('UPDATE session_category SET date_start = NULL WHERE date_start = "0000-00-00"');
        $this->addSql('UPDATE session_category SET date_end = NULL WHERE date_end = "0000-00-00"');

        $table = $schema->getTable('session_rel_course_rel_user');

        if (!$table->hasColumn('progress')) {
            $this->addSql('ALTER TABLE session_rel_course_rel_user ADD progress INT NOT NULL');
        }

        if ($table->hasForeignKey('FK_720167E91D79BD3')) {
            $this->addSql('ALTER TABLE session_rel_course_rel_user DROP FOREIGN KEY FK_720167E91D79BD3');
            $this->addSql('ALTER TABLE session_rel_course_rel_user ADD CONSTRAINT FK_720167E91D79BD3 FOREIGN KEY (c_id) REFERENCES course (id) ON DELETE CASCADE');
        } else {
            $this->addSql('ALTER TABLE session_rel_course_rel_user ADD CONSTRAINT FK_720167E91D79BD3 FOREIGN KEY (c_id) REFERENCES course (id) ON DELETE CASCADE');
        }

        if (!$table->hasIndex('course_session_unique')) {
            $this->addSql(' CREATE UNIQUE INDEX course_session_unique ON session_rel_course_rel_user (session_id, c_id, user_id, status);');
        }

        $table = $schema->getTable('session_rel_course');
        if (!$table->hasIndex('UNIQ_12D110D391D79BD3')) {
            $this->addSql('CREATE UNIQUE INDEX course_session_unique ON session_rel_course (session_id, c_id)');
        }

        $table = $schema->getTable('session_rel_user');
        if (!$table->hasIndex('session_user_unique')) {
            $this->addSql('CREATE UNIQUE INDEX session_user_unique ON session_rel_user (session_id, user_id, relation_type);');
        }
    }

    public function down(Schema $schema): void
    {
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			/* For licensing terms, see /license.txt */
6
7			namespace Chamilo\CoreBundle\Migrations\Schema\V200;
8
9			use Chamilo\CoreBundle\Migrations\AbstractMigrationChamilo;
10			use Doctrine\DBAL\Schema\Schema;
11
12			class Version20190210182615 extends AbstractMigrationChamilo
13			{
14			public function getDescription(): string
15			{
16			return 'Session changes';
17			}
18
19			public function up(Schema $schema): void
20			{
21			$table = $schema->getTable('session');
22			if (false === $table->hasColumn('position')) {
23			$this->addSql('ALTER TABLE session ADD COLUMN position INT DEFAULT 0 NOT NULL');
24			} else {
25			$this->addSql('ALTER TABLE session CHANGE position position INT DEFAULT 0 NOT NULL');
26			}
27
28			$this->addSql('UPDATE session SET promotion_id = NULL WHERE promotion_id = 0');
29			if (false === $table->hasForeignKey('FK_D044D5D4139DF194')) {
30			$this->addSql('ALTER TABLE session ADD CONSTRAINT FK_D044D5D4139DF194 FOREIGN KEY (promotion_id) REFERENCES promotion (id) ON DELETE CASCADE');
31			$this->addSql('CREATE INDEX IDX_D044D5D4139DF194 ON session (promotion_id);');
32			}
33
34			if (false === $table->hasColumn('status')) {
35			$this->addSql('ALTER TABLE session ADD COLUMN status INT NOT NULL');
36			}
37
38			if (false === $table->hasForeignKey('FK_D044D5D4EF87E278')) {
39			$this->addSql('ALTER TABLE session ADD CONSTRAINT FK_D044D5D4EF87E278 FOREIGN KEY(session_admin_id) REFERENCES user(id);');
40			}
41
42			$this->addSql('UPDATE session_category SET date_start = NULL WHERE date_start = "0000-00-00"');
43			$this->addSql('UPDATE session_category SET date_end = NULL WHERE date_end = "0000-00-00"');
44
45			$table = $schema->getTable('session_rel_course_rel_user');
46
47			if (!$table->hasColumn('progress')) {
48			$this->addSql('ALTER TABLE session_rel_course_rel_user ADD progress INT NOT NULL');
49			}
50
51			if ($table->hasForeignKey('FK_720167E91D79BD3')) {
52			$this->addSql('ALTER TABLE session_rel_course_rel_user DROP FOREIGN KEY FK_720167E91D79BD3');
53			$this->addSql('ALTER TABLE session_rel_course_rel_user ADD CONSTRAINT FK_720167E91D79BD3 FOREIGN KEY (c_id) REFERENCES course (id) ON DELETE CASCADE');
54			} else {
55			$this->addSql('ALTER TABLE session_rel_course_rel_user ADD CONSTRAINT FK_720167E91D79BD3 FOREIGN KEY (c_id) REFERENCES course (id) ON DELETE CASCADE');
56			}
57
58			if (!$table->hasIndex('course_session_unique')) {
59			$this->addSql(' CREATE UNIQUE INDEX course_session_unique ON session_rel_course_rel_user (session_id, c_id, user_id, status);');
60			}
61
62			$table = $schema->getTable('session_rel_course');
63			if (!$table->hasIndex('UNIQ_12D110D391D79BD3')) {
64			$this->addSql('CREATE UNIQUE INDEX course_session_unique ON session_rel_course (session_id, c_id)');
65			}
66
67			$table = $schema->getTable('session_rel_user');
68			if (!$table->hasIndex('session_user_unique')) {
69			$this->addSql('CREATE UNIQUE INDEX session_user_unique ON session_rel_user (session_id, user_id, relation_type);');
70			}
71			}
72
73			public function down(Schema $schema): void
74			{
75			}
76			}
77

chamilo / chamilo-lms

Push — master ( c37697...7551f3 )

Version20190210182615::up() D

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like