QueryTrait::usersNotIn() - Code Metrics - Inspection of "Revert "Misc: added .DS_Store to .gitignore"" - satrun77/tinyissue - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Branch — develop (a31570)

by Mohamed

created 2016-01-10 07:04 UTC

QueryTrait::usersNotIn() A

↳ Parent: QueryTrait

Complexity

Conditions	2
Paths	2

Size

Total Lines	11
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	4
CRAP Score	2.1481

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
c	2
b	0
f	0
dl	0
loc	11
ccs	4
cts	6
cp	0.6667
rs	9.4286
cc	2
eloc	7
nc	2
nop	0
crap	2.1481

<?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\Tag;
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 ($status, $sortOrder) {
                    $status = $status == Project\Issue::STATUS_OPEN ? Tag::STATUS_OPEN : Tag::STATUS_CLOSED;
$x = 1;
$callable = function() use ($x) {
    $x = 2; // Not visible in outer scope. If you would like this, how
            // about using a different variable name than $x?
};

$callable();
var_dump($x); // integer(1)
                    $query->where('name', '!=',
                        ($status == Project\Issue::STATUS_OPEN ? Tag::STATUS_OPEN : Tag::STATUS_CLOSED));
                    $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, 'tags'));

        // 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;
    }

}


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\Tag;
20			use Tinyissue\Model\User;
21
22			/**
23			* QueryTrait is trait class containing the database queries methods for the Project model
24			*
25			* @author Mohamed Alsharaf <[email protected]>
26			*
27			* @property int $id
28			*
29			* @method Eloquent\Model where($column, $operator = null, $value = null, $boolean = 'and')
30			* @method Query\Builder join($table, $one, $operator = null, $two = null, $type = 'inner', $where = false)
31			* @method Relations\HasMany users()
32			* @method Relations\HasMany issues()
33			* @method void filterAssignTo(Query\Builder $query, $userId)
34			* @method void filterTitleOrBody(Query\Builder $query, $keyword)
35			* @method void filterTags(Eloquent\Builder $query, array $tags)
36			* @method void sortByUpdated(Query\Builder $query, $order = 'asc')
37			* @method Eloquent\Collection sortByTag(Query\Builder $query, $tagGroup, $order = 'asc')
38			*/
39			trait QueryTrait
40			{
41			/**
42			* Returns collection of active projects
43			*
44			* @return Eloquent\Collection
45			*/
46			public static function activeProjects()
47			{
48			return static::where('status', '=', Project::STATUS_OPEN)
49			->orderBy('name', 'ASC')
50			->get();
51			}
52
53			/**
54			* Returns collection of public projects
55			*
56			* @return Eloquent\Collection
57			*/
58			public function publicProjects()
59			{
60			return $this->where('private', '=', Project::PRIVATE_NO)
61			->orderBy('name', 'ASC')
62			->get();
63			}
64
65			/**
66			* Returns all users that are not assigned in the current project.
67			*
68			* @return array
69			*/
70	1		public function usersNotIn()
71			{
72	1		if ($this->id > 0) {
73			$userIds = $this->users()->lists('user_id')->all();
74			$users = User::where('deleted', '=', User::NOT_DELETED_USERS)->whereNotIn('id', $userIds)->get();
75			} else {
76	1		$users = User::where('deleted', '=', User::NOT_DELETED_USERS)->get();
77			}
78
79	1		return $users->lists('fullname', 'id')->all();
80			}
81
82			/**
83			* Fetch and filter issues in the project
84			*
85			* @param int $status
86			* @param array $filter
87			*
88			* @return \Illuminate\Database\Eloquent\Collection
89			*/
90	2		public function listIssues($status = Project\Issue::STATUS_OPEN, array $filter = [])
91			{
92	2		$sortOrder = array_get($filter, 'sort.sortorder', 'desc');
93	2		$sortBy = array_get($filter, 'sort.sortby', null);
94
95	2		$query = $this->issues()
96	2		->with('countComments', 'user', 'updatedBy', 'tags', 'tags.parent')
97	2		->with([
98			'tags' => function (Relation $query) use ($status, $sortOrder) {
99	2		$status = $status == Project\Issue::STATUS_OPEN ? Tag::STATUS_OPEN : Tag::STATUS_CLOSED;
			0 ignored issues – show Bug introduced 2015-12-15 16:27 UTC by Report Bug Copy Issue Report Consider using a different name than the imported variable `$status`, or did you forget to import by reference? It seems like you are assigning to a variable which was imported through a `use` statement which was not imported by reference. For clarity, we suggest to use a different name or import by reference depending on whether you would like to have the change visibile in outer-scope. Change not visible in outer-scope $x = 1; $callable = function() use ($x) { $x = 2; // Not visible in outer scope. If you would like this, how // about using a different variable name than $x? }; $callable(); var_dump($x); // integer(1) Change visible in outer-scope $x = 1; $callable = function() use (&$x) { $x = 2; }; $callable(); var_dump($x); // integer(2) Loading history...
100	2		$query->where('name', '!=',
101	2		($status == Project\Issue::STATUS_OPEN ? Tag::STATUS_OPEN : Tag::STATUS_CLOSED));
102	2		$query->orderBy('name', $sortOrder);
103	2		},
104			])
105	2		->where('status', '=', $status);
106
107			// Filter issues
108	2		$this->filterAssignTo($query, array_get($filter, 'assignto'));
109	2		$this->filterTitleOrBody($query, array_get($filter, 'keyword'));
110	2		$this->filterTags($query, array_get($filter, 'tags'));
111
112			// Sort
113	2		if ($sortBy == 'updated') {
114			$this->sortByUpdated($query, $sortOrder);
115	2		} elseif (($tagGroup = substr($sortBy, strlen('tag:'))) > 0) {
116			return $this->sortByTag($query, $tagGroup, $sortOrder);
117			}
118
119	2		return $query->get();
120			}
121
122			/**
123			* Fetch issues assigned to a user
124			*
125			* @param int $userId
126			*
127			* @return \Illuminate\Database\Eloquent\Collection
128			*/
129	1		public function listAssignedIssues($userId)
130			{
131	1		return $this->issues()
132	1		->with('countComments', 'user', 'updatedBy')
133	1		->where('status', '=', Project\Issue::STATUS_OPEN)
134	1		->where('assigned_to', '=', $userId)
135	1		->orderBy('updated_at', 'DESC')
136	1		->get();
137			}
138
139			/**
140			* Returns projects with issues details eager loaded
141			*
142			* @param int $status
143			* @param int $private
144			*
145			* @return Relations\HasMany
146			*/
147			public function projectsWidthIssues($status = Project::STATUS_OPEN, $private = Project::PRIVATE_NO)
148			{
149			$query = $this
150			->where('status', '=', $status)
151			->orderBy('name');
152
153			if ($private !== Project::PRIVATE_ALL) {
154			$query->where('private', '=', $private);
155			}
156
157			$query->with([
158		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...
159			$query->with('updatedBy');
160			if ($status === Project::STATUS_OPEN) {
161			$query->where('status', '=', Project\Issue::STATUS_OPEN);
162			}
163			},
164			'issues.user' => function () {
165			},
166			'issues.countComments' => function () {
167			},
168			]);
169
170			return $query;
171			}
172
173			}
174

satrun77 / tinyissue

Branch — develop (a31570)

QueryTrait::usersNotIn() A

Complexity

Size

Duplication

Code Coverage

Importance

Change not visible in outer-scope

Change visible in outer-scope

Duplication Side-by-Side

Filter issues like