QueryTrait::listAssignedIssues() - Code Metrics - Inspection of "Bug: apply CS fixes & fix type hint issue" - satrun77/tinyissue - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — develop ( fea319...7a0090 )

by Mohamed

created 2016-06-18 10:41 UTC

QueryTrait::listAssignedIssues() A

↳ Parent: QueryTrait

Complexity

Conditions	1
Paths	1

Size

Total Lines	9
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	7
CRAP Score	1

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	9
ccs	7
cts	7
cp	1
rs	9.6666
cc	1
eloc	7
nc	1
nop	1
crap	1

<?php

/*
 * This file is part of the Tinyissue package.
 *
 * (c) Mohamed Alsharaf <[email protected]>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Tinyissue\Model\Traits\Project;

use Illuminate\Database\Eloquent;
use Illuminate\Database\Eloquent\Relations;
use Illuminate\Database\Eloquent\Relations\Relation;
use Illuminate\Database\Query;
use Tinyissue\Model\Project;
use Tinyissue\Model\User;

/**
 * QueryTrait is trait class containing the database queries methods for the Project model.
 *
 * @author Mohamed Alsharaf <[email protected]>
 *
 * @property int $id
 *
 * @method   Eloquent\Model      where($column, $operator = null, $value = null, $boolean = 'and')
 * @method   Query\Builder       join($table, $one, $operator = null, $two = null, $type = 'inner', $where = false)
 * @method   Relations\HasMany   users()
 * @method   Relations\HasMany   issues()
 * @method   void                filterAssignTo(Query\Builder $query, $userId)
 * @method   void                filterTitleOrBody(Query\Builder $query, $keyword)
 * @method   void                filterTags(Eloquent\Builder $query, array $tags)
 * @method   void                sortByUpdated(Query\Builder $query, $order = 'asc')
 * @method   Eloquent\Collection sortByTag(Query\Builder $query, $tagGroup, $order = 'asc')
 */
trait QueryTrait
{
    /**
     * Returns collection of active projects.
     *
     * @return Eloquent\Collection
     */
    public static function activeProjects()
    {
        return static::where('status', '=', Project::STATUS_OPEN)
            ->orderBy('name', 'ASC')
            ->get();
    }

    /**
     * Returns collection of public projects.
     *
     * @return Eloquent\Collection
     */
    public function publicProjects()
    {
        return $this->where('private', '=', Project::PRIVATE_NO)
            ->orderBy('name', 'ASC')
            ->get();
    }

    /**
     * Returns all users that are not assigned in the current project.
     *
     * @return array
     */
    public function usersNotIn()
    {
        if ($this->id > 0) {
            $userIds = $this->users()->lists('user_id')->all();
            $users   = User::where('deleted', '=', User::NOT_DELETED_USERS)->whereNotIn('id', $userIds)->get();
        } else {
            $users = User::where('deleted', '=', User::NOT_DELETED_USERS)->get();
        }

        return $users->lists('fullname', 'id')->all();
    }

    /**
     * Fetch and filter issues in the project.
     *
     * @param int   $status
     * @param array $filter
     *
     * @return \Illuminate\Database\Eloquent\Collection
     */
    public function listIssues($status = Project\Issue::STATUS_OPEN, array $filter = [])
    {
        $sortOrder = array_get($filter, 'sort.sortorder', 'desc');
        $sortBy    = array_get($filter, 'sort.sortby', null);

        $query = $this->issues()
            ->with('countComments', 'user', 'updatedBy', 'tags', 'tags.parent')
            ->with([
                'tags' => function (Relation $query) use ($sortOrder) {
                    $query->orderBy('name', $sortOrder);
                },
            ])
            ->where('status', '=', $status);

        // Filter issues
        $this->filterAssignTo($query, array_get($filter, 'assignto'));
        $this->filterTitleOrBody($query, array_get($filter, 'keyword'));
        $this->filterTags($query, array_get($filter, 'tag_status'));
        $this->filterTags($query, array_get($filter, 'tag_type'));

        // Sort
        if ($sortBy == 'updated') {
            $this->sortByUpdated($query, $sortOrder);
        } elseif (($tagGroup = substr($sortBy, strlen('tag:'))) > 0) {
            return $this->sortByTag($query, $tagGroup, $sortOrder);
        }

        return $query->get();
    }

    /**
     * Fetch issues assigned to a user.
     *
     * @param int $userId
     *
     * @return \Illuminate\Database\Eloquent\Collection
     */
    public function listAssignedIssues($userId)
    {
        return $this->issues()
            ->with('countComments', 'user', 'updatedBy')
            ->where('status', '=', Project\Issue::STATUS_OPEN)
            ->where('assigned_to', '=', $userId)
            ->orderBy('updated_at', 'DESC')
            ->get();
    }

