Doctrine\ODM\CouchDB\View\ODMQuery

Complexity

Total Complexity

11

Size/Duplication

Total Lines	63
Duplicated Lines	7.94 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
B	execute()	5	20	7
A	setDocumentManager()	0	4	1
A	onlyDocs()	0	8	2
A	toArray()	0	5	1

<?php


namespace Doctrine\ODM\CouchDB\View;

use Doctrine\CouchDB\View\Query;
use Doctrine\ODM\CouchDB\DocumentManager;

class ODMQuery extends Query
{
    /**
     * @var DocumentManager
     */
    private $dm;

    /**
     * @var bool
     */
    private $onlyDocs = false;

    private $toArray = false;

    public function execute()
    {
        $response = $this->doExecute();
        $data = array();
        if ($this->dm && $this->getParameter('include_docs') === true) {
            $uow = $this->dm->getUnitOfWork();
            foreach ($response->body['rows'] AS $k => $v) {
                $doc = $uow->createDocument(null, $v['doc']);
                if ($this->toArray) {
                    $data[] = $doc;
                } else if ($this->onlyDocs) {

This code seems to be duplicated across your project.

                    $response->body['rows'][$k] = $doc;
                } else {
                    $response->body['rows'][$k]['doc'] = $doc;
                }
            }
        }

        return ($this->toArray) ? $data : $this->createResult($response);

The expression $this->toArray ? $data : $this->createResult($response); of type array|Doctrine\CouchDB\View\Result adds the type array to the return on line 41 which is incompatible with the return type of the parent method Doctrine\CouchDB\View\AbstractQuery::execute of type Doctrine\CouchDB\View\Result.

    }


    /**
     * @param DocumentManager $dm
     */
    public function setDocumentManager(DocumentManager $dm)
    {
        $this->dm = $dm;
    }

    /**
     * @param  bool $flag
     * @return Query
     */
    public function onlyDocs($flag)
    {
        if ($flag) {
            $this->setIncludeDocs(true);
        }
        $this->onlyDocs = $flag;
        return $this;
    }

    public function toArray($flag)
    {
        $this->toArray = $flag;
        return $this;
    }
}