Version20240826220000::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

Version20240826220000::up() A

↳ Parent: Version20240826220000

Complexity

Conditions	5
Paths	16

Size

Total Lines	51
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	5
eloc	11
c	1
b	0
f	0
nc	16
nop	1
dl	0
loc	51
rs	9.6111

How to fix Long Method

<?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;

final class Version20240826220000 extends AbstractMigrationChamilo
{
    public function getDescription(): string
    {
        return 'Migration to create justification_document and justification_document_rel_users tables with their foreign key relationships.';
    }

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

        // Create justification_document table
        if (!$schemaManager->tablesExist(['justification_document'])) {
            $this->addSql('
                CREATE TABLE justification_document (
                    id INT AUTO_INCREMENT NOT NULL,
                    code LONGTEXT DEFAULT NULL,
                    name LONGTEXT DEFAULT NULL,
                    validity_duration INT DEFAULT NULL,
                    comment LONGTEXT DEFAULT NULL,
                    date_manual_on INT DEFAULT NULL,
                    PRIMARY KEY(id)
                ) DEFAULT CHARACTER SET utf8mb4 COLLATE `utf8mb4_unicode_ci` ENGINE = InnoDB ROW_FORMAT = DYNAMIC;
            ');
        }

        // Create justification_document_rel_users table
        if (!$schemaManager->tablesExist(['justification_document_rel_users'])) {
            $this->addSql('
                CREATE TABLE justification_document_rel_users (
                    id INT AUTO_INCREMENT NOT NULL,
                    justification_document_id INT DEFAULT NULL,
                    user_id INT DEFAULT NULL,
                    file_path VARCHAR(255) DEFAULT NULL,
                    date_validity DATE DEFAULT NULL,
                    INDEX IDX_D1BB19421F2B6144 (justification_document_id),
                    INDEX IDX_D1BB1942A76ED395 (user_id),
                    PRIMARY KEY(id)
                ) DEFAULT CHARACTER SET utf8mb4 COLLATE `utf8mb4_unicode_ci` ENGINE = InnoDB ROW_FORMAT = DYNAMIC;
            ');
        }

        // Add foreign key constraints
        $foreignKeys = $schemaManager->listTableForeignKeys('justification_document_rel_users');
        $foreignKeyNames = array_map(fn($fk) => $fk->getName(), $foreignKeys);

        if (!in_array('FK_D1BB19421F2B6144', $foreignKeyNames, true)) {
            $this->addSql('
                ALTER TABLE justification_document_rel_users
                ADD CONSTRAINT FK_D1BB19421F2B6144
                FOREIGN KEY (justification_document_id)
                REFERENCES justification_document (id)
                ON DELETE CASCADE;
            ');
        }

        if (!in_array('FK_D1BB1942A76ED395', $foreignKeyNames, true)) {
            $this->addSql('
                ALTER TABLE justification_document_rel_users
                ADD CONSTRAINT FK_D1BB1942A76ED395
                FOREIGN KEY (user_id)
                REFERENCES user (id)
                ON DELETE CASCADE;
            ');
        }
    }

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

        // Drop foreign key constraints
        if ($schemaManager->tablesExist(['justification_document_rel_users'])) {
            $foreignKeys = $schemaManager->listTableForeignKeys('justification_document_rel_users');
            $foreignKeyNames = array_map(fn($fk) => $fk->getName(), $foreignKeys);

            if (in_array('FK_D1BB19421F2B6144', $foreignKeyNames, true)) {
                $this->addSql('
                    ALTER TABLE justification_document_rel_users
                    DROP FOREIGN KEY FK_D1BB19421F2B6144;
                ');
            }

            if (in_array('FK_D1BB1942A76ED395', $foreignKeyNames, true)) {
                $this->addSql('
                    ALTER TABLE justification_document_rel_users
                    DROP FOREIGN KEY FK_D1BB1942A76ED395;
                ');
            }
        }

        // Drop justification_document_rel_users table
        if ($schemaManager->tablesExist(['justification_document_rel_users'])) {
            $this->addSql('DROP TABLE justification_document_rel_users;');
        }

        // Drop justification_document table
        if ($schemaManager->tablesExist(['justification_document'])) {
            $this->addSql('DROP TABLE justification_document;');
        }
    }
}


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			final class Version20240826220000 extends AbstractMigrationChamilo
13			{
14			public function getDescription(): string
15			{
16			return 'Migration to create justification_document and justification_document_rel_users tables with their foreign key relationships.';
17			}
18
19			public function up(Schema $schema): void
20			{
21			$schemaManager = $this->connection->createSchemaManager();
22
23			// Create justification_document table
24			if (!$schemaManager->tablesExist(['justification_document'])) {
25			$this->addSql('
26			CREATE TABLE justification_document (
27			id INT AUTO_INCREMENT NOT NULL,
28			code LONGTEXT DEFAULT NULL,
29			name LONGTEXT DEFAULT NULL,
30			validity_duration INT DEFAULT NULL,
31			comment LONGTEXT DEFAULT NULL,
32			date_manual_on INT DEFAULT NULL,
33			PRIMARY KEY(id)
34			) DEFAULT CHARACTER SET utf8mb4 COLLATE `utf8mb4_unicode_ci` ENGINE = InnoDB ROW_FORMAT = DYNAMIC;
35			');
36			}
37
38			// Create justification_document_rel_users table
39			if (!$schemaManager->tablesExist(['justification_document_rel_users'])) {
40			$this->addSql('
41			CREATE TABLE justification_document_rel_users (
42			id INT AUTO_INCREMENT NOT NULL,
43			justification_document_id INT DEFAULT NULL,
44			user_id INT DEFAULT NULL,
45			file_path VARCHAR(255) DEFAULT NULL,
46			date_validity DATE DEFAULT NULL,
47			INDEX IDX_D1BB19421F2B6144 (justification_document_id),
48			INDEX IDX_D1BB1942A76ED395 (user_id),
49			PRIMARY KEY(id)
50			) DEFAULT CHARACTER SET utf8mb4 COLLATE `utf8mb4_unicode_ci` ENGINE = InnoDB ROW_FORMAT = DYNAMIC;
51			');
52			}
53
54			// Add foreign key constraints
55			$foreignKeys = $schemaManager->listTableForeignKeys('justification_document_rel_users');
56			$foreignKeyNames = array_map(fn($fk) => $fk->getName(), $foreignKeys);
57
58			if (!in_array('FK_D1BB19421F2B6144', $foreignKeyNames, true)) {
59			$this->addSql('
60			ALTER TABLE justification_document_rel_users
61			ADD CONSTRAINT FK_D1BB19421F2B6144
62			FOREIGN KEY (justification_document_id)
63			REFERENCES justification_document (id)
64			ON DELETE CASCADE;
65			');
66			}
67
68			if (!in_array('FK_D1BB1942A76ED395', $foreignKeyNames, true)) {
69			$this->addSql('
70			ALTER TABLE justification_document_rel_users
71			ADD CONSTRAINT FK_D1BB1942A76ED395
72			FOREIGN KEY (user_id)
73			REFERENCES user (id)
74			ON DELETE CASCADE;
75			');
76			}
77			}
78
79			public function down(Schema $schema): void
80			{
81			$schemaManager = $this->connection->createSchemaManager();
82
83			// Drop foreign key constraints
84			if ($schemaManager->tablesExist(['justification_document_rel_users'])) {
85			$foreignKeys = $schemaManager->listTableForeignKeys('justification_document_rel_users');
86			$foreignKeyNames = array_map(fn($fk) => $fk->getName(), $foreignKeys);
87
88			if (in_array('FK_D1BB19421F2B6144', $foreignKeyNames, true)) {
89			$this->addSql('
90			ALTER TABLE justification_document_rel_users
91			DROP FOREIGN KEY FK_D1BB19421F2B6144;
92			');
93			}
94
95			if (in_array('FK_D1BB1942A76ED395', $foreignKeyNames, true)) {
96			$this->addSql('
97			ALTER TABLE justification_document_rel_users
98			DROP FOREIGN KEY FK_D1BB1942A76ED395;
99			');
100			}
101			}
102
103			// Drop justification_document_rel_users table
104			if ($schemaManager->tablesExist(['justification_document_rel_users'])) {
105			$this->addSql('DROP TABLE justification_document_rel_users;');
106			}
107
108			// Drop justification_document table
109			if ($schemaManager->tablesExist(['justification_document'])) {
110			$this->addSql('DROP TABLE justification_document;');
111			}
112			}
113			}
114

chamilo / chamilo-lms

Pull Request — master (#5720)

Version20240826220000::up() A

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like