CursorArgs::extractArgs() - Code Metrics - DutchCodingCompany/cursor-pagination - Measure and Improve Code Quality continuously with Scrutinizer

CursorArgs::extractArgs() A
last analyzed 2020-06-26 10:00 UTC

↳ Parent: CursorArgs

Complexity

Conditions	5
Paths	4

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	29
rs	9.1448
c	0
b	0
f	0
cc	5
nc	4
nop	3

<?php

namespace DutchCodingCompany\CursorPagination\Pagination;

use GraphQL\Error\Error;
use Illuminate\Contracts\Pagination\LengthAwarePaginator;
use Illuminate\Support\Arr;
use Laravel\Scout\Builder as ScoutBuilder;
use DutchCodingCompany\CursorPagination\Cursor;
use DutchCodingCompany\CursorPagination\CursorPaginator;
use Nuwave\Lighthouse\Pagination\PaginationArgs;
use Nuwave\Lighthouse\Pagination\PaginationType;

class CursorArgs
{
    /**
     * @var Cursor
     */
    public $cursor;

    /**
     * @var int
     */
    public $first;

    /**
     * Create a new instance from user given args.
     *
     * @param  mixed[]  $args
     * @param  \Nuwave\Lighthouse\Pagination\PaginationType|null  $paginationType
     * @return static
     *
     * @throws \GraphQL\Error\Error
     */
    public static function extractArgs(array $args, ?PaginationType $paginationType, ?int $paginateMaxCount): self
    {
        $instance = new static();

        if ($paginationType->isConnection()) {
function someFunction(A $objectMaybe = null)
{
    if ($objectMaybe instanceof A) {
        $objectMaybe->doSomething();
    }
}
            $instance->first = $args['first'];
            $instance->cursor = Cursor::decode($args);
        } else {
            return PaginationArgs::extractArgs($args, $paginationType, $paginateMaxCount);
function notNullable(stdClass $x) { }

// Unsafe
function withoutCheck(stdClass $x = null) {
    notNullable($x);
}

// Safe - Alternative 1: Adding Additional Type-Check
function withCheck(stdClass $x = null) {
    if ($x instanceof stdClass) {
        notNullable($x);
    }
}

// Safe - Alternative 2: Changing Parameter
function withNonNullableParam(stdClass $x) {
    notNullable($x);
}
        }

        if ($instance->first <= 0) {
            throw new Error(
                self::requestedZeroOrLessItems($instance->first)
            );
        }

        // Make sure the maximum pagination count is not exceeded
        if (
            $paginateMaxCount !== null
            && $instance->first > $paginateMaxCount
        ) {
            throw new Error(
                self::requestedTooManyItems($paginateMaxCount, $instance->first)
            );
        }

        return $instance;
    }

    public static function requestedZeroOrLessItems(int $amount): string
    {
        return "Requested pagination amount must be more than 0, got {$amount}.";
    }

    public static function requestedTooManyItems(int $maxCount, int $actualCount): string
    {
        return "Maximum number of {$maxCount} requested items exceeded, got {$actualCount}. Fetch smaller chunks.";
    }

    public static function scoutBuilderNotSupported(): string
    {
        return "Laravel Scout Builder is not supported for connection based pagination";
    }

    /**
     * Apply the args to a builder, constructing a paginator.
     *
     * @param \Illuminate\Database\Query\Builder|\Laravel\Scout\Builder|\Illuminate\Database\Eloquent\Relations\Relation $builder
     */
    public function applyToBuilder($builder): CursorPaginator
    {
        if ($builder instanceof ScoutBuilder) {
if ($x instanceof DoesNotExist) {
    // Do something.
}
            throw new Error(
                self::scoutBuilderNotSupported()
            );
        }

        return $builder->cursorPaginate($this->first, $this->cursor);
    }
}


