PhraseanetSDK\Repository\Record

Complexity

Total Complexity

7

Size/Duplication

Total Lines	74
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	6

Test Coverage

Coverage

92.86%

Importance

Changes	8
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	findById()	0	17	3
A	find()	0	16	2
A	search()	0	13	2

<?php

/*
 * This file is part of Phraseanet SDK.
 *
 * (c) Alchemy <info@alchemy.fr>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace PhraseanetSDK\Repository;

use PhraseanetSDK\AbstractRepository;
use PhraseanetSDK\Entity\Query;
use PhraseanetSDK\Exception\RuntimeException;
use Doctrine\Common\Collections\ArrayCollection;

class Record extends AbstractRepository
{
    /**
     * Find the record by its id that belongs to the provided databox
     *
     * @param  integer                      $databoxId The record databox id
     * @param  integer                      $recordId  The record id
     * @return \PhraseanetSDK\Entity\Record
     * @throws RuntimeException
     */
    public function findById($databoxId, $recordId, $disableCache = false)
    {
        $path = sprintf('v1/records/%s/%s/', $databoxId, $recordId);
        $query = [];

        if ($disableCache) {
            $query['t'] = time();
        }

        $response = $this->query('GET', $path, $query);

        if (true !== $response->hasProperty('record')) {
            throw new RuntimeException('Missing "record" property in response content');
        }

        return \PhraseanetSDK\Entity\Record::fromValue($response->getProperty('record'));

It seems like $response->getProperty('record') targeting PhraseanetSDK\Http\APIResponse::getProperty() can also be of type array<integer,object<stdClass>> or null; however, PhraseanetSDK\Entity\Record::fromValue() does only seem to accept object<stdClass>, maybe add an additional type check?

    }

    /**
     * Find records
     *
     * @param  integer          $offsetStart The offset
     * @param  integer          $perPage     The number of item per page
     * @return ArrayCollection
     * @throws RuntimeException
     */
    public function find($offsetStart, $perPage)
    {
        $response = $this->query('POST', 'v1/records/search/', array(), array(
            'query'        => 'all',
            'offset_start' => (int) $offsetStart,
            'per_page'     => (int) $perPage,
        ));

        if (true !== $response->hasProperty('results')) {
            throw new RuntimeException('Missing "results" property in response content');
        }

        return new ArrayCollection(\PhraseanetSDK\Entity\Record::fromList(
            $response->getProperty('results')

It seems like $response->getProperty('results') targeting PhraseanetSDK\Http\APIResponse::getProperty() can also be of type null or object<stdClass>; however, PhraseanetSDK\Entity\Record::fromList() does only seem to accept array<integer,object<stdClass>>, maybe add an additional type check?

        ));
    }

    /**
     * Search for records
     *
     * @param  array                       $parameters Query parameters
     * @return \PhraseanetSDK\Entity\Query object
     * @throws RuntimeException
     */
    public function search(array $parameters = array())
    {
        $response = $this->query('POST', 'v1/search/', array(), array_merge(
            array('search_type' => 0),
            $parameters
        ));

        if ($response->isEmpty()) {
            throw new RuntimeException('Response content is empty');
        }

        return Query::fromValue($this->em, $response->getResult());
    }
}