KitLoong\MigrationsGenerator\Generators\Modifier\DefaultModifier::generate()

Complexity

Conditions	12
Paths	12

Size

Total Lines	24
Code Lines	20

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	20
CRAP Score	12

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function generate(string $dbalType, Column $column): string
    {
        switch ($dbalType) {
            case DBALTypes::SMALLINT:
            case DBALTypes::INTEGER:
            case DBALTypes::BIGINT:
            case DBALTypes::MEDIUMINT:
            case DBALTypes::TINYINT:
            case DBALTypes::DECIMAL:
            case DBALTypes::FLOAT:
            case DBALTypes::DOUBLE:
                $default = $column->getDefault();
                break;
            case DBALTypes::BOOLEAN:
                $default = $this->booleanField->makeDefault($column);
                break;
            case DBALTypes::DATETIME_MUTABLE:
            case DBALTypes::TIMESTAMP:
                return $this->datetimeField->makeDefault($column);
            default:
                $default = $this->decorator->columnDefaultToString($column->getDefault());

It seems like $column->getDefault() can also be of type null; however, parameter $args of KitLoong\MigrationsGenerator\Generators\Decorator::columnDefaultToString() does only seem to accept string, maybe add an additional type check?

        }

        return $this->decorator->decorate(ColumnModifier::DEFAULT, [$default]);
    }