Applications\Repository\Application

Complexity

Total Complexity

27

Size/Duplication

Total Lines	217
Duplicated Lines	20.74 %

Coupling/Cohesion

Components	1
Dependencies	7

Importance

Changes	1
Bugs	0	Features	0

13 Methods

Rating	Name	Duplication	Size	Complexity
A	findBy()	9	9	3
A	findOneBy()	9	9	3
A	createQueryBuilder()	8	8	2
A	getPaginatorCursor()	0	6	1
A	getPaginationQueryBuilder()	7	7	1
A	getPaginationList()	0	11	1
A	loadApplicationsForJob()	0	7	1
A	loadUnreadApplicationsForJob()	0	8	1
A	findComment()	0	15	4
A	findProfile()	0	10	3
A	getStates()	0	7	1
B	findDraft()	0	25	4
A	getUserApplications()	12	12	2

<?php
/**
 * YAWIK
 *
 * @filesource
 * @copyright (c) 2013 - 2016 Cross Solution (http://cross-solution.de)
 * @license   MIT
 * @author Carsten Bleek <bleek@cross-solution.de>
 * @author Mathias Gelhausen <gelhausen@cross-solution.de>
 * @author Mathias Weitz <weitz@xenon>
 * @author Miroslav Fedeleš <miroslav.fedeles@gmail.com>
 */
namespace Applications\Repository;

use Core\Repository\AbstractRepository;
use Applications\Entity\Application as ApplicationEntity;
use Applications\Entity\CommentInterface;
use Zend\Stdlib\ArrayUtils;
use Auth\Entity\UserInterface;

/**
 * class for accessing applications
 *
 * @todo find better mechanism for loading drafts or applications or both states.
 *
 * @package Applications
 */
class Application extends AbstractRepository
{
   
    /**
     * {@inheritDoc}
     */
    public function findBy(array $criteria, array $sort = null, $limit = null, $skip = null)

This method seems to be duplicated in your project.

    {
        if (!array_key_exists('isDraft', $criteria)) {
            $criteria['isDraft'] = false;
        } elseif (null === $criteria['isDraft']) {
            unset($criteria['isDraft']);
        }
        return parent::findBy($criteria, $sort, $limit, $skip);
    }
    
    /**
     * {@inheritDoc}
     */
    public function findOneBy(array $criteria)

This method seems to be duplicated in your project.

    {
        if (!array_key_exists('isDraft', $criteria)) {
            $criteria['isDraft'] = false;
        } elseif (null === $criteria['isDraft']) {
            unset($criteria['isDraft']);
        }
        return parent::findOneBy($criteria);
    }
    
    /**
     * {@inheritDoc}
     */
    public function createQueryBuilder($findDrafts = false)

This method seems to be duplicated in your project.

    {
        $qb = parent::createQueryBuilder();
        if (null !== $findDrafts) {
            $qb->field('isDraft')->equals($findDrafts);
        }
        return $qb;
    }
    
    /**
     * Gets a pointer to an application
     *
     * @param array $params
     */
    public function getPaginatorCursor($params)
    {
        return $this->getPaginationQueryBuilder($params)
                    ->getQuery()
                    ->execute();
    }
    
    /**
     * Gets a query builder to search for applications
     *
     * @param array $params
     * @return unknown
     */
    protected function getPaginationQueryBuilder($params)

This method seems to be duplicated in your project.

    {
        $filter = $this->getService('filterManager')->get('PaginationQuery/Applications');
        $qb = $filter->filter($params, $this->createQueryBuilder());
        
        return $qb;
    }
    
    /**
     * Gets a result list of applications
     *
     * @param array $params
     * @return \Applications\Repository\PaginationList
     */
    public function getPaginationList($params)
    {
        $qb = $this->getPaginationQueryBuilder($params);
        $cursor = $qb->hydrate(false)
                     ->select('_id')
                     ->getQuery()
                     ->execute();
        
        $list = new PaginationList(array_keys(ArrayUtils::iteratorToArray($cursor)));
        return $list;
    }

    /**
     * @param $job
     *
     * @return mixed
     */
    public function loadApplicationsForJob($job)
    {
        return $this->createQueryBuilder()
                    ->field("job")->equals(new \MongoId($job->id))
                    ->getQuery()
                    ->execute();
    }
    
    /**
     * Get unread applications
     *
     * @param \Jobs\Entity\JobInterface $job
     * @return array|bool|\Doctrine\MongoDB\ArrayIterator|\Doctrine\MongoDB\Cursor|\Doctrine\MongoDB\EagerCursor|mixed|null
     */
    public function loadUnreadApplicationsForJob($job)
    {
        $auth=$this->getService('AuthenticationService');
        $qb=$this->createQueryBuilder()
                  ->field("readBy")->notIn(array($auth->getUser()->id))
                  ->field("job")->equals(new \MongoId($job->id));

Accessing id on the interface Jobs\Entity\JobInterface suggest that you code against a concrete implementation. How about adding an instanceof check?

        return $qb->getQuery()->execute();
    }

    /**
     * Get comments of an applications
     *
     * @param $commentOrId
     * @internal param \Application\Entity\Comment $comment | Id
     * @return \Applications\Entity\Comment|NULL
     */
    public function findComment($commentOrId)
    {
        if ($commentOrId instanceof CommentInterface) {
            $commentOrId = $commentOrId->getId();
        }
        
        $application = $this->findOneBy(array('comments.id' => $commentOrId));
        /* @var \Applications\Entity\Comment $comment */
        foreach ($application->getComments() as $comment) {
            if ($comment->getId() == $commentOrId) {
                return $comment;
            }
        }
        return null;
    }
    
    /**
     * Gets social profiles of an application
     *
     * @param String $profileId
     * @return $profile|NULL

The doc-type $profile|NULL could not be parsed: Unknown type name "$profile" at position 0. (view supported doc-types)

     */
    public function findProfile($profileId)
    {
        $application = $this->findOneBy(array('isDraft' => null, 'profiles._id' => new \MongoId($profileId)));
        foreach ($application->getProfiles() as $profile) {
            if ($profile->getId() == $profileId) {
                return $profile;
            }
        }
        return null;
    }

    /**
     * @return mixed
     * @throws \Doctrine\ODM\MongoDB\MongoDBException
     */
    public function getStates()
    {
        $qb = $this->createQueryBuilder();
        $qb->hydrate(false)->distinct('status.name');
        $result = $qb->getQuery()->execute();
        return $result;
    }

    /**
     * @param $user UserInterface
     * @param $applyId
     * @return ApplicationEntity|null
     */
    public function findDraft($user, $applyId)
    {
        if ($user instanceof UserInterface) {
            $user = $user->getId();
        }

        $documents = $this->findBy(
            [
                'isDraft' => true,
                '$or'     => [
                    ['user' => $user],
                    ['permissions.change' => $user]
                ]
            ]
        );

        /* @var $document ApplicationEntity */
        foreach ($documents as $document) {
            if ($applyId == $document->getJob()->getApplyId()) {
                return $document;
            }
        }
        
        return null;
    }

    /**
     * Get applications for given user ID
     *
     * @param string $userId
     * @param int $limit

Should the type for parameter $limit not be integer|null?

     * @return Cursor
     * @since 0.27
     */
    public function getUserApplications($userId, $limit = null)

This method seems to be duplicated in your project.

    {
        $qb = $this->createQueryBuilder(null)

null is of type null, but the function expects a boolean.

            ->field('user')->equals($userId)
            ->sort(['date' => -1]);
    
        if (isset($limit)) {
            $qb->limit($limit);
        }
    
        return $qb->getQuery()->execute();
    }
}