1			<?php
2
3			namespace DutchCodingCompany\CursorPagination\Pagination;
4
5			use GraphQL\Error\Error;
6			use Illuminate\Contracts\Pagination\LengthAwarePaginator;
7			use Illuminate\Support\Arr;
8			use Laravel\Scout\Builder as ScoutBuilder;
9			use DutchCodingCompany\CursorPagination\Cursor;
10			use DutchCodingCompany\CursorPagination\CursorPaginator;
11			use Nuwave\Lighthouse\Pagination\PaginationArgs;
12			use Nuwave\Lighthouse\Pagination\PaginationType;
13
14			class CursorArgs
15			{
16			/**
17			* @var Cursor
18			*/
19			public $cursor;
20
21			/**
22			* @var int
23			*/
24			public $first;
25
26			/**
27			* Create a new instance from user given args.
28			*
29			* @param mixed[] $args
30			* @param \Nuwave\Lighthouse\Pagination\PaginationType\|null $paginationType
31			* @return static
32			*
33			* @throws \GraphQL\Error\Error
34			*/
35			public static function extractArgs(array $args, ?PaginationType $paginationType, ?int $paginateMaxCount): self
36			{
37			$instance = new static();
38
39			if ($paginationType->isConnection()) {
			0 ignored issues – show Bug introduced 2020-06-26 08:00 UTC by Report Bug Copy Issue Report It seems like `$paginationType` is not always an object, but can also be of type `null`. Maybe add an additional type check? If a variable is not always an object, we recommend to add an additional type check to ensure your method call is safe: function someFunction(A $objectMaybe = null) { if ($objectMaybe instanceof A) { $objectMaybe->doSomething(); } } Loading history...
40			$instance->first = $args['first'];
41			$instance->cursor = Cursor::decode($args);
42			} else {
43			return PaginationArgs::extractArgs($args, $paginationType, $paginateMaxCount);
			0 ignored issues – show Bug introduced 2020-06-26 08:00 UTC by Report Bug Copy Issue Report It seems like `$paginationType` defined by parameter `$paginationType` on line `35` can be `null`; however, `Nuwave\Lighthouse\Pagina...tionArgs::extractArgs()` does not accept `null`, maybe add an additional type check? It seems like you allow that null is being passed for a parameter, however the function which is called does not seem to accept null. We recommend to add an additional type check (or disallow null for the parameter): function notNullable(stdClass $x) { } // Unsafe function withoutCheck(stdClass $x = null) { notNullable($x); } // Safe - Alternative 1: Adding Additional Type-Check function withCheck(stdClass $x = null) { if ($x instanceof stdClass) { notNullable($x); } } // Safe - Alternative 2: Changing Parameter function withNonNullableParam(stdClass $x) { notNullable($x); } Loading history...
44			}
45
46			if ($instance->first <= 0) {
47			throw new Error(
48			self::requestedZeroOrLessItems($instance->first)
49			);
50			}
51
52			// Make sure the maximum pagination count is not exceeded
53			if (
54			$paginateMaxCount !== null
55			&& $instance->first > $paginateMaxCount
56			) {
57			throw new Error(
58			self::requestedTooManyItems($paginateMaxCount, $instance->first)
59			);
60			}
61
62			return $instance;
63			}
64
65			public static function requestedZeroOrLessItems(int $amount): string
66			{
67			return "Requested pagination amount must be more than 0, got {$amount}.";
68			}
69
70			public static function requestedTooManyItems(int $maxCount, int $actualCount): string
71			{
72			return "Maximum number of {$maxCount} requested items exceeded, got {$actualCount}. Fetch smaller chunks.";
73			}
74
75			public static function scoutBuilderNotSupported(): string
76			{
77			return "Laravel Scout Builder is not supported for connection based pagination";
78			}
79
80			/**
81			* Apply the args to a builder, constructing a paginator.
82			*
83			* @param \Illuminate\Database\Query\Builder\|\Laravel\Scout\Builder\|\Illuminate\Database\Eloquent\Relations\Relation $builder
84			*/
85			public function applyToBuilder($builder): CursorPaginator
86			{
87			if ($builder instanceof ScoutBuilder) {
			0 ignored issues – show Bug introduced 2020-06-26 08:00 UTC by Report Bug Copy Issue Report The class `Laravel\Scout\Builder` does not exist. Did you forget a USE statement, or did you not list all dependencies? This error could be the result of: 1. Missing dependencies PHP Analyzer uses your `composer.json` file (if available) to determine the dependencies of your project and to determine all the available classes and functions. It expects the `composer.json` to be in the root folder of your repository. Are you sure this class is defined by one of your dependencies, or did you maybe not list a dependency in either the `require` or `require-dev` section? 2. Missing use statement PHP does not complain about undefined classes in `ìnstanceof` checks. For example, the following PHP code will work perfectly fine: if ($x instanceof DoesNotExist) { // Do something. } If you have not tested against this specific condition, such errors might go unnoticed. Loading history...
88			throw new Error(
89			self::scoutBuilderNotSupported()
90			);
91			}
92
93			return $builder->cursorPaginate($this->first, $this->cursor);
94			}
95			}
96

DutchCodingCompany / cursor-pagination

CursorArgs::extractArgs() A last analyzed 2020-06-26 10:00 UTC

Complexity

Size

Duplication

Importance

1. Missing dependencies

2. Missing use statement

Duplication Side-by-Side

Filter issues like

CursorArgs::extractArgs() A
last analyzed 2020-06-26 10:00 UTC