mav3rick177\RapidPagination\Base\Query\SelectOrUnionAll

Complexity

Total Complexity

6

Size/Duplication

Total Lines	33
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	7

Test Coverage

Coverage

100%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
B	create()	0	21	6

<?php

namespace mav3rick177\RapidPagination\Base\Query;

use mav3rick177\RapidPagination\Base\ArrayCursor;
use mav3rick177\RapidPagination\Base\Contracts\Cursor;

/**
 * Class SelectOrUnionAll
 */
abstract class SelectOrUnionAll implements \IteratorAggregate
{
    /**
     * @param  Order[]                $orders
     * @param  Cursor|int[]|string[]  $cursor
     * @param  Limit                  $limit
     * @param  Direction              $direction
     * @param  bool                   $exclusive
     * @param  bool                   $seekable
     * @return Select|static|UnionAll
     */
    public static function create(array $orders, $cursor, Limit $limit, Direction $direction, $exclusive, $seekable)
    {
        $cursor = $cursor instanceof Cursor ? $cursor : new ArrayCursor($cursor);
        $mainQuery = new Select(
            $cursor->has() ? ConditionGroup::createMany($orders, $cursor, $direction, $exclusive) : [],
            $direction->backward()
                ? array_map(static function (Order $order) {
                    return $order->inverse();
                }, $orders)
                : $orders,
            $limit
        );

        if (!$cursor->has() || !$seekable) {

The expression $cursor->has() of type null|boolean is loosely compared to false; this is ambiguous if the boolean can be false. You might want to explicitly use !== null instead.

            // We don't need UNION ALL and support query when cursor parameters are empty,
            // or it does not need to be seekable.
            return $mainQuery;
        }

        return new UnionAll($mainQuery);
    }
}