Version20240811221750::up() - Code Metrics - Inspection of "Migration: Check schema differences post-migration..." - chamilo/chamilo-lms - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#5720)

unknown

created 2024-08-26 20:05 UTC

Version20240811221750::up() C

↳ Parent: Version20240811221750

Complexity

Conditions	11
Paths	255

Size

Total Lines	65
Code Lines	21

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	11
eloc	21
c	1
b	0
f	0
nc	255
nop	1
dl	0
loc	65
rs	5.7958

How to fix Long Method Complexity

<?php

declare(strict_types=1);

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

namespace Chamilo\CoreBundle\Migrations\Schema\V200;

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

final class Version20240811221750 extends AbstractMigrationChamilo
{
    public function getDescription(): string
    {
        return 'Migration to add foreign key constraints to LTI-related tables.';
    }

    public function up(Schema $schema): void
    {
        $schemaManager = $this->connection->createSchemaManager();

        if ($schema->hasTable('lti_external_tool')) {
            $foreignKeys = $schemaManager->listTableForeignKeys('lti_external_tool');

            if (!$this->hasForeignKey($foreignKeys, 'FK_DB0E04E41BAD783F')) {
                $this->addSql('
                    ALTER TABLE lti_external_tool
                    ADD CONSTRAINT FK_DB0E04E41BAD783F FOREIGN KEY (resource_node_id)
                    REFERENCES resource_node (id) ON DELETE CASCADE;
                ');
            }

            if (!$this->hasForeignKey($foreignKeys, 'FK_DB0E04E491D79BD3')) {
                $this->addSql('
                    ALTER TABLE lti_external_tool
                    ADD CONSTRAINT FK_DB0E04E491D79BD3 FOREIGN KEY (c_id)
                    REFERENCES course (id);
                ');
            }

            if (!$this->hasForeignKey($foreignKeys, 'FK_DB0E04E482F80D8B')) {
                $this->addSql('
                    ALTER TABLE lti_external_tool
                    ADD CONSTRAINT FK_DB0E04E482F80D8B FOREIGN KEY (gradebook_eval_id)
                    REFERENCES gradebook_evaluation (id) ON DELETE SET NULL;
                ');
            }

            if (!$this->hasForeignKey($foreignKeys, 'FK_DB0E04E4727ACA70')) {
                $this->addSql('
                    ALTER TABLE lti_external_tool
                    ADD CONSTRAINT FK_DB0E04E4727ACA70 FOREIGN KEY (parent_id)
                    REFERENCES lti_external_tool (id);
                ');
            }
        }

        if ($schema->hasTable('lti_token')) {
            $foreignKeys = $schemaManager->listTableForeignKeys('lti_token');

            if (!$this->hasForeignKey($foreignKeys, 'FK_EA71C468F7B22CC')) {
                $this->addSql('
                    ALTER TABLE lti_token
                    ADD CONSTRAINT FK_EA71C468F7B22CC FOREIGN KEY (tool_id)
                    REFERENCES lti_external_tool (id) ON DELETE CASCADE;
                ');
            }
        }

        if ($schema->hasTable('lti_lineitem')) {
            $foreignKeys = $schemaManager->listTableForeignKeys('lti_lineitem');

            if (!$this->hasForeignKey($foreignKeys, 'FK_5C76B75D8F7B22CC')) {
                $this->addSql('
                    ALTER TABLE lti_lineitem
                    ADD CONSTRAINT FK_5C76B75D8F7B22CC FOREIGN KEY (tool_id)
                    REFERENCES lti_external_tool (id) ON DELETE CASCADE;
                ');
            }

            if (!$this->hasForeignKey($foreignKeys, 'FK_5C76B75D1323A575')) {
                $this->addSql('
                    ALTER TABLE lti_lineitem
                    ADD CONSTRAINT FK_5C76B75D1323A575 FOREIGN KEY (evaluation)
                    REFERENCES gradebook_evaluation (id) ON DELETE CASCADE;
                ');
            }
        }
    }

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

    private function hasForeignKey(array $foreignKeys, string $foreignKeyName): bool
    {
        foreach ($foreignKeys as $foreignKey) {
            if ($foreignKey->getName() === $foreignKeyName) {
                return true;
            }
        }
        return false;
    }
}


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 Doctrine\DBAL\Schema\Schema;
10			use Chamilo\CoreBundle\Migrations\AbstractMigrationChamilo;
11
12			final class Version20240811221750 extends AbstractMigrationChamilo
13			{
14			public function getDescription(): string
15			{
16			return 'Migration to add foreign key constraints to LTI-related tables.';
17			}
18
19			public function up(Schema $schema): void
20			{
21			$schemaManager = $this->connection->createSchemaManager();
22
23			if ($schema->hasTable('lti_external_tool')) {
24			$foreignKeys = $schemaManager->listTableForeignKeys('lti_external_tool');
25
26			if (!$this->hasForeignKey($foreignKeys, 'FK_DB0E04E41BAD783F')) {
27			$this->addSql('
28			ALTER TABLE lti_external_tool
29			ADD CONSTRAINT FK_DB0E04E41BAD783F FOREIGN KEY (resource_node_id)
30			REFERENCES resource_node (id) ON DELETE CASCADE;
31			');
32			}
33
34			if (!$this->hasForeignKey($foreignKeys, 'FK_DB0E04E491D79BD3')) {
35			$this->addSql('
36			ALTER TABLE lti_external_tool
37			ADD CONSTRAINT FK_DB0E04E491D79BD3 FOREIGN KEY (c_id)
38			REFERENCES course (id);
39			');
40			}
41
42			if (!$this->hasForeignKey($foreignKeys, 'FK_DB0E04E482F80D8B')) {
43			$this->addSql('
44			ALTER TABLE lti_external_tool
45			ADD CONSTRAINT FK_DB0E04E482F80D8B FOREIGN KEY (gradebook_eval_id)
46			REFERENCES gradebook_evaluation (id) ON DELETE SET NULL;
47			');
48			}
49
50			if (!$this->hasForeignKey($foreignKeys, 'FK_DB0E04E4727ACA70')) {
51			$this->addSql('
52			ALTER TABLE lti_external_tool
53			ADD CONSTRAINT FK_DB0E04E4727ACA70 FOREIGN KEY (parent_id)
54			REFERENCES lti_external_tool (id);
55			');
56			}
57			}
58
59			if ($schema->hasTable('lti_token')) {
60			$foreignKeys = $schemaManager->listTableForeignKeys('lti_token');
61
62			if (!$this->hasForeignKey($foreignKeys, 'FK_EA71C468F7B22CC')) {
63			$this->addSql('
64			ALTER TABLE lti_token
65			ADD CONSTRAINT FK_EA71C468F7B22CC FOREIGN KEY (tool_id)
66			REFERENCES lti_external_tool (id) ON DELETE CASCADE;
67			');
68			}
69			}
70
71			if ($schema->hasTable('lti_lineitem')) {
72			$foreignKeys = $schemaManager->listTableForeignKeys('lti_lineitem');
73
74			if (!$this->hasForeignKey($foreignKeys, 'FK_5C76B75D8F7B22CC')) {
75			$this->addSql('
76			ALTER TABLE lti_lineitem
77			ADD CONSTRAINT FK_5C76B75D8F7B22CC FOREIGN KEY (tool_id)
78			REFERENCES lti_external_tool (id) ON DELETE CASCADE;
79			');
80			}
81
82			if (!$this->hasForeignKey($foreignKeys, 'FK_5C76B75D1323A575')) {
83			$this->addSql('
84			ALTER TABLE lti_lineitem
85			ADD CONSTRAINT FK_5C76B75D1323A575 FOREIGN KEY (evaluation)
86			REFERENCES gradebook_evaluation (id) ON DELETE CASCADE;
87			');
88			}
89			}
90			}
91
92			public function down(Schema $schema): void {}
93
94			private function hasForeignKey(array $foreignKeys, string $foreignKeyName): bool
95			{
96			foreach ($foreignKeys as $foreignKey) {
97			if ($foreignKey->getName() === $foreignKeyName) {
98			return true;
99			}
100			}
101			return false;
102			}
103			}
104

chamilo / chamilo-lms

Pull Request — master (#5720)

Version20240811221750::up() C

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like