CMSNicetiesTraitForValidation::validateForUniqueValues()

Complexity

Conditions	8
Paths	21

Size

Total Lines	33
Code Lines	20

Duplication

Lines	0
Ratio	0 %

Importance

Changes	3
Bugs	0	Features	0

<?php
    public function validateForUniqueValues(ValidationResult $result): ValidationResult
    {
        foreach ($this->Config()->get('indexes') as $index) {

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

            $isUniqueEntry = isset($index['type']) && 'unique' === $index['type'];
            if ($isUniqueEntry) {
                $fields = $index['columns'] ?? [];
                if (count($fields) > 0) {
                    $filter = [];
                    foreach ($fields as $field) {
                        $filter[$field] = $this->{$field};
                    }

                    $id = (empty($this->ID) ? 0 : $this->ID);
                    // https://stackoverflow.com/questions/63227834/return-self-for-the-return-type-of-a-function-inside-a-php-trait
                    $exists = self::get()
                        ->filter($filter)
                        ->exclude(['ID' => $id])
                        ->exists()
                    ;
                    if ($exists) {
                        $result->addError(
                            _t(
                                self::class . '.' . $index['type'] . '_UNIQUE',
                                $index['type'] . ' needs to be unique'
                            ),
                            'UNIQUE_' . self::class . '.' . implode('_', $fields)
                        );
                    }
                }
            }
        }

        return $result;
    }