Mado\QueryBundle\Services\Router::createRouter()

Complexity

Conditions	4
Paths	8

Size

Total Lines	25
Code Lines	15

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	11
CRAP Score	4.0092

Importance

Changes

0

<?php
    public function createRouter(QueryBuilderOptions $queryOptions, $routeName) :Route
    {
        $params = [];
        $routeParams = [];

        if (null != $queryOptions->get('_route_params')) {
            $routeParams = array_keys($queryOptions->get('_route_params'));

It seems like $queryOptions->get('_route_params') can also be of type null; however, parameter $input of array_keys() does only seem to accept array, maybe add an additional type check?

        }

        $list = array_merge([
            'filtering',
            'limit',
            'page',
            'sorting',
        ], $routeParams);

        foreach ($list as $itemKey => $itemValue) {
            $params[$itemValue] = $queryOptions->get($itemValue);
        }

        if (!isset($routeName)) {
            $routeName = $queryOptions->get('_route');
        }

        return new Route($routeName, $params);
    }
}