    /**
     * Returns projects with issues details eager loaded.
     *
     * @param int $status
     * @param int $private
     *
     * @return Relations\HasMany
     */
    public function projectsWidthIssues($status = Project::STATUS_OPEN, $private = Project::PRIVATE_NO)
    {
        $query = $this
            ->where('status', '=', $status)
            ->orderBy('name');

        if ($private !== Project::PRIVATE_ALL) {
            $query->where('private', '=', $private);
        }

        $query->with([
            'issues' => function (Relations\Relation $query) use ($status) {

                $query->with('updatedBy');
                if ($status === Project::STATUS_OPEN) {
                    $query->where('status', '=', Project\Issue::STATUS_OPEN);
                }
            },
            'issues.user' => function () {
            },
            'issues.countComments' => function () {
            },
        ]);

        return $query;
    }

    /**
     * Returns collection of tags for Kanban view.
     *
     * @param User $user
     *
     * @return mixed
     */
    public function getKanbanTagsForUser(User $user)
    {
        $tags = $this->kanbanTags()

            ->where(function (Eloquent\Builder $query) use ($user) {
                $query->where('role_limit', '<=', $user->role_id);
                $query->orWhere('role_limit', '=', null);
            })
            ->get();

        return $tags;
    }

    /**
     * Returns collection of issues grouped by tags.
     *
     * @param $tagIds
     *
     * @return mixed
     */
    public function issuesGroupByTags($tagIds)
    {
        $issues = $this->issues()
        ->with('user', 'tags')
            ->where('status', '=', Project\Issue::STATUS_OPEN)
            ->whereIn('projects_issues_tags.tag_id', $tagIds)
            ->join('projects_issues_tags', 'issue_id', '=', 'id')
            ->orderBy('id')
            ->get()
            ->groupBy(function (Project\Issue $issue) {
                return $issue->tags->last()->name;
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
            });

        return $issues;
    }

    /**
     * Returns users assigned to the project that can fix issues (with edit permission).
     *
     * @return Relations\BelongsToMany
     */
    public function usersCanFixIssue()
    {
        return $this->users()->where('users.role_id', '>', 1);
    }
}


