crocodicstudio\crudbooster\controllers\FormValidator::parseValidationRules()

Complexity

Conditions	10
Paths	15

Size

Total Lines	43
Code Lines	26

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    private function parseValidationRules($id, $rules, $name)
    {
        if (is_string($rules)) {
            $exp = explode('|', $rules);
        } elseif(is_array($rules)) {
            $exp = $rules;
        }

        $uniqueRules = array_filter($exp, function($item){

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

            return starts_with($item, 'unique');
        });

        foreach ($uniqueRules as &$validationItem) {
            $parseUnique = explode(',', str_replace('unique:', '', $validationItem));
            $uniqueTable = ($parseUnique[0]) ?: $this->table;
            $uniqueColumn = ($parseUnique[1]) ?: $name;
            $uniqueIgnoreId = ($parseUnique[2]) ?: (($id) ?: '');

            //Make sure table name
            $uniqueTable = CRUDBooster::parseSqlTable($uniqueTable)['table'];

            //Rebuild unique rule
            $uniqueRebuild = [];
            $uniqueRebuild[] = $uniqueTable;
            $uniqueRebuild[] = $uniqueColumn;

            if ($uniqueIgnoreId) {
                $uniqueRebuild[] = $uniqueIgnoreId;
            } else {
                $uniqueRebuild[] = 'NULL';
            }

            //Check whether deleted_at exists or not
            if (\Schema::hasColumn($uniqueTable, 'deleted_at')) {
                $uniqueRebuild[] = DbInspector::findPK($uniqueTable);
                $uniqueRebuild[] = 'deleted_at';
                $uniqueRebuild[] = 'NULL';
            }
            $uniqueRebuild = array_filter($uniqueRebuild);
            $validationItem = 'unique:'.implode(',', $uniqueRebuild);
        }

        return implode('|', $exp);

The expression return implode('|', $exp) returns the type string which is incompatible with the documented return type array.

    }
}