DatabaseKeyDefinition::buildKeyDefinitionFromArray()

Complexity

Conditions	11
Paths	78

Size

Total Lines	51
Code Lines	32

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function buildKeyDefinitionFromArray($k, $options)
    {
        if (is_string($options)) {
            $options = ['type' => $options];
        } elseif (!is_array($options)) {

The condition is_array($options) is always true.

            throw new DatabaseStatementException('Key value can only be a string or an array');
        } elseif (!isset($options['type'])) {
            throw new DatabaseStatementException('Key type must be defined.');
        }
        $type = strtolower($options['type']);
        $k = $this->asTickedString($k);

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

        $cols = isset($options['cols']) ? $options['cols'] : $k;
        $typeIndex = in_array($type, [
            'key', 'unique', 'primary', 'fulltext', 'index'
        ]);
        if ($typeIndex === false) {
            throw new DatabaseStatementException("Key of type `$type` is not valid");
        }
        switch ($type) {
            case 'unique':
                $type = strtoupper($type) . ' KEY';
                break;
            case 'primary':
                // Use the key name as the KEY keyword
                // since the primary key does not have a name
                $k = 'KEY';
                // fall through
            default:
                $type = strtoupper($type);
                break;
        }

        // Format columns including size
        $colsFormatted = [];

        if (!is_array($cols)) {
            $cols = [$cols];
        }

        foreach ($cols as $key => $value) {
            // No Size
            if (General::intval($key) !== -1) {
                $colsFormatted[] = $this->asTickedString($value);
            // Size
            } else {
                $colsFormatted[] = $this->asTickedString($key) . '(' . $value . ')';
            }
        }
        $colsFormatted = implode(self::LIST_DELIMITER, $colsFormatted);

The constant DatabaseKeyDefinition::LIST_DELIMITER was not found. Maybe you did not declare it correctly or list all dependencies?

        return "$type $k ($colsFormatted)";
    }