KitLoong\MigrationsGenerator\MigrationGeneratorSetting

Complexity

Total Complexity

8

Size/Duplication

Total Lines	73
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	getConnection()	0	4	1
A	setConnection()	0	28	5
A	getDatabasePlatform()	0	4	1
A	getPlatform()	0	4	1

<?php
/**
 * Created by PhpStorm.
 * User: liow.kitloong
 * Date: 2020/03/31
 * Time: 22:55
 */

namespace KitLoong\MigrationsGenerator;

use Illuminate\Support\Facades\DB;
use KitLoong\MigrationsGenerator\Generators\Platform;

class MigrationGeneratorSetting
{
    /**
     * @var string
     */
    private $connection;

    /**
     * @var \Doctrine\DBAL\Platforms\AbstractPlatform
     */
    private $databasePlatform;

    /**
     * @var string
     */
    private $platform;

    /**
     * @return string
     */
    public function getConnection(): string
    {
        return $this->connection;
    }

    /**
     * @param  string  $connection
     */
    public function setConnection(string $connection): void
    {
        $this->connection = $connection;

        /** @var \Doctrine\DBAL\Connection $doctConn */
        $doctConn = DB::connection($this->connection)->getDoctrineConnection();
        $this->databasePlatform = $doctConn->getDatabasePlatform();
        $classPath = explode('\\', get_class($this->databasePlatform));
        $platform = end($classPath);

        switch (true) {

It seems like you are loosely comparing preg_match('/mysql/i', $platform) of type integer to the boolean true. If you are specifically checking for non-zero, consider using something more explicit like > 0 or !== 0 instead.

It seems like you are loosely comparing preg_match('/postgresql/i', $platform) of type integer to the boolean true. If you are specifically checking for non-zero, consider using something more explicit like > 0 or !== 0 instead.

It seems like you are loosely comparing preg_match('/sqlserver/i', $platform) of type integer to the boolean true. If you are specifically checking for non-zero, consider using something more explicit like > 0 or !== 0 instead.

It seems like you are loosely comparing preg_match('/sqlite/i', $platform) of type integer to the boolean true. If you are specifically checking for non-zero, consider using something more explicit like > 0 or !== 0 instead.

            case preg_match('/mysql/i', $platform):
                $this->platform = Platform::MYSQL;
                break;
            case preg_match('/postgresql/i', $platform):
                $this->platform = Platform::POSTGRESQL;
                break;
            case preg_match('/sqlserver/i', $platform):
                $this->platform = Platform::SQLSERVER;
                break;
            case preg_match('/sqlite/i', $platform):
                $this->platform = Platform::SQLITE;
                break;
            default:
                $this->platform = Platform::OTHERS;
                break;
        }
    }

    /**
     * @return \Doctrine\DBAL\Platforms\AbstractPlatform
     */
    public function getDatabasePlatform(): \Doctrine\DBAL\Platforms\AbstractPlatform
    {
        return $this->databasePlatform;
    }

    /**
     * @return string
     */
    public function getPlatform(): string
    {
        return $this->platform;
    }
}