HandlesLimit::getLimitParameter() - Code Metrics - Inspection of "Merge remote-tracking branch 'origin/master'" - deviouspk/larapi - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 802ddf...d5fac0 )

by Arthur

created 2018-11-16 15:50 UTC

HandlesLimit::getLimitParameter() A

↳ Parent: HandlesLimit

Complexity

Conditions	3
Paths	3

Size

Total Lines	11
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	5
CRAP Score	3.4746

Importance

Changes

Metric	Value
cc	3
eloc	7
nc	3
nop	0
dl	0
loc	11
ccs	5
cts	8
cp	0.625
crap	3.4746
rs	10
c	0
b	0
f	0

<?php
/**
 * Created by PhpStorm.
 * User: arthur
 * Date: 04.11.18
 * Time: 19:59.
 */

namespace Foundation\Abstracts\Transformers;

use Illuminate\Database\Eloquent\Collection;

/**
 * Trait IncludesRelations.
 */
trait HandlesLimit
{

    /**
     * @return int
     */
    protected function parseRequestLimitParameter()
    {
        $request = request();
        if (isset($request->limit) && is_numeric($request->limit)) {
            return (int)$request->limit;
        }

        return -1;
    }

    public function getLimitParameter() :int
    {
        $requestedLimit = $this->parseRequestLimitParameter();
        $maxLimit = $this->limit;

        if ($maxLimit === -1)
            return $requestedLimit;
        elseif ($requestedLimit > $maxLimit) {
            return $maxLimit;
        }
        return $requestedLimit;
    }

    private static function processLimit(Collection $resource){
        $class = static::class;
        $limit = call_class_function($class,'getLimitParameter');
        if($limit === -1)
            return $resource;
       return $resource->take((int) $limit);
    }


}


1		<?php
2		/**
3		* Created by PhpStorm.
4		* User: arthur
5		* Date: 04.11.18
6		* Time: 19:59.
7		*/
8
9		namespace Foundation\Abstracts\Transformers;
10
11		use Illuminate\Database\Eloquent\Collection;
12
13		/**
14		* Trait IncludesRelations.
15		*/
16		trait HandlesLimit
17		{
18
19		/**
20		* @return int
21		*/
22	10	protected function parseRequestLimitParameter()
23		{
24	10	$request = request();
25	10	if (isset($request->limit) && is_numeric($request->limit)) {
26		return (int)$request->limit;
27		}
28
29	10	return -1;
30		}
31
32	10	public function getLimitParameter() :int
33		{
34	10	$requestedLimit = $this->parseRequestLimitParameter();
35	10	$maxLimit = $this->limit;
36
37	10	if ($maxLimit === -1)
38	10	return $requestedLimit;
39		elseif ($requestedLimit > $maxLimit) {
40		return $maxLimit;
41		}
42		return $requestedLimit;
43		}
44
45	10	private static function processLimit(Collection $resource){
46	10	$class = static::class;
47	10	$limit = call_class_function($class,'getLimitParameter');
48	10	if($limit === -1)
49	10	return $resource;
50		return $resource->take((int) $limit);
51		}
52
53
54		}
55

deviouspk / larapi

Push — master ( 802ddf...d5fac0 )

HandlesLimit::getLimitParameter() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like