App\Repositories\Repository

Complexity

Total Complexity

5

Size/Duplication

Total Lines	49
Duplicated Lines	14.29 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	find()	7	7	2
A	lists()	0	13	3

<?php

namespace App\Repositories;

use App\Contracts\RepositoryContract;
use Illuminate\Database\Eloquent\Model;

abstract class Repository implements RepositoryContract
{
    /**
     * Ascendent.
     */
    const ASC = 'asc';

    /**
     * Descendent.
     */
    const DESC = 'desc';

    /**
     * Find model by id/slug.
     *
     * @param $slug
     * @return Model
     */
    public function find($slug)

This method seems to be duplicated in your project.

    {
        if (is_numeric($slug))
            return $this->getModel()->active()->whereId((int) $slug)->first();

        return $this->getModel()->active()->whereSlug($slug)->first();
    }

    /**
     * Lists the columns.
     * @example: $collection->lists('name') will return ['id' => 'name', ...] etc.
     *
     * @param $column
     * @param null $key
     * @param bool $active
     * @return array
     */
    public function lists($column, $key = null, $active = false)
    {
        $items = [];

        $collection = $active ? $this->getModel()->active()->get() : $this->getModel()->all();

        foreach ($collection as $item)
        {
            $items[$item->$key] = $item->$column;
        }

        return $items;
    }
}