1			<?php
2
3			/*
4			* This file is part of the Tinyissue package.
5			*
6			* (c) Mohamed Alsharaf <[email protected]>
7			*
8			* For the full copyright and license information, please view the LICENSE
9			* file that was distributed with this source code.
10			*/
11
12			namespace Tinyissue\Model\Traits\Project;
13
14			use Illuminate\Database\Eloquent;
15			use Illuminate\Database\Eloquent\Relations;
16			use Illuminate\Database\Eloquent\Relations\Relation;
17			use Illuminate\Database\Query;
18			use Tinyissue\Model\Project;
19			use Tinyissue\Model\User;
20
21			/**
22			* QueryTrait is trait class containing the database queries methods for the Project model.
23			*
24			* @author Mohamed Alsharaf <[email protected]>
25			*
26			* @property int $id
27			*
28			* @method Eloquent\Model where($column, $operator = null, $value = null, $boolean = 'and')
29			* @method Query\Builder join($table, $one, $operator = null, $two = null, $type = 'inner', $where = false)
30			* @method Relations\HasMany users()
31			* @method Relations\HasMany issues()
32			* @method void filterAssignTo(Query\Builder $query, $userId)
33			* @method void filterTitleOrBody(Query\Builder $query, $keyword)
34			* @method void filterTags(Eloquent\Builder $query, array $tags)
35			* @method void sortByUpdated(Query\Builder $query, $order = 'asc')
36			* @method Eloquent\Collection sortByTag(Query\Builder $query, $tagGroup, $order = 'asc')
37			*/
38			trait QueryTrait
39			{
40			/**
41			* Returns collection of active projects.
42			*
43			* @return Eloquent\Collection
44			*/
45			public static function activeProjects()
46			{
47			return static::where('status', '=', Project::STATUS_OPEN)
48			->orderBy('name', 'ASC')
49			->get();
50			}
51
52			/**
53			* Returns collection of public projects.
54			*
55			* @return Eloquent\Collection
56			*/
57			public function publicProjects()
58			{
59			return $this->where('private', '=', Project::PRIVATE_NO)
60			->orderBy('name', 'ASC')
61			->get();
62			}
63
64			/**
65			* Returns all users that are not assigned in the current project.
66			*
67			* @return array
68			*/
69	1		public function usersNotIn()
70			{
71	1		if ($this->id > 0) {
72			$userIds = $this->users()->lists('user_id')->all();
73			$users = User::where('deleted', '=', User::NOT_DELETED_USERS)->whereNotIn('id', $userIds)->get();
74			} else {
75	1		$users = User::where('deleted', '=', User::NOT_DELETED_USERS)->get();
76			}
77
78	1		return $users->lists('fullname', 'id')->all();
79			}
80
81			/**
82			* Fetch and filter issues in the project.
83			*
84			* @param int $status
85			* @param array $filter
86			*
87			* @return \Illuminate\Database\Eloquent\Collection
88			*/
89			public function listIssues($status = Project\Issue::STATUS_OPEN, array $filter = [])
90			{
91			$sortOrder = array_get($filter, 'sort.sortorder', 'desc');
92			$sortBy = array_get($filter, 'sort.sortby', null);
93
94			$query = $this->issues()
95			->with('countComments', 'user', 'updatedBy', 'tags', 'tags.parent')
96			->with([
97			'tags' => function (Relation $query) use ($sortOrder) {
98			$query->orderBy('name', $sortOrder);
99			},
100			])
101			->where('status', '=', $status);
102
103			// Filter issues
104			$this->filterAssignTo($query, array_get($filter, 'assignto'));
105			$this->filterTitleOrBody($query, array_get($filter, 'keyword'));
106			$this->filterTags($query, array_get($filter, 'tag_status'));
107			$this->filterTags($query, array_get($filter, 'tag_type'));
108
109			// Sort
110			if ($sortBy == 'updated') {
111			$this->sortByUpdated($query, $sortOrder);
112			} elseif (($tagGroup = substr($sortBy, strlen('tag:'))) > 0) {
113			return $this->sortByTag($query, $tagGroup, $sortOrder);
114			}
115
116			return $query->get();
117			}
118
119			/**
120			* Fetch issues assigned to a user.
121			*
122			* @param int $userId
123			*
124			* @return \Illuminate\Database\Eloquent\Collection
125			*/
126	1		public function listAssignedIssues($userId)
127			{
128	1		return $this->issues()
129	1		->with('countComments', 'user', 'updatedBy')
130	1		->where('status', '=', Project\Issue::STATUS_OPEN)
131	1		->where('assigned_to', '=', $userId)
132	1		->orderBy('updated_at', 'DESC')
133	1		->get();
134			}
135
136			/**
137			* Returns projects with issues details eager loaded.
138			*
139			* @param int $status
140			* @param int $private
141			*
142			* @return Relations\HasMany
143			*/
144	4		public function projectsWidthIssues($status = Project::STATUS_OPEN, $private = Project::PRIVATE_NO)
145			{
146			$query = $this
147	4		->where('status', '=', $status)
148	4		->orderBy('name');
149
150	4		if ($private !== Project::PRIVATE_ALL) {
151	4		$query->where('private', '=', $private);
152			}
153
154	4		$query->with([
155		View Code Duplication	'issues' => function (Relations\Relation $query) use ($status) {
			0 ignored issues – show Duplication introduced 2016-01-10 07:09 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
156	3		$query->with('updatedBy');
157	3		if ($status === Project::STATUS_OPEN) {
158	3		$query->where('status', '=', Project\Issue::STATUS_OPEN);
159			}
160	4		},
161			'issues.user' => function () {
162	4		},
163			'issues.countComments' => function () {
164	4		},
165			]);
166
167	4		return $query;
168			}
169
170			/**
171			* Returns collection of tags for Kanban view.
172			*
173			* @param User $user
174			*
175			* @return mixed
176			*/
177			public function getKanbanTagsForUser(User $user)
178			{
179			$tags = $this->kanbanTags()
			0 ignored issues – show Bug introduced 2016-05-02 10:04 UTC by Report Bug Copy Issue Report The method `kanbanTags()` does not exist on `Tinyissue\Model\Traits\Project\QueryTrait`. Did you maybe mean `getKanbanTagsForUser()`? This check marks calls to methods that do not seem to exist on an object. This is most likely the result of a method being renamed without all references to it being renamed likewise. Loading history...
180			->where(function (Eloquent\Builder $query) use ($user) {
181			$query->where('role_limit', '<=', $user->role_id);
182			$query->orWhere('role_limit', '=', null);
183			})
184			->get();
185
186			return $tags;
187			}
188
189			/**
190			* Returns collection of issues grouped by tags.
191			*
192			* @param $tagIds
193			*
194			* @return mixed
195			*/
196			public function issuesGroupByTags($tagIds)
197			{
198			$issues = $this->issues()
199			->with('user', 'tags')
200			->where('status', '=', Project\Issue::STATUS_OPEN)
201			->whereIn('projects_issues_tags.tag_id', $tagIds)
202			->join('projects_issues_tags', 'issue_id', '=', 'id')
203			->orderBy('id')
204			->get()
205			->groupBy(function (Project\Issue $issue) {
206			return $issue->tags->last()->name;
			0 ignored issues – show Documentation introduced 2016-05-04 10:54 UTC by Report Bug Copy Issue Report The property `tags` does not exist on `object<Tinyissue\Model\Project\Issue>`. Since you implemented `__get`, maybe consider adding a @property annotation. Since your code implements the magic getter `_get`, this function will be called for any read access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } If the property has read access only, you can use the @property-read annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
207			});
208
209			return $issues;
210			}
211
212			/**
213			* Returns users assigned to the project that can fix issues (with edit permission).
214			*
215			* @return Relations\BelongsToMany
216			*/
217	7		public function usersCanFixIssue()
218			{
219	7		return $this->users()->where('users.role_id', '>', 1);
220			}
221			}
222

satrun77 / tinyissue

Push — develop ( fea319...7a0090 )

QueryTrait::listAssignedIssues() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like