Bankiru\Api\Doctrine\Rpc\DoctrineApi

Complexity

Total Complexity

3

Size/Duplication

Total Lines	40
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

0%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	createCountRequest()	0	7	1
A	createFindRequest()	0	7	1
A	createSearchRequest()	0	7	1

<?php

namespace Bankiru\Api\Doctrine\Rpc;

use Bankiru\Api\Doctrine\Mapping\ApiMetadata;

/**
 * Example implementation for API-capable communicator
 * following Doctrine arguments layout for objects fetching
 *
 * @internal
 */
final class DoctrineApi extends SingleRequestApi

There is at least one abstract method in this class. Maybe declare it as abstract, or implement the remaining methods: create, getClient, getMetadata, patch, remove

{
    const METHOD_FIND   = 'find';
    const METHOD_SEARCH = 'search';
    const METHOD_COUNT  = 'count';

    private static $params = [
        'criteria',
        'sort',
        'limit',
        'offset',
    ];

    /** {@inheritdoc} */
    protected function createCountRequest(ApiMetadata $metadata, array $criteria)
    {
        return new RpcRequest(
            $metadata->getMethodContainer()->getMethod(self::METHOD_COUNT),
            ['criteria' => $criteria]
        );
    }

    /** {@inheritdoc} */
    protected function createFindRequest(ApiMetadata $metadata, array $identifier)
    {
        return new RpcRequest(
            $metadata->getMethodContainer()->getMethod(self::METHOD_FIND),
            $identifier
        );
    }

    /** {@inheritdoc} */
    protected function createSearchRequest(ApiMetadata $metadata, array $parameters)
    {
        return new RpcRequest(
            $metadata->getMethodContainer()->getMethod(self::METHOD_SEARCH),
            array_combine(self::$params, $parameters)
        );
    }
}