Position::paginate() - Code Metrics - Inspection of "Merge pull request #13 from bristol-su/develop" - bristol-su/control - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 348781...c5b82c )

by Toby

created 2020-04-22 13:29 UTC

Position::paginate() A

↳ Parent: Position

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	2
CRAP Score	1

Importance

Changes

Metric	Value
eloc	1
c	0
b	0
f	0
dl	0
loc	3
ccs	2
cts	2
cp	1
rs	10
cc	1
nc	1
nop	2
crap	1

<?php


namespace BristolSU\ControlDB\Repositories;

use BristolSU\ControlDB\Contracts\Models\Position as PositionModel;
use BristolSU\ControlDB\Contracts\Repositories\Position as PositionContract;
use Illuminate\Support\Collection;

/**
 * Handles positions
 */

class Position implements PositionContract
{

    /**
     * Get a position by ID
     *
     * @param int $id ID of the position

     * @return PositionModel

     */
    public function getById(int $id): PositionModel
    {
        return \BristolSU\ControlDB\Models\Position::findOrFail($id);
    }

    /**
     * Get all positions
     *
     * @return Collection|PositionModel[]
     */
    public function all(): Collection
    {
        return \BristolSU\ControlDB\Models\Position::all();
    }

    /**
     * Create a new position
     *
     * @param int $dataProviderId

     * @return PositionModel

     */
    public function create(int $dataProviderId): PositionModel
    {
        return \BristolSU\ControlDB\Models\Position::create([

            'data_provider_id' => $dataProviderId
        ]);
someFunctionCall(
    $firstArgument,
    $secondArgument,
    $thirdArgument
); // Closing parenthesis on a new line.
    }

    /**
     * Delete a position by ID
     *
     * @param int $id

     */

    public function delete(int $id): void
    {
        \BristolSU\ControlDB\Models\Position::findOrFail($id)->delete();
    }

    /**
     * Get a position by its data provider ID
     *
     * @param int $dataProviderId

     * @return PositionModel

     */
    public function getByDataProviderId(int $dataProviderId): PositionModel {

        return \BristolSU\ControlDB\Models\Position::where('data_provider_id', $dataProviderId)->firstOrFail();
    }

    /**
     * Paginate through all the positions
     *
     * @param int $page The page number to return

     * @param int $perPage The number of results to return per page
     *
     * @return Collection|PositionModel[]
     */
    public function paginate(int $page, int $perPage): Collection
    {
        return \BristolSU\ControlDB\Models\Position::paginate($perPage, ['*'], 'page', $page)->getCollection();
    }

    /**
     * Get the number of positions
     *
     * @return int
     */
    public function count(): int
    {
        return \BristolSU\ControlDB\Models\Position::count();

    }
    
}


1		<?php
2
		0 ignored issues – show Coding Style introduced 2020-01-29 23:13 UTC by Report Bug Copy Issue Report Missing file doc comment Loading history...
3		namespace BristolSU\ControlDB\Repositories;
4
5		use BristolSU\ControlDB\Contracts\Models\Position as PositionModel;
6		use BristolSU\ControlDB\Contracts\Repositories\Position as PositionContract;
7		use Illuminate\Support\Collection;
8
9		/**
10		* Handles positions
11		*/
		0 ignored issues – show Coding Style introduced 2020-01-29 23:13 UTC by Report Bug Copy Issue Report Missing @category tag in class comment Loading history... Coding Style introduced 2020-01-29 23:13 UTC by Report Bug Copy Issue Report Missing @package tag in class comment Loading history... Coding Style introduced 2020-01-29 23:13 UTC by Report Bug Copy Issue Report Missing @author tag in class comment Loading history... Coding Style introduced 2020-01-29 23:13 UTC by Report Bug Copy Issue Report Missing @license tag in class comment Loading history... Coding Style introduced 2020-01-29 23:13 UTC by Report Bug Copy Issue Report Missing @link tag in class comment Loading history...
12		class Position implements PositionContract
13		{
14
15		/**
16		* Get a position by ID
17		*
18		* @param int $id ID of the position
		0 ignored issues – show Coding Style introduced 2020-01-29 23:13 UTC by Report Bug Copy Issue Report Tag value for @param tag indented incorrectly; expected 2 spaces but found 1 Loading history...
19		* @return PositionModel
		0 ignored issues – show Coding Style introduced 2020-01-29 23:13 UTC by Report Bug Copy Issue Report Tag @return cannot be grouped with parameter tags in a doc comment Loading history...
20		*/
21	20	public function getById(int $id): PositionModel
22		{
23	20	return \BristolSU\ControlDB\Models\Position::findOrFail($id);
24		}
25
26		/**
27		* Get all positions
28		*
29		* @return Collection\|PositionModel[]
30		*/
31	2	public function all(): Collection
32		{
33	2	return \BristolSU\ControlDB\Models\Position::all();
34		}
35
36		/**
37		* Create a new position
38		*
39		* @param int $dataProviderId
		0 ignored issues – show Coding Style introduced 2020-01-29 23:13 UTC by Report Bug Copy Issue Report Missing parameter comment Loading history... Coding Style introduced 2020-01-29 23:13 UTC by Report Bug Copy Issue Report Tag value for @param tag indented incorrectly; expected 2 spaces but found 1 Loading history...
40		* @return PositionModel
		0 ignored issues – show Coding Style introduced 2020-01-29 23:13 UTC by Report Bug Copy Issue Report Tag @return cannot be grouped with parameter tags in a doc comment Loading history...
41		*/
42	4	public function create(int $dataProviderId): PositionModel
43		{
44	4	return \BristolSU\ControlDB\Models\Position::create([
		0 ignored issues – show Coding Style introduced 2020-01-29 23:13 UTC by Report Bug Copy Issue Report The opening parenthesis of a multi-line function call should be the last content on the line. Loading history...
45	4	'data_provider_id' => $dataProviderId
46		]);
		0 ignored issues – show Coding Style introduced 2020-01-29 23:13 UTC by Report Bug Copy Issue Report For multi-line function calls, the closing parenthesis should be on a new line. If a function call spawns multiple lines, the coding standard suggests to move the closing parenthesis to a new line: someFunctionCall( $firstArgument, $secondArgument, $thirdArgument ); // Closing parenthesis on a new line. Loading history...
47		}
48
49		/**
50		* Delete a position by ID
51		*
52		* @param int $id
		0 ignored issues – show Coding Style introduced 2020-01-29 23:13 UTC by Report Bug Copy Issue Report Missing parameter comment Loading history...
53		*/
		0 ignored issues – show Coding Style introduced 2020-01-29 23:13 UTC by Report Bug Copy Issue Report Missing @return tag in function comment Loading history...
54	2	public function delete(int $id): void
55		{
56	2	\BristolSU\ControlDB\Models\Position::findOrFail($id)->delete();
57	2	}
58
59		/**
60		* Get a position by its data provider ID
61		*
62		* @param int $dataProviderId
		0 ignored issues – show Coding Style introduced 2020-01-29 23:13 UTC by Report Bug Copy Issue Report Missing parameter comment Loading history... Coding Style introduced 2020-01-29 23:13 UTC by Report Bug Copy Issue Report Tag value for @param tag indented incorrectly; expected 2 spaces but found 1 Loading history...
63		* @return PositionModel
		0 ignored issues – show Coding Style introduced 2020-01-29 23:13 UTC by Report Bug Copy Issue Report Tag @return cannot be grouped with parameter tags in a doc comment Loading history...
64		*/
65	4	public function getByDataProviderId(int $dataProviderId): PositionModel {
		0 ignored issues – show Coding Style introduced 2020-01-29 23:13 UTC by Report Bug Copy Issue Report Opening brace should be on a new line Loading history...
66	4	return \BristolSU\ControlDB\Models\Position::where('data_provider_id', $dataProviderId)->firstOrFail();
67		}
68
69		/**
70		* Paginate through all the positions
71		*
72		* @param int $page The page number to return
		0 ignored issues – show Coding Style introduced 2020-04-22 13:25 UTC by Report Bug Copy Issue Report Expected 4 spaces after parameter name; 1 found Loading history...
73		* @param int $perPage The number of results to return per page
74		*
75		* @return Collection\|PositionModel[]
76		*/
77	3	public function paginate(int $page, int $perPage): Collection
78		{
79	3	return \BristolSU\ControlDB\Models\Position::paginate($perPage, ['*'], 'page', $page)->getCollection();
80		}
81
82		/**
83		* Get the number of positions
84		*
85		* @return int
86		*/
87	3	public function count(): int
88		{
89	3	return \BristolSU\ControlDB\Models\Position::count();
		0 ignored issues – show Bug Best Practice introduced 2020-04-22 13:24 UTC by Report Bug Copy Issue Report The expression `return BristolSU\Control...odels\Position::count()` could return the type `Illuminate\Database\Eloquent\Builder` which is incompatible with the type-hinted return `integer`. Consider adding an additional type-check to rule them out. Loading history...
90		}
91
92		}
93

bristol-su / control

GitHub Access Token became invalid

Push — master ( 348781...c5b82c )

Position::paginate() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like