Backpack\CRUD\app\Models\Traits\HasEnumFields::getPossibleEnumValues()

Complexity

Conditions	4
Paths	12

Size

Total Lines	25
Code Lines	15

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public static function getPossibleEnumValues($field_name)
    {
        $instance = new static(); // create an instance of the model to be able to get the table name

        $connection = $instance->getConnection();

It seems like getConnection() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $table_prefix = Config::get('database.connections.'.$connection->getName().'.prefix');

        try {
            $select = app()->version() < 10 ?

The method version() does not exist on Illuminate\Container\Container. Are you sure you never get this type here, but always one of the subclasses?

                        DB::raw('SHOW COLUMNS FROM `'.$table_prefix.$instance->getTable().'` WHERE Field = "'.$field_name.'"') :

It seems like getTable() must be provided by classes using this trait. How about adding it as abstract method to this trait?

                        DB::raw('SHOW COLUMNS FROM `'.$table_prefix.$instance->getTable().'` WHERE Field = "'.$field_name.'"')->getValue($connection->getQueryGrammar());

            $type = $connection->select($select)[0]->Type;
        } catch (\Exception $e) {
            abort(500, 'Enum field type is not supported - it only works on MySQL. Please use select_from_array instead.', ['developer-error-exception']);
        }

        preg_match('/^enum\((.*)\)$/', $type, $matches);

The variable $type does not seem to be defined for all execution paths leading up to this point.

        $enum = [];
        foreach (explode(',', $matches[1]) as $value) {
            $enum[] = trim($value, "'");
        }

        return $enum;
    }