Tinyissue\Repository\User\Counter

Complexity

Total Complexity

8

Size/Duplication

Total Lines	62
Duplicated Lines	4.84 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	countNotDeleted()	0	4	1
A	countDeleted()	0	4	1
A	createdIssuesCount()	0	11	2
A	countProjectsByStatus()	3	8	3

<?php

/*
 * This file is part of the Tinyissue package.
 *
 * (c) Mohamed Alsharaf <mohamed.alsharaf@gmail.com>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Tinyissue\Repository\User;

use Illuminate\Database\Eloquent\Relations;
use Tinyissue\Contracts\Model\UserInterface;
use Tinyissue\Model\Project;
use Tinyissue\Repository\Repository;

class Counter extends Repository
{
    public function __construct(UserInterface $model)
    {
        $this->model = $model;

It seems like $model of type object<Tinyissue\Contracts\Model\UserInterface> is incompatible with the declared type object<Illuminate\Database\Eloquent\Model> of property $model.

    }

    /**
     * Count active users.
     *
     * @return int
     */
    public function countNotDeleted()
    {
        return $this->model->notDeleted()->count();

The method notDeleted() does not exist on Illuminate\Database\Eloquent\Model. Did you maybe mean delete()?

    }

    /**
     * Count deleted users.
     *
     * @return int
     */
    public function countDeleted()
    {
        return $this->model->deleted()->count();

The call to deleted() misses a required argument $callback.

The method count cannot be called on $this->model->deleted() (of type null).

    }

    /**
     * Count number of created issues in a project.
     *
     * @param int $projectId
     *
     * @return int
     */
    public function createdIssuesCount($projectId = 0)
    {
        $issues = $this->model->issuesCreatedBy();

The method issuesCreatedBy() does not exist on Illuminate\Database\Eloquent\Model. Did you maybe mean create()?

        if (0 < $projectId) {
            $issues = $issues->where('project_id', '=', $projectId);
        }
        $issues->where('status', '=', Project\Issue::STATUS_OPEN);

        return $issues->count();
    }

    /**
     * Count number of archived projects.
     *
     * @param $status
     *
     * @return int
     */
    public function countProjectsByStatus($status)
    {
        if ($this->model->isManager() || $this->model->isAdmin()) {

This code seems to be duplicated across your project.

            return app()->make(Project::class)->countProjectsByStatus($status);
        }

        return $this->model->projects()->status($status)->count();
    }
}