Tinyissue\Repository\User\Counter

Complexity

Total Complexity

11

Size/Duplication

Total Lines	79
Duplicated Lines	17.72 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	countNotDeleted()	0	4	1
A	countDeleted()	0	4	1
A	createdIssuesCount()	11	11	2
A	projectsWithCountOpenIssues()	0	11	3
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\Permission;
use Tinyissue\Model\Project;
use Tinyissue\Repository\Repository;

/**
 * Class Counter.
 *
 * @package Tinyissue\\User
 */
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<Tinyissue\Repository\Model> of property $model.

    }

    /**
     * @param int $deleted

There is no parameter named $deleted. Was it maybe removed?

     *
     * @return int
     */
    public function countNotDeleted()
    {
        return $this->model->notDeleted()->count();
    }

    /**
     * @param int $deleted

There is no parameter named $deleted. Was it maybe removed?

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

    /**
     * Count number of created issues in a project.
     *
     * @param int $projectId
     *
     * @return int
     */
    public function createdIssuesCount($projectId = 0)

This method seems to be duplicated in your project.

    {
        $issues = $this->model->issuesCreatedBy();

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

        return $issues->count();
    }

    /**
     * Returns all projects with open issue count.
     *
     * @param int $status
     *
     * @return Builder|Relations\BelongsToMany
     */
    public function projectsWithCountOpenIssues($status = Project::STATUS_OPEN)
    {
        // TODO not nice
        if ($this->model->isManager() || $this->model->isAdmin()) {
            $project = new Project();

            return $project->projectsWithOpenIssuesCount($status, Project::PRIVATE_ALL);

The method projectsWithOpenIssuesCount() does not exist on Tinyissue\Model\Project. Did you maybe mean openIssuesCount()?

        }

        return $this->model->projects($status)->with('openIssuesCount');
    }

    /**
     * Count number of archived projects.
     *
     * @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();
    }
}