StingerSoft\EntitySearchBundle\Services\DoctrineListener

Complexity

Total Complexity

13

Size/Duplication

Total Lines	110
Duplicated Lines	12.73 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes

0

10 Methods

Rating	Name	Duplication	Size	Complexity
A	getSubscribedEvents()	0	8	1
A	__construct()	0	4	1
A	postPersist()	0	3	1
A	postFlush()	0	6	2
A	preRemove()	0	3	1
A	postUpdate()	0	3	1
A	getEntityToDocumentMapper()	0	3	1
A	getSearchService()	0	4	1
A	updateEntity()	7	7	2
A	removeEntity()	7	7	2

<?php

/*
 * This file is part of the Stinger Entity Search package.
 *
 * (c) Oliver Kotte <oliver.kotte@stinger-soft.net>
 * (c) Florian Meyer <florian.meyer@stinger-soft.net>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */
namespace StingerSoft\EntitySearchBundle\Services;

use Doctrine\Common\Persistence\Event\LifecycleEventArgs;
use Doctrine\Common\Persistence\ObjectManager;
use Doctrine\Common\EventSubscriber;
use StingerSoft\EntitySearchBundle\Services\Mapping\EntityToDocumentMapperInterface;
use Doctrine\ORM\Event\PostFlushEventArgs;

class DoctrineListener implements EventSubscriber {

	/**
	 *
	 * @var EntityToDocumentMapperInterface
	 */
	protected $entityToDocumentMapper;

	/**
	 *
	 * @var SearchService
	 */
	protected $searchService;

	protected $needsFlush = false;

	public function getSubscribedEvents() {
		return array(
			'postPersist',
			'postUpdate',
			'preRemove',
			'postFlush' 
		);
	}

	/**
	 * Constructor
	 *
	 * @param SearchService $searchService        	
	 */
	public function __construct(EntityToDocumentMapperInterface $entityToDocumentMapper, SearchService $searchService) {
		$this->entityToDocumentMapper = $entityToDocumentMapper;
		$this->searchService = $searchService;
	}

	/**
	 * Index the entity after it is persisted for the first time
	 *
	 * @param LifecycleEventArgs $args        	
	 */
	public function postPersist(LifecycleEventArgs $args) {
		$this->updateEntity($args->getObject(), $args->getObjectManager());
	}

	public function postFlush(PostFlushEventArgs $eventArgs) {
		if($this->needsFlush) {
			$this->needsFlush = false;
			$eventArgs->getEntityManager()->flush();
		}
	}

	/**
	 * Removes the entity from the index when it marked for deletion
	 *
	 * @param LifecycleEventArgs $args        	
	 */
	public function preRemove(LifecycleEventArgs $args) {
		$this->removeEntity($args->getObject(), $args->getObjectManager());
	}

	/**
	 * Updates the entity in the index after it is updated
	 *
	 * @param LifecycleEventArgs $args        	
	 */
	public function postUpdate(LifecycleEventArgs $args) {
		$this->updateEntity($args->getObject(), $args->getObjectManager());
	}

	/**
	 *
	 * @return EntityToDocumentMapperInterface
	 */
	protected function getEntityToDocumentMapper() {
		return $this->entityToDocumentMapper;
	}

	/**
	 *
	 * @return SearchService
	 */
	protected function getSearchService(ObjectManager $manager) {
		$this->searchService->setObjectManager($manager);
		return $this->searchService;
	}

	/**
	 *
	 * @param object $entity        	
	 */
	protected function updateEntity($entity, ObjectManager $manager) {

This method seems to be duplicated in your project.

		$document = $this->getEntityToDocumentMapper()->createDocument($manager, $entity);
		if($document !== false) {
			$this->getSearchService($manager)->saveDocument($document);

It seems like $document defined by $this->getEntityToDocumentMapper()->createDocument($manager, $entity) on line 111 can also be of type boolean; however, StingerSoft\EntitySearchBundle\Services\SearchService::saveDocument() does only seem to accept object<StingerSoft\EntitySearchBundle\Model\Document>, maybe add an additional type check?

			$this->needsFlush = true;
		}
	}

	/**
	 *
	 * @param object $entity        	
	 */
	protected function removeEntity($entity, ObjectManager $manager) {

This method seems to be duplicated in your project.

		$document = $this->getEntityToDocumentMapper()->createDocument($manager, $entity);
		if($document !== false) {
			$this->getSearchService($manager)->removeDocument($document);

It seems like $document defined by $this->getEntityToDocumentMapper()->createDocument($manager, $entity) on line 123 can also be of type boolean; however, StingerSoft\EntitySearchBundle\Services\SearchService::removeDocument() does only seem to accept object<StingerSoft\EntitySearchBundle\Model\Document>, maybe add an additional type check?

			$this->needsFlush = true;
		}
	}
}