Version20240806120000::up() - Code Metrics - Inspection of "Internal: Migrate settings from configuration.php..." - chamilo/chamilo-lms - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#5711)

unknown

created 2024-08-08 20:09 UTC

Version20240806120000::up() B

↳ Parent: Version20240806120000

Complexity

Conditions	9
Paths	24

Size

Total Lines	51
Code Lines	30

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
cc	9
eloc	30
nc	24
nop	1
dl	0
loc	51
rs	8.0555
c	2
b	0
f	0

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;
use Symfony\Component\Yaml\Yaml;

final class Version20240806120000 extends AbstractMigrationChamilo
{
    public function getDescription(): string
    {
        return 'Migrate settings from configuration.php to .env and hosting_limits.yml files';
    }

    public function up(Schema $schema): void
    {
        global $_configuration;

        $rootPath = $this->getRootPath();
        $oldConfigPath = $rootPath . '/app/config/configuration.php';
        if (!\in_array($oldConfigPath, get_included_files(), true)) {
            include_once $oldConfigPath;
        }

        // Update .env and .env.local files
        $this->updateEnvFiles($rootPath, [
            "DB_MANAGER_ENABLED" => $_configuration['db_manager_enabled'] ? '1' : '0',
            "SOFTWARE_NAME" => $_configuration['software_name'],
            "SOFTWARE_URL" => $_configuration['software_url'],
            "DENY_DELETE_USERS" => $_configuration['deny_delete_users'] ? '1' : '0',
            "HOSTING_TOTAL_SIZE_LIMIT" => $_configuration['hosting_total_size_limit']
        ]);

        // Ensure the hosting_limits.yml file exists
        $hostingLimitsFile = $rootPath . '/config/hosting_limits.yml';
        $hostingLimits = ['hosting_limits' => ['urls' => []]];

        // Prepare hosting limits
        foreach ($_configuration as $urlId => $config) {
            if (is_array($config)) {
                $hostingLimits['hosting_limits']['urls'][$urlId] = [
                    ['hosting_limit_users' => $config['hosting_limit_users'] ?? 0],
                    ['hosting_limit_teachers' => $config['hosting_limit_teachers'] ?? 0],
                    ['hosting_limit_courses' => $config['hosting_limit_courses'] ?? 0],
                    ['hosting_limit_sessions' => $config['hosting_limit_sessions'] ?? 0],
                    ['hosting_limit_disk_space' => $config['hosting_limit_disk_space'] ?? 0],
                    ['hosting_limit_active_courses' => $config['hosting_limit_active_courses'] ?? 0],
                    ['hosting_total_size_limit' => $_configuration['hosting_total_size_limit'] ?? 0]
                ];
            }
        }

        // Format hosting limits as YAML
        $yamlContent = "hosting_limits:\n    urls:\n";
        foreach ($hostingLimits['hosting_limits']['urls'] as $urlId => $limits) {
            $yamlContent .= "        {$urlId}:\n";
            foreach ($limits as $limit) {
                foreach ($limit as $key => $value) {
                    $yamlContent .= "            - {$key}: {$value}\n";
                }
            }
        }

        // Write hosting limits to hosting_limits.yml
        file_put_contents($hostingLimitsFile, $yamlContent);
    }

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

    private function getRootPath(): string
    {
        return $this->container->getParameter('kernel.project_dir');
        return $this->container->/** @scrutinizer ignore-call */ getParameter('kernel.project_dir');
    }

    private function updateEnvFiles(string $rootPath, array $envSettings): void
    {
        $envFiles = [$rootPath . '/.env', $rootPath . '/.env.local'];

        foreach ($envFiles as $envFile) {
            if (file_exists($envFile)) {
                $lines = file($envFile, FILE_IGNORE_NEW_LINES);
                $updatedLines = [];
                $existingKeys = [];

                foreach ($lines as $line) {
                    if (strpos($line, '=') !== false) {
                        [$key, $value] = explode('=', $line, 2);
                        $key = trim($key);
                        if (array_key_exists($key, $envSettings)) {
                            $value = $envSettings[$key];
                            unset($envSettings[$key]);
                        }
                        $updatedLines[] = "{$key}={$value}";
                        $existingKeys[] = $key;
                    } else {
                        $updatedLines[] = $line;
                    }
                }

                // Add remaining new settings
                foreach ($envSettings as $key => $value) {
                    if (!in_array($key, $existingKeys)) {
                        $updatedLines[] = "{$key}={$value}";
                    }
                }

                file_put_contents($envFile, implode(PHP_EOL, $updatedLines) . PHP_EOL);
            } else {
                // If the file does not exist, create it with the settings
                $newContent = [];
                foreach ($envSettings as $key => $value) {
                    $newContent[] = "{$key}={$value}";
                }
                file_put_contents($envFile, implode(PHP_EOL, $newContent) . PHP_EOL);
            }
        }
    }
}


chamilo / chamilo-lms

Pull Request — master (#5711)

Version20240806120000::up() B